Messages in this thread | | | Date | Tue, 5 May 2015 21:46:19 -0400 | From | Steven Rostedt <> | Subject | Re: [PATCH] tracing: Adds handler name to irq_handler_entry |
| |
On Tue, 5 May 2015 17:46:58 -0700 Badhri Jagan Sridharan <badhri@google.com> wrote:
> This patch includes the ISR function name to > irq_handler_entry trace point. > > Signed-off-by: Badhri Jagan Sridharan <Badhri@google.com> > --- > include/trace/events/irq.h | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/include/trace/events/irq.h b/include/trace/events/irq.h > index ff8f6c0..9994ba4 100644 > --- a/include/trace/events/irq.h > +++ b/include/trace/events/irq.h > @@ -58,14 +58,17 @@ TRACE_EVENT(irq_handler_entry, > TP_STRUCT__entry( > __field( int, irq ) > __string( name, action->name ) > + __field(void*, handler)
Please follow the format that the other fields have. This isn't a function call to use the "checkpatch" version of indentation.
__field( void *, handler )
-- Steve
> ), > > TP_fast_assign( > __entry->irq = irq; > __assign_str(name, action->name); > + __entry->handler = action->handler; > ), > > - TP_printk("irq=%d name=%s", __entry->irq, __get_str(name)) > + TP_printk("irq=%d name=%s handler=%pf", > + __entry->irq, __get_str(name), __entry->handler) > ); > > /**
| |