Re: [PATCHv4 00/19] perf metric: Add support to reuse metric

[John Garry]

On 24/03/2021 01:54, Paul A. Clarke wrote:
> > > --
> > Since commit 8989f5f07605 ("perf stat: Update POWER9 metrics to utilize
> > other metrics"), power9 has reused metrics.
> >
> > And I am finding that subtest 10.3 caused problems when I tried to introduce
> > metric reuse on arm64, so I was just asking you to check.
> >
> > Now I am a bit confused...
> I now understand your original request!:-)
>
> The above test was run on a POWER8 system.
>
> I do see failures on a POWER9 system:
> --
> $ ./perf test 10
> 10: PMU events                                                      :
> 10.1: PMU event table sanity                                        : Ok
> 10.2: PMU event map aliases                                         : Ok
> 10.3: Parsing of PMU event table metrics                            : Skip (some metrics failed)
> 10.4: Parsing of PMU event table metrics with fake PMUs             : Ok
> $ ./perf test --verbose 10 2>&1 | grep -i '^Parse event failed' | wc -l
> 112

ok, thanks for confirming.

I'm looking at fixing it, and the solution doesn't look simple :(

John