]> git.baikalelectronics.ru Git - kernel.git/commit
tracing: Remove parent recording in latency tracer graph options
authorSteven Rostedt <srostedt@redhat.com>
Wed, 6 Oct 2010 03:22:19 +0000 (23:22 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Mon, 18 Oct 2010 14:53:38 +0000 (10:53 -0400)
commit9f21ebfc6597606510c5a6ecb1340278d18e7f61
tree08f529d73a27513014a15c0a21dfa2d106a68368
parent9a0b17adb566d55d0162bcec6fc685df692c67ad
tracing: Remove parent recording in latency tracer graph options

Even though the parent is recorded with the normal function tracing
of the latency tracers (irqsoff and wakeup), the function graph
recording is bogus.

This is due to the function graph messing with the return stack.
The latency tracers pass in as the parent CALLER_ADDR0, which
works fine for plain function tracing. But this causes bogus output
with the graph tracer:

 3)    <idle>-0    |  d.s3.  0.000 us    |  return_to_handler();
 3)    <idle>-0    |  d.s3.  0.000 us    |  _raw_spin_unlock_irqrestore();
 3)    <idle>-0    |  d.s3.  0.000 us    |  return_to_handler();
 3)    <idle>-0    |  d.s3.  0.000 us    |  trace_hardirqs_on();

The "return_to_handle()" call is the trampoline of the
function graph tracer, and is meaningless in this context.

Cc: Jiri Olsa <jolsa@redhat.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
kernel/trace/trace_functions_graph.c