From: Besar Wicaksono <bwicaksono@nvidia.com>
To: <acme@kernel.org>, <namhyung@kernel.org>, <irogers@google.com>,
<ctshao@google.com>
Cc: <linux-tegra@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-perf-users@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <vsethi@nvidia.com>,
<rwiley@nvidia.com>, <skelley@nvidia.com>, <mochs@nvidia.com>,
<rajjoshi@nvidia.com>, <treding@nvidia.com>,
<jonathanh@nvidia.com>, <john.garry@linux.dev>, <will@kernel.org>,
<james.clark@linaro.org>, <mike.leach@arm.com>,
<leo.yan@linux.dev>, <mark.rutland@arm.com>,
<alexander.shishkin@linux.intel.com>, <jolsa@kernel.org>,
<adrian.hunter@intel.com>,
Besar Wicaksono <bwicaksono@nvidia.com>
Subject: [PATCH v2 0/9] perf: Add NVIDIA Tegra410 uncore PMU events and metrics
Date: Fri, 30 Oct 2026 17:48:07 +0000 [thread overview]
Message-ID: <20261030174816.1544471-1-bwicaksono@nvidia.com> (raw)
Add event aliases and derived metrics for the NVIDIA Tegra410 uncore
PMUs, and fix perf's aggregation of PMU instances that are numbered in
more than one parts in their name.
The series covers the UCF, PCIe, PCIe target, CPU memory latency, and
chip-to-chip interconnect PMUs. It provides symbolic names for the raw
event encodings and metrics for bandwidth, frequency, and average read
latency.
The PCIe and PCIe target PMUs are named nvidia_pcie_pmu_<socket>_rc_<rc>
and nvidia_pcie_tgt_pmu_<socket>_rc_<rc>. Their json Unit uses the
existing comma-separated form ("nvidia_pcie_pmu_,rc_"), but perf only
merged instances that differ in the trailing number, so these events and
metrics gave one result per socket and were reported incorrectly (e.g.
with --metric-only), as discussed in v1. Patches 6-9 fix this:
- Patch 6 adds a check of the cpumasks of all merged PMU instances to the
zero-count skip in perf stat, so a socket that counts nothing shows 0
instead of having its row hidden.
- Patch 7 merges PMU instances that match the same comma-separated json
Unit. This also applies to the HiSilicon hisi_sccl PMUs, which use the
same json form; not tested on HiSilicon hardware.
- Patch 8 labels a merged uncore counter after all of its PMUs, e.g.
nvidia_pcie_pmu_*_rc_*/cycles/ or nvidia_ucf_pmu/cycles/, instead of
after the first instance, such as nvidia_pcie_pmu_0_rc_0/cycles/.
Output with --no-aggr is unchanged.
- Patch 9 describes these PMU names in the ABI document.
Changes in v2:
- Move the uncore JSON out of the sys directory and remove the
"Compat" fields. Thanks to Ian Rogers for the feedback.
- Drop "perf jevents: Process model files before sys subdirectories",
which this series no longer needs.
- Add UCF SLC and memory bandwidth metrics from all sources to local
CMEM (read, write and read+write) and from local sources to local
CMEM and to remote memory, PCIe bandwidth metrics to local
destinations, and read+write bandwidth metrics for UCF SLC, UCF
memory and PCIe. Thanks to Chun-Tse Shao for the suggestion.
- Add patches 6-9 to fix the PCIe instance aggregation. Thanks to Ian
Rogers and Chun-Tse Shao for the pointers.
v1: https://lore.kernel.org/all/20261019164241.403644-1-bwicaksono@nvidia.com/
Besar Wicaksono (9):
perf jevents: Add support for NVIDIA Tegra410 UCF PMU aliasing
perf jevents: Add support for NVIDIA Tegra410 PCIE PMU aliasing
perf jevents: Add support for NVIDIA Tegra410 PCIE-TGT PMU aliasing
perf jevents: Add support for NVIDIA Tegra410 CPU Memory Latency PMU
aliasing
perf jevents: Add support for NVIDIA Tegra410 C2C PMU aliasing
perf stat: Check merged PMU instance's cpumasks when skipping zero
counts
perf pmu: Merge PMU instances that match the same comma-separated json
Unit
perf evsel: Name a merged uncore counter after all of its PMUs
perf Documentation: Describe PMU names numbered in more than one place
.../testing/sysfs-bus-event_source-devices | 14 +
.../arm64/nvidia/t410/uncore-cpu_mem.json | 43 +
.../arm64/nvidia/t410/uncore-nvclink.json | 67 ++
.../arm64/nvidia/t410/uncore-nvdlink.json | 43 +
.../arm64/nvidia/t410/uncore-nvlink_c2c.json | 115 +++
.../arch/arm64/nvidia/t410/uncore-pcie.json | 249 ++++++
.../arm64/nvidia/t410/uncore-pcie_tgt.json | 83 ++
.../arch/arm64/nvidia/t410/uncore-ucf.json | 764 ++++++++++++++++++
tools/perf/tests/pmu.c | 134 +++
tools/perf/util/evlist.c | 2 +-
tools/perf/util/evsel.c | 83 +-
tools/perf/util/evsel.h | 2 +-
tools/perf/util/parse-events.c | 2 +-
tools/perf/util/pmu.c | 65 ++
tools/perf/util/pmu.h | 9 +
tools/perf/util/pmus.c | 47 ++
tools/perf/util/pmus.h | 1 +
tools/perf/util/stat-display.c | 20 +
18 files changed, 1737 insertions(+), 6 deletions(-)
create mode 100644 tools/perf/pmu-events/arch/arm64/nvidia/t410/uncore-cpu_mem.json
create mode 100644 tools/perf/pmu-events/arch/arm64/nvidia/t410/uncore-nvclink.json
create mode 100644 tools/perf/pmu-events/arch/arm64/nvidia/t410/uncore-nvdlink.json
create mode 100644 tools/perf/pmu-events/arch/arm64/nvidia/t410/uncore-nvlink_c2c.json
create mode 100644 tools/perf/pmu-events/arch/arm64/nvidia/t410/uncore-pcie.json
create mode 100644 tools/perf/pmu-events/arch/arm64/nvidia/t410/uncore-pcie_tgt.json
create mode 100644 tools/perf/pmu-events/arch/arm64/nvidia/t410/uncore-ucf.json
--
2.43.0
next reply other threads:[~2026-10-30 17:49 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-30 17:48 Besar Wicaksono [this message]
2026-10-30 17:48 ` [PATCH v2 1/9] perf jevents: Add support for NVIDIA Tegra410 UCF PMU aliasing Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 2/9] perf jevents: Add support for NVIDIA Tegra410 PCIE " Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 3/9] perf jevents: Add support for NVIDIA Tegra410 PCIE-TGT " Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 4/9] perf jevents: Add support for NVIDIA Tegra410 CPU Memory Latency " Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 5/9] perf jevents: Add support for NVIDIA Tegra410 C2C " Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 6/9] perf stat: Check merged PMU instance's cpumasks when skipping zero counts Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 7/9] perf pmu: Merge PMU instances that match the same comma-separated json Unit Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 8/9] perf evsel: Name a merged uncore counter after all of its PMUs Besar Wicaksono
2026-10-30 17:48 ` [PATCH v2 9/9] perf Documentation: Describe PMU names numbered in more than one place Besar Wicaksono
2026-10-30 18:07 ` [PATCH v2 0/9] perf: Add NVIDIA Tegra410 uncore PMU events and metrics Ian Rogers
2026-10-30 22:18 ` Besar Wicaksono
2026-10-31 4:17 ` Ian Rogers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261030174816.1544471-1-bwicaksono@nvidia.com \
--to=bwicaksono@nvidia.com \
--cc=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=ctshao@google.com \
--cc=irogers@google.com \
--cc=james.clark@linaro.org \
--cc=john.garry@linux.dev \
--cc=jolsa@kernel.org \
--cc=jonathanh@nvidia.com \
--cc=leo.yan@linux.dev \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mike.leach@arm.com \
--cc=mochs@nvidia.com \
--cc=namhyung@kernel.org \
--cc=rajjoshi@nvidia.com \
--cc=rwiley@nvidia.com \
--cc=skelley@nvidia.com \
--cc=treding@nvidia.com \
--cc=vsethi@nvidia.com \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®