diff options
| author | Dapeng Mi <[email protected]> | 2025-04-15 10:41:34 +0000 |
|---|---|---|
| committer | Ingo Molnar <[email protected]> | 2025-04-17 12:19:07 +0000 |
| commit | a5f5e1238f4ff919816f69e77d2537a48911767b (patch) | |
| tree | 7d2b3929f9bda5f2c30ecbc9b454a8134418d2cf /rust/helpers/platform.c | |
| parent | perf/x86/intel/uncore: Fix the scale of IIO free running counters on SPR (diff) | |
| download | kernel-a5f5e1238f4ff919816f69e77d2537a48911767b.tar.gz kernel-a5f5e1238f4ff919816f69e77d2537a48911767b.zip | |
perf/x86/intel: Don't clear perf metrics overflow bit unconditionally
The below code would always unconditionally clear other status bits like
perf metrics overflow bit once PEBS buffer overflows:
status &= intel_ctrl | GLOBAL_STATUS_TRACE_TOPAPMI;
This is incorrect. Perf metrics overflow bit should be cleared only when
fixed counter 3 in PEBS counter group. Otherwise perf metrics overflow
could be missed to handle.
Closes: https://lore.kernel.org/all/[email protected]/
Fixes: 7b2c05a15d29 ("perf/x86/intel: Generic support for hardware TopDown metrics")
Signed-off-by: Dapeng Mi <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>
Reviewed-by: Kan Liang <[email protected]>
Cc: [email protected]
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'rust/helpers/platform.c')
0 files changed, 0 insertions, 0 deletions
