diff options
author | Will Deacon <will.deacon@arm.com> | 2013-08-20 06:47:41 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-09-26 20:18:03 -0400 |
commit | 1f96d83b38937294584ede9473c2b2961528f287 (patch) | |
tree | b5805eb5d4f38584d28bca1e37136bbb9a0a02c8 /arch/arm64 | |
parent | 008fe511c11fa20d1060c9e2ddb9c829ac44e0a7 (diff) |
arm64: perf: fix group validation when using enable_on_exec
commit 8455e6ec70f33b0e8c3ffd47067e00481f09f454 upstream.
This is a port of cb2d8b342aa0 ("ARM: 7698/1: perf: fix group validation
when using enable_on_exec") to arm64, which fixes the event validation
checking so that events in the OFF state are still considered when
enable_on_exec is true.
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'arch/arm64')
-rw-r--r-- | arch/arm64/kernel/perf_event.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/arch/arm64/kernel/perf_event.c b/arch/arm64/kernel/perf_event.c index 12e6ccb88691..2a1e9163d67a 100644 --- a/arch/arm64/kernel/perf_event.c +++ b/arch/arm64/kernel/perf_event.c | |||
@@ -325,7 +325,10 @@ validate_event(struct pmu_hw_events *hw_events, | |||
325 | if (is_software_event(event)) | 325 | if (is_software_event(event)) |
326 | return 1; | 326 | return 1; |
327 | 327 | ||
328 | if (event->pmu != leader_pmu || event->state <= PERF_EVENT_STATE_OFF) | 328 | if (event->pmu != leader_pmu || event->state < PERF_EVENT_STATE_OFF) |
329 | return 1; | ||
330 | |||
331 | if (event->state == PERF_EVENT_STATE_OFF && !event->attr.enable_on_exec) | ||
329 | return 1; | 332 | return 1; |
330 | 333 | ||
331 | return armpmu->get_event_idx(hw_events, &fake_event) >= 0; | 334 | return armpmu->get_event_idx(hw_events, &fake_event) >= 0; |