diff options
author | Alexander Shishkin <alexander.shishkin@linux.intel.com> | 2016-04-27 11:44:42 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2016-05-05 04:13:55 -0400 |
commit | c796bbbe8dccd9c91ebbb99ffef33e0f73ced7bf (patch) | |
tree | 5f015cba5b91a034fe189a302d47aa4272d402af | |
parent | 1a618c2cfe4c02190d8d1f9c5052f0defc58de20 (diff) |
perf/core: Move set_filter() out of CONFIG_EVENT_TRACING
For instruction trace filtering, namely, for communicating filter
definitions from userspace, I'd like to re-use the SET_FILTER code
that the tracepoints are using currently.
To that end, move the relevant code out from behind the
CONFIG_EVENT_TRACING dependency.
Signed-off-by: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Vince Weaver <vincent.weaver@maine.edu>
Cc: vince@deater.net
Link: http://lkml.kernel.org/r/1461771888-10409-2-git-send-email-alexander.shishkin@linux.intel.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | kernel/events/core.c | 45 |
1 files changed, 22 insertions, 23 deletions
diff --git a/kernel/events/core.c b/kernel/events/core.c index 9de459a4dac7..13956ee883fd 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c | |||
@@ -7235,24 +7235,6 @@ static inline void perf_tp_register(void) | |||
7235 | perf_pmu_register(&perf_tracepoint, "tracepoint", PERF_TYPE_TRACEPOINT); | 7235 | perf_pmu_register(&perf_tracepoint, "tracepoint", PERF_TYPE_TRACEPOINT); |
7236 | } | 7236 | } |
7237 | 7237 | ||
7238 | static int perf_event_set_filter(struct perf_event *event, void __user *arg) | ||
7239 | { | ||
7240 | char *filter_str; | ||
7241 | int ret; | ||
7242 | |||
7243 | if (event->attr.type != PERF_TYPE_TRACEPOINT) | ||
7244 | return -EINVAL; | ||
7245 | |||
7246 | filter_str = strndup_user(arg, PAGE_SIZE); | ||
7247 | if (IS_ERR(filter_str)) | ||
7248 | return PTR_ERR(filter_str); | ||
7249 | |||
7250 | ret = ftrace_profile_set_filter(event, event->attr.config, filter_str); | ||
7251 | |||
7252 | kfree(filter_str); | ||
7253 | return ret; | ||
7254 | } | ||
7255 | |||
7256 | static void perf_event_free_filter(struct perf_event *event) | 7238 | static void perf_event_free_filter(struct perf_event *event) |
7257 | { | 7239 | { |
7258 | ftrace_profile_free_filter(event); | 7240 | ftrace_profile_free_filter(event); |
@@ -7307,11 +7289,6 @@ static inline void perf_tp_register(void) | |||
7307 | { | 7289 | { |
7308 | } | 7290 | } |
7309 | 7291 | ||
7310 | static int perf_event_set_filter(struct perf_event *event, void __user *arg) | ||
7311 | { | ||
7312 | return -ENOENT; | ||
7313 | } | ||
7314 | |||
7315 | static void perf_event_free_filter(struct perf_event *event) | 7292 | static void perf_event_free_filter(struct perf_event *event) |
7316 | { | 7293 | { |
7317 | } | 7294 | } |
@@ -7339,6 +7316,28 @@ void perf_bp_event(struct perf_event *bp, void *data) | |||
7339 | } | 7316 | } |
7340 | #endif | 7317 | #endif |
7341 | 7318 | ||
7319 | static int perf_event_set_filter(struct perf_event *event, void __user *arg) | ||
7320 | { | ||
7321 | char *filter_str; | ||
7322 | int ret = -EINVAL; | ||
7323 | |||
7324 | if (event->attr.type != PERF_TYPE_TRACEPOINT || | ||
7325 | !IS_ENABLED(CONFIG_EVENT_TRACING)) | ||
7326 | return -EINVAL; | ||
7327 | |||
7328 | filter_str = strndup_user(arg, PAGE_SIZE); | ||
7329 | if (IS_ERR(filter_str)) | ||
7330 | return PTR_ERR(filter_str); | ||
7331 | |||
7332 | if (IS_ENABLED(CONFIG_EVENT_TRACING) && | ||
7333 | event->attr.type == PERF_TYPE_TRACEPOINT) | ||
7334 | ret = ftrace_profile_set_filter(event, event->attr.config, | ||
7335 | filter_str); | ||
7336 | |||
7337 | kfree(filter_str); | ||
7338 | return ret; | ||
7339 | } | ||
7340 | |||
7342 | /* | 7341 | /* |
7343 | * hrtimer based swevent callback | 7342 | * hrtimer based swevent callback |
7344 | */ | 7343 | */ |