Messages in this thread | | | Date | Mon, 14 Jul 2014 21:23:54 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH 7/7] tracing: change filter_free_subsystem_*() to check ->subsystem rather then ->name |
| |
On 07/14, Steven Rostedt wrote: > > On Fri, 11 Jul 2014 21:06:55 +0200 > Oleg Nesterov <oleg@redhat.com> wrote: > > > @@ -846,13 +846,10 @@ static void filter_free_subsystem_preds(struct event_subsystem *system, > > struct trace_array *tr) > > { > > struct ftrace_event_file *file; > > - struct ftrace_event_call *call; > > > > list_for_each_entry(file, &tr->events, list) { > > - call = file->event_call; > > - if (strcmp(call->class->system, system->name) != 0) > > + if (file->system->subsystem != system) > > This is only used on one place. What about just passing in dir instead > of system and then do:
I swear, I too thought about this ;) OK, in v2.
Oleg.
| |