mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Yoshihiro Furudera (Fujitsu)" <fj5100bi@fujitsu.com>
To: 'James Clark' <james.clark@linaro.org>
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-arm-kernel@lists.infradead.org>,
	"'linux-perf-users@vger.kernel.org'"
	<linux-perf-users@vger.kernel.org>,
	"'linux-kernel@vger.kernel.org'" <linux-kernel@vger.kernel.org>,
	"Akio Kakuno (Fujitsu)" <fj3333bs@fujitsu.com>
Subject: RE: [PATCH] Remove some PMU events for FUJITSU-MONAKA
Date: Wed, 5 Mar 2025 06:40:28 +0000	[thread overview]
Message-ID: <OS3PR01MB6903726FDA70250F72A73410D4CB2@OS3PR01MB6903.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <OSZPR01MB6908AA7CEB24ED38F9289D6DD4CC2@OSZPR01MB6908.jpnprd01.prod.outlook.com>

Hi, James

> 
> Hi, James
> Thank you for your comment.
> 
> > 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.
> 
> I will leave these 2 events and
> add a warning message to the description in the JSON file.
> I will handle other events in the same way as these 2 events.

I'm thinking of adding one of the following warnings
to the description of events where inaccurate counts 
occur on FUJITSU-MONAKA. 
Is this okay?

1.Simple version
"Note: This event does not count accurately."

2.Detailed version
"Note: This event does not count accurately because it counts as a miss regardless of whether the L3 prefetch is a hit or miss."

I think "2.Detailed version" is better.

example:
{
    "EventCode": "0x0396",
    "EventName": "L2D_CACHE_REFILL_L3D_MISS",
    "BriefDescription": "This event counts operations that cause a miss of the L3 cache. Note: This event does not count accurately because it counts as a miss regardless of whether the L3 prefetch is a hit or miss."
}

Best Regards,
Yoshihiro Furudera

  reply	other threads:[~2025-03-05  6:41 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
2025-02-28  9:55   ` Yoshihiro Furudera (Fujitsu)
2025-03-05  6:40     ` Yoshihiro Furudera (Fujitsu) [this message]
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=OS3PR01MB6903726FDA70250F72A73410D4CB2@OS3PR01MB6903.jpnprd01.prod.outlook.com \
    --to=fj5100bi@fujitsu.com \
    --cc=acme@kernel.org \
    --cc=adrian.hunter@intel.com \
    --cc=alexander.shishkin@linux.intel.com \
    --cc=fj3333bs@fujitsu.com \
    --cc=irogers@google.com \
    --cc=james.clark@linaro.org \
    --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®