perf record: Do not save pathname in ./debug/.build-id directory for vmlinux
authorNamhyung Kim <namhyung@kernel.org>
Tue, 4 Nov 2014 01:14:32 +0000 (10:14 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 5 Nov 2014 13:14:08 +0000 (10:14 -0300)
When perf record finishes a session, it pre-processes samples in order
to write build-id info from DSOs that had samples.

During this process it'll call map__load() for the kernel map, and it
ends up calling dso__load_vmlinux_path() which replaces dso->long_name.

But this function checks kernel's build-id before searching vmlinux path
so it'll end up with a cryptic name, the pathname for the entry in the
~/.debug cache, which can be confusing to users.

This patch adds a flag to skip the build-id check during record, so
that it'll have the original vmlinux path for the kernel dso->long_name,
not the entry in the ~/.debug cache.

Before:
  # perf record -va sleep 3
  mmap size 528384B
  [ perf record: Woken up 1 times to write data ]
  [ perf record: Captured and wrote 0.196 MB perf.data (~8545 samples) ]
  Looking at the vmlinux_path (7 entries long)
  Using /home/namhyung/.debug/.build-id/f0/6e17aa50adf4d00b88925e03775de107611551 for symbols

After:
  # perf record -va sleep 3
  mmap size 528384B
  [ perf record: Woken up 1 times to write data ]
  [ perf record: Captured and wrote 0.193 MB perf.data (~8432 samples) ]
  Looking at the vmlinux_path (7 entries long)
  Using /lib/modules/3.16.4-1-ARCH/build/vmlinux for symbols

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung.kim@lge.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1415063674-17206-7-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-record.c
tools/perf/util/symbol.c
tools/perf/util/symbol.h

index 5091a27e6d28256628e14fc01b3a7ed5edfd693f..582c4da155ea98c5ead221a0531e69034ccad9b6 100644 (file)
@@ -200,6 +200,17 @@ static int process_buildids(struct record *rec)
        if (size == 0)
                return 0;
 
+       /*
+        * During this process, it'll load kernel map and replace the
+        * dso->long_name to a real pathname it found.  In this case
+        * we prefer the vmlinux path like
+        *   /lib/modules/3.16.4/build/vmlinux
+        *
+        * rather than build-id path (in debug directory).
+        *   $HOME/.debug/.build-id/f0/6e17aa50adf4d00b88925e03775de107611551
+        */
+       symbol_conf.ignore_vmlinux_buildid = true;
+
        return __perf_session__process_events(session, start,
                                              size - start,
                                              size, &build_id__mark_dso_hit_ops);
index c69915c9d5bc202b699793e6833860f172a75093..c24c5b83156cd92ec5ba5c457153e43706b81424 100644 (file)
@@ -1511,12 +1511,10 @@ int dso__load_vmlinux_path(struct dso *dso, struct map *map,
                           symbol_filter_t filter)
 {
        int i, err = 0;
-       char *filename;
+       char *filename = NULL;
 
-       pr_debug("Looking at the vmlinux_path (%d entries long)\n",
-                vmlinux_path__nr_entries + 1);
-
-       filename = dso__build_id_filename(dso, NULL, 0);
+       if (!symbol_conf.ignore_vmlinux_buildid)
+               filename = dso__build_id_filename(dso, NULL, 0);
        if (filename != NULL) {
                err = dso__load_vmlinux(dso, map, filename, true, filter);
                if (err > 0)
@@ -1524,6 +1522,9 @@ int dso__load_vmlinux_path(struct dso *dso, struct map *map,
                free(filename);
        }
 
+       pr_debug("Looking at the vmlinux_path (%d entries long)\n",
+                vmlinux_path__nr_entries + 1);
+
        for (i = 0; i < vmlinux_path__nr_entries; ++i) {
                err = dso__load_vmlinux(dso, map, vmlinux_path[i], false, filter);
                if (err > 0)
index eb2c19bf8d90d4901703381c3f216ba9576f109f..ded3ca7266dea4d122660bfdbe9c357befec993a 100644 (file)
@@ -105,6 +105,7 @@ struct symbol_conf {
        unsigned short  nr_events;
        bool            try_vmlinux_path,
                        ignore_vmlinux,
+                       ignore_vmlinux_buildid,
                        show_kernel_path,
                        use_modules,
                        sort_by_name,