After applying the previous patch, TRACE_EVENT_FL_USE_CALL_FILTER and the codes relate to it can be removed.
Signed-off-by: He Kuang <heku...@huawei.com> --- include/linux/ftrace_event.h | 2 -- kernel/trace/trace_events_filter.c | 68 ++++++-------------------------------- 2 files changed, 11 insertions(+), 59 deletions(-) diff --git a/include/linux/ftrace_event.h b/include/linux/ftrace_event.h index c674ee8..fd6bf18 100644 --- a/include/linux/ftrace_event.h +++ b/include/linux/ftrace_event.h @@ -250,7 +250,6 @@ enum { TRACE_EVENT_FL_NO_SET_FILTER_BIT, TRACE_EVENT_FL_IGNORE_ENABLE_BIT, TRACE_EVENT_FL_WAS_ENABLED_BIT, - TRACE_EVENT_FL_USE_CALL_FILTER_BIT, TRACE_EVENT_FL_TRACEPOINT_BIT, }; @@ -272,7 +271,6 @@ enum { TRACE_EVENT_FL_NO_SET_FILTER = (1 << TRACE_EVENT_FL_NO_SET_FILTER_BIT), TRACE_EVENT_FL_IGNORE_ENABLE = (1 << TRACE_EVENT_FL_IGNORE_ENABLE_BIT), TRACE_EVENT_FL_WAS_ENABLED = (1 << TRACE_EVENT_FL_WAS_ENABLED_BIT), - TRACE_EVENT_FL_USE_CALL_FILTER = (1 << TRACE_EVENT_FL_USE_CALL_FILTER_BIT), TRACE_EVENT_FL_TRACEPOINT = (1 << TRACE_EVENT_FL_TRACEPOINT_BIT), }; diff --git a/kernel/trace/trace_events_filter.c b/kernel/trace/trace_events_filter.c index ced69da..cc61e0b 100644 --- a/kernel/trace/trace_events_filter.c +++ b/kernel/trace/trace_events_filter.c @@ -645,10 +645,7 @@ static void append_filter_err(struct filter_parse_state *ps, static inline struct event_filter *event_filter(struct ftrace_event_file *file) { - if (file->event_call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - return file->event_call->filter; - else - return file->filter; + return file->filter; } /* caller must hold event_mutex */ @@ -782,12 +779,7 @@ static void __free_preds(struct event_filter *filter) static void filter_disable(struct ftrace_event_file *file) { - struct ftrace_event_call *call = file->event_call; - - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - call->flags &= ~TRACE_EVENT_FL_FILTERED; - else - file->flags &= ~FTRACE_EVENT_FL_FILTERED; + file->flags &= ~FTRACE_EVENT_FL_FILTERED; } static void __free_filter(struct event_filter *filter) @@ -839,13 +831,8 @@ static int __alloc_preds(struct event_filter *filter, int n_preds) static inline void __remove_filter(struct ftrace_event_file *file) { - struct ftrace_event_call *call = file->event_call; - filter_disable(file); - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - remove_filter_string(call->filter); - else - remove_filter_string(file->filter); + remove_filter_string(file->filter); } static void filter_free_subsystem_preds(struct ftrace_subsystem_dir *dir, @@ -862,15 +849,8 @@ static void filter_free_subsystem_preds(struct ftrace_subsystem_dir *dir, static inline void __free_subsystem_filter(struct ftrace_event_file *file) { - struct ftrace_event_call *call = file->event_call; - - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) { - __free_filter(call->filter); - call->filter = NULL; - } else { - __free_filter(file->filter); - file->filter = NULL; - } + __free_filter(file->filter); + file->filter = NULL; } static void filter_free_subsystem_filters(struct ftrace_subsystem_dir *dir, @@ -1664,55 +1644,30 @@ fail: static inline void event_set_filtered_flag(struct ftrace_event_file *file) { - struct ftrace_event_call *call = file->event_call; - - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - call->flags |= TRACE_EVENT_FL_FILTERED; - else - file->flags |= FTRACE_EVENT_FL_FILTERED; + file->flags |= FTRACE_EVENT_FL_FILTERED; } static inline void event_set_filter(struct ftrace_event_file *file, struct event_filter *filter) { - struct ftrace_event_call *call = file->event_call; - - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - rcu_assign_pointer(call->filter, filter); - else - rcu_assign_pointer(file->filter, filter); + rcu_assign_pointer(file->filter, filter); } static inline void event_clear_filter(struct ftrace_event_file *file) { - struct ftrace_event_call *call = file->event_call; - - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - RCU_INIT_POINTER(call->filter, NULL); - else - RCU_INIT_POINTER(file->filter, NULL); + RCU_INIT_POINTER(file->filter, NULL); } static inline void event_set_no_set_filter_flag(struct ftrace_event_file *file) { - struct ftrace_event_call *call = file->event_call; - - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - call->flags |= TRACE_EVENT_FL_NO_SET_FILTER; - else - file->flags |= FTRACE_EVENT_FL_NO_SET_FILTER; + file->flags |= FTRACE_EVENT_FL_NO_SET_FILTER; } static inline void event_clear_no_set_filter_flag(struct ftrace_event_file *file) { - struct ftrace_event_call *call = file->event_call; - - if (call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) - call->flags &= ~TRACE_EVENT_FL_NO_SET_FILTER; - else - file->flags &= ~FTRACE_EVENT_FL_NO_SET_FILTER; + file->flags &= ~FTRACE_EVENT_FL_NO_SET_FILTER; } static inline bool @@ -1723,8 +1678,7 @@ event_no_set_filter_flag(struct ftrace_event_file *file) if (file->flags & FTRACE_EVENT_FL_NO_SET_FILTER) return true; - if ((call->flags & TRACE_EVENT_FL_USE_CALL_FILTER) && - (call->flags & TRACE_EVENT_FL_NO_SET_FILTER)) + if (call->flags & TRACE_EVENT_FL_NO_SET_FILTER) return true; return false; -- 2.2.0.33.gc18b867 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/