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 | |
| 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>
| -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 |
