diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2008-12-09 17:54:20 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-12-12 05:14:08 -0500 |
commit | bcbc4f20b52c2c40c43a4d2337707dcdfe81bc3a (patch) | |
tree | 897774cc2f53691b2736e3d1dbadc9d1e263d47a | |
parent | a0343e823184070f55364d8359f832dcb33c57c7 (diff) |
tracing/function-graph-tracer: annotate do_IRQ and smp_apic_timer_interrupt
Impact: move most important x86 irq entry-points to a separate subsection
Annotate do_IRQ and smp_apic_timer_interrupt to put them into the .irqentry.text
subsection. These function will so be recognized as hardirq entrypoints for the
function-graph-tracer. We could also annotate other irq entries but the others
are far less important but they can be added on request.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | arch/x86/kernel/apic.c | 3 | ||||
-rw-r--r-- | arch/x86/kernel/irq_64.c | 3 | ||||
-rw-r--r-- | include/linux/ftrace.h | 11 |
3 files changed, 15 insertions, 2 deletions
diff --git a/arch/x86/kernel/apic.c b/arch/x86/kernel/apic.c index 16f94879b525..b946ac19753b 100644 --- a/arch/x86/kernel/apic.c +++ b/arch/x86/kernel/apic.c | |||
@@ -30,6 +30,7 @@ | |||
30 | #include <linux/module.h> | 30 | #include <linux/module.h> |
31 | #include <linux/dmi.h> | 31 | #include <linux/dmi.h> |
32 | #include <linux/dmar.h> | 32 | #include <linux/dmar.h> |
33 | #include <linux/ftrace.h> | ||
33 | 34 | ||
34 | #include <asm/atomic.h> | 35 | #include <asm/atomic.h> |
35 | #include <asm/smp.h> | 36 | #include <asm/smp.h> |
@@ -800,7 +801,7 @@ static void local_apic_timer_interrupt(void) | |||
800 | * [ if a single-CPU system runs an SMP kernel then we call the local | 801 | * [ if a single-CPU system runs an SMP kernel then we call the local |
801 | * interrupt as well. Thus we cannot inline the local irq ... ] | 802 | * interrupt as well. Thus we cannot inline the local irq ... ] |
802 | */ | 803 | */ |
803 | void smp_apic_timer_interrupt(struct pt_regs *regs) | 804 | void __irq_entry smp_apic_timer_interrupt(struct pt_regs *regs) |
804 | { | 805 | { |
805 | struct pt_regs *old_regs = set_irq_regs(regs); | 806 | struct pt_regs *old_regs = set_irq_regs(regs); |
806 | 807 | ||
diff --git a/arch/x86/kernel/irq_64.c b/arch/x86/kernel/irq_64.c index 60eb84eb77a0..11c65e811ffe 100644 --- a/arch/x86/kernel/irq_64.c +++ b/arch/x86/kernel/irq_64.c | |||
@@ -13,6 +13,7 @@ | |||
13 | #include <linux/seq_file.h> | 13 | #include <linux/seq_file.h> |
14 | #include <linux/module.h> | 14 | #include <linux/module.h> |
15 | #include <linux/delay.h> | 15 | #include <linux/delay.h> |
16 | #include <linux/ftrace.h> | ||
16 | #include <asm/uaccess.h> | 17 | #include <asm/uaccess.h> |
17 | #include <asm/io_apic.h> | 18 | #include <asm/io_apic.h> |
18 | #include <asm/idle.h> | 19 | #include <asm/idle.h> |
@@ -47,7 +48,7 @@ static inline void stack_overflow_check(struct pt_regs *regs) | |||
47 | * SMP cross-CPU interrupts have their own specific | 48 | * SMP cross-CPU interrupts have their own specific |
48 | * handlers). | 49 | * handlers). |
49 | */ | 50 | */ |
50 | asmlinkage unsigned int do_IRQ(struct pt_regs *regs) | 51 | asmlinkage unsigned int __irq_entry do_IRQ(struct pt_regs *regs) |
51 | { | 52 | { |
52 | struct pt_regs *old_regs = set_irq_regs(regs); | 53 | struct pt_regs *old_regs = set_irq_regs(regs); |
53 | struct irq_desc *desc; | 54 | struct irq_desc *desc; |
diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h index 11cac81eed08..44020f31bd81 100644 --- a/include/linux/ftrace.h +++ b/include/linux/ftrace.h | |||
@@ -377,6 +377,16 @@ struct ftrace_graph_ret { | |||
377 | */ | 377 | */ |
378 | #define __notrace_funcgraph notrace | 378 | #define __notrace_funcgraph notrace |
379 | 379 | ||
380 | /* | ||
381 | * We want to which function is an entrypoint of a hardirq. | ||
382 | * That will help us to put a signal on output. | ||
383 | */ | ||
384 | #define __irq_entry __attribute__((__section__(".irqentry.text"))) | ||
385 | |||
386 | /* Limits of hardirq entrypoints */ | ||
387 | extern char __irqentry_text_start[]; | ||
388 | extern char __irqentry_text_end[]; | ||
389 | |||
380 | #define FTRACE_RETFUNC_DEPTH 50 | 390 | #define FTRACE_RETFUNC_DEPTH 50 |
381 | #define FTRACE_RETSTACK_ALLOC_SIZE 32 | 391 | #define FTRACE_RETSTACK_ALLOC_SIZE 32 |
382 | /* Type of the callback handlers for tracing function graph*/ | 392 | /* Type of the callback handlers for tracing function graph*/ |
@@ -414,6 +424,7 @@ static inline void unpause_graph_tracing(void) | |||
414 | #else | 424 | #else |
415 | 425 | ||
416 | #define __notrace_funcgraph | 426 | #define __notrace_funcgraph |
427 | #define __irq_entry | ||
417 | 428 | ||
418 | static inline void ftrace_graph_init_task(struct task_struct *t) { } | 429 | static inline void ftrace_graph_init_task(struct task_struct *t) { } |
419 | static inline void ftrace_graph_exit_task(struct task_struct *t) { } | 430 | static inline void ftrace_graph_exit_task(struct task_struct *t) { } |