Messages in this thread | | | Date | Wed, 18 May 2022 09:15:21 +0100 | Subject | Re: [PATCH 00/20] perf vendors events arm64: Multiple Arm CPUs | From | John Garry <> |
| |
On 17/05/2022 15:32, Robin Murphy wrote: > > On 2022-05-10 11:47, Nick Forrington wrote: >> Add Performance Monitoring Unit event data for the Arm CPUs listed >> below. >> >> Changesets are dependent due to incremental updates to the common events >> file and mapfile.csv. >> >> Data is sourced from https://github.com/ARM-software/data >> >> Nick Forrington (20): >> perf vendors events arm64: Arm Cortex-A5 >> perf vendors events arm64: Arm Cortex-A7 >> perf vendors events arm64: Arm Cortex-A8 >> perf vendors events arm64: Arm Cortex-A9 >> perf vendors events arm64: Arm Cortex-A15 >> perf vendors events arm64: Arm Cortex-A17 >> perf vendors events arm64: Arm Cortex-A32 > > Obligatory question over anything relating to the above CPUs being in an > "arch/arm64" directory... ;)
If we were to add to arm32/arm then the common event numbers and maybe other JSONs in future would need to be duplicated.
Would there be any reason to add to arm32/arm apart to from being strictly proper? Maybe if lots of other 32b support for other vendors came along then it could make sense (to separate them out).
thanks, John
| |