diff options
author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2014-06-25 10:32:48 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2014-07-18 13:56:57 -0400 |
commit | 7fa322dba30ced1a357d53abb4ade5535e4e24fc (patch) | |
tree | 90dcd684aa48f6280d568eb69c0f2ac8f138631a /arch/sh/kernel | |
parent | 96d4f43e3d012824f7e61920a340a1c03b6aacb0 (diff) |
sh: ftrace: Add call to ftrace_graph_is_dead() in function graph code
ftrace_stop() is going away as it disables parts of function tracing
that affects users that should not be affected. But ftrace_graph_stop()
is built on ftrace_stop(). Here's another example of killing all of
function tracing because something went wrong with function graph
tracing.
Instead of disabling all users of function tracing on function graph
error, disable only function graph tracing. To do this, the arch code
must call ftrace_graph_is_dead() before it implements function graph.
Cc: Paul Mundt <lethal@linux-sh.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'arch/sh/kernel')
-rw-r--r-- | arch/sh/kernel/ftrace.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/arch/sh/kernel/ftrace.c b/arch/sh/kernel/ftrace.c index 3c74f53db6db..079d70e6d74b 100644 --- a/arch/sh/kernel/ftrace.c +++ b/arch/sh/kernel/ftrace.c | |||
@@ -344,6 +344,9 @@ void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr) | |||
344 | struct ftrace_graph_ent trace; | 344 | struct ftrace_graph_ent trace; |
345 | unsigned long return_hooker = (unsigned long)&return_to_handler; | 345 | unsigned long return_hooker = (unsigned long)&return_to_handler; |
346 | 346 | ||
347 | if (unlikely(ftrace_graph_is_dead())) | ||
348 | return; | ||
349 | |||
347 | if (unlikely(atomic_read(¤t->tracing_graph_pause))) | 350 | if (unlikely(atomic_read(¤t->tracing_graph_pause))) |
348 | return; | 351 | return; |
349 | 352 | ||