1. Patchew
  2. Xen
  3. View series

[PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum

Julien Grall posted 1 patch 3 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/xen tags/patchew/20200825174208.11070-1-julien@xen.org
Maintainers: Julien Grall <julien@xen.org>, Stefano Stabellini <sstabellini@kernel.org>, Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>
docs/misc/arm/silicon-errata.txt | 1 +
1 file changed, 1 insertion(+)
[PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
Posted by Julien Grall 3 years, 8 months ago 
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
Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
Posted by Bertrand Marquis 3 years, 8 months ago 

> 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.
Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
Posted by Bertrand Marquis 3 years, 8 months ago 

> 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
Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
Posted by Julien Grall 3 years, 8 months ago 

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
Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
Posted by Bertrand Marquis 3 years, 8 months ago 

> 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.
Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum
Posted by Stefano Stabellini 3 years, 8 months ago 
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>

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.