]> git.karo-electronics.de Git - linux-beck.git/commitdiff
perf kmem: Fixup the symbol address before using it
authorArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 2 Apr 2010 00:24:38 +0000 (21:24 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 2 Apr 2010 19:28:25 +0000 (16:28 -0300)
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>
tools/perf/builtin-kmem.c

index 7cbb5eb15101b4ea27a27ee0fe854227f59c6502..513aa8a55db66be69e89c4f9063b1dc30211d783 100644 (file)
@@ -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);