diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-03-04 19:49:22 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-03-05 06:14:41 -0500 |
commit | 0012693ad4f636c720fed3802027f9427962f540 (patch) | |
tree | 355277518dbdcc8385f374db1a5d5e15301940e0 | |
parent | c4ef144a9d0803eb0a2d4110ae87e7f34e667ded (diff) |
tracing/function-graph-tracer: use the more lightweight local clock
Impact: decrease hangs risks with the graph tracer on slow systems
Since the function graph tracer can spend too much time on timer
interrupts, it's better now to use the more lightweight local
clock. Anyway, the function graph traces are more reliable on a
per cpu trace.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Peter Zijlstra <peterz@infradead.org>
LKML-Reference: <49af243d.06e9300a.53ad.ffff840c@mx.google.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | arch/x86/kernel/ftrace.c | 2 | ||||
-rw-r--r-- | include/linux/ftrace.h | 13 | ||||
-rw-r--r-- | kernel/trace/trace_functions_graph.c | 2 |
3 files changed, 9 insertions, 8 deletions
diff --git a/arch/x86/kernel/ftrace.c b/arch/x86/kernel/ftrace.c index 3925ec0184b1..a85da1764b1c 100644 --- a/arch/x86/kernel/ftrace.c +++ b/arch/x86/kernel/ftrace.c | |||
@@ -436,7 +436,7 @@ void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr) | |||
436 | return; | 436 | return; |
437 | } | 437 | } |
438 | 438 | ||
439 | calltime = cpu_clock(raw_smp_processor_id()); | 439 | calltime = trace_clock_local(); |
440 | 440 | ||
441 | if (ftrace_push_return_trace(old, calltime, | 441 | if (ftrace_push_return_trace(old, calltime, |
442 | self_addr, &trace.depth) == -EBUSY) { | 442 | self_addr, &trace.depth) == -EBUSY) { |
diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h index 1f69ac7c1587..6ea62acbe4b6 100644 --- a/include/linux/ftrace.h +++ b/include/linux/ftrace.h | |||
@@ -1,15 +1,16 @@ | |||
1 | #ifndef _LINUX_FTRACE_H | 1 | #ifndef _LINUX_FTRACE_H |
2 | #define _LINUX_FTRACE_H | 2 | #define _LINUX_FTRACE_H |
3 | 3 | ||
4 | #include <linux/linkage.h> | 4 | #include <linux/trace_clock.h> |
5 | #include <linux/fs.h> | ||
6 | #include <linux/ktime.h> | ||
7 | #include <linux/init.h> | ||
8 | #include <linux/types.h> | ||
9 | #include <linux/module.h> | ||
10 | #include <linux/kallsyms.h> | 5 | #include <linux/kallsyms.h> |
6 | #include <linux/linkage.h> | ||
11 | #include <linux/bitops.h> | 7 | #include <linux/bitops.h> |
8 | #include <linux/module.h> | ||
9 | #include <linux/ktime.h> | ||
12 | #include <linux/sched.h> | 10 | #include <linux/sched.h> |
11 | #include <linux/types.h> | ||
12 | #include <linux/init.h> | ||
13 | #include <linux/fs.h> | ||
13 | 14 | ||
14 | #include <asm/ftrace.h> | 15 | #include <asm/ftrace.h> |
15 | 16 | ||
diff --git a/kernel/trace/trace_functions_graph.c b/kernel/trace/trace_functions_graph.c index c009553a8e81..e527f2f66c73 100644 --- a/kernel/trace/trace_functions_graph.c +++ b/kernel/trace/trace_functions_graph.c | |||
@@ -112,7 +112,7 @@ unsigned long ftrace_return_to_handler(void) | |||
112 | unsigned long ret; | 112 | unsigned long ret; |
113 | 113 | ||
114 | ftrace_pop_return_trace(&trace, &ret); | 114 | ftrace_pop_return_trace(&trace, &ret); |
115 | trace.rettime = cpu_clock(raw_smp_processor_id()); | 115 | trace.rettime = trace_clock_local(); |
116 | ftrace_graph_return(&trace); | 116 | ftrace_graph_return(&trace); |
117 | 117 | ||
118 | if (unlikely(!ret)) { | 118 | if (unlikely(!ret)) { |