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

Jiri Olsa
2020-07-20T08:19:56Z

On Mon, Jul 20, 2020 at 01:39:24PM +0530, kajoljain wrote:

SNIP

> This is with your perf/metric branch:
> command# ./perf stat -M PowerBUS_Frequency -C 0 -I 1000
> assertion failed at util/metricgroup.c:709
> # time counts unit events
> 1.000054545 7,807,505 hv_24x7/pm_pb_cyc,chip=0/ # 2.0 GHz PowerBUS_Frequency_0
> 1.000054545 7,807,485 hv_24x7/pm_pb_cyc,chip=1/
> 2.000232761 7,807,500 hv_24x7/pm_pb_cyc,chip=0/ # 2.0 GHz PowerBUS_Frequency_0
> 2.000232761 7,807,478 hv_24x7/pm_pb_cyc,chip=1/
> 3.000363762 7,799,665 hv_24x7/pm_pb_cyc,chip=0/ # 1.9 GHz PowerBUS_Frequency_0
> 3.000363762 7,807,502 hv_24x7/pm_pb_cyc,chip=1/
> ^C 3.259418599 2,022,150 hv_24x7/pm_pb_cyc,chip=0/ # 0.5 GHz PowerBUS_Frequency_0
> 3.259418599 2,022,164 hv_24x7/pm_pb_cyc,chip=1/
>
> Performance counter stats for 'CPU(s) 0':
>
> 25,436,820 hv_24x7/pm_pb_cyc,chip=0/ # 6.4 GHz PowerBUS_Frequency_0
> 25,444,629 hv_24x7/pm_pb_cyc,chip=1/
>
> 3.259505529 seconds time elapsed

I found the bug, we are not adding runtime metrics as standalone ones,
but as referenced metrics.. will fix and try to add test for that

as for testing.. do I need some special ppc server to have support for this?

thanks,
jirka