From: Ian Rogers <irogers@google.com>
To: irogers@google.com, acme@kernel.org, namhyung@kernel.org
Cc: adrian.hunter@intel.com, bvanassche@acm.org, elver@google.com,
james.clark@linaro.org, jolsa@kernel.org,
justinstitt@google.com, linux-kernel@vger.kernel.org,
linux-perf-users@vger.kernel.org, llvm@lists.linux.dev,
mingo@redhat.com, morbo@google.com, nathan@kernel.org,
ndesaulniers@google.com, peterz@infradead.org
Subject: [PATCH v10 08/20] perf dsos: Skip BPF and OOL DSOs in dsos__find_kernel_dso()
Date: Thu, 5 Nov 2026 23:03:32 -0800 [thread overview]
Message-ID: <20261106070344.329897-9-irogers@google.com> (raw)
In-Reply-To: <20261106070344.329897-1-irogers@google.com>
In machine__process_ksymbol_register(), set the binary type of non-OOL
ksymbol DSOs to DSO_BINARY_TYPE__BPF_IMAGE (which
machine__process_bpf_event_load() upgrades to
DSO_BINARY_TYPE__BPF_PROG_INFO on PERF_BPF_EVENT_PROG_LOAD), and update
dsos__find_kernel_dso_cb() to skip DSO_BINARY_TYPE__BPF_PROG_INFO,
DSO_BINARY_TYPE__BPF_IMAGE, and DSO_BINARY_TYPE__OOL so that
dsos__find_kernel_dso() does not match ksymbol DSOs in machine->dsos.
Signed-off-by: Ian Rogers <irogers@google.com>
---
tools/perf/util/dsos.c | 3 +++
tools/perf/util/machine.c | 9 ++++-----
2 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/tools/perf/util/dsos.c b/tools/perf/util/dsos.c
index 1574c8885d78..fbbf98c4ccae 100644
--- a/tools/perf/util/dsos.c
+++ b/tools/perf/util/dsos.c
@@ -475,6 +475,9 @@ static int dsos__find_kernel_dso_cb(struct dso *dso, void *data)
* is_kernel_module() treats it as a kernel cpumode.
*/
if (!dso__kernel(dso) ||
+ dso__binary_type(dso) == DSO_BINARY_TYPE__BPF_PROG_INFO ||
+ dso__binary_type(dso) == DSO_BINARY_TYPE__BPF_IMAGE ||
+ dso__binary_type(dso) == DSO_BINARY_TYPE__OOL ||
is_kernel_module(dso__long_name(dso), PERF_RECORD_MISC_CPUMODE_UNKNOWN))
return 0;
diff --git a/tools/perf/util/machine.c b/tools/perf/util/machine.c
index 076d167380f2..b958c6935a56 100644
--- a/tools/perf/util/machine.c
+++ b/tools/perf/util/machine.c
@@ -769,6 +769,10 @@ static int machine__process_ksymbol_register(struct machine *machine,
dso__set_binary_type(dso, DSO_BINARY_TYPE__OOL);
dso__data(dso)->file_size = event->ksymbol.len;
dso__set_loaded(dso);
+ } else {
+ dso__set_binary_type(dso, DSO_BINARY_TYPE__BPF_IMAGE);
+ if (is_bpf_image(event->ksymbol.name))
+ dso__set_long_name(dso, "", false);
}
map__set_start(map, event->ksymbol.addr);
@@ -780,11 +784,6 @@ static int machine__process_ksymbol_register(struct machine *machine,
}
dso__set_loaded(dso);
-
- if (is_bpf_image(event->ksymbol.name)) {
- dso__set_binary_type(dso, DSO_BINARY_TYPE__BPF_IMAGE);
- dso__set_long_name(dso, "", false);
- }
} else {
dso = dso__get(map__dso(map));
}
--
2.56.0.455.g6c5a503b38-goog
next prev parent reply other threads:[~2026-11-06 7:04 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-06 7:03 [PATCH v10 00/20] tools/perf: Adopt kernel compiler-context-analysis.h annotations Ian Rogers
2026-11-06 7:03 ` [PATCH v10 01/20] compiler-context-analysis: Avoid unused parameter warning in _context_unsafe_alias() Ian Rogers
2026-11-06 7:09 ` sashiko-bot
2026-11-06 18:22 ` Bart Van Assche
2026-11-06 7:03 ` [PATCH v10 02/20] tools headers compiler: Synchronize compiler_types.h with the kernel sources Ian Rogers
2026-11-06 7:13 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 03/20] tools headers: Synchronize linux/compiler-context-analysis.h " Ian Rogers
2026-11-06 7:10 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 04/20] perf mutex: Use kernel style compiler-context-analysis.h annotations Ian Rogers
2026-11-06 7:14 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 05/20] perf build: Enable additional Clang thread-safety flags Ian Rogers
2026-11-06 7:13 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 06/20] perf comm: Add __guarded_by annotations to struct comm_strs Ian Rogers
2026-11-06 7:10 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 07/20] perf vdso: Fix unlocked __dsos__add() and race in __machine__addnew_vdso() Ian Rogers
2026-11-06 7:11 ` sashiko-bot
2026-11-06 7:03 ` Ian Rogers [this message]
2026-11-06 7:17 ` [PATCH v10 08/20] perf dsos: Skip BPF and OOL DSOs in dsos__find_kernel_dso() sashiko-bot
2026-11-06 7:03 ` [PATCH v10 09/20] perf dsos: Ensure DSOs in auxtrace, machine, and symbol are in dsos Ian Rogers
2026-11-06 7:19 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 10/20] perf dso: Pass dso_data to close_data_fd() and dso__list_del() Ian Rogers
2026-11-06 7:10 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 11/20] perf dso: Use DEFINE_MUTEX for dso__data_open_lock Ian Rogers
2026-11-06 7:07 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 12/20] perf dsos: Use read lock fast path in dsos__findnew_id() Ian Rogers
2026-11-06 7:09 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 13/20] perf dso: Expand context analysis annotations Ian Rogers
2026-11-06 7:11 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 14/20] perf thread: Expand context analysis annotations to threads and namespaces_lock Ian Rogers
2026-11-06 7:12 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 15/20] perf intel-tpebs: Fix race in tpebs_stop() and tpebs_cmd.out fd handling Ian Rogers
2026-11-06 7:27 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 16/20] perf intel-tpebs: Use DEFINE_MUTEX for tpebs_mtx Ian Rogers
2026-11-06 7:12 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 17/20] perf config, intel-tpebs, env: Add __guarded_by and __must_hold annotations Ian Rogers
2026-11-06 7:15 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 18/20] perf maps: Fix lock inversion in maps__load_first() Ian Rogers
2026-11-06 7:21 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 19/20] perf maps: Fix map index lookup fallback and removal Ian Rogers
2026-11-06 7:17 ` sashiko-bot
2026-11-06 7:03 ` [PATCH v10 20/20] perf maps: Add context analysis annotations Ian Rogers
2026-11-06 7:13 ` sashiko-bot
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=20261106070344.329897-9-irogers@google.com \
--to=irogers@google.com \
--cc=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=bvanassche@acm.org \
--cc=elver@google.com \
--cc=james.clark@linaro.org \
--cc=jolsa@kernel.org \
--cc=justinstitt@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=mingo@redhat.com \
--cc=morbo@google.com \
--cc=namhyung@kernel.org \
--cc=nathan@kernel.org \
--cc=ndesaulniers@google.com \
--cc=peterz@infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.