Commit e05a43b7 authored by Steven Rostedt's avatar Steven Rostedt Committed by Ingo Molnar

trace: better use of stack_trace_enabled for boot up code

Impact: clean up

Andrew Morton suggested to use the stack_tracer_enabled variable
to decide whether or not to start stack tracing on bootup.
This lets us remove the start_stack_trace variable.
Reported-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarSteven Rostedt <srostedt@redhat.com>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent f38f1d2a
...@@ -308,7 +308,7 @@ stack_trace_sysctl(struct ctl_table *table, int write, ...@@ -308,7 +308,7 @@ stack_trace_sysctl(struct ctl_table *table, int write,
mutex_lock(&stack_sysctl_mutex); mutex_lock(&stack_sysctl_mutex);
ret = proc_dointvec(table, write, file, buffer, lenp, ppos); ret = proc_dointvec(table, write, file, buffer, lenp, ppos);
if (ret || !write || if (ret || !write ||
(last_stack_tracer_enabled == stack_tracer_enabled)) (last_stack_tracer_enabled == stack_tracer_enabled))
...@@ -326,11 +326,10 @@ stack_trace_sysctl(struct ctl_table *table, int write, ...@@ -326,11 +326,10 @@ stack_trace_sysctl(struct ctl_table *table, int write,
return ret; return ret;
} }
static int start_stack_trace __initdata;
static __init int enable_stacktrace(char *str) static __init int enable_stacktrace(char *str)
{ {
start_stack_trace = 1; stack_tracer_enabled = 1;
last_stack_tracer_enabled = 1;
return 1; return 1;
} }
__setup("stacktrace", enable_stacktrace); __setup("stacktrace", enable_stacktrace);
...@@ -352,10 +351,8 @@ static __init int stack_trace_init(void) ...@@ -352,10 +351,8 @@ static __init int stack_trace_init(void)
if (!entry) if (!entry)
pr_warning("Could not create debugfs 'stack_trace' entry\n"); pr_warning("Could not create debugfs 'stack_trace' entry\n");
if (start_stack_trace) { if (stack_tracer_enabled)
register_ftrace_function(&trace_ops); register_ftrace_function(&trace_ops);
stack_tracer_enabled = 1;
}
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment