Messages in this thread | | | Date | Tue, 13 Oct 2009 15:35:26 +0200 | Subject | Re: [PATCH 2/3] tracing: Rename set_ftrace to set_bootup_ftrace | From | Frederic Weisbecker <> |
| |
2009/10/13 Steven Rostedt <rostedt@goodmis.org>: > On Tue, 2009-10-13 at 09:38 +0200, Frederic Weisbecker wrote: >> Do this rename because set_ftrace is too much generic and not enough >> self-explainable as a name. >> >> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com> >> Cc: Steven Rostedt <rostedt@goodmis.org> >> Cc: Li Zefan <lizf@cn.fujitsu.com> >> --- >> kernel/trace/trace.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c >> index 4506826..866daf8 100644 >> --- a/kernel/trace/trace.c >> +++ b/kernel/trace/trace.c >> @@ -129,7 +129,7 @@ static int tracing_set_tracer(const char *buf); >> static char bootup_tracer_buf[MAX_TRACER_SIZE] __initdata; >> static char *default_bootup_tracer; >> >> -static int __init set_ftrace(char *str) >> +static int __init set_bootup_ftrace(char *str) >> { >> strncpy(bootup_tracer_buf, str, MAX_TRACER_SIZE); >> default_bootup_tracer = bootup_tracer_buf; >> @@ -137,7 +137,7 @@ static int __init set_ftrace(char *str) >> ring_buffer_expanded = 1; >> return 1; >> } >> -__setup("ftrace=", set_ftrace); >> +__setup("ftrace=", set_bootup_ftrace); > > Actually, it probably should be called, set_cmdline_ftrace. Since it > came from the command line, and that is what is matching. > > -- Steve
No problem, I can change that in another request that also addresses Li's suggestion. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |