From: Namhyung Kim <[email protected]>

If an user gives both of --symfs and --vmlinux option, the vmlinux
will be searched under the symfs directory.  This is somewhat
confusing since vmlinux often lives in kernel build directory or
somewhere other than user space binaries.

So it'd be better not adding symfs prefix for a vmlinux if it has an
absolute pathname.

Reported-by: Kwanghyun Yoo <[email protected]>
Signed-off-by: Namhyung Kim <[email protected]>
---
 tools/perf/util/symbol.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
index d5528e1cc03a..02718e728d59 100644
--- a/tools/perf/util/symbol.c
+++ b/tools/perf/util/symbol.c
@@ -888,8 +888,11 @@ int dso__load_vmlinux(struct dso *dso, struct map *map,
        char symfs_vmlinux[PATH_MAX];
        enum dso_binary_type symtab_type;
 
-       snprintf(symfs_vmlinux, sizeof(symfs_vmlinux), "%s%s",
-                symbol_conf.symfs, vmlinux);
+       if (vmlinux[0] == '/')
+               snprintf(symfs_vmlinux, sizeof(symfs_vmlinux), "%s", vmlinux);
+       else
+               snprintf(symfs_vmlinux, sizeof(symfs_vmlinux), "%s%s",
+                        symbol_conf.symfs, vmlinux);
 
        if (dso->kernel == DSO_TYPE_GUEST_KERNEL)
                symtab_type = DSO_BINARY_TYPE__GUEST_VMLINUX;
-- 
1.7.11.7

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to