diff options
| author | Mark Rutland <mark.rutland@arm.com> | 2011-08-08 14:56:02 +0100 |
|---|---|---|
| committer | Will Deacon <will.deacon@arm.com> | 2011-08-31 10:50:06 +0100 |
| commit | 1b69beb7684c79673995607939d8acab51056b63 (patch) | |
| tree | 20e15b0d5d54866190e8fb2599ff5ebfb73317ba /arch/arm/kernel | |
| parent | 92f701e1f429e007f9619469d548022061c41ecc (diff) | |
| download | talos-op-linux-1b69beb7684c79673995607939d8acab51056b63.tar.gz talos-op-linux-1b69beb7684c79673995607939d8acab51056b63.zip | |
ARM: perf: remove unnecessary armpmu->stop
As armpmu_disable will call armpmu->stop when the last event has been
removed, this is pointless and simply adds to the noise when debugging.
Additionally, due to this call occurring in a preemptible context, this
is problematic for per-cpu locking of PMU registers (where we will
attempt to access per-cpu spinlock for use with raw_spin_lock_irqsave).
This patch removes the call to armpmu->stop.
Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Reviewed-by: Will Deacon <will.deacon@arm.com>
Reviewed-by: Jamie Iles <jamie@jamieiles.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Diffstat (limited to 'arch/arm/kernel')
| -rw-r--r-- | arch/arm/kernel/perf_event.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/arm/kernel/perf_event.c b/arch/arm/kernel/perf_event.c index 9d6ac99de30b..5ce6c3332915 100644 --- a/arch/arm/kernel/perf_event.c +++ b/arch/arm/kernel/perf_event.c @@ -396,7 +396,6 @@ armpmu_release_hardware(void) free_irq(irq, NULL); } - armpmu->stop(); release_pmu(ARM_PMU_DEVICE_CPU); } |
