diff options
author | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2018-11-19 07:40:39 -0500 |
---|---|---|
committer | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2018-11-27 20:31:54 -0500 |
commit | d125f3f866df88da5a85df00291f88f0baa89f7c (patch) | |
tree | 77af76a4456a5831f27b2c96ecd78b818bdc6bf2 | |
parent | 9c4bf5e0db164f330a2d3e128e9832661f69f0e9 (diff) |
function_graph: Make ftrace_push_return_trace() static
As all architectures now call function_graph_enter() to do the entry work,
no architecture should ever call ftrace_push_return_trace(). Make it static.
This is needed to prepare for a fix of a design bug on how the curr_ret_stack
is used.
Cc: stable@kernel.org
Fixes: 03274a3ffb449 ("tracing/fgraph: Adjust fgraph depth before calling trace return callback")
Reviewed-by: Masami Hiramatsu <mhiramat@kernel.org>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
-rw-r--r-- | include/linux/ftrace.h | 3 | ||||
-rw-r--r-- | kernel/trace/trace_functions_graph.c | 2 |
2 files changed, 1 insertions, 4 deletions
diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h index 5717e8f81c59..dd16e8218db3 100644 --- a/include/linux/ftrace.h +++ b/include/linux/ftrace.h | |||
@@ -777,9 +777,6 @@ struct ftrace_ret_stack { | |||
777 | extern void return_to_handler(void); | 777 | extern void return_to_handler(void); |
778 | 778 | ||
779 | extern int | 779 | extern int |
780 | ftrace_push_return_trace(unsigned long ret, unsigned long func, int *depth, | ||
781 | unsigned long frame_pointer, unsigned long *retp); | ||
782 | extern int | ||
783 | function_graph_enter(unsigned long ret, unsigned long func, | 780 | function_graph_enter(unsigned long ret, unsigned long func, |
784 | unsigned long frame_pointer, unsigned long *retp); | 781 | unsigned long frame_pointer, unsigned long *retp); |
785 | 782 | ||
diff --git a/kernel/trace/trace_functions_graph.c b/kernel/trace/trace_functions_graph.c index 28f2602435d0..88ca787a1cdc 100644 --- a/kernel/trace/trace_functions_graph.c +++ b/kernel/trace/trace_functions_graph.c | |||
@@ -118,7 +118,7 @@ print_graph_duration(struct trace_array *tr, unsigned long long duration, | |||
118 | struct trace_seq *s, u32 flags); | 118 | struct trace_seq *s, u32 flags); |
119 | 119 | ||
120 | /* Add a function return address to the trace stack on thread info.*/ | 120 | /* Add a function return address to the trace stack on thread info.*/ |
121 | int | 121 | static int |
122 | ftrace_push_return_trace(unsigned long ret, unsigned long func, int *depth, | 122 | ftrace_push_return_trace(unsigned long ret, unsigned long func, int *depth, |
123 | unsigned long frame_pointer, unsigned long *retp) | 123 | unsigned long frame_pointer, unsigned long *retp) |
124 | { | 124 | { |