diff options
author | Jason Baron <jbaron@redhat.com> | 2009-02-26 10:11:05 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-02-26 12:43:50 -0500 |
commit | af39241b90a345556b8884adff87096afe71b050 (patch) | |
tree | a913e3f15b1fb91555b5cb5899270274ce4c2c46 /kernel/irq | |
parent | 8656e7a2fa6afcd8682990f804a2a9674568738f (diff) |
tracing, genirq: add irq enter and exit trace events
Impact: add new tracepoints
Add them to the generic IRQ code, that way every architecture
gets these new tracepoints, not just x86.
Using Steve's new 'TRACE_FORMAT', I can get function graph
trace as follows using the original two IRQ tracepoints:
3) | handle_IRQ_event() {
3) | /* (irq_handler_entry) irq=28 handler=eth0 */
3) | e1000_intr_msi() {
3) 2.460 us | __napi_schedule();
3) 9.416 us | }
3) | /* (irq_handler_exit) irq=28 handler=eth0 return=handled */
3) + 22.935 us | }
Signed-off-by: Jason Baron <jbaron@redhat.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Acked-by: Peter Zijlstra <peterz@infradead.org>
Acked-by: Masami Hiramatsu <mhiramat@redhat.com>
Cc: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Mathieu Desnoyers <compudj@krystal.dyndns.org>
Cc: "Frank Ch. Eigler" <fche@redhat.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/irq')
-rw-r--r-- | kernel/irq/handle.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/kernel/irq/handle.c b/kernel/irq/handle.c index 3aba8d12f328..4709a7c870d7 100644 --- a/kernel/irq/handle.c +++ b/kernel/irq/handle.c | |||
@@ -17,6 +17,7 @@ | |||
17 | #include <linux/kernel_stat.h> | 17 | #include <linux/kernel_stat.h> |
18 | #include <linux/rculist.h> | 18 | #include <linux/rculist.h> |
19 | #include <linux/hash.h> | 19 | #include <linux/hash.h> |
20 | #include <trace/irq.h> | ||
20 | 21 | ||
21 | #include "internals.h" | 22 | #include "internals.h" |
22 | 23 | ||
@@ -316,6 +317,9 @@ irqreturn_t no_action(int cpl, void *dev_id) | |||
316 | return IRQ_NONE; | 317 | return IRQ_NONE; |
317 | } | 318 | } |
318 | 319 | ||
320 | DEFINE_TRACE(irq_handler_entry); | ||
321 | DEFINE_TRACE(irq_handler_exit); | ||
322 | |||
319 | /** | 323 | /** |
320 | * handle_IRQ_event - irq action chain handler | 324 | * handle_IRQ_event - irq action chain handler |
321 | * @irq: the interrupt number | 325 | * @irq: the interrupt number |
@@ -332,7 +336,9 @@ irqreturn_t handle_IRQ_event(unsigned int irq, struct irqaction *action) | |||
332 | local_irq_enable_in_hardirq(); | 336 | local_irq_enable_in_hardirq(); |
333 | 337 | ||
334 | do { | 338 | do { |
339 | trace_irq_handler_entry(irq, action); | ||
335 | ret = action->handler(irq, action->dev_id); | 340 | ret = action->handler(irq, action->dev_id); |
341 | trace_irq_handler_exit(irq, action, ret); | ||
336 | if (ret == IRQ_HANDLED) | 342 | if (ret == IRQ_HANDLED) |
337 | status |= action->flags; | 343 | status |= action->flags; |
338 | retval |= ret; | 344 | retval |= ret; |