From: John Garry <john.g.garry@oracle.com>
To: Jing Zhang <renyu.zj@linux.alibaba.com>, Ian Rogers <irogers@google.com>
Cc: Will Deacon <will@kernel.org>, James Clark <james.clark@arm.com>,
Arnaldo Carvalho de Melo <acme@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Mike Leach <mike.leach@linaro.org>, Leo Yan <leo.yan@linaro.org>,
Namhyung Kim <namhyung@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Jiri Olsa <jolsa@kernel.org>,
Adrian Hunter <adrian.hunter@intel.com>,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-perf-users@vger.kernel.org, linux-doc@vger.kernel.org,
Zhuo Song <zhuo.song@linux.alibaba.com>,
Shuai Xue <xueshuai@linux.alibaba.com>
Subject: Re: [PATCH v7 0/8] perf vendor events: Add JSON metrics for Arm CMN
Date: Wed, 23 Aug 2023 09:12:55 +0100 [thread overview]
Message-ID: <566ec1ff-fdea-336e-3cb8-503fc593f8ec@oracle.com> (raw)
In-Reply-To: <1692606977-92009-1-git-send-email-renyu.zj@linux.alibaba.com>
On 21/08/2023 09:36, Jing Zhang wrote:
I'm hoping that Ian can check the outstanding patches here, but I'll
also have a look.
> Changes since v6:
> - Supplement the omitted EventCode;
> - Keep the original way of ConfigCode;
> - Supplement the test in empty-pmu-events.c, so that the pmu event test
> can succeed when build with NO_JEVENT=1.
> - Link: https://urldefense.com/v3/__https://lore.kernel.org/all/1691394685-61240-1-git-send-email-renyu.zj@linux.alibaba.com/__;!!ACWV5N9M2RV99hQ!Lakh7Lf-6l6Hovm4Tt5S5pqV1xHm-LAW2ICVl6gLONlN4CNk-BvyADBfjwQe5yQQj89yMK7s7rSjMNHCyFsIUxnHXg$
>
> Jing Zhang (8):
> perf pmu: "Compat" supports matching multiple identifiers
> perf metric: "Compat" supports matching multiple identifiers
> perf vendor events: Supplement the omitted EventCode
> perf jevents: Support more event fields
> perf test: Make matching_pmu effective
> perf test: Add pmu-event test for "Compat" and new event_field.
> perf jevents: Add support for Arm CMN PMU aliasing
> perf vendor events: Add JSON metrics for Arm CMN
>
> .../pmu-events/arch/arm64/arm/cmn/sys/cmn.json | 266 +++++++++++++++++++++
> .../pmu-events/arch/arm64/arm/cmn/sys/metric.json | 74 ++++++
> .../pmu-events/arch/test/test_soc/sys/uncore.json | 8 +
> .../pmu-events/arch/x86/alderlake/pipeline.json | 9 +
> .../pmu-events/arch/x86/alderlaken/pipeline.json | 3 +
> .../pmu-events/arch/x86/broadwell/pipeline.json | 4 +
> .../pmu-events/arch/x86/broadwellde/pipeline.json | 4 +
> .../arch/x86/broadwellde/uncore-cache.json | 2 +
> .../arch/x86/broadwellde/uncore-interconnect.json | 1 +
> .../arch/x86/broadwellde/uncore-memory.json | 1 +
> .../arch/x86/broadwellde/uncore-power.json | 1 +
> .../pmu-events/arch/x86/broadwellx/pipeline.json | 4 +
> .../arch/x86/broadwellx/uncore-cache.json | 2 +
> .../arch/x86/broadwellx/uncore-interconnect.json | 13 +
> .../arch/x86/broadwellx/uncore-memory.json | 2 +
> .../arch/x86/broadwellx/uncore-power.json | 1 +
> .../pmu-events/arch/x86/cascadelakex/pipeline.json | 4 +
> .../arch/x86/cascadelakex/uncore-cache.json | 2 +
> .../arch/x86/cascadelakex/uncore-interconnect.json | 1 +
> .../arch/x86/cascadelakex/uncore-io.json | 1 +
> .../arch/x86/cascadelakex/uncore-memory.json | 1 +
> .../arch/x86/cascadelakex/uncore-power.json | 1 +
> .../pmu-events/arch/x86/elkhartlake/pipeline.json | 2 +
> .../pmu-events/arch/x86/goldmont/pipeline.json | 3 +
> .../pmu-events/arch/x86/goldmontplus/pipeline.json | 3 +
> .../pmu-events/arch/x86/grandridge/pipeline.json | 3 +
> .../arch/x86/graniterapids/pipeline.json | 4 +
> .../perf/pmu-events/arch/x86/haswell/pipeline.json | 4 +
> .../pmu-events/arch/x86/haswellx/pipeline.json | 4 +
> .../pmu-events/arch/x86/haswellx/uncore-cache.json | 2 +
> .../arch/x86/haswellx/uncore-interconnect.json | 14 ++
> .../arch/x86/haswellx/uncore-memory.json | 2 +
> .../pmu-events/arch/x86/haswellx/uncore-power.json | 1 +
> .../perf/pmu-events/arch/x86/icelake/pipeline.json | 4 +
> .../pmu-events/arch/x86/icelakex/pipeline.json | 4 +
> .../pmu-events/arch/x86/icelakex/uncore-cache.json | 1 +
> .../arch/x86/icelakex/uncore-interconnect.json | 1 +
> .../arch/x86/icelakex/uncore-memory.json | 1 +
> .../pmu-events/arch/x86/icelakex/uncore-power.json | 1 +
> .../pmu-events/arch/x86/ivybridge/pipeline.json | 3 +
> .../perf/pmu-events/arch/x86/ivytown/pipeline.json | 4 +
> .../pmu-events/arch/x86/ivytown/uncore-cache.json | 2 +
> .../arch/x86/ivytown/uncore-interconnect.json | 11 +
> .../pmu-events/arch/x86/ivytown/uncore-memory.json | 1 +
> .../pmu-events/arch/x86/ivytown/uncore-power.json | 1 +
> .../pmu-events/arch/x86/jaketown/pipeline.json | 4 +
> .../pmu-events/arch/x86/jaketown/uncore-cache.json | 2 +
> .../arch/x86/jaketown/uncore-interconnect.json | 12 +
> .../arch/x86/jaketown/uncore-memory.json | 1 +
> .../pmu-events/arch/x86/jaketown/uncore-power.json | 2 +
> .../arch/x86/knightslanding/pipeline.json | 3 +
> .../arch/x86/knightslanding/uncore-cache.json | 1 +
> .../arch/x86/knightslanding/uncore-memory.json | 4 +
> .../pmu-events/arch/x86/meteorlake/pipeline.json | 8 +
> .../pmu-events/arch/x86/sandybridge/pipeline.json | 4 +
> .../arch/x86/sapphirerapids/pipeline.json | 5 +
> .../pmu-events/arch/x86/sierraforest/pipeline.json | 4 +
> .../pmu-events/arch/x86/silvermont/pipeline.json | 3 +
> .../perf/pmu-events/arch/x86/skylake/pipeline.json | 4 +
> .../pmu-events/arch/x86/skylakex/pipeline.json | 4 +
> .../pmu-events/arch/x86/skylakex/uncore-cache.json | 2 +
> .../arch/x86/skylakex/uncore-interconnect.json | 1 +
> .../pmu-events/arch/x86/skylakex/uncore-io.json | 1 +
> .../arch/x86/skylakex/uncore-memory.json | 1 +
> .../pmu-events/arch/x86/skylakex/uncore-power.json | 1 +
> .../pmu-events/arch/x86/snowridgex/pipeline.json | 2 +
> .../arch/x86/snowridgex/uncore-cache.json | 1 +
> .../arch/x86/snowridgex/uncore-interconnect.json | 1 +
> .../arch/x86/snowridgex/uncore-memory.json | 1 +
> .../arch/x86/snowridgex/uncore-power.json | 1 +
> .../pmu-events/arch/x86/tigerlake/pipeline.json | 5 +
> tools/perf/pmu-events/empty-pmu-events.c | 8 +
> tools/perf/pmu-events/jevents.py | 21 +-
> tools/perf/tests/pmu-events.c | 64 ++++-
> tools/perf/util/metricgroup.c | 2 +-
> tools/perf/util/pmu.c | 33 ++-
> tools/perf/util/pmu.h | 1 +
> 77 files changed, 679 insertions(+), 9 deletions(-)
> create mode 100644 tools/perf/pmu-events/arch/arm64/arm/cmn/sys/cmn.json
> create mode 100644 tools/perf/pmu-events/arch/arm64/arm/cmn/sys/metric.json
>
next prev parent reply other threads:[~2023-08-23 10:14 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-21 8:36 [PATCH v7 0/8] perf vendor events: Add JSON metrics for Arm CMN Jing Zhang
2023-08-21 8:36 ` [PATCH v7 1/8] perf pmu: "Compat" supports matching multiple identifiers Jing Zhang
2023-08-24 15:05 ` Robin Murphy
2023-08-25 8:40 ` Jing Zhang
2023-08-25 4:11 ` Ian Rogers
2023-08-25 6:12 ` Jing Zhang
2023-08-21 8:36 ` [PATCH v7 2/8] perf metric: " Jing Zhang
2023-08-25 4:16 ` Ian Rogers
2023-08-21 8:36 ` [PATCH v7 4/8] perf jevents: Support more event fields Jing Zhang
2023-08-23 9:12 ` Robin Murphy
2023-08-25 4:42 ` Ian Rogers
2023-08-21 8:36 ` [PATCH v7 5/8] perf test: Make matching_pmu effective Jing Zhang
2023-08-25 4:27 ` Ian Rogers
2023-08-25 6:30 ` Jing Zhang
2023-08-21 8:36 ` [PATCH v7 6/8] perf test: Add pmu-event test for "Compat" and new event_field Jing Zhang
2023-08-25 4:30 ` Ian Rogers
2023-08-25 6:30 ` Jing Zhang
2023-08-21 8:36 ` [PATCH v7 7/8] perf jevents: Add support for Arm CMN PMU aliasing Jing Zhang
2023-08-23 9:33 ` Robin Murphy
2023-08-24 2:12 ` Jing Zhang
2023-08-21 8:36 ` [PATCH v7 8/8] perf vendor events: Add JSON metrics for Arm CMN Jing Zhang
2023-08-25 4:13 ` Ian Rogers
2023-08-25 6:47 ` Jing Zhang
2023-08-23 8:12 ` John Garry [this message]
2023-08-24 2:33 ` [PATCH v7 0/8] " Jing Zhang
2023-09-06 16:05 ` Arnaldo Carvalho de Melo
2023-09-07 2:42 ` Jing Zhang
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=566ec1ff-fdea-336e-3cb8-503fc593f8ec@oracle.com \
--to=john.g.garry@oracle.com \
--cc=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=irogers@google.com \
--cc=james.clark@arm.com \
--cc=jolsa@kernel.org \
--cc=leo.yan@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mike.leach@linaro.org \
--cc=mingo@redhat.com \
--cc=namhyung@kernel.org \
--cc=peterz@infradead.org \
--cc=renyu.zj@linux.alibaba.com \
--cc=will@kernel.org \
--cc=xueshuai@linux.alibaba.com \
--cc=zhuo.song@linux.alibaba.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).