Received-SPF: pass (google.com: domain of linux-kernel+bounces-138081-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
From: Xu Yang <xu.yang_2@nxp.com>
To: Will Deacon <will@kernel.org>
CC: Frank Li <frank.li@nxp.com>, "mark.rutland@arm.com"
	<mark.rutland@arm.com>, "robh+dt@kernel.org" <robh+dt@kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>,
	"conor+dt@kernel.org" <conor+dt@kernel.org>, "shawnguo@kernel.org"
	<shawnguo@kernel.org>, "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>, "festevam@gmail.com"
	<festevam@gmail.com>, "john.g.garry@oracle.com" <john.g.garry@oracle.com>,
	"jolsa@kernel.org" <jolsa@kernel.org>, "namhyung@kernel.org"
	<namhyung@kernel.org>, "irogers@google.com" <irogers@google.com>,
	"mike.leach@linaro.org" <mike.leach@linaro.org>, "peterz@infradead.org"
	<peterz@infradead.org>, "mingo@redhat.com" <mingo@redhat.com>,
	"acme@kernel.org" <acme@kernel.org>, "alexander.shishkin@linux.intel.com"
	<alexander.shishkin@linux.intel.com>, "adrian.hunter@intel.com"
	<adrian.hunter@intel.com>, "linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, "devicetree@vger.kernel.org"
	<devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "linux-perf-users@vger.kernel.org"
	<linux-perf-users@vger.kernel.org>, "imx@lists.linux.dev"
	<imx@lists.linux.dev>
Subject: RE: [EXT] Re: [PATCH v8 3/8] perf: imx_perf: let the driver manage
 the counter usage rather the user
Thread-Topic: [EXT] Re: [PATCH v8 3/8] perf: imx_perf: let the driver manage
 the counter usage rather the user
Thread-Index: AQHafCOvkc1sQxct2EGL+7AcOKkKWbFgLE0AgAEKnEA=
Date: Wed, 10 Apr 2024 07:39:46 +0000
Message-ID:
 <DU2PR04MB882209413AB62531713166B38C062@DU2PR04MB8822.eurprd04.prod.outlook.com>
References: <20240322063930.749126-1-xu.yang_2@nxp.com>
 <20240322063930.749126-3-xu.yang_2@nxp.com>
 <20240409152627.GA23621@willie-the-truck>
In-Reply-To: <20240409152627.GA23621@willie-the-truck>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?IErx0OpBR0qrg3H959TIhd12umo380wL+yERiXXIDdZ1YsAA4/iE2C2OvYFV?=
 =?us-ascii?Q?ua6rREaNtOV+8TXGVYC2rlu534m+roAmUUsPSclXLXUrNISY7gZOoaPv0rPr?=
 =?us-ascii?Q?sVX7Qg2zGkNyhNHKYQDMm8HlEbVrYCxc1vrm9CLrTD3hLycTrk2SlPaZjNco?=
 =?us-ascii?Q?LJrIniz4EnRsPcAmMc5zBAngCS3gNipCJ/EpiGQqjGxeiYNiyhnEGFK3OIH6?=
 =?us-ascii?Q?ECYtCl4/BPW5QVOIPq4EMn9rMIcM/JBe9rQ1Jv8GfMGJHxJC6PbcgsKwu7gq?=
 =?us-ascii?Q?aAs5m+tDCBp6gzFORZS7T8UPyv0Z4joWF+MM5H7/+hF1StSJMzIOk+HqbNAC?=
 =?us-ascii?Q?NRhEHDbgNrx0vUddTh1UvRLHFHnKew1xbZPGOWnQB4BIGC2WeTLfFGv/cU+0?=
 =?us-ascii?Q?xgdml0fR1fVrsI0efJoqR7Odc7qhThVAmRY4FGqhElyKfJnaQFn2MMtrIJhT?=
 =?us-ascii?Q?z0fRqhbZpqcUkiYE0M0dN+qjmqwieyT0Rvhr+fCzClWlrY0AnkK7uveVXqt5?=
 =?us-ascii?Q?QSjUuA/B8OSUcV3BdUmb5L7W5n8NbEx05oddX5WKnLc4JDqnpQeOGJIaIwEu?=
 =?us-ascii?Q?gV1GTTqJzpwpHfGXmSByAmmY4jlG1JRtRZruIsd7FnzbCHFEkl6QdYJfWIX8?=
 =?us-ascii?Q?ZGVoZhZcQRauFfAgTRYtRTIUJS1Kd0Tgkwyj2Y3AaJmxsnDEaILqRA21jFk4?=
 =?us-ascii?Q?22TxeFAgjTg+/xMQJoq6JSz5XcXKqNH53AXloBSLKkXgxjHvYN5wrvCAX0Xf?=
 =?us-ascii?Q?J90TIYYP7fiuun7nHyXij7ew90LdIu6ssGiijZfwbCE327GFY9VFE9lcPyLa?=
 =?us-ascii?Q?zJYZQ4OGXFaVfNu89g8C1jp/F00Hzw5B4UBdjN3o4tuNxA8nyY7lYqfbc494?=
 =?us-ascii?Q?O9Uyfa/5CKNT8VbwOYwMH/w8D4ewzaE7xw3rB3HLie1RPRHRS8Z8RJkuexFO?=
 =?us-ascii?Q?d/VoGMvHCjWr5hwns+mvSEcm4OCfP65av2ttlqdWRVMOonj/Y6R7HVpbKhTh?=
 =?us-ascii?Q?zr/HSHAtyjz14zhtSOAfG7AS28Do5TX4Y7VyXzidOU2nqQcqUwwoud1P2+Tg?=
 =?us-ascii?Q?X4iq8WDJsjzAcdNfNdReAi17dax/8xI0aF7kU9vbv1kTxtpa/bxUKmAWA2qe?=
 =?us-ascii?Q?nECtENzKYu9UuGgv6j2+aZy4i8n6FgkccgVGXvBzRFcDZfwuTq4fcd/7UYIk?=
 =?us-ascii?Q?keTGpDywvldsw8ZLlxhJW3RdisL0K8OlHRo2uDrUCSiknAzz3nWPulaMTHNU?=
 =?us-ascii?Q?HH5OJGTDiBrY4O8mCvIf0dngAgZM+4jwIOnWrKoLncGY56DRrE88xlII7mOv?=
 =?us-ascii?Q?AFPDo9bwCcaNYYVoUdzyN6tEIJG4IcaZg/1FVWI0jUQwvSnVQ6pXFgI1k4DY?=
 =?us-ascii?Q?jZ2KMa0SYRBjdJofRZuyylHF1OkvsfOkZ9GG8p0i4/aGRST21Bt/Awcyi7A/?=
 =?us-ascii?Q?JOs4+ozeONjI82s6NBIUWA8qoD+Og47pn062Ly/nqCDG7COgkIrYKQ/+VsoF?=
 =?us-ascii?Q?pGa//Y2uXteHqUd/u2rBzBw+7WYzS5LSFhxXwyicUebOdSEEIHKCtsIG8llU?=
 =?us-ascii?Q?ngRgQaOy7K3wqJJnOa8=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8822.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e1158f4a-db3c-4c41-dc4a-08dc59316543
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Apr 2024 07:39:46.6530
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: KFUxsXhcQEQLje2WGhWdjF2zYa4wUmVm0Mb/CWb+UJAaWhkWQPq8UrhyeRWksA0nInmc7SsfgBYBcLf05Uh48w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB9082

Hi Will,

>=20
> On Fri, Mar 22, 2024 at 02:39:25PM +0800, Xu Yang wrote:
> > In current design, the user of perf app needs to input counter ID to co=
unt
> > events. However, this is not user-friendly since the user needs to look=
up
> > the map table to find the counter. Instead of letting the user to input
> > the counter, let this driver to manage the counters in this patch.
>=20
> I think we still have to support the old interface so that we don't break
> those existing users (even if the driver just ignores whatever counter ID
> is provided in a backwards-compatible way).
>=20
> > This will be implemented by:
> >  1. allocate counter 0 for cycle event.
> >  2. find unused counter from 1-10 for reference events.
> >  3. allocate specific counter for counter-specific events.
> >
> > In this patch, counter attribute is removed too. To mark counter-specif=
ic
> > events, counter ID will be encoded into perf_pmu_events_attr.id.
> >
> > Reviewed-by: Frank Li <Frank.Li@nxp.com>
> > Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
> >
> > ---
> > Changes in v6:
> >  - new patch
> > Changes in v7:
> >  - no changes
> > Changes in v8:
> >  - add Rb tag
> > ---
> >  drivers/perf/fsl_imx9_ddr_perf.c | 168 ++++++++++++++++++-------------
> >  1 file changed, 99 insertions(+), 69 deletions(-)
> >
> > diff --git a/drivers/perf/fsl_imx9_ddr_perf.c b/drivers/perf/fsl_imx9_d=
dr_perf.c
> > index 0017f2c9ef91..b728719b494c 100644
> > --- a/drivers/perf/fsl_imx9_ddr_perf.c
> > +++ b/drivers/perf/fsl_imx9_ddr_perf.c
> > @@ -245,14 +249,12 @@ static const struct attribute_group ddr_perf_even=
ts_attr_group =3D {
> >       .attrs =3D ddr_perf_events_attrs,
> >  };
> >
> > -PMU_FORMAT_ATTR(event, "config:0-7");
> > -PMU_FORMAT_ATTR(counter, "config:8-15");
> > +PMU_FORMAT_ATTR(event, "config:0-15");
>=20
> Sadly, this is a user-visible change so I think it will break old tools,
> won't it?

For imx ddr pmu, most of the people will use metrics rather event itself,
and we have speficy the format of event parameters in metrics table.
The parameters is also updated in this patchset.

And to easy use for user, the counter should be hidden (transparent) to
user after I had talk with Frank. Then, the user need't to look up the even=
t
table to find which counter to use.=20

So this patchset will basically not break the usage of perf tools and will
improve practicality.

Thanks,
Xu Yang


>=20
> Will