diff options
| author | Namhyung Kim <namhyung@kernel.org> | 2025-12-02 15:57:15 -0800 |
|---|---|---|
| committer | Namhyung Kim <namhyung@kernel.org> | 2025-12-02 21:59:14 -0800 |
| commit | ad0b9c4865b98dc37f4d606d26b1c19808796805 (patch) | |
| tree | 31b71179b232b262bd87f961c83769a19fa865e7 | |
| parent | 7da4d60db33cccd8f4c445ab20bba71531435ee5 (diff) | |
perf tools: Fix split kallsyms DSO counting
It's counted twice as it's increased after calling maps__insert(). I
guess we want to increase it only after it's added properly.
Reviewed-by: Ian Rogers <irogers@google.com>
Fixes: 2e538c4a1847291cf ("perf tools: Improve kernel/modules symbol lookup")
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
| -rw-r--r-- | tools/perf/util/symbol.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c index 5a38435d90c9..8eea49c50453 100644 --- a/tools/perf/util/symbol.c +++ b/tools/perf/util/symbol.c @@ -960,11 +960,11 @@ static int maps__split_kallsyms(struct maps *kmaps, struct dso *dso, u64 delta, if (dso__kernel(dso) == DSO_SPACE__KERNEL_GUEST) snprintf(dso_name, sizeof(dso_name), "[guest.kernel].%d", - kernel_range++); + kernel_range); else snprintf(dso_name, sizeof(dso_name), "[kernel].%d", - kernel_range++); + kernel_range); ndso = dso__new(dso_name); map__zput(curr_map); |
