[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Zs-OMNr74getypNN@x1>
Date: Wed, 28 Aug 2024 17:53:04 -0300
From: Arnaldo Carvalho de Melo <acme@...nel.org>
To: Namhyung Kim <namhyung@...nel.org>
Cc: Ian Rogers <irogers@...gle.com>, Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>, Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...nel.org>,
Adrian Hunter <adrian.hunter@...el.com>,
Kan Liang <kan.liang@...ux.intel.com>,
linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1] perf report: Name events in stats for pipe mode
On Wed, Aug 28, 2024 at 10:06:37AM -0700, Namhyung Kim wrote:
> Hi Ian,
>
> On Tue, Aug 27, 2024 at 2:37 PM Ian Rogers <irogers@...gle.com> wrote:
> >
> > In stats mode PERF_RECORD_EVENT_UPDATE isn't being handled meaning the
> > evsels aren't named when handling pipe mode output.
> >
> > Before:
> > ```
> > $ perf record -e inst_retired.any -a -o - sleep 0.1|perf report --stats -i -
> > ...
> > Aggregated stats:
> > TOTAL events: 23358
> > COMM events: 2608 (11.2%)
> > EXIT events: 1 ( 0.0%)
> > FORK events: 2607 (11.2%)
> > SAMPLE events: 174 ( 0.7%)
> > MMAP2 events: 17936 (76.8%)
> > ATTR events: 2 ( 0.0%)
> > FINISHED_ROUND events: 2 ( 0.0%)
> > ID_INDEX events: 1 ( 0.0%)
> > THREAD_MAP events: 1 ( 0.0%)
> > CPU_MAP events: 1 ( 0.0%)
> > EVENT_UPDATE events: 3 ( 0.0%)
> > TIME_CONV events: 1 ( 0.0%)
> > FEATURE events: 20 ( 0.1%)
> > FINISHED_INIT events: 1 ( 0.0%)
> > raw 0xc0 stats:
> > SAMPLE events: 174
> > ```
> >
> > After:
> > ```
> > $ perf record -e inst_retired.any -a -o - sleep 0.1|perf report --stats -i -
> > ...
> > Aggregated stats:
> > TOTAL events: 23742
> > COMM events: 2620 (11.0%)
> > EXIT events: 2 ( 0.0%)
> > FORK events: 2619 (11.0%)
> > SAMPLE events: 165 ( 0.7%)
> > MMAP2 events: 18304 (77.1%)
> > ATTR events: 2 ( 0.0%)
> > FINISHED_ROUND events: 2 ( 0.0%)
> > ID_INDEX events: 1 ( 0.0%)
> > THREAD_MAP events: 1 ( 0.0%)
> > CPU_MAP events: 1 ( 0.0%)
> > EVENT_UPDATE events: 3 ( 0.0%)
> > TIME_CONV events: 1 ( 0.0%)
> > FEATURE events: 20 ( 0.1%)
> > FINISHED_INIT events: 1 ( 0.0%)
> > inst_retired.any stats:
> > SAMPLE events: 165
> > ```
> >
> > This makes the pipe output match the regular output.
> >
> > Signed-off-by: Ian Rogers <irogers@...gle.com>
>
> Acked-by: Namhyung Kim <namhyung@...nel.org>
Thanks, applied to perf-tools-next,
- Arnaldo
Powered by blists - more mailing lists