arm: perf: cleanups and initial refactoring
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest2 of 8 patches 0 repliesNew
2 of 8 patches are in this archive. The cover letter announces 8.
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 7/8 | arm: perf: add more specific platform device IDs | +0 | 0 replies | |
| 8/8 | arm: perf: krait: stop using singleton PMU | +0 | 0 replies |
Reviews trailers per patch
All 2 patches have a Reviewed-by or Acked-by.
| Patch | Will Deacon |
|---|---|
| 7/8 | A-b |
| 8/8 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/8] arm: perf: cleanups and initial refactoring
Quoted 27 lines · Christopher Covington
Hi Mark,
Quoted 20 lines · Mark Rutland
While reorganising the ARM perf backend to provide support for heterogeneous PMUs in big.LITTLE systems, I encountered a couple of issues requiring some preparatory work in addition to some opportunities for general cleanup in the backend. This series consists of those initial cleanups.
Currently the event mapping code in the ARM perf backend requires multi-dimensional tables to map from common event encodings to raw hardware values, where empty slots must be initialised to non-zero values. The current way of organising this results in very large support files which we can simplify with some macro use to remove a few hundred lines and make things more legible.
The PMU naming is somewhat of a mess, with most names not being supported by the perf tool. Given that no-one has complained, it seems that the PMUs are not being accessed by name in a perf context, and we can fix these to be consistent and usable with current tools. We update the OProfile name mangling to ensure that OProfile (which accesses the PMUs in its own namespace) are not adversely affected.
Quoted 3 lines
To prepare for reorganisation of the PMU probing code, new
platform_device_idvalues are added to allow ARMv6, ARMv7, and XScale PMUplatform_devicesto be distinguished.
Quoted 5 lines
I ran Vince Weaver's perf events test suite [1] against a kernel with these patches applied on an 8074 Dragonboard [2] and didn't see any regressions. The patches also look good to me.
Thanks for testing, Christopher!
Will