Re: [PATCH 00/20] perf vendors events arm64: Multiple Arm CPUs

[Nick Forrington]

On 18/05/2022 09:15, John Garry wrote:
> 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).

Not that I'm aware of, although I don't have these available to test.

I'm happy to re-submit without these CPUs if it simplifies things.

Thanks,
Nick