> On 7 Oct 2020, at 14:00, Julien Grall <jul...@xen.org> wrote:
>
> Hi Bertrand,
>
> On 07/10/2020 12:05, Bertrand Marquis wrote:
>> Fix enable_smccc_arch_workaround_1 to only print the warning asking to
>> update the firmware once.
>> Signed-off-by: Bertrand Marquis <bertrand.marq...@arm.com>
>> ---
>> xen/arch/arm/cpuerrata.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> diff --git a/xen/arch/arm/cpuerrata.c b/xen/arch/arm/cpuerrata.c
>> index 6c09017515..0c63dfa779 100644
>> --- a/xen/arch/arm/cpuerrata.c
>> +++ b/xen/arch/arm/cpuerrata.c
>> @@ -187,7 +187,7 @@ warn:
>> ASSERT(system_state < SYS_STATE_active);
>> warning_add("No support for ARM_SMCCC_ARCH_WORKAROUND_1.\n"
>> "Please update your firmware.\n");
>> - warned = false;
>> + warned = true;
>
> Thanks for spotting it. It looks like I introduced this bug in commit
> 976319fa3de7f98b558c87b350699fffc278effc "xen/arm64: Kill PSCI_GET_VERSION as
> a variant-2 workaround".
>
> I would suggest to add a fixes tag (can be done on commit).
If you can do it during the commit that works for me.
>
> Reviewed-by: Julien Grall <jgr...@amazon.com>
Thanks
Cheers
Bertrand