perf: fix handling of .debug cache for kernel symbols

PATCH In review Stephane Eranian 2 participants 2 messages Dec 2, 2011 perf events

lore

Share & tools

lore

Stephane Eranian

author

2011-12-02 04:48 UTC #0 lore raw

PATCH 1/1 perf: fix handling of .debug cache for kernel symbols

There was a bug in the way perf was looking for kernel symbols when the .debug cache was not in $HOME.

With the /etc/perfconfig or $HOME/.perfconfig files, it is possible to use an alternate location for the .debug buildid + binary cache.

The problem is that the alternate location was ignored for kernel symbols, leading to partial profiles in case the host kernel was not the one used to collect the profile, e.g., with remote profiling.

This patch fixes the problem by adding the buildi_dir prefix to the lookup path used in dso__load_kernel_sym().

The change to util.h is necessary to avoid a redefinition of _GNU_SOURCE.

Signed-off-by Stephane Eranian <[email protected]>

---

tools/perf/util/symbol.c +3 -2

index 632b50c..8ace954 100644
@@ -16,6 +16,7 @@
 #include "debug.h"
 #include "symbol.h"
 #include "strlist.h"
+#include "util.h"
 
 #include <libelf.h>
 #include <gelf.h>

@@ -2082,8 +2083,8 @@ static int dso__load_kernel_sym(struct dso *dso, struct map *map,
 				  sbuild_id);
 
 		if (asprintf(&kallsyms_allocated_filename,
-			     "%s/.debug/[kernel.kallsyms]/%s",
-			     getenv("HOME"), sbuild_id) == -1) {
+			     "%s/[kernel.kallsyms]/%s",
+			     buildid_dir, sbuild_id) == -1) {
 			pr_err("Not enough memory for kallsyms file lookup\n");
 			return -1;
 		}

tools/perf/util/util.h +2 -0

index 0128906..4b90e93 100644
@@ -40,7 +40,9 @@
 #define decimal_length(x)	((int)(sizeof(x) * 2.56 + 0.5) + 1)
 
 #define _ALL_SOURCE 1
+#ifndef _GNU_SOURCE
 #define _GNU_SOURCE 1
+#endif
 #define _BSD_SOURCE 1
 #define HAS_BOOL

David Ahern

2011-12-02 06:05 UTC #1 lore raw

Quoted 2 lines · Stephane Eranian

There was a bug in the way perf was looking for kernel symbols when the .debug cache was not in $HOME.

By chance have you encountered problems with annotate in this scenario?
i.e., remote analysis with a copy of /proc/kallsyms from the target and
using the --vmlinux argument.

For example, 'perf report --kallsyms kallsyms --vmlinux vmlinux ...' and then attempting to use the annotate option? I have a hit a few issues with this path - and kernel modules with a symfs path -- that I have not had time to debug yet.

David

Quoted 56 lines

With the /etc/perfconfig or $HOME/.perfconfig files, it is possible to use an alternate location for the .debug buildid + binary cache.
The problem is that the alternate location was ignored for kernel symbols, leading to partial profiles in case the host kernel was not the one used to collect the profile, e.g., with remote profiling.
This patch fixes the problem by adding the buildi_dir prefix to the lookup path used in dso__load_kernel_sym().
The change to util.h is necessary to avoid a redefinition of _GNU_SOURCE.
Signed-off-by Stephane Eranian <[email protected]>
---
tools/perf/util/symbol.c +3 -2
index 632b50c..8ace954 100644
@@ -16,6 +16,7 @@
 #include "debug.h"
 #include "symbol.h"
 #include "strlist.h"
+#include "util.h"
 
 #include <libelf.h>
 #include <gelf.h>
@@ -2082,8 +2083,8 @@ static int dso__load_kernel_sym(struct dso *dso, struct map *map,
 				  sbuild_id);
 
 		if (asprintf(&kallsyms_allocated_filename,
-			     "%s/.debug/[kernel.kallsyms]/%s",
-			     getenv("HOME"), sbuild_id) == -1) {
+			     "%s/[kernel.kallsyms]/%s",
+			     buildid_dir, sbuild_id) == -1) {
 			pr_err("Not enough memory for kallsyms file lookup\n");
 			return -1;
 		}
tools/perf/util/util.h +2 -0
index 0128906..4b90e93 100644
@@ -40,7 +40,9 @@
 #define decimal_length(x)	((int)(sizeof(x) * 2.56 + 0.5) + 1)
 
 #define _ALL_SOURCE 1
+#ifndef _GNU_SOURCE
 #define _GNU_SOURCE 1
+#endif
 #define _BSD_SOURCE 1
 #define HAS_BOOL
 