path: root/tools/perf/builtin-script.c
diff options
authorArnaldo Carvalho de Melo <>2015-07-22 16:43:37 -0300
committerArnaldo Carvalho de Melo <>2015-07-23 22:51:07 -0300
commitccb3a8294a7d346257b526702eabcb2faeb721ac (patch)
treed0121abbb14921ffd4c5ccd4bbc661806e6378c7 /tools/perf/builtin-script.c
parent706c3da409dfc6964740ac503c040c4c621954fd (diff)
perf script: Switch from's kallsyms to perf's symbol resolver
We were storing a copy of kallsyms inside file so that we could resolve kernel addresses to function (start, name, mod) tuples, but that can be achieved using the symbol resolving routines we have in symbols.c, and that are used elsewhere in tools/perf. So, do just like 'perf trace' did and ask libtraceevent to use perf's symbol resolution routines. The next step is to just skip whatever kallsyms data is embedded in older files and finally to stop storing kallsyms in the perf data file, as the 20-bytes build-id stored in's header is enough to find out the right symtab (be it ELF, kcore, kallsyms, etc) to use. Acked-by: David Ahern <> Cc: Adrian Hunter <> Cc: Borislav Petkov <> Cc: Frederic Weisbecker <> Cc: Jiri Olsa <> Cc: Namhyung Kim <> Cc: Stephane Eranian <> Cc: Steven Rostedt <> Link: Signed-off-by: Arnaldo Carvalho de Melo <>
Diffstat (limited to 'tools/perf/builtin-script.c')
1 files changed, 7 insertions, 0 deletions
diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
index 24809787369f..2777c6444bc3 100644
--- a/tools/perf/builtin-script.c
+++ b/tools/perf/builtin-script.c
@@ -1830,6 +1830,13 @@ int cmd_script(int argc, const char **argv, const char *prefix __maybe_unused)
symbol_conf.use_callchain = false;
+ if (pevent_set_function_resolver(session->tevent.pevent,
+ machine__resolve_kernel_addr,
+ &session-> < 0) {
+ pr_err("%s: failed to set libtraceevent function resolver\n", __func__);
+ return -1;
+ }
if (generate_script_lang) {
struct stat perf_stat;
int input;