diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-01-22 11:35:02 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-01-27 06:46:49 -0500 |
commit | 19fc2dedff448120a7aeaa3c136689c6b71777c6 (patch) | |
tree | 814df9313b778bf8c73aabfe1f274127b8cdb9eb /tools | |
parent | e1c7c6a40c8037478742ce134190c1a955853bfb (diff) |
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')
-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, | |||
1650 | getenv("HOME"), sbuild_id) == -1) | 1650 | getenv("HOME"), sbuild_id) == -1) |
1651 | return -1; | 1651 | return -1; |
1652 | 1652 | ||
1653 | kallsyms_filename = kallsyms_allocated_filename; | ||
1654 | |||
1653 | if (access(kallsyms_filename, F_OK)) { | 1655 | if (access(kallsyms_filename, F_OK)) { |
1654 | free(kallsyms_allocated_filename); | 1656 | free(kallsyms_allocated_filename); |
1655 | return -1; | 1657 | return -1; |
1656 | } | 1658 | } |
1657 | |||
1658 | kallsyms_filename = kallsyms_allocated_filename; | ||
1659 | } else { | 1659 | } else { |
1660 | /* | 1660 | /* |
1661 | * Last resort, if we don't have a build-id and couldn't find | 1661 | * Last resort, if we don't have a build-id and couldn't find |