RE: [PATCH RFC 0/7] perf pmu-events: Support event aliasing for system PMUs

[Joakim Zhang]

> -----Original Message-----
> From: Mark Rutland <mark.rutland@xxxxxxx>
> Sent: 2020年2月19日 2:14
> To: John Garry <john.garry@xxxxxxxxxx>
> Cc: ak@xxxxxxxxxxxxxxx; Joakim Zhang <qiangqing.zhang@xxxxxxx>;
> suzuki.poulose@xxxxxxx; peterz@xxxxxxxxxxxxx; Will Deacon
> <will@xxxxxxxxxx>; Linuxarm <linuxarm@xxxxxxxxxx>; acme@xxxxxxxxxx;
> linux-kernel@xxxxxxxxxxxxxxx; Zhangshaokun <zhangshaokun@xxxxxxxxxxxxx>;
> alexander.shishkin@xxxxxxxxxxxxxxx; mingo@xxxxxxxxxx;
> james.clark@xxxxxxx; namhyung@xxxxxxxxxx; jolsa@xxxxxxxxxx;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; robin.murphy@xxxxxxx; Sudeep Holla
> <sudeep.holla@xxxxxxx>
> Subject: Re: [PATCH RFC 0/7] perf pmu-events: Support event aliasing for
> system PMUs

[...]
> > And typically most PMU HW would have no ID reg, so where to even get
> > this identification info? Joakim Zhang seems to have this problem for
> > the imx8 DDRC PMU driver.
> 
> For imx8, the DT compat string or additional properties on the DDRC node
> could be used to imply the id.

Hi Mark,

Yes, actually we can expose something like DDRC_ID to indicate each specific DDR controller, to point out the filter feature.
But, even the SoCs integrated the same DDRC_ID, just say that they have the same DDRC controller.

From user space, the usage is different, for example:

i.MX8MM and i.MX8MN, they use the same driver(DDRC_ID) and cortex-a53 integrated.

If we want to monitor VPU, their *master id* is different from SoCs.
On i.MX8MM, event is imx8_ddr0/axid-read,axi_id=0x08/
On i.MX8MN, event is imx8_ddr0/axid-read,axi_id=0x12/

I try to write a JSON file to use these events, for now, I only can locate the file at the directory: tools/perf/pmu-events/arch/arm64/arm/cortex-a53/

Perf tool loads all events when CPUID matched, which is now unreasonable, we want related events are loaded for specific SoC.

All events will also be loaded if we use DDRC_ID to match in the future, this seems to not be a good ideal.

Best Regards,
Joakim Zhang

[....]