On Tue,  5 May 2015 17:46:58 -0700
Badhri Jagan Sridharan <[email protected]> wrote:

> This patch includes the ISR function name to
> irq_handler_entry trace point.
> 
> Signed-off-by: Badhri Jagan Sridharan <[email protected]>
> ---
>  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)
>  );
>  
>  /**

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to