[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAM9d7cjSynKt+CfoPqHQHNJt=to3LmdQqh_x6rKJSUcx6qFbsA@mail.gmail.com>
Date: Tue, 14 Jun 2022 10:59:10 -0700
From: Namhyung Kim <namhyung@...nel.org>
To: Xing Zhengjun <zhengjun.xing@...ux.intel.com>
Cc: Arnaldo Carvalho de Melo <acme@...nel.org>,
Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>, alexander.shishkin@...el.com,
Jiri Olsa <jolsa@...nel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
linux-perf-users <linux-perf-users@...r.kernel.org>,
Ian Rogers <irogers@...gle.com>,
Andi Kleen <ak@...ux.intel.com>,
Kan Liang <kan.liang@...ux.intel.com>
Subject: Re: [PATCH v3 0/5] Add perf stat default events for hybrid machines
Hello,
On Thu, Jun 9, 2022 at 7:55 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: Namhyung Kim <namhyung@...nel.org>
Thanks,
Namhyung
>
> 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