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

[Bertrand Marquis <Bertrand.Marquis@xxxxxxx>]

> On 26 Aug 2020, at 10:37, Julien Grall <julien@xxxxxxx> wrote:
>
>
>
> On 26/08/2020 08:46, Bertrand Marquis wrote:
>>> On 25 Aug 2020, at 18:42, Julien Grall <julien@xxxxxxx> wrote:
>>>
>>> From: Julien Grall <jgrall@xxxxxxxxxx>
>>>
>>> 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@xxxxxxxxxx>
>>> Cc: Bertrand Marquis <bertrand.marquis@xxxxxxx>
>>> ---
>>> 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@xxxxxxx>

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.