Re: [PATCH] perf metricgroup: Fix system PMU metrics

[John Garry]

On 19/01/2021 10:56, Joakim Zhang wrote:
root@imx8mmevk:~# ./perf stat -a -I 1000 -M 
imx8mm_ddr_read.all,imx8mm_ddr_write                                            
                                                        .all
#           time             counts unit events
      1.001446500              40832      imx8mm_ddr.read_cycles    #    638.0 
KB  imx8mm_ddr_read.all
      1.001446500              16973      imx8mm_ddr.write_cycles   #    265.2 
KB  imx8mm_ddr_write.all
      2.003150250              28836      imx8mm_ddr.read_cycles    #    450.6 
KB  imx8mm_ddr_read.all
      2.003150250               6705      imx8mm_ddr.write_cycles   #    104.8 
KB  imx8mm_ddr_write.all
For this issue, Tested-by: Joakim Zhang<qiangqing.zh...@nxp.com>

Hi John,

It seems have other issue compared to 5.10 kernel after switching to this 
framework, below metric can't work.
"MetricExpr": "(( imx8_ddr0@read\\-cycles@ + imx8_ddr0@write\\-cycles@ ) * 4 * 4 / 
duration_time) / (750 * 1000000 * 4 * 4)"
After change to:
"MetricExpr": "(( imx8mm_ddr.read_cycles + imx8mm_ddr.write_cycles ) * 4 * 4 / 
duration_time) / (750 * 1000000 * 4 * 4)",


OK, I can have a look. So is there anything else, apart from this, which 
doesn't work now?

Thanks,
john