On Wed, Dec 06, 2017 at 02:45:15PM -0800, Song Liu wrote: > @@ -8537,7 +8620,7 @@ static int perf_event_set_filter(struct perf_event > *event, void __user *arg) > char *filter_str; > int ret = -EINVAL; > > - if ((event->attr.type != PERF_TYPE_TRACEPOINT || > + if ((!perf_event_is_tracing(event) || > !IS_ENABLED(CONFIG_EVENT_TRACING)) && > !has_addr_filter(event)) > return -EINVAL;
You actually missed an instance later in this same function... fixing that.