* [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field
@ 2022-08-08 19:54 Ilkka Koskinen
2022-08-09 10:05 ` Robin Murphy
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Ilkka Koskinen @ 2022-08-08 19:54 UTC (permalink / raw)
To: Will Deacon, Mark Rutland, Robin Murphy
Cc: linux-arm-kernel, linux-kernel, Ilkka Koskinen, patches
CMN-600 uses bits [27:0] for child node address offset while bits [30:28]
are required to be zero.
For CMN-650, the child node address offset field has been increased
to include bits [29:0] while leaving only bit 30 set to zero.
Let's include the missing two bits and assume older implementations
comply with the spec and set bits [29:28] to 0.
Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
---
I have tested this patch on CMN-600 and CMN-650.
drivers/perf/arm-cmn.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/perf/arm-cmn.c b/drivers/perf/arm-cmn.c
index 80d8309652a4..b80a9b74662b 100644
--- a/drivers/perf/arm-cmn.c
+++ b/drivers/perf/arm-cmn.c
@@ -36,7 +36,7 @@
#define CMN_CI_CHILD_COUNT GENMASK_ULL(15, 0)
#define CMN_CI_CHILD_PTR_OFFSET GENMASK_ULL(31, 16)
-#define CMN_CHILD_NODE_ADDR GENMASK(27, 0)
+#define CMN_CHILD_NODE_ADDR GENMASK(29, 0)
#define CMN_CHILD_NODE_EXTERNAL BIT(31)
#define CMN_MAX_DIMENSION 12
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field
2022-08-08 19:54 [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field Ilkka Koskinen
@ 2022-08-09 10:05 ` Robin Murphy
2022-08-31 4:46 ` Ilkka Koskinen
2022-09-21 16:17 ` Robin Murphy
2022-09-22 20:33 ` Will Deacon
2 siblings, 1 reply; 6+ messages in thread
From: Robin Murphy @ 2022-08-09 10:05 UTC (permalink / raw)
To: Ilkka Koskinen, Will Deacon, Mark Rutland
Cc: linux-arm-kernel, linux-kernel, patches
On 2022-08-08 20:54, Ilkka Koskinen wrote:
> CMN-600 uses bits [27:0] for child node address offset while bits [30:28]
> are required to be zero.
>
> For CMN-650, the child node address offset field has been increased
> to include bits [29:0] while leaving only bit 30 set to zero.
>
> Let's include the missing two bits and assume older implementations
> comply with the spec and set bits [29:28] to 0.
Ah, I don't think it ever even crossed my mind to look for changes here,
but of course if the node regions have grown from 16KB to 64KB then it's
only natural that offsets grow to match. I expect the CMN-700 config
that we tested on internally would have been small enough for this not
to make a difference. Thanks for the catch!
Reviewed-by: Robin Murphy <robin.murphy@arm.com>
> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
> ---
>
> I have tested this patch on CMN-600 and CMN-650.
>
> drivers/perf/arm-cmn.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/perf/arm-cmn.c b/drivers/perf/arm-cmn.c
> index 80d8309652a4..b80a9b74662b 100644
> --- a/drivers/perf/arm-cmn.c
> +++ b/drivers/perf/arm-cmn.c
> @@ -36,7 +36,7 @@
> #define CMN_CI_CHILD_COUNT GENMASK_ULL(15, 0)
> #define CMN_CI_CHILD_PTR_OFFSET GENMASK_ULL(31, 16)
>
> -#define CMN_CHILD_NODE_ADDR GENMASK(27, 0)
> +#define CMN_CHILD_NODE_ADDR GENMASK(29, 0)
> #define CMN_CHILD_NODE_EXTERNAL BIT(31)
>
> #define CMN_MAX_DIMENSION 12
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field
2022-08-09 10:05 ` Robin Murphy
@ 2022-08-31 4:46 ` Ilkka Koskinen
0 siblings, 0 replies; 6+ messages in thread
From: Ilkka Koskinen @ 2022-08-31 4:46 UTC (permalink / raw)
To: Will Deacon, Robin Murphy
Cc: Ilkka Koskinen, Mark Rutland, linux-arm-kernel, linux-kernel,
patches
Will,
Would you need something more or could you pick up this patch?
Cheers, Ilkka
On Tue, 9 Aug 2022, Robin Murphy wrote:
> On 2022-08-08 20:54, Ilkka Koskinen wrote:
>> CMN-600 uses bits [27:0] for child node address offset while bits [30:28]
>> are required to be zero.
>>
>> For CMN-650, the child node address offset field has been increased
>> to include bits [29:0] while leaving only bit 30 set to zero.
>>
>> Let's include the missing two bits and assume older implementations
>> comply with the spec and set bits [29:28] to 0.
>
> Ah, I don't think it ever even crossed my mind to look for changes here, but
> of course if the node regions have grown from 16KB to 64KB then it's only
> natural that offsets grow to match. I expect the CMN-700 config that we
> tested on internally would have been small enough for this not to make a
> difference. Thanks for the catch!
>
> Reviewed-by: Robin Murphy <robin.murphy@arm.com>
>
>> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
>> ---
>>
>> I have tested this patch on CMN-600 and CMN-650.
>>
>> drivers/perf/arm-cmn.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/perf/arm-cmn.c b/drivers/perf/arm-cmn.c
>> index 80d8309652a4..b80a9b74662b 100644
>> --- a/drivers/perf/arm-cmn.c
>> +++ b/drivers/perf/arm-cmn.c
>> @@ -36,7 +36,7 @@
>> #define CMN_CI_CHILD_COUNT GENMASK_ULL(15, 0)
>> #define CMN_CI_CHILD_PTR_OFFSET GENMASK_ULL(31, 16)
>> -#define CMN_CHILD_NODE_ADDR GENMASK(27, 0)
>> +#define CMN_CHILD_NODE_ADDR GENMASK(29, 0)
>> #define CMN_CHILD_NODE_EXTERNAL BIT(31)
>> #define CMN_MAX_DIMENSION 12
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field
2022-08-08 19:54 [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field Ilkka Koskinen
2022-08-09 10:05 ` Robin Murphy
@ 2022-09-21 16:17 ` Robin Murphy
2022-09-21 19:56 ` Ilkka Koskinen
2022-09-22 20:33 ` Will Deacon
2 siblings, 1 reply; 6+ messages in thread
From: Robin Murphy @ 2022-09-21 16:17 UTC (permalink / raw)
To: Ilkka Koskinen, Will Deacon, Mark Rutland
Cc: linux-arm-kernel, linux-kernel, patches
On 08/08/2022 8:54 pm, Ilkka Koskinen wrote:
> CMN-600 uses bits [27:0] for child node address offset while bits [30:28]
> are required to be zero.
>
> For CMN-650, the child node address offset field has been increased
> to include bits [29:0] while leaving only bit 30 set to zero.
>
> Let's include the missing two bits and assume older implementations
> comply with the spec and set bits [29:28] to 0.
Just to add that other users have run into this as well, so getting it
applied as a fix would be just lovely. Technically I think it's:
Fixes: 60d1504070c2 ("perf/arm-cmn: Support new IP features")
Cheers,
Robin.
> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
> ---
>
> I have tested this patch on CMN-600 and CMN-650.
>
> drivers/perf/arm-cmn.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/perf/arm-cmn.c b/drivers/perf/arm-cmn.c
> index 80d8309652a4..b80a9b74662b 100644
> --- a/drivers/perf/arm-cmn.c
> +++ b/drivers/perf/arm-cmn.c
> @@ -36,7 +36,7 @@
> #define CMN_CI_CHILD_COUNT GENMASK_ULL(15, 0)
> #define CMN_CI_CHILD_PTR_OFFSET GENMASK_ULL(31, 16)
>
> -#define CMN_CHILD_NODE_ADDR GENMASK(27, 0)
> +#define CMN_CHILD_NODE_ADDR GENMASK(29, 0)
> #define CMN_CHILD_NODE_EXTERNAL BIT(31)
>
> #define CMN_MAX_DIMENSION 12
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field
2022-09-21 16:17 ` Robin Murphy
@ 2022-09-21 19:56 ` Ilkka Koskinen
0 siblings, 0 replies; 6+ messages in thread
From: Ilkka Koskinen @ 2022-09-21 19:56 UTC (permalink / raw)
To: Robin Murphy, Will Deacon
Cc: Ilkka Koskinen, Mark Rutland, linux-arm-kernel, linux-kernel,
patches
Hi Robin,
On Wed, 21 Sep 2022, Robin Murphy wrote:
> On 08/08/2022 8:54 pm, Ilkka Koskinen wrote:
>> CMN-600 uses bits [27:0] for child node address offset while bits [30:28]
>> are required to be zero.
>>
>> For CMN-650, the child node address offset field has been increased
>> to include bits [29:0] while leaving only bit 30 set to zero.
>>
>> Let's include the missing two bits and assume older implementations
>> comply with the spec and set bits [29:28] to 0.
>
> Just to add that other users have run into this as well, so getting it
> applied as a fix would be just lovely. Technically I think it's:
>
> Fixes: 60d1504070c2 ("perf/arm-cmn: Support new IP features")
Ah, that's true. Should I resend the patch with your reviewed-by and the
fixes lines?
Cheers, Ilkka
>
> Cheers,
> Robin.
>
>> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
>> ---
>>
>> I have tested this patch on CMN-600 and CMN-650.
>>
>> drivers/perf/arm-cmn.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/perf/arm-cmn.c b/drivers/perf/arm-cmn.c
>> index 80d8309652a4..b80a9b74662b 100644
>> --- a/drivers/perf/arm-cmn.c
>> +++ b/drivers/perf/arm-cmn.c
>> @@ -36,7 +36,7 @@
>> #define CMN_CI_CHILD_COUNT GENMASK_ULL(15, 0)
>> #define CMN_CI_CHILD_PTR_OFFSET GENMASK_ULL(31, 16)
>> -#define CMN_CHILD_NODE_ADDR GENMASK(27, 0)
>> +#define CMN_CHILD_NODE_ADDR GENMASK(29, 0)
>> #define CMN_CHILD_NODE_EXTERNAL BIT(31)
>> #define CMN_MAX_DIMENSION 12
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field
2022-08-08 19:54 [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field Ilkka Koskinen
2022-08-09 10:05 ` Robin Murphy
2022-09-21 16:17 ` Robin Murphy
@ 2022-09-22 20:33 ` Will Deacon
2 siblings, 0 replies; 6+ messages in thread
From: Will Deacon @ 2022-09-22 20:33 UTC (permalink / raw)
To: Mark Rutland, Robin Murphy, Ilkka Koskinen
Cc: catalin.marinas, kernel-team, Will Deacon, patches, linux-kernel,
linux-arm-kernel
On Mon, 8 Aug 2022 12:54:55 -0700, Ilkka Koskinen wrote:
> CMN-600 uses bits [27:0] for child node address offset while bits [30:28]
> are required to be zero.
>
> For CMN-650, the child node address offset field has been increased
> to include bits [29:0] while leaving only bit 30 set to zero.
>
> Let's include the missing two bits and assume older implementations
> comply with the spec and set bits [29:28] to 0.
>
> [...]
Applied to arm64 (for-next/fixes), thanks!
[1/1] perf/arm-cmn: Add more bits to child node address offset field
https://git.kernel.org/arm64/c/05d6f6d346fe
Cheers,
--
Will
https://fixes.arm64.dev
https://next.arm64.dev
https://will.arm64.dev
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-09-22 20:33 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-08-08 19:54 [PATCH 1/1] perf/arm-cmn: Add more bits to child node address offset field Ilkka Koskinen
2022-08-09 10:05 ` Robin Murphy
2022-08-31 4:46 ` Ilkka Koskinen
2022-09-21 16:17 ` Robin Murphy
2022-09-21 19:56 ` Ilkka Koskinen
2022-09-22 20:33 ` Will Deacon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox