diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2008-11-23 00:22:56 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-11-23 03:17:26 -0500 |
commit | f201ae2356c74bcae130b2177b3dca903ea98071 (patch) | |
tree | c4b1b43fbe0a4594cb86749b2e7098fe15eb86ba /include/linux/ftrace.h | |
parent | a0a70c735ef714fe1b6777b571630c3d50c7b008 (diff) |
tracing/function-return-tracer: store return stack into task_struct and allocate it dynamically
Impact: use deeper function tracing depth safely
Some tests showed that function return tracing needed a more deeper depth
of function calls. But it could be unsafe to store these return addresses
to the stack.
So these arrays will now be allocated dynamically into task_struct of current
only when the tracer is activated.
Typical scheme when tracer is activated:
- allocate a return stack for each task in global list.
- fork: allocate the return stack for the newly created task
- exit: free return stack of current
- idle init: same as fork
I chose a default depth of 50. I don't have overruns anymore.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'include/linux/ftrace.h')
-rw-r--r-- | include/linux/ftrace.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h index f7ba4ea5e128..2ba259b2defa 100644 --- a/include/linux/ftrace.h +++ b/include/linux/ftrace.h | |||
@@ -323,6 +323,8 @@ struct ftrace_retfunc { | |||
323 | }; | 323 | }; |
324 | 324 | ||
325 | #ifdef CONFIG_FUNCTION_RET_TRACER | 325 | #ifdef CONFIG_FUNCTION_RET_TRACER |
326 | #define FTRACE_RETFUNC_DEPTH 50 | ||
327 | #define FTRACE_RETSTACK_ALLOC_SIZE 32 | ||
326 | /* Type of a callback handler of tracing return function */ | 328 | /* Type of a callback handler of tracing return function */ |
327 | typedef void (*trace_function_return_t)(struct ftrace_retfunc *); | 329 | typedef void (*trace_function_return_t)(struct ftrace_retfunc *); |
328 | 330 | ||
@@ -330,6 +332,9 @@ extern int register_ftrace_return(trace_function_return_t func); | |||
330 | /* The current handler in use */ | 332 | /* The current handler in use */ |
331 | extern trace_function_return_t ftrace_function_return; | 333 | extern trace_function_return_t ftrace_function_return; |
332 | extern void unregister_ftrace_return(void); | 334 | extern void unregister_ftrace_return(void); |
335 | |||
336 | extern void ftrace_retfunc_init_task(struct task_struct *t); | ||
337 | extern void ftrace_retfunc_exit_task(struct task_struct *t); | ||
333 | #endif | 338 | #endif |
334 | 339 | ||
335 | #endif /* _LINUX_FTRACE_H */ | 340 | #endif /* _LINUX_FTRACE_H */ |