Messages in this thread | | | Date | Wed, 14 Jan 2015 12:23:29 -0500 | From | Steven Rostedt <> | Subject | Re: [PATCH 1/2] tracing: Move enabling tracepoints to just after rcu_init() |
| |
On Wed, 14 Jan 2015 10:09:05 +0900 Namhyung Kim <namhyung@kernel.org> 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
| |