Messages in this thread | | | Subject | Re: [PATCH 7/7] tracing: change filter_free_subsystem_*() to check ->subsystem rather then ->name | From | Namhyung Kim <> | Date | Mon, 14 Jul 2014 22:58:19 +0900 |
| |
2014-07-11 (금), 21:06 +0200, Oleg Nesterov: > filter_free_subsystem_preds() and filter_free_subsystem_filters() > can rely on file->system->subsystem and avoid strcmp().
I guess the same can go to the replace_system_preds() too.
Thanks, Namhyung
> > Signed-off-by: Oleg Nesterov <oleg@redhat.com> > --- > kernel/trace/trace_events_filter.c | 9 ++------- > 1 files changed, 2 insertions(+), 7 deletions(-) > > diff --git a/kernel/trace/trace_events_filter.c b/kernel/trace/trace_events_filter.c > index 54a125c..59655bc 100644 > --- a/kernel/trace/trace_events_filter.c > +++ b/kernel/trace/trace_events_filter.c > @@ -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) > continue; > - > __remove_filter(file); > } > } > @@ -874,11 +871,9 @@ static void filter_free_subsystem_filters(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) > continue; > __free_subsystem_filter(file); > }
-- 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/
| |