I had an agent review our linux-nvidia-bos kernel tree for patches that would result in different perf binary and it had 2 findings (the T410 one that I reported with this bug and another):
I found three flavor-only commits representing two actual perf-binary differences. 1. Tegra410/Olympus PMU event definitions The surviving patch is: feafe6f224d5 perf vendor events arm64: Add Tegra410 Olympus PMU events It adds 18 Tegra410 JSON files plus the map and common-event changes. The initial SAUCE version and its subsequent revert are only historical: 2a12c77919ba NVIDIA: VR: SAUCE: ... Add Tegra410 Olympus PMU events 97a981a22381 Revert "NVIDIA: VR: SAUCE: ... Add Tegra410 Olympus PMU events" feafe6f224d5 perf vendor events arm64: Add Tegra410 Olympus PMU events This patch definitely affects the binary. During an arm64 build: - JEVENTS_ARCH defaults to arm64 and JEVENTS_MODEL to all. - All arm64 JSON/CSV inputs are passed through jevents.py; see tools/perf/pmu-events/Build:24. - The resulting generated C is archived as libpmu-events.a and linked into perf; see tools/perf/Makefile.perf:457. The flavor source has this MIDR mapping: 0x000000004e0f0100,v1,nvidia/t410,core in tools/perf/pmu-events/arch/arm64/mapfile.csv:49. The generic branch has neither that entry nor the nvidia/t410 files. Consequently, generic perf will lack the Tegra410 CPU named events and metrics in perf list, even though raw event encodings can still be used. 2. OpenCSD-backed CoreSight decoding These two commits form one functional change: 8d0498847f49 UBUNTU: [Packaging] Enable coresight in Perf if arm64 18f162428e8f UBUNTU: [Packaging] Add libopencsd-dev as a build dependency The flavor packaging passes CORESIGHT=1 on arm64 in debian/rules.d/2-binary-arch.mk:626, and adds the dependency in debian.nvidia-bos/control.stub.in:61. That enables the OpenCSD check, defines HAVE_CSTRACE_SUPPORT, and links OpenCSD; see tools/perf/Makefile.config:517. It also includes cs-etm.o and the decoder sources conditionally in tools/ perf/util/Build:145. The generic packaging currently does not pass CORESIGHT=1 and does not depend on libopencsd-dev. Its perf binary will therefore lack OpenCSD- backed CoreSight trace decoding. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/2162778 Title: Perf binary for linux-nvidia kernel not including event aliases To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/linux-nvidia-bos/+bug/2162778/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
