diff options
Diffstat (limited to 'kernel/trace/trace_events_trigger.c')
-rw-r--r-- | kernel/trace/trace_events_trigger.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/kernel/trace/trace_events_trigger.c b/kernel/trace/trace_events_trigger.c index 12ac8a5d4d95..f6dd115b958d 100644 --- a/kernel/trace/trace_events_trigger.c +++ b/kernel/trace/trace_events_trigger.c | |||
@@ -67,6 +67,7 @@ event_triggers_call(struct ftrace_event_file *file, void *rec) | |||
67 | { | 67 | { |
68 | struct event_trigger_data *data; | 68 | struct event_trigger_data *data; |
69 | enum event_trigger_type tt = ETT_NONE; | 69 | enum event_trigger_type tt = ETT_NONE; |
70 | struct event_filter *filter; | ||
70 | 71 | ||
71 | if (list_empty(&file->triggers)) | 72 | if (list_empty(&file->triggers)) |
72 | return tt; | 73 | return tt; |
@@ -76,7 +77,8 @@ event_triggers_call(struct ftrace_event_file *file, void *rec) | |||
76 | data->ops->func(data); | 77 | data->ops->func(data); |
77 | continue; | 78 | continue; |
78 | } | 79 | } |
79 | if (data->filter && !filter_match_preds(data->filter, rec)) | 80 | filter = rcu_dereference(data->filter); |
81 | if (filter && !filter_match_preds(filter, rec)) | ||
80 | continue; | 82 | continue; |
81 | if (data->cmd_ops->post_trigger) { | 83 | if (data->cmd_ops->post_trigger) { |
82 | tt |= data->cmd_ops->trigger_type; | 84 | tt |= data->cmd_ops->trigger_type; |
@@ -703,7 +705,7 @@ static int set_trigger_filter(char *filter_str, | |||
703 | if (ret) | 705 | if (ret) |
704 | goto out; | 706 | goto out; |
705 | assign: | 707 | assign: |
706 | tmp = data->filter; | 708 | tmp = rcu_access_pointer(data->filter); |
707 | 709 | ||
708 | rcu_assign_pointer(data->filter, filter); | 710 | rcu_assign_pointer(data->filter, filter); |
709 | 711 | ||
@@ -719,7 +721,7 @@ static int set_trigger_filter(char *filter_str, | |||
719 | if (filter_str) { | 721 | if (filter_str) { |
720 | data->filter_str = kstrdup(filter_str, GFP_KERNEL); | 722 | data->filter_str = kstrdup(filter_str, GFP_KERNEL); |
721 | if (!data->filter_str) { | 723 | if (!data->filter_str) { |
722 | free_event_filter(data->filter); | 724 | free_event_filter(rcu_access_pointer(data->filter)); |
723 | data->filter = NULL; | 725 | data->filter = NULL; |
724 | ret = -ENOMEM; | 726 | ret = -ENOMEM; |
725 | } | 727 | } |