diff mbox series

[v2,2/2] perf vendor events: Rename arm64 arch std event files

Message ID 20211210123706.7490-3-andrew.kilroy@arm.com
State New
Headers show
Series perf vendor events: Arm Neoverse N2 | expand

Commit Message

Andrew Kilroy Dec. 10, 2021, 12:37 p.m. UTC
A previous commit adds pmu events into the files

  armv8-common-and-microarch.json
  armv8-recommended.json

that are actually specified in an armv9 reference supplement, not armv8.
As such, naming the files with the armv8 prefix seems artificial.

This patch renames the files to reflect that these two files are for
arch std events regardless of whether they are defined in armv8 or
armv9.

Signed-off-by: Andrew Kilroy <andrew.kilroy@arm.com>
---
 ...{armv8-common-and-microarch.json => common-and-microarch.json} | 0
 .../arch/arm64/{armv8-recommended.json => recommended.json}       | 0
 2 files changed, 0 insertions(+), 0 deletions(-)
 rename tools/perf/pmu-events/arch/arm64/{armv8-common-and-microarch.json => common-and-microarch.json} (100%)
 rename tools/perf/pmu-events/arch/arm64/{armv8-recommended.json => recommended.json} (100%)

Comments

John Garry Dec. 10, 2021, 1:46 p.m. UTC | #1
On 10/12/2021 12:37, Andrew Kilroy wrote:
> A previous commit adds pmu events into the files
> 
>    armv8-common-and-microarch.json
>    armv8-recommended.json
> 
> that are actually specified in an armv9 reference supplement, not armv8.
> As such, naming the files with the armv8 prefix seems artificial.
> 
> This patch renames the files to reflect that these two files are for
> arch std events regardless of whether they are defined in armv8 or
> armv9.
> 
> Signed-off-by: Andrew Kilroy<andrew.kilroy@arm.com>

Reviewed-by: John Garry <john.garry@huawei.com>
Arnaldo Carvalho de Melo Dec. 10, 2021, 7:01 p.m. UTC | #2
Em Fri, Dec 10, 2021 at 01:46:48PM +0000, John Garry escreveu:
> On 10/12/2021 12:37, Andrew Kilroy wrote:
> > A previous commit adds pmu events into the files
> > 
> >    armv8-common-and-microarch.json
> >    armv8-recommended.json
> > 
> > that are actually specified in an armv9 reference supplement, not armv8.
> > As such, naming the files with the armv8 prefix seems artificial.
> > 
> > This patch renames the files to reflect that these two files are for
> > arch std events regardless of whether they are defined in armv8 or
> > armv9.
> > 
> > Signed-off-by: Andrew Kilroy<andrew.kilroy@arm.com>
> 
> Reviewed-by: John Garry <john.garry@huawei.com>


Thanks, applied both patches.

- Arnaldo
diff mbox series

Patch

diff --git a/tools/perf/pmu-events/arch/arm64/armv8-common-and-microarch.json b/tools/perf/pmu-events/arch/arm64/common-and-microarch.json
similarity index 100%
rename from tools/perf/pmu-events/arch/arm64/armv8-common-and-microarch.json
rename to tools/perf/pmu-events/arch/arm64/common-and-microarch.json
diff --git a/tools/perf/pmu-events/arch/arm64/armv8-recommended.json b/tools/perf/pmu-events/arch/arm64/recommended.json
similarity index 100%
rename from tools/perf/pmu-events/arch/arm64/armv8-recommended.json
rename to tools/perf/pmu-events/arch/arm64/recommended.json