diff options
author | Steven Rostedt <srostedt@redhat.com> | 2012-07-20 11:04:44 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2012-07-31 10:29:52 -0400 |
commit | 4740974a6844156c14d741b0080b59d275679a23 (patch) | |
tree | 7272f577c74a185d9fe037a424d0401e4ec7c126 /kernel/trace/trace_functions.c | |
parent | 5767cfeaa9ec7b67c802143394f3ad9f8b174eb8 (diff) |
ftrace: Add default recursion protection for function tracing
As more users of the function tracer utility are being added, they do
not always add the necessary recursion protection. To protect from
function recursion due to tracing, if the callback ftrace_ops does not
specifically specify that it protects against recursion (by setting
the FTRACE_OPS_FL_RECURSION_SAFE flag), the list operation will be
called by the mcount trampoline which adds recursion protection.
If the flag is set, then the function will be called directly with no
extra protection.
Note, the list operation is called if more than one function callback
is registered, or if the arch does not support all of the function
tracer features.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace_functions.c')
-rw-r--r-- | kernel/trace/trace_functions.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/trace/trace_functions.c b/kernel/trace/trace_functions.c index 5675ebd541f0..fdff65dff1bb 100644 --- a/kernel/trace/trace_functions.c +++ b/kernel/trace/trace_functions.c | |||
@@ -153,13 +153,13 @@ function_stack_trace_call(unsigned long ip, unsigned long parent_ip, | |||
153 | static struct ftrace_ops trace_ops __read_mostly = | 153 | static struct ftrace_ops trace_ops __read_mostly = |
154 | { | 154 | { |
155 | .func = function_trace_call, | 155 | .func = function_trace_call, |
156 | .flags = FTRACE_OPS_FL_GLOBAL, | 156 | .flags = FTRACE_OPS_FL_GLOBAL | FTRACE_OPS_FL_RECURSION_SAFE, |
157 | }; | 157 | }; |
158 | 158 | ||
159 | static struct ftrace_ops trace_stack_ops __read_mostly = | 159 | static struct ftrace_ops trace_stack_ops __read_mostly = |
160 | { | 160 | { |
161 | .func = function_stack_trace_call, | 161 | .func = function_stack_trace_call, |
162 | .flags = FTRACE_OPS_FL_GLOBAL, | 162 | .flags = FTRACE_OPS_FL_GLOBAL | FTRACE_OPS_FL_RECURSION_SAFE, |
163 | }; | 163 | }; |
164 | 164 | ||
165 | /* Our two options */ | 165 | /* Our two options */ |