[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ZC9qT42KyeiUbc3A@kernel.org>
Date: Thu, 6 Apr 2023 21:56:47 -0300
From: Arnaldo Carvalho de Melo <acme@...nel.org>
To: Ian Rogers <irogers@...gle.com>
Cc: 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>,
Namhyung Kim <namhyung@...nel.org>,
Adrian Hunter <adrian.hunter@...el.com>,
Kan Liang <kan.liang@...ux.intel.com>,
Zhengjun Xing <zhengjun.xing@...ux.intel.com>,
linux-kernel@...r.kernel.org, linux-perf-users@...r.kernel.org
Subject: Re: [PATCH v2 1/5] perf vendor events intel: Update free running
alderlake events
Em Thu, Apr 06, 2023 at 05:13:18PM -0700, Ian Rogers escreveu:
> Fix the PMU name, event code and umask.
>
> These updates were generated by:
> https://github.com/intel/perfmon/blob/main/scripts/create_perf_json.py
> with this PR:
> https://github.com/intel/perfmon/pull/66
Thanks, applied.
- Arnaldo
> Signed-off-by: Ian Rogers <irogers@...gle.com>
> ---
> .../arch/x86/alderlake/uncore-memory.json | 16 ++++++++++++----
> .../arch/x86/alderlaken/uncore-memory.json | 16 ++++++++++++----
> 2 files changed, 24 insertions(+), 8 deletions(-)
>
> diff --git a/tools/perf/pmu-events/arch/x86/alderlake/uncore-memory.json b/tools/perf/pmu-events/arch/x86/alderlake/uncore-memory.json
> index 2ccd9cf96957..163d7e7755c4 100644
> --- a/tools/perf/pmu-events/arch/x86/alderlake/uncore-memory.json
> +++ b/tools/perf/pmu-events/arch/x86/alderlake/uncore-memory.json
> @@ -1,29 +1,37 @@
> [
> {
> "BriefDescription": "Counts every 64B read request entering the Memory Controller 0 to DRAM (sum of all channels).",
> + "EventCode": "0xff",
> "EventName": "UNC_MC0_RDCAS_COUNT_FREERUN",
> "PerPkg": "1",
> "PublicDescription": "Counts every 64B read request entering the Memory Controller 0 to DRAM (sum of all channels).",
> - "Unit": "iMC"
> + "UMask": "0x20",
> + "Unit": "imc_free_running_0"
> },
> {
> "BriefDescription": "Counts every 64B write request entering the Memory Controller 0 to DRAM (sum of all channels). Each write request counts as a new request incrementing this counter. However, same cache line write requests (both full and partial) are combined to a single 64 byte data transfer to DRAM.",
> + "EventCode": "0xff",
> "EventName": "UNC_MC0_WRCAS_COUNT_FREERUN",
> "PerPkg": "1",
> - "Unit": "iMC"
> + "UMask": "0x30",
> + "Unit": "imc_free_running_0"
> },
> {
> "BriefDescription": "Counts every 64B read request entering the Memory Controller 1 to DRAM (sum of all channels).",
> + "EventCode": "0xff",
> "EventName": "UNC_MC1_RDCAS_COUNT_FREERUN",
> "PerPkg": "1",
> "PublicDescription": "Counts every 64B read entering the Memory Controller 1 to DRAM (sum of all channels).",
> - "Unit": "iMC"
> + "UMask": "0x20",
> + "Unit": "imc_free_running_1"
> },
> {
> "BriefDescription": "Counts every 64B write request entering the Memory Controller 1 to DRAM (sum of all channels). Each write request counts as a new request incrementing this counter. However, same cache line write requests (both full and partial) are combined to a single 64 byte data transfer to DRAM.",
> + "EventCode": "0xff",
> "EventName": "UNC_MC1_WRCAS_COUNT_FREERUN",
> "PerPkg": "1",
> - "Unit": "iMC"
> + "UMask": "0x30",
> + "Unit": "imc_free_running_1"
> },
> {
> "BriefDescription": "ACT command for a read request sent to DRAM",
> diff --git a/tools/perf/pmu-events/arch/x86/alderlaken/uncore-memory.json b/tools/perf/pmu-events/arch/x86/alderlaken/uncore-memory.json
> index 2ccd9cf96957..163d7e7755c4 100644
> --- a/tools/perf/pmu-events/arch/x86/alderlaken/uncore-memory.json
> +++ b/tools/perf/pmu-events/arch/x86/alderlaken/uncore-memory.json
> @@ -1,29 +1,37 @@
> [
> {
> "BriefDescription": "Counts every 64B read request entering the Memory Controller 0 to DRAM (sum of all channels).",
> + "EventCode": "0xff",
> "EventName": "UNC_MC0_RDCAS_COUNT_FREERUN",
> "PerPkg": "1",
> "PublicDescription": "Counts every 64B read request entering the Memory Controller 0 to DRAM (sum of all channels).",
> - "Unit": "iMC"
> + "UMask": "0x20",
> + "Unit": "imc_free_running_0"
> },
> {
> "BriefDescription": "Counts every 64B write request entering the Memory Controller 0 to DRAM (sum of all channels). Each write request counts as a new request incrementing this counter. However, same cache line write requests (both full and partial) are combined to a single 64 byte data transfer to DRAM.",
> + "EventCode": "0xff",
> "EventName": "UNC_MC0_WRCAS_COUNT_FREERUN",
> "PerPkg": "1",
> - "Unit": "iMC"
> + "UMask": "0x30",
> + "Unit": "imc_free_running_0"
> },
> {
> "BriefDescription": "Counts every 64B read request entering the Memory Controller 1 to DRAM (sum of all channels).",
> + "EventCode": "0xff",
> "EventName": "UNC_MC1_RDCAS_COUNT_FREERUN",
> "PerPkg": "1",
> "PublicDescription": "Counts every 64B read entering the Memory Controller 1 to DRAM (sum of all channels).",
> - "Unit": "iMC"
> + "UMask": "0x20",
> + "Unit": "imc_free_running_1"
> },
> {
> "BriefDescription": "Counts every 64B write request entering the Memory Controller 1 to DRAM (sum of all channels). Each write request counts as a new request incrementing this counter. However, same cache line write requests (both full and partial) are combined to a single 64 byte data transfer to DRAM.",
> + "EventCode": "0xff",
> "EventName": "UNC_MC1_WRCAS_COUNT_FREERUN",
> "PerPkg": "1",
> - "Unit": "iMC"
> + "UMask": "0x30",
> + "Unit": "imc_free_running_1"
> },
> {
> "BriefDescription": "ACT command for a read request sent to DRAM",
> --
> 2.40.0.577.gac1e443424-goog
>
--
- Arnaldo
Powered by blists - more mailing lists