Messages in this thread Patch in this message | | | Date | Wed, 22 Apr 2020 12:17:38 -0000 | From | "tip-bot2 for Jiri Olsa" <> | Subject | [tip: perf/core] perf machine: Set ksymbol dso as loaded on arrival |
| |
The following commit has been merged into the perf/core branch of tip:
Commit-ID: 7eddf7e74e54aea3b24410b3fb8911927836632f Gitweb: https://git.kernel.org/tip/7eddf7e74e54aea3b24410b3fb8911927836632f Author: Jiri Olsa <jolsa@kernel.org> AuthorDate: Thu, 12 Mar 2020 20:56:09 +01:00 Committer: Arnaldo Carvalho de Melo <acme@redhat.com> CommitterDate: Thu, 16 Apr 2020 12:19:06 -03:00
perf machine: Set ksymbol dso as loaded on arrival
There's no special load action for ksymbol data on map__load/dso__load action, where the kernel is getting loaded. It only gets confused with kernel kallsyms/vmlinux load for bpf object, which fails and could mess up with the map.
Disabling any further load of the map for ksymbol related dso/map.
Signed-off-by: Jiri Olsa <jolsa@kernel.org> Acked-by: Song Liu <songliubraving@fb.com> Cc: Alexei Starovoitov <ast@kernel.org> Cc: Andrii Nakryiko <andriin@fb.com> Cc: Björn Töpel <bjorn.topel@intel.com> Cc: Daniel Borkmann <daniel@iogearbox.net> Cc: David S. Miller <davem@redhat.com> Cc: Jakub Kicinski <kuba@kernel.org> Cc: Jesper Dangaard Brouer <hawk@kernel.org> Cc: John Fastabend <john.fastabend@gmail.com> Cc: Martin KaFai Lau <kafai@fb.com> Cc: Yonghong Song <yhs@fb.com> Link: https://lore.kernel.org/bpf/20200312195610.346362-15-jolsa@kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> --- tools/perf/util/machine.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/tools/perf/util/machine.c b/tools/perf/util/machine.c index 97142e9..06aa4e4 100644 --- a/tools/perf/util/machine.c +++ b/tools/perf/util/machine.c @@ -759,6 +759,7 @@ static int machine__process_ksymbol_register(struct machine *machine, map->start = event->ksymbol.addr; map->end = map->start + event->ksymbol.len; maps__insert(&machine->kmaps, map); + dso__set_loaded(dso); } sym = symbol__new(map->map_ip(map, map->start),
| |