From: James Clark <james.clark@linaro.org>
To: Yoshihiro Furudera <fj5100bi@fujitsu.com>
Cc: John Garry <john.g.garry@oracle.com>,
Will Deacon <will@kernel.org>, Mike Leach <mike.leach@linaro.org>,
Leo Yan <leo.yan@linux.dev>,
Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
Arnaldo Carvalho de Melo <acme@kernel.org>,
Namhyung Kim <namhyung@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Jiri Olsa <jolsa@kernel.org>, Ian Rogers <irogers@google.com>,
Adrian Hunter <adrian.hunter@intel.com>,
"Liang, Kan" <kan.liang@linux.intel.com>,
linux-arm-kernel@lists.infradead.org,
linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org,
Akio Kakuno <fj3333bs@aa.jp.fujitsu.com>
Subject: Re: [PATCH] Remove some PMU events for FUJITSU-MONAKA
Date: Thu, 27 Feb 2025 10:53:53 +0000 [thread overview]
Message-ID: <96b323eb-15b2-4b60-8522-83bf2f57694b@linaro.org> (raw)
In-Reply-To: <20250227054045.1340090-1-fj5100bi@fujitsu.com>
On 27/02/2025 5:40 am, Yoshihiro Furudera wrote:
> The following events are not counted properly:
>
> 0x0037 LL_CACHE_MISS_RD
> 0x400B L3D_CACHE_LMISS_RD
These two are discoverable so will still appear in
/sys/bus/event_source/devices/armv8_pmuv3_0/events/ if the hardware says
they exist. It might be better to change the json strings of these two
to warn that they don't work if that's the case, otherwise Perf will
still list them and you'll be worse off.
> 0x0396 L2D_CACHE_REFILL_L3D_MISS
> 0x039A L2D_CACHE_REFILL_L3D_MISS_PRF
> 0x039B L2D_CACHE_REFILL_L3D_MISS_HWPRF
> 0x039C L2D_CACHE_REFILL_L3D_HIT
> 0x03A0 L2D_CACHE_REFILL_L3D_HIT_PRF
> 0x03A1 L2D_CACHE_REFILL_L3D_HIT_HWPRF
>
> Specifically, these events are always counted as misses,
> regardless of whether the L3 prefetch is a hit or a miss.
> So I remove these events in l3_cache.json, ll_cache.json
>
> Signed-off-by: Yoshihiro Furudera <fj5100bi@fujitsu.com>
> ---
> .../arch/arm64/fujitsu/monaka/l3_cache.json | 34 -------------------
> .../arch/arm64/fujitsu/monaka/ll_cache.json | 4 ---
> 2 files changed, 38 deletions(-)
>
> diff --git a/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/l3_cache.json b/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/l3_cache.json
> index 3f3e0d22ac68..917b9b5bf8bb 100644
> --- a/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/l3_cache.json
> +++ b/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/l3_cache.json
> @@ -37,11 +37,6 @@
> "EventName": "L2D_CACHE_REFILL_L3D_CACHE_HWPRF",
> "BriefDescription": "This event counts L2D_CACHE_REFILL_L3D_CACHE caused by hardware prefetch access."
> },
> - {
> - "EventCode": "0x0396",
> - "EventName": "L2D_CACHE_REFILL_L3D_MISS",
> - "BriefDescription": "This event counts operations that cause a miss of the L3 cache."
> - },
> {
> "EventCode": "0x0397",
> "EventName": "L2D_CACHE_REFILL_L3D_MISS_DM",
> @@ -57,21 +52,6 @@
> "EventName": "L2D_CACHE_REFILL_L3D_MISS_DM_WR",
> "BriefDescription": "This event counts L2D_CACHE_REFILL_L3D_MISS caused by demand write access."
> },
> - {
> - "EventCode": "0x039A",
> - "EventName": "L2D_CACHE_REFILL_L3D_MISS_PRF",
> - "BriefDescription": "This event counts L2D_CACHE_REFILL_L3D_MISS caused by prefetch access."
> - },
> - {
> - "EventCode": "0x039B",
> - "EventName": "L2D_CACHE_REFILL_L3D_MISS_HWPRF",
> - "BriefDescription": "This event counts L2D_CACHE_REFILL_L3D_MISS caused by hardware prefetch access."
> - },
> - {
> - "EventCode": "0x039C",
> - "EventName": "L2D_CACHE_REFILL_L3D_HIT",
> - "BriefDescription": "This event counts operations that cause a hit of the L3 cache."
> - },
> {
> "EventCode": "0x039D",
> "EventName": "L2D_CACHE_REFILL_L3D_HIT_DM",
> @@ -87,16 +67,6 @@
> "EventName": "L2D_CACHE_REFILL_L3D_HIT_DM_WR",
> "BriefDescription": "This event counts L2D_CACHE_REFILL_L3D_HIT caused by demand write access."
> },
> - {
> - "EventCode": "0x03A0",
> - "EventName": "L2D_CACHE_REFILL_L3D_HIT_PRF",
> - "BriefDescription": "This event counts L2D_CACHE_REFILL_L3D_HIT caused by prefetch access."
> - },
> - {
> - "EventCode": "0x03A1",
> - "EventName": "L2D_CACHE_REFILL_L3D_HIT_HWPRF",
> - "BriefDescription": "This event counts L2D_CACHE_REFILL_L3D_HIT caused by hardware prefetch access."
> - },
> {
> "EventCode": "0x03A2",
> "EventName": "L2D_CACHE_REFILL_L3D_MISS_PFTGT_HIT",
> @@ -151,9 +121,5 @@
> "EventCode": "0x03AC",
> "EventName": "L2D_CACHE_REFILL_L3D_MISS_FR_L3",
> "BriefDescription": "This event counts the number of L3 cache misses where the requests access L3 cache in the different socket from the requests."
> - },
> - {
> - "ArchStdEvent": "L3D_CACHE_LMISS_RD",
> - "BriefDescription": "This event counts access counted by L3D_CACHE that is not completed by the L3D cache, and a Memory-read operation, as defined by the L2D_CACHE_REFILL_L3D_MISS events."
> }
> ]
> diff --git a/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/ll_cache.json b/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/ll_cache.json
> index a441b84729ab..c2325855eda3 100644
> --- a/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/ll_cache.json
> +++ b/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/ll_cache.json
> @@ -2,9 +2,5 @@
> {
> "ArchStdEvent": "LL_CACHE_RD",
> "BriefDescription": "This event counts access counted by L3D_CACHE that is a Memory-read operation, as defined by the L2D_CACHE_REFILL_L3D_CACHE events."
> - },
> - {
> - "ArchStdEvent": "LL_CACHE_MISS_RD",
> - "BriefDescription": "This event counts access counted by L3D_CACHE that is not completed by the L3D cache, and a Memory-read operation, as defined by the L2D_CACHE_REFILL_L3D_MISS events."
> }
> ]
next prev parent reply other threads:[~2025-02-27 10:53 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-27 5:40 Yoshihiro Furudera
2025-02-27 10:53 ` James Clark [this message]
2025-02-28 9:55 ` Yoshihiro Furudera (Fujitsu)
2025-03-05 6:40 ` Yoshihiro Furudera (Fujitsu)
2025-03-05 9:37 ` James Clark
2025-03-07 3:10 ` Yoshihiro Furudera (Fujitsu)
2025-03-27 4:24 ` Koichi Okuno (Fujitsu)
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=96b323eb-15b2-4b60-8522-83bf2f57694b@linaro.org \
--to=james.clark@linaro.org \
--cc=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=fj3333bs@aa.jp.fujitsu.com \
--cc=fj5100bi@fujitsu.com \
--cc=irogers@google.com \
--cc=john.g.garry@oracle.com \
--cc=jolsa@kernel.org \
--cc=kan.liang@linux.intel.com \
--cc=leo.yan@linux.dev \
--cc=linux-arm-kernel@lists.infradead.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=will@kernel.org \
/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
all inboxes | Powered by JetHome®