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).
Reviewed-by: Julien Grall <jgr...@amazon.com>
}
return 0;
--
Julien Grall