diff options
author | Steven Rostedt <srostedt@redhat.com> | 2008-11-07 22:36:02 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-11-08 03:51:24 -0500 |
commit | 451931702017951f74624ddc4f7f02e4641b0e20 (patch) | |
tree | b581305e32588e4bff5da8cc66ea2db9e0aeb1c9 | |
parent | e168e0516e476070faa9e8e7b23dfcba79b76d82 (diff) |
ftrace: irqsoff tracer incorrect reset
Impact: fix to irqsoff tracer output
In converting to the new start / stop ftrace handling, the irqsoff
tracer start called the irqsoff reset function. irqsoff tracer is
not the same as the other traces, and it resets the buffers while
searching for the longest latency.
The reset that the irqsoff stop method calls disables the function
tracing. That means that, by starting the tracer, the function
tracer is disabled incorrectly.
This patch simply removes the call to reset which keeps the function
tracing enabled. Reset is not needed for the irqsoff stop method.
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/trace/trace_irqsoff.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/kernel/trace/trace_irqsoff.c b/kernel/trace/trace_irqsoff.c index a87a20fa3fc6..3509086cdc4c 100644 --- a/kernel/trace/trace_irqsoff.c +++ b/kernel/trace/trace_irqsoff.c | |||
@@ -404,7 +404,6 @@ static void irqsoff_tracer_ctrl_update(struct trace_array *tr) | |||
404 | 404 | ||
405 | static void irqsoff_tracer_start(struct trace_array *tr) | 405 | static void irqsoff_tracer_start(struct trace_array *tr) |
406 | { | 406 | { |
407 | irqsoff_tracer_reset(tr); | ||
408 | tracer_enabled = 1; | 407 | tracer_enabled = 1; |
409 | save_tracer_enabled = 1; | 408 | save_tracer_enabled = 1; |
410 | } | 409 | } |