Commit 6759788b authored by Will Deacon's avatar Will Deacon Committed by Russell King

ARM: 6865/1: perf: ensure pass through zero is counted on overflow

Commit a737823d ("ARM: perf: ensure overflows aren't missed due to IRQ
latency") changed the way that event deltas are calculated on overflow
so that we don't miss events when the new count value overtakes the
previous one.

Unfortunately, we forget to count the event that passes through zero so
we end up being off by 1. This patch adds on the correction.
Reported-by: default avatarChris Moore <moore@free.fr>
Signed-off-by: default avatarWill Deacon <will.deacon@arm.com>
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent e89c0d70
...@@ -221,7 +221,7 @@ armpmu_event_update(struct perf_event *event, ...@@ -221,7 +221,7 @@ armpmu_event_update(struct perf_event *event,
prev_raw_count &= armpmu->max_period; prev_raw_count &= armpmu->max_period;
if (overflow) if (overflow)
delta = armpmu->max_period - prev_raw_count + new_raw_count; delta = armpmu->max_period - prev_raw_count + new_raw_count + 1;
else else
delta = new_raw_count - prev_raw_count; delta = new_raw_count - prev_raw_count;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment