Re: [PATCH 1/2] tracing: Move enabling tracepoints to just after rcu_init()

[Steven Rostedt]

On Wed, 14 Jan 2015 10:09:05 +0900
Namhyung Kim <namhyung@xxxxxxxxxx> wrote:


> > >> +
> > >> +void __init trace_event_init(void)
> > >> +{
> > >> +	event_trace_memsetup();
> > >> +	init_ftrace_syscalls();
> > >> +	event_trace_enable();
> > >> +}
> > >> +
> > > 
> > > init_ftrace_syscalls() get called twice by trace_init() and trace_event_init(), some resources are wasted.
> > > At lease one of them can be removed.
> 
> I think it's more natural to keep it in the trace_event_init().

I agree, I'll remove it. I'm fixing another bug that this change caused
to system call events.

-- Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/