diff options
| author | Robin Murphy <robin.murphy@arm.com> | 2026-09-15 13:24:22 +0100 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2026-09-18 11:11:34 +0000 |
| commit | 406aa2b186d3f13a35bc1ad6aff4274917851bc4 (patch) | |
| tree | e311663c647604f010bc775065d9e00e3fd1ad4e /tools/perf/tests/shell/test_stackcollapse_python.sh | |
| parent | fdb9ebc7fb7788b8371fbef6c17dc5c8291e1429 (diff) | |
| download | linux-next-406aa2b186d3f13a35bc1ad6aff4274917851bc4.tar.gz linux-next-406aa2b186d3f13a35bc1ad6aff4274917851bc4.zip | |
perf/arm-cmn: Fix multi-filter encoding
The current special-case for EVICT_STATE_SEL filtering effectively
assigns the "filter" and "filter2" controls in the opposite order from
how the CMN S3 r2 TRM states "Filtering is programmed in pmu_hbt_lbt_sel
and pmu_evict_state_sel". On reflection, not only does this seem
unnecessarily non-obvious to users, but it's also likely to be a problem
for scaling to a full multi-filter abstraction in future. There is a
logical order to filters based on their bitfield positions in the
pmu_event_sel register, which the TRM descriptions allude to, and the
cmn_filter_select enum already (almost) follows, so let's fix the UABI
to follow suit while it's still unreleased.
Fixes: 09178f536bb9 ("perf/arm-cmn: Plumb in new filter types")
Signed-off-by: Robin Murphy <robin.murphy@arm.com>
Reviewed-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
Reviewed-by: Leo Yan <leo.yan@arm.com>
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'tools/perf/tests/shell/test_stackcollapse_python.sh')
0 files changed, 0 insertions, 0 deletions
