perf symbols: Remove unneeded call to dso__set_long_name()
authorCody P Schafer <cody@linux.vnet.ibm.com>
Fri, 10 Aug 2012 22:22:52 +0000 (15:22 -0700)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 13 Aug 2012 17:04:32 +0000 (14:04 -0300)
dso__set_long_name() is already called by dso__load_vmlinux(), avoid
calling it a second time unnecessarily.

Signed-off-by: Cody P Schafer <cody@linux.vnet.ibm.com>
Cc: David Hansen <dave@linux.vnet.ibm.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Matt Hellsley <matthltc@us.ibm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Link: http://lkml.kernel.org/r/1344637382-22789-7-git-send-email-cody@linux.vnet.ibm.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/symbol.c

index f02de8a..42c0d94 100644 (file)
@@ -1388,10 +1388,8 @@ int dso__load_vmlinux_path(struct dso *dso, struct map *map,
        filename = dso__build_id_filename(dso, NULL, 0);
        if (filename != NULL) {
                err = dso__load_vmlinux(dso, map, filename, filter);
-               if (err > 0) {
-                       dso__set_long_name(dso, filename);
+               if (err > 0)
                        goto out;
-               }
                free(filename);
        }