perf record: Fix wrong comm in system-wide mode with delay
authorNamhyung Kim <namhyung@kernel.org>
Fri, 27 Aug 2021 23:32:12 +0000 (16:32 -0700)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 30 Aug 2021 21:22:23 +0000 (18:22 -0300)
commitbb07d62e039b592f8006c9faedab48cd627e20c4
tree032b20b72ed0a23762c8dd56dc91a98fde758c4b
parent1c02f6c9043e9a6f359278cc2f17b4283ac0bd67
perf record: Fix wrong comm in system-wide mode with delay

Stephane found that the name of the forked process in a system-wide
mode is wrong when --delay option is used.  For example,

  # perf record -a --delay=1000  noploop 3

The noploop process will run a busy loop for 3 second.  And on an idle
machine it should show up at the top in the perf report.  It works
well without the --delay option.  But if I add the option, it showed
'perf' not 'noploop'.

  # perf report -s comm -q | head -3
      52.94%  perf
      16.65%  swapper
      12.04%  chrome

It turned out that the dummy event didn't work at all and it missed
COMM and MMAP events for the noploop process (and others too).  We
should enable the dummy event immediately in system-wide mode, as the
enable-on-exec would work only for task events.

With this change,

  # perf report -s comm -q | head -3
      52.75%  noploop
      17.03%  swapper
      12.83%  chrome

Reported-by: Stephane Eranian <eranian@google.com>
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Acked-by: Ian Rogers <irogers@google.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lore.kernel.org/lkml/20210827233212.3121037-1-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-record.c