[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAP-5=fUJ2KaxLXwUhyTtAZhwQ=M=QEv4_tP6y5CU08tf_uKAnA@mail.gmail.com>
Date: Fri, 29 Jul 2022 08:03:20 -0700
From: Ian Rogers <irogers@...gle.com>
To: zhengjun.xing@...ux.intel.com
Cc: acme@...nel.org, peterz@...radead.org, mingo@...hat.com,
alexander.shishkin@...el.com, jolsa@...nel.org,
namhyung@...nel.org, linux-kernel@...r.kernel.org,
linux-perf-users@...r.kernel.org, ak@...ux.intel.com,
kan.liang@...ux.intel.com
Subject: Re: [PATCH v4 0/5] Add perf stat default events for hybrid machines
On Wed, Jul 20, 2022 at 11:56 PM <zhengjun.xing@...ux.intel.com> wrote:
>
> From: Zhengjun Xing <zhengjun.xing@...ux.intel.com>
>
> The patch series is to clean up the existing perf stat default and support
> the perf metrics Topdown for the p-core PMU in the perf stat default. The
> first 4 patches are the clean-up patch and fixing the "--detailed" issue.
> The last patch adds support for the perf metrics Topdown, the perf metrics
> Topdown support for e-core PMU will be implemented later separately.
>
> Kan Liang (4):
> perf stat: Revert "perf stat: Add default hybrid events"
> perf evsel: Add arch_evsel__hw_name()
> perf evlist: Always use arch_evlist__add_default_attrs()
> perf x86 evlist: Add default hybrid events for perf stat
>
> Zhengjun Xing (1):
> perf stat: Add topdown metrics in the default perf stat on the hybrid
> machine
Acked-by: Ian Rogers <irogers@...gle.com>
Thanks,
Ian
> tools/perf/arch/x86/util/evlist.c | 64 +++++++++++++++++++++++++-----
> tools/perf/arch/x86/util/evsel.c | 20 ++++++++++
> tools/perf/arch/x86/util/topdown.c | 51 ++++++++++++++++++++++++
> tools/perf/arch/x86/util/topdown.h | 1 +
> tools/perf/builtin-stat.c | 50 ++++-------------------
> tools/perf/util/evlist.c | 11 +++--
> tools/perf/util/evlist.h | 9 ++++-
> tools/perf/util/evsel.c | 7 +++-
> tools/perf/util/evsel.h | 1 +
> tools/perf/util/stat-display.c | 2 +-
> tools/perf/util/topdown.c | 7 ++++
> tools/perf/util/topdown.h | 3 +-
> 12 files changed, 166 insertions(+), 60 deletions(-)
>
> --
> 2.25.1
>
Powered by blists - more mailing lists