perf symbols: Unexport kernel_map__functions
authorArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 27 Nov 2009 18:29:15 +0000 (16:29 -0200)
committerIngo Molnar <mingo@elte.hu>
Fri, 27 Nov 2009 19:21:58 +0000 (20:21 +0100)
perf annotate was the only user, and it doesn't really need it.

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1259346563-12568-4-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
tools/perf/builtin-annotate.c
tools/perf/util/symbol.c
tools/perf/util/symbol.h

index 377cb7c9bdda2c88faeba5c35ddf2b26c5b3ea05..0846c8a155ed797a8b5f16c2e4c5f37566b9e5e2 100644 (file)
@@ -169,7 +169,6 @@ process_sample_event(event_t *event, unsigned long offset, unsigned long head)
                level = '.';
                map = thread__find_map(thread, ip);
                if (map != NULL) {
-got_map:
                        ip = map->map_ip(map, ip);
                        sym = map__find_function(map, ip, symbol_filter);
                } else {
@@ -183,10 +182,9 @@ got_map:
                         * the "[vdso]" dso, but for now lets use the old
                         * trick of looking in the whole kernel symbol list.
                         */
-                       if ((long long)ip < 0) {
-                               map = kernel_map__functions;
-                               goto got_map;
-                       }
+                       if ((long long)ip < 0)
+                               sym = kernel_maps__find_function(ip, &map,
+                                                                symbol_filter);
                }
                dump_printf(" ...... dso: %s\n",
                            map ? map->dso->long_name : "<not found>");
index dc25231813eec953e05796b996b5e2708ac41ec4..0b8a298d41adf02fd1b74dbb785595abd8aab8fb 100644 (file)
@@ -36,6 +36,7 @@ static int dso__load_kernel_sym(struct dso *self, struct map *map,
 unsigned int symbol__priv_size;
 static int vmlinux_path__nr_entries;
 static char **vmlinux_path;
+static struct map *kernel_map__functions;
 
 static struct symbol_conf symbol_conf__defaults = {
        .use_modules      = true,
@@ -1164,8 +1165,6 @@ out:
        return ret;
 }
 
-struct map *kernel_map__functions;
-
 static void kernel_maps__insert(struct map *map)
 {
        maps__insert(&kernel_maps__functions, map);
index 5d0371fe8a06bcbe0c27071b95eb4fc8d62bc776..fb0be9e92bf379fa319f8f5ae282036e27d43272 100644 (file)
@@ -105,6 +105,5 @@ size_t kernel_maps__fprintf(FILE *fp);
 int symbol__init(struct symbol_conf *conf);
 
 extern struct list_head dsos__user, dsos__kernel;
-extern struct map *kernel_map__functions;
 extern struct dso *vdso;
 #endif /* __PERF_SYMBOL */