perf record: Add --buildid-all option
authorNamhyung Kim <namhyung@kernel.org>
Mon, 11 Jan 2016 13:37:09 +0000 (22:37 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 12 Jan 2016 15:42:07 +0000 (12:42 -0300)
The --buildid-all option is to record build-id of all DSOs in the file.
It might be very costly to postprocess samples to find which DSO hits.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1452519429-31779-1-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/Documentation/perf-record.txt
tools/perf/builtin-record.c

index 3a1a32f5479f99d2d1744ec9135daae63fa34979..fbceb631387c31c44002080130b7ff6e282229d3 100644 (file)
@@ -338,6 +338,9 @@ Options passed to clang when compiling BPF scriptlets.
 Specify vmlinux path which has debuginfo.
 (enabled when BPF prologue is on)
 
+--buildid-all::
+Record build-id of all DSOs regardless whether it's actually hit or not.
+
 SEE ALSO
 --------
 linkperf:perf-stat[1], linkperf:perf-list[1]
index dc4e0adf5c5b5c60cf96cb22120918c575f2b113..319712a4e02b73de7359fc2ed772bed1d9cd424f 100644 (file)
@@ -50,6 +50,7 @@ struct record {
        int                     realtime_prio;
        bool                    no_buildid;
        bool                    no_buildid_cache;
+       bool                    buildid_all;
        unsigned long long      samples;
 };
 
@@ -362,6 +363,13 @@ static int process_buildids(struct record *rec)
         */
        symbol_conf.ignore_vmlinux_buildid = true;
 
+       /*
+        * If --buildid-all is given, it marks all DSO regardless of hits,
+        * so no need to process samples.
+        */
+       if (rec->buildid_all)
+               rec->tool.sample = NULL;
+
        return perf_session__process_events(session);
 }
 
@@ -756,12 +764,8 @@ out_child:
 
                if (!rec->no_buildid) {
                        process_buildids(rec);
-                       /*
-                        * We take all buildids when the file contains
-                        * AUX area tracing data because we do not decode the
-                        * trace because it would take too long.
-                        */
-                       if (rec->opts.full_auxtrace)
+
+                       if (rec->buildid_all)
                                dsos__hit_all(rec->session);
                }
                perf_session__write_header(rec->session, rec->evlist, fd, true);
@@ -1138,6 +1142,8 @@ struct option __record_options[] = {
                   "options passed to clang when compiling BPF scriptlets"),
        OPT_STRING(0, "vmlinux", &symbol_conf.vmlinux_name,
                   "file", "vmlinux pathname"),
+       OPT_BOOLEAN(0, "buildid-all", &record.buildid_all,
+                   "Record build-id of all DSOs regardless of hits"),
        OPT_END()
 };
 
@@ -1255,6 +1261,14 @@ int cmd_record(int argc, const char **argv, const char *prefix __maybe_unused)
        if (err)
                goto out_symbol_exit;
 
+       /*
+        * We take all buildids when the file contains
+        * AUX area tracing data because we do not decode the
+        * trace because it would take too long.
+        */
+       if (rec->opts.full_auxtrace)
+               rec->buildid_all = true;
+
        if (record_opts__config(&rec->opts)) {
                err = -EINVAL;
                goto out_symbol_exit;