From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Andi Kleen <ak@linux.intel.com>
Cc: Ian Rogers <irogers@google.com>,
vineet.singh@intel.com, perry.taylor@intel.com,
caleb.biggers@intel.com, asaf.yaffe@intel.com,
kshipra.bopardikar@intel.com,
Kan Liang <kan.liang@linux.intel.com>,
Zhengjun Xing <zhengjun.xing@linux.intel.com>,
Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
Mark Rutland <mark.rutland@arm.com>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Jiri Olsa <jolsa@redhat.com>, Namhyung Kim <namhyung@kernel.org>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
James Clark <james.clark@arm.com>,
John Garry <john.garry@huawei.com>,
linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org,
Stephane Eranian <eranian@google.com>
Subject: Re: [PATCH] perf stat: Fix display of grouped aliased events.
Date: Sun, 6 Feb 2022 08:55:17 -0300 [thread overview]
Message-ID: <Yf+3JfzBsH0Z/aNY@kernel.org> (raw)
In-Reply-To: <e3bf95b1-bb9a-91be-1816-59c96c4e2024@linux.intel.com>
Em Fri, Feb 04, 2022 at 05:48:20PM -0800, Andi Kleen escreveu:
>
> On 2/4/2022 5:09 PM, Ian Rogers wrote:
> > An event may have a number of uncore aliases that when added to
> > the evlist are consecutive. If there are multiple uncore events
> > in a group then parse_events__set_leader_for_uncore_aliase will
> > reorder the evlist so that events on the same PMU are
> > adjacent. collect_all_aliases assumes that aliases are in blocks
> > so that only the first counter is printed and all others are
> > marked merged. The reordering for groups breaks the assumption
> > and so all counts are printed. This change removes the assumption
> > from collect_all_aliases that the events are in blocks and
> > instead processes the entire evlist.
>
> Looks good to me.
>
> Reviewed-by: Andi Kleen <ak@linux.intel.com>
Thanks, applied.
- Arnaldo
prev parent reply other threads:[~2022-02-06 11:55 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-05 1:09 [PATCH] perf stat: Fix display of grouped aliased events Ian Rogers
2022-02-05 1:48 ` Andi Kleen
2022-02-06 11:55 ` Arnaldo Carvalho de Melo [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Yf+3JfzBsH0Z/aNY@kernel.org \
--to=acme@kernel.org \
--cc=ak@linux.intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=alexandre.torgue@foss.st.com \
--cc=asaf.yaffe@intel.com \
--cc=caleb.biggers@intel.com \
--cc=eranian@google.com \
--cc=irogers@google.com \
--cc=james.clark@arm.com \
--cc=john.garry@huawei.com \
--cc=jolsa@redhat.com \
--cc=kan.liang@linux.intel.com \
--cc=kshipra.bopardikar@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=mingo@redhat.com \
--cc=namhyung@kernel.org \
--cc=perry.taylor@intel.com \
--cc=peterz@infradead.org \
--cc=vineet.singh@intel.com \
--cc=zhengjun.xing@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.