* [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
@ 2020-08-25 17:42 Julien Grall
2020-08-26 7:46 ` Bertrand Marquis
0 siblings, 1 reply; 6+ messages in thread
From: Julien Grall @ 2020-08-25 17:42 UTC (permalink / raw)
To: xen-devel
Cc: julien, Julien Grall, Stefano Stabellini, Volodymyr Babchuk,
Bertrand Marquis
From: Julien Grall <jgrall@amazon.com>
Commit 858c0be8c2fa "xen/arm: Enable CPU Erratum 1165522 for Neoverse"
added a new erratum but forgot to update silicon-errata.txt.
Update the file accordingly to keep track of errata workaround in Xen.
Signed-off-by: Julien Grall <jgrall@amazon.com>
Cc: Bertrand Marquis <bertrand.marquis@arm.com>
---
docs/misc/arm/silicon-errata.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/docs/misc/arm/silicon-errata.txt b/docs/misc/arm/silicon-errata.txt
index 11e5a9dcec2c..ee070e723a5f 100644
--- a/docs/misc/arm/silicon-errata.txt
+++ b/docs/misc/arm/silicon-errata.txt
@@ -51,4 +51,5 @@ stable hypervisors.
| ARM | Cortex-A57 | #1319537 | N/A |
| ARM | Cortex-A72 | #1319367 | N/A |
| ARM | Cortex-A76 | #1165522 | N/A |
+| ARM | Neoverse | #1165522 | N/A
| ARM | MMU-500 | #842869 | N/A |
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
2020-08-25 17:42 [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum Julien Grall
@ 2020-08-26 7:46 ` Bertrand Marquis
2020-08-26 7:55 ` Bertrand Marquis
2020-08-26 9:37 ` Julien Grall
0 siblings, 2 replies; 6+ messages in thread
From: Bertrand Marquis @ 2020-08-26 7:46 UTC (permalink / raw)
To: Julien Grall
Cc: Xen-devel, Julien Grall, Stefano Stabellini, Volodymyr Babchuk
> On 25 Aug 2020, at 18:42, Julien Grall <julien@xen.org> wrote:
>
> From: Julien Grall <jgrall@amazon.com>
>
> Commit 858c0be8c2fa "xen/arm: Enable CPU Erratum 1165522 for Neoverse"
> added a new erratum but forgot to update silicon-errata.txt.
>
> Update the file accordingly to keep track of errata workaround in Xen.
Oh i should have done that as part of my patch.
Nice catch.
>
> Signed-off-by: Julien Grall <jgrall@amazon.com>
> Cc: Bertrand Marquis <bertrand.marquis@arm.com>
> ---
> docs/misc/arm/silicon-errata.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/docs/misc/arm/silicon-errata.txt b/docs/misc/arm/silicon-errata.txt
> index 11e5a9dcec2c..ee070e723a5f 100644
> --- a/docs/misc/arm/silicon-errata.txt
> +++ b/docs/misc/arm/silicon-errata.txt
> @@ -51,4 +51,5 @@ stable hypervisors.
> | ARM | Cortex-A57 | #1319537 | N/A |
> | ARM | Cortex-A72 | #1319367 | N/A |
> | ARM | Cortex-A76 | #1165522 | N/A |
> +| ARM | Neoverse | #1165522 | N/A
Should be Neoverse-N1 here (E1 for example is not impacted by this errata)
Cheers
Bertrand
> | ARM | MMU-500 | #842869 | N/A |
> --
> 2.17.1
>
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
2020-08-26 7:46 ` Bertrand Marquis
@ 2020-08-26 7:55 ` Bertrand Marquis
2020-08-26 9:37 ` Julien Grall
1 sibling, 0 replies; 6+ messages in thread
From: Bertrand Marquis @ 2020-08-26 7:55 UTC (permalink / raw)
To: Julien Grall
Cc: Xen-devel, Julien Grall, Stefano Stabellini, Volodymyr Babchuk, nd
> On 26 Aug 2020, at 08:46, Bertrand Marquis <Bertrand.Marquis@arm.com> wrote:
>
>
>
>> On 25 Aug 2020, at 18:42, Julien Grall <julien@xen.org> wrote:
>>
>> From: Julien Grall <jgrall@amazon.com>
>>
>> Commit 858c0be8c2fa "xen/arm: Enable CPU Erratum 1165522 for Neoverse"
>> added a new erratum but forgot to update silicon-errata.txt.
>>
>> Update the file accordingly to keep track of errata workaround in Xen.
>
> Oh i should have done that as part of my patch.
> Nice catch.
>
>>
>> Signed-off-by: Julien Grall <jgrall@amazon.com>
>> Cc: Bertrand Marquis <bertrand.marquis@arm.com>
>> ---
>> docs/misc/arm/silicon-errata.txt | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/docs/misc/arm/silicon-errata.txt b/docs/misc/arm/silicon-errata.txt
>> index 11e5a9dcec2c..ee070e723a5f 100644
>> --- a/docs/misc/arm/silicon-errata.txt
>> +++ b/docs/misc/arm/silicon-errata.txt
>> @@ -51,4 +51,5 @@ stable hypervisors.
>> | ARM | Cortex-A57 | #1319537 | N/A |
>> | ARM | Cortex-A72 | #1319367 | N/A |
>> | ARM | Cortex-A76 | #1165522 | N/A |
>> +| ARM | Neoverse | #1165522 | N/A
>
> Should be Neoverse-N1 here (E1 for example is not impacted by this errata)
>
> Cheers
> Bertrand
>
>> | ARM | MMU-500 | #842869 | N/A |
>> --
>> 2.17.1
>>
>
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
Sorry forgot to remove the disclaimer (again).
Bertrand
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
2020-08-26 7:46 ` Bertrand Marquis
2020-08-26 7:55 ` Bertrand Marquis
@ 2020-08-26 9:37 ` Julien Grall
2020-08-26 10:15 ` Bertrand Marquis
1 sibling, 1 reply; 6+ messages in thread
From: Julien Grall @ 2020-08-26 9:37 UTC (permalink / raw)
To: Bertrand Marquis
Cc: Xen-devel, Julien Grall, Stefano Stabellini, Volodymyr Babchuk
On 26/08/2020 08:46, Bertrand Marquis wrote:
>
>
>> On 25 Aug 2020, at 18:42, Julien Grall <julien@xen.org> wrote:
>>
>> From: Julien Grall <jgrall@amazon.com>
>>
>> Commit 858c0be8c2fa "xen/arm: Enable CPU Erratum 1165522 for Neoverse"
>> added a new erratum but forgot to update silicon-errata.txt.
>>
>> Update the file accordingly to keep track of errata workaround in Xen.
>
> Oh i should have done that as part of my patch.
> Nice catch.
>
>>
>> Signed-off-by: Julien Grall <jgrall@amazon.com>
>> Cc: Bertrand Marquis <bertrand.marquis@arm.com>
>> ---
>> docs/misc/arm/silicon-errata.txt | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/docs/misc/arm/silicon-errata.txt b/docs/misc/arm/silicon-errata.txt
>> index 11e5a9dcec2c..ee070e723a5f 100644
>> --- a/docs/misc/arm/silicon-errata.txt
>> +++ b/docs/misc/arm/silicon-errata.txt
>> @@ -51,4 +51,5 @@ stable hypervisors.
>> | ARM | Cortex-A57 | #1319537 | N/A |
>> | ARM | Cortex-A72 | #1319367 | N/A |
>> | ARM | Cortex-A76 | #1165522 | N/A |
>> +| ARM | Neoverse | #1165522 | N/A
>
> Should be Neoverse-N1 here (E1 for example is not impacted by this errata)
Ah, right. Would you be happy if I do the change on commit?
Cheers,
--
Julien Grall
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
2020-08-26 9:37 ` Julien Grall
@ 2020-08-26 10:15 ` Bertrand Marquis
2020-08-26 16:17 ` Stefano Stabellini
0 siblings, 1 reply; 6+ messages in thread
From: Bertrand Marquis @ 2020-08-26 10:15 UTC (permalink / raw)
To: Julien Grall
Cc: Xen-devel, Julien Grall, Stefano Stabellini, Volodymyr Babchuk
> On 26 Aug 2020, at 10:37, Julien Grall <julien@xen.org> wrote:
>
>
>
> On 26/08/2020 08:46, Bertrand Marquis wrote:
>>> On 25 Aug 2020, at 18:42, Julien Grall <julien@xen.org> wrote:
>>>
>>> From: Julien Grall <jgrall@amazon.com>
>>>
>>> Commit 858c0be8c2fa "xen/arm: Enable CPU Erratum 1165522 for Neoverse"
>>> added a new erratum but forgot to update silicon-errata.txt.
>>>
>>> Update the file accordingly to keep track of errata workaround in Xen.
>> Oh i should have done that as part of my patch.
>> Nice catch.
>>>
>>> Signed-off-by: Julien Grall <jgrall@amazon.com>
>>> Cc: Bertrand Marquis <bertrand.marquis@arm.com>
>>> ---
>>> docs/misc/arm/silicon-errata.txt | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/docs/misc/arm/silicon-errata.txt b/docs/misc/arm/silicon-errata.txt
>>> index 11e5a9dcec2c..ee070e723a5f 100644
>>> --- a/docs/misc/arm/silicon-errata.txt
>>> +++ b/docs/misc/arm/silicon-errata.txt
>>> @@ -51,4 +51,5 @@ stable hypervisors.
>>> | ARM | Cortex-A57 | #1319537 | N/A |
>>> | ARM | Cortex-A72 | #1319367 | N/A |
>>> | ARM | Cortex-A76 | #1165522 | N/A |
>>> +| ARM | Neoverse | #1165522 | N/A
>> Should be Neoverse-N1 here (E1 for example is not impacted by this errata)
>
> Ah, right. Would you be happy if I do the change on commit?
Sure then you have my:
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
Cheers
Bertrand
>
> Cheers,
>
> --
> Julien Grall
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
2020-08-26 10:15 ` Bertrand Marquis
@ 2020-08-26 16:17 ` Stefano Stabellini
0 siblings, 0 replies; 6+ messages in thread
From: Stefano Stabellini @ 2020-08-26 16:17 UTC (permalink / raw)
To: Bertrand Marquis
Cc: Julien Grall, Xen-devel, Julien Grall, Stefano Stabellini,
Volodymyr Babchuk
On Wed, 26 Aug 2020, Bertrand Marquis wrote:
> > On 26/08/2020 08:46, Bertrand Marquis wrote:
> >>> On 25 Aug 2020, at 18:42, Julien Grall <julien@xen.org> wrote:
> >>>
> >>> From: Julien Grall <jgrall@amazon.com>
> >>>
> >>> Commit 858c0be8c2fa "xen/arm: Enable CPU Erratum 1165522 for Neoverse"
> >>> added a new erratum but forgot to update silicon-errata.txt.
> >>>
> >>> Update the file accordingly to keep track of errata workaround in Xen.
> >> Oh i should have done that as part of my patch.
> >> Nice catch.
> >>>
> >>> Signed-off-by: Julien Grall <jgrall@amazon.com>
> >>> Cc: Bertrand Marquis <bertrand.marquis@arm.com>
> >>> ---
> >>> docs/misc/arm/silicon-errata.txt | 1 +
> >>> 1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/docs/misc/arm/silicon-errata.txt b/docs/misc/arm/silicon-errata.txt
> >>> index 11e5a9dcec2c..ee070e723a5f 100644
> >>> --- a/docs/misc/arm/silicon-errata.txt
> >>> +++ b/docs/misc/arm/silicon-errata.txt
> >>> @@ -51,4 +51,5 @@ stable hypervisors.
> >>> | ARM | Cortex-A57 | #1319537 | N/A |
> >>> | ARM | Cortex-A72 | #1319367 | N/A |
> >>> | ARM | Cortex-A76 | #1165522 | N/A |
> >>> +| ARM | Neoverse | #1165522 | N/A
> >> Should be Neoverse-N1 here (E1 for example is not impacted by this errata)
> >
> > Ah, right. Would you be happy if I do the change on commit?
>
> Sure then you have my:
> Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-08-26 16:18 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-25 17:42 [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum Julien Grall
2020-08-26 7:46 ` Bertrand Marquis
2020-08-26 7:55 ` Bertrand Marquis
2020-08-26 9:37 ` Julien Grall
2020-08-26 10:15 ` Bertrand Marquis
2020-08-26 16:17 ` Stefano Stabellini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.