summaryrefslogtreecommitdiffstats
path: root/tools/perf/builtin-kmem.c
diff options
context:
space:
mode:
authorArnaldo Carvalho de Melo <acme@redhat.com>2010-04-01 21:24:38 -0300
committerArnaldo Carvalho de Melo <acme@redhat.com>2010-04-02 16:28:25 -0300
commit71cf8b8ff7d6a79af086be9e4c72628da9d62d58 (patch)
tree8a8d6f97995133971d8f8bec3f4a885070ce103f /tools/perf/builtin-kmem.c
parente727ca73f85d4c5be3547eda674168219d1c22d8 (diff)
downloadop-kernel-dev-71cf8b8ff7d6a79af086be9e4c72628da9d62d58.zip
op-kernel-dev-71cf8b8ff7d6a79af086be9e4c72628da9d62d58.tar.gz
perf kmem: Fixup the symbol address before using it
We get absolute addresses in the events, but relative ones from the symbol subsystem, so calculate the absolute address by asking for the map where the symbol was found, that has the place where the DSO was actually loaded. For the core kernel this poses no problems if the kernel is not relocated by things like kexec, or if we use /proc/kallsyms, but for modules we were getting really large, negative offsets. LKML-Reference: <new-submission> Cc: Frédéric Weisbecker <fweisbec@gmail.com> Cc: Li Zefan <lizf@cn.fujitsu.com> Cc: Mike Galbraith <efault@gmx.de> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Paul Mackerras <paulus@samba.org> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-kmem.c')
-rw-r--r--tools/perf/builtin-kmem.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/perf/builtin-kmem.c b/tools/perf/builtin-kmem.c
index 7cbb5eb..513aa8a 100644
--- a/tools/perf/builtin-kmem.c
+++ b/tools/perf/builtin-kmem.c
@@ -363,6 +363,7 @@ static void __print_result(struct rb_root *root, struct perf_session *session,
struct alloc_stat *data = rb_entry(next, struct alloc_stat,
node);
struct symbol *sym = NULL;
+ struct map *map;
char buf[BUFSIZ];
u64 addr;
@@ -370,13 +371,13 @@ static void __print_result(struct rb_root *root, struct perf_session *session,
addr = data->call_site;
if (!raw_ip)
sym = map_groups__find_function(&session->kmaps,
- addr, NULL, NULL);
+ addr, &map, NULL);
} else
addr = data->ptr;
if (sym != NULL)
snprintf(buf, sizeof(buf), "%s+%Lx", sym->name,
- addr - sym->start);
+ addr - map->unmap_ip(map, sym->start));
else
snprintf(buf, sizeof(buf), "%#Lx", addr);
printf(" %-34s |", buf);
OpenPOWER on IntegriCloud