Skip to contentSkip to conversation

tracing: Adds handler name to irq_handler_entry

PATCH Superseded Badhri Jagan Sridharan 2 participants 3 messages May 6, 2015 Tracing

v1 of 3 Superseded Latest is v3 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 tracing: Adds handler name to irq_handler_entry

This patch includes the ISR function name to
irq_handler_entry trace point.

---
 include/trace/events/irq.h | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
include/trace/events/irq.h +4 -1
index ff8f6c0..9994ba4 100644
@@ -58,14 +58,17 @@ TRACE_EVENT(irq_handler_entry,
 	TP_STRUCT__entry(
 		__field(	int,	irq		)
 		__string(	name,	action->name	)
+		__field(void*,	handler)
 	),
 
 	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)
 );
 
 /**
Signature · 1 line
2.2.0.rc0.207.ga3a616c
maintainer
#1 lore raw
Quoted 13 lines · On Tue, 5 May 2015 17:46:58 -0700 Badhri Jagan Sridharan <[email protected]>…

This patch includes the ISR function name to
irq_handler_entry trace point.

---
 include/trace/events/irq.h | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
include/trace/events/irq.h +0 -0
index ff8f6c0..9994ba4 100644

include/trace/events/irq.h TRACE_EVENT(irq_handler_entry, · line 61

Quoted 4 lines
include/trace/events/irq.h +1 -0
@@ -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

Quoted 14 lines
include/trace/events/irq.h +3 -1
 	),
 
 	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)
 );
 
 /**
#2 lore raw

Done!!
Uploaded a new patch v2.

Quoted 43 lines · Steven Rostedt
Quoted 13 lines · On Tue, 5 May 2015 17:46:58 -0700 Badhri Jagan Sridharan <[email protected]>…

This patch includes the ISR function name to
irq_handler_entry trace point.

---
  include/trace/events/irq.h | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)
include/trace/events/irq.h +0 -0
index ff8f6c0..9994ba4 100644

include/trace/events/irq.h TRACE_EVENT(irq_handler_entry, · line 61

Quoted 4 lines
include/trace/events/irq.h +1 -0
@@ -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

Quoted 14 lines
include/trace/events/irq.h +3 -1
  	),
 
  	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)
  );
 
  /**