diff options
-rw-r--r-- | kernel/perf_event.c | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/kernel/perf_event.c b/kernel/perf_event.c index 3f2cc313ee25..7a932526946f 100644 --- a/kernel/perf_event.c +++ b/kernel/perf_event.c | |||
@@ -4920,6 +4920,13 @@ static int perf_event_set_output(struct perf_event *event, int output_fd) | |||
4920 | int fput_needed = 0; | 4920 | int fput_needed = 0; |
4921 | int ret = -EINVAL; | 4921 | int ret = -EINVAL; |
4922 | 4922 | ||
4923 | /* | ||
4924 | * Don't allow output of inherited per-task events. This would | ||
4925 | * create performance issues due to cross cpu access. | ||
4926 | */ | ||
4927 | if (event->cpu == -1 && event->attr.inherit) | ||
4928 | return -EINVAL; | ||
4929 | |||
4923 | if (!output_fd) | 4930 | if (!output_fd) |
4924 | goto set; | 4931 | goto set; |
4925 | 4932 | ||
@@ -4940,6 +4947,18 @@ static int perf_event_set_output(struct perf_event *event, int output_fd) | |||
4940 | if (event->data) | 4947 | if (event->data) |
4941 | goto out; | 4948 | goto out; |
4942 | 4949 | ||
4950 | /* | ||
4951 | * Don't allow cross-cpu buffers | ||
4952 | */ | ||
4953 | if (output_event->cpu != event->cpu) | ||
4954 | goto out; | ||
4955 | |||
4956 | /* | ||
4957 | * If its not a per-cpu buffer, it must be the same task. | ||
4958 | */ | ||
4959 | if (output_event->cpu == -1 && output_event->ctx != event->ctx) | ||
4960 | goto out; | ||
4961 | |||
4943 | atomic_long_inc(&output_file->f_count); | 4962 | atomic_long_inc(&output_file->f_count); |
4944 | 4963 | ||
4945 | set: | 4964 | set: |