diff options
| author |   Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-01-22 14:35:02 -0200 |
|---|---|---|
| committer |   Ingo Molnar <mingo@elte.hu> | 2010-01-27 12:46:49 +0100 |
| commit | 19fc2dedff448120a7aeaa3c136689c6b71777c6 (patch) | |
| tree | 814df9313b778bf8c73aabfe1f274127b8cdb9eb /tools/perf/util | |
| parent | perf symbols: Fix inverted logic for showing kallsyms as the source of symbols (diff) | |
| download | linux-dev-19fc2dedff448120a7aeaa3c136689c6b71777c6.tar.xz linux-dev-19fc2dedff448120a7aeaa3c136689c6b71777c6.zip | |
perf symbols: Use the right variable to check for kallsyms in the cache
Probably this wasn't noticed when testing this on my parisc
machine because I must have copied manually to its cache the
vmlinux file used in the x86_64 machine, now that I tried
looking on a x86-32 machine with a fresh cache, kernel symbols
weren't being resolved even with the right kallsyms copy on its
cache, duh.
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: <1264178102-4203-2-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf/util')
| -rw-r--r-- | tools/perf/util/symbol.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c index 1270cf867e61..f1f609dcf9a1 100644 --- a/tools/perf/util/symbol.c +++ b/tools/perf/util/symbol.c @@ -1650,12 +1650,12 @@ static int dso__load_kernel_sym(struct dso *self, struct map *map, getenv("HOME"), sbuild_id) == -1) return -1; + kallsyms_filename = kallsyms_allocated_filename; + if (access(kallsyms_filename, F_OK)) { free(kallsyms_allocated_filename); return -1; } - - kallsyms_filename = kallsyms_allocated_filename; } else { /* * Last resort, if we don't have a build-id and couldn't find |