perf_counter tools: Consolidate dso methods to load kernel symbols
authorArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 28 May 2009 17:55:19 +0000 (14:55 -0300)
committerIngo Molnar <mingo@elte.hu>
Thu, 28 May 2009 21:25:44 +0000 (23:25 +0200)
Now one has just to use dso__load_kernel() optionally passing a vmlinux
filename.

Will make things easier for perf top that will want to pass a callback
to filter some symbols.

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Documentation/perf_counter/builtin-report.c
Documentation/perf_counter/util/symbol.c
Documentation/perf_counter/util/symbol.h

index 889067eb2b690daee1904ea209374d9a2b98f72b..4e9f2bc104527d9860cd9adf52bf75251860d70f 100644 (file)
@@ -118,18 +118,13 @@ static void dsos__fprintf(FILE *fp)
 
 static int load_kernel(void)
 {
-       int err = -1;
+       int err;
 
        kernel_dso = dso__new("[kernel]", 0);
        if (!kernel_dso)
                return -1;
 
-       if (vmlinux)
-               err = dso__load_vmlinux(kernel_dso, vmlinux);
-
-       if (err)
-               err = dso__load_kallsyms(kernel_dso);
-
+       err = dso__load_kernel(kernel_dso, vmlinux);
        if (err) {
                dso__delete(kernel_dso);
                kernel_dso = NULL;
index 7088206244ac7b294262d8ed0372291633e83b56..504ac3132019ea866ac012333acb31a3c8087e0b 100644 (file)
@@ -155,7 +155,7 @@ static int hex2long(char *ptr, unsigned long *long_val)
        return p - ptr;
 }
 
-int dso__load_kallsyms(struct dso *self)
+static int dso__load_kallsyms(struct dso *self)
 {
        struct rb_node *nd, *prevnd;
        char *line = NULL;
@@ -410,7 +410,7 @@ out:
        return ret;
 }
 
-int dso__load_vmlinux(struct dso *self, const char *vmlinux)
+static int dso__load_vmlinux(struct dso *self, const char *vmlinux)
 {
        int err, fd = open(vmlinux, O_RDONLY);
 
@@ -423,6 +423,19 @@ int dso__load_vmlinux(struct dso *self, const char *vmlinux)
        return err;
 }
 
+int dso__load_kernel(struct dso *self, const char *vmlinux)
+{
+       int err = -1;
+
+       if (vmlinux)
+               err = dso__load_vmlinux(self, vmlinux);
+
+       if (err)
+               err = dso__load_kallsyms(self);
+
+       return err;
+}
+
 void symbol__init(void)
 {
        elf_version(EV_CURRENT);
index 9e120af9c71f93f026152ed5230ef1bc15329dbf..db2fdf9f70a20eff869ad9126328e82133f099a3 100644 (file)
@@ -29,8 +29,7 @@ static inline void *dso__sym_priv(struct dso *self, struct symbol *sym)
 
 struct symbol *dso__find_symbol(struct dso *self, uint64_t ip);
 
-int dso__load_kallsyms(struct dso *self);
-int dso__load_vmlinux(struct dso *self, const char *vmlinux);
+int dso__load_kernel(struct dso *self, const char *vmlinux);
 int dso__load(struct dso *self);
 
 size_t dso__fprintf(struct dso *self, FILE *fp);