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)
commit78c89ba121221d9224a5747803d7fffe51cd6e44
tree08f529d73a27513014a15c0a21dfa2d106a68368
parent5e6d2b9cfa3a6e7fe62fc0135bc1bd778f5db564
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