Skip to content

Commit 2a21cf4

Browse files
committed
perf report: Do not process non-JIT BPF ksymbol events
JIRA: https://issues.redhat.com/browse/RHEL-78197 upstream ======== commit b0920ab Author: Namhyung Kim <namhyung@kernel.org> Date: Wed Mar 5 15:28:38 2025 -0800 description =========== The length of PERF_RECORD_KSYMBOL for BPF is a size of JITed code so it'd be 0 when it's not JITed. The ksymbol is needed to symbolize the code when it gets samples in the region but non-JITed code cannot get samples. Thus it'd be ok to ignore them. Actually it caused a performance issue in the perf tools on old ARM kernels where it can refuse to JIT some BPF codes. It ended up splitting the existing kernel map (kallsyms). And later lookup for a kernel symbol would create a new kernel map from kallsyms and then split it again and again. :( Probably there's a bug in the kernel map/symbol handling in perf tools. But I think we need to fix this anyway. Reported-by: Kevin Nomura <nomurak@google.com> Acked-by: Song Liu <song@kernel.org> Link: https://lore.kernel.org/r/20250305232838.128692-1-namhyung@kernel.org Signed-off-by: Namhyung Kim <namhyung@kernel.org> Signed-off-by: Michael Petlan <mpetlan@redhat.com>
1 parent 288f6b8 commit 2a21cf4

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

tools/perf/util/machine.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -779,6 +779,10 @@ int machine__process_ksymbol(struct machine *machine __maybe_unused,
779779
if (dump_trace)
780780
perf_event__fprintf_ksymbol(event, stdout);
781781

782+
/* no need to process non-JIT BPF as it cannot get samples */
783+
if (event->ksymbol.len == 0)
784+
return 0;
785+
782786
if (event->ksymbol.flags & PERF_RECORD_KSYMBOL_FLAGS_UNREGISTER)
783787
return machine__process_ksymbol_unregister(machine, event,
784788
sample);

0 commit comments

Comments
 (0)