Re: [PATCH v3 7/7] perf pmus: Treat both numbers of NVIDIA PCIe PMU names as suffix

[sashiko-bot@kernel.org]

> The NVIDIA PCIe and PCIe target PMUs are named <name>_<socket>_rc_<rc>,
> like nvidia_pcie_pmu_1_rc_3. pmu_name_len_no_suffix() only strips the
> last number, so perf treats the PMUs of each socket as a different PMU.
> An event or metric on all PCIe PMUs then gives one result per socket,
> each labelled with the first RC of its socket.
> 
> Make pmu_name_len_no_suffix() strip both numbers of these names, so the
> instances on all sockets are merged as one PMU, nvidia_pcie_pmu or
> nvidia_pcie_tgt_pmu. Also accept that name on the command line, as perf
> list shows it, and sort these PMUs by socket and then by RC.
> 
> Suggested-by: Ian Rogers <irogers@google.com>
> Signed-off-by: Besar Wicaksono <bwicaksono@nvidia.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261104160716.2910873-1-bwicaksono@nvidia.com?part=7