434a83c3fb
Now that we can filter based on fields via perf record, people will start using filter expressions and will expect them to be obvious. The primary way to see which fields are available is by looking at the trace output, such as: gcc-18676 [000] 343.011728: irq_handler_entry: irq=0 handler=timer cc1-18677 [000] 343.012727: irq_handler_entry: irq=0 handler=timer cc1-18677 [000] 343.032692: irq_handler_entry: irq=0 handler=timer cc1-18677 [000] 343.033690: irq_handler_entry: irq=0 handler=timer cc1-18677 [000] 343.034687: irq_handler_entry: irq=0 handler=timer cc1-18677 [000] 343.035686: irq_handler_entry: irq=0 handler=timer cc1-18677 [000] 343.036684: irq_handler_entry: irq=0 handler=timer While 'irq==0' filters work, the 'handler==<x>' filter expression does not work: $ perf record -R -f -a -e irq:irq_handler_entry --filter handler=timer sleep 1 Error: failed to set filter with 22 (Invalid argument) The problem is that while an 'irq' field exists and is recognized as a filter field - 'handler' does not exist - its name is 'name' in the output. To solve this, we need to synchronize the printout and the field names, wherever possible. In cases where the printout prints a non-field, we enclose that information in square brackets, such as: perf-1380 [013] 724.903505: softirq_exit: vec=9 [action=RCU] perf-1380 [013] 724.904482: softirq_exit: vec=1 [action=TIMER] This way users can use filter expressions more intuitively: all fields that show up as 'primary' (non-bracketed) information is filterable. This patch harmonizes the field names for all irq, bkl, power, sched and timer events. We might in fact think about dropping the print format bit of generic tracepoints altogether, and just print the fields that are being recorded. Cc: Li Zefan <lizf@cn.fujitsu.com> Cc: Tom Zanussi <tzanussi@gmail.com> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Mike Galbraith <efault@gmx.de> Cc: Paul Mackerras <paulus@samba.org> Cc: Arnaldo Carvalho de Melo <acme@redhat.com> LKML-Reference: <new-submission> Signed-off-by: Ingo Molnar <mingo@elte.hu>
80 lines
1.3 KiB
C
80 lines
1.3 KiB
C
#undef TRACE_SYSTEM
|
|
#define TRACE_SYSTEM power
|
|
|
|
#if !defined(_TRACE_POWER_H) || defined(TRACE_HEADER_MULTI_READ)
|
|
#define _TRACE_POWER_H
|
|
|
|
#include <linux/ktime.h>
|
|
#include <linux/tracepoint.h>
|
|
|
|
#ifndef _TRACE_POWER_ENUM_
|
|
#define _TRACE_POWER_ENUM_
|
|
enum {
|
|
POWER_NONE = 0,
|
|
POWER_CSTATE = 1,
|
|
POWER_PSTATE = 2,
|
|
};
|
|
#endif
|
|
|
|
TRACE_EVENT(power_start,
|
|
|
|
TP_PROTO(unsigned int type, unsigned int state),
|
|
|
|
TP_ARGS(type, state),
|
|
|
|
TP_STRUCT__entry(
|
|
__field( u64, type )
|
|
__field( u64, state )
|
|
),
|
|
|
|
TP_fast_assign(
|
|
__entry->type = type;
|
|
__entry->state = state;
|
|
),
|
|
|
|
TP_printk("type=%lu state=%lu", (unsigned long)__entry->type, (unsigned long)__entry->state)
|
|
);
|
|
|
|
TRACE_EVENT(power_end,
|
|
|
|
TP_PROTO(int dummy),
|
|
|
|
TP_ARGS(dummy),
|
|
|
|
TP_STRUCT__entry(
|
|
__field( u64, dummy )
|
|
),
|
|
|
|
TP_fast_assign(
|
|
__entry->dummy = 0xffff;
|
|
),
|
|
|
|
TP_printk("dummy=%lu", (unsigned long)__entry->dummy)
|
|
|
|
);
|
|
|
|
|
|
TRACE_EVENT(power_frequency,
|
|
|
|
TP_PROTO(unsigned int type, unsigned int state),
|
|
|
|
TP_ARGS(type, state),
|
|
|
|
TP_STRUCT__entry(
|
|
__field( u64, type )
|
|
__field( u64, state )
|
|
),
|
|
|
|
TP_fast_assign(
|
|
__entry->type = type;
|
|
__entry->state = state;
|
|
),
|
|
|
|
TP_printk("type=%lu state=%lu", (unsigned long)__entry->type, (unsigned long) __entry->state)
|
|
);
|
|
|
|
#endif /* _TRACE_POWER_H */
|
|
|
|
/* This part must be outside protection */
|
|
#include <trace/define_trace.h>
|