Messages in this thread | | | Date | Sat, 18 Apr 2009 10:18:52 -0400 (EDT) | From | Steven Rostedt <> | Subject | Re: [PATCH 1/5] tracing/events: enable code with EVENT_TRACING not EVENT_TRACER |
| |
On Sat, 18 Apr 2009, Ingo Molnar wrote:
> > * Steven Rostedt <rostedt@goodmis.org> wrote: > > > -#ifdef CONFIG_EVENT_TRACER > > +#ifdef CONFIG_EVENT_TRACING > > Hm, my crystal ball predicts many such incidents in future years to > come. > > This: > > s/CONFIG_EVENT_TRACER/CONFIG_EVENT_TRACER_PLUGIN > > might help. Or not.
The problem is that a plugin is defined (by me anyway) as something in the "available_tracers" file. The event tracer does not qualify as one. But I see the confusion with the current naming.
What about:
s/CONFIG_EVENT_TRACER/CONFIG_ENABLE_EVENT_TRACING/
It pretty much says what it does.
??
-- Steve
| |