On Tue, Nov 9, 2021 at 5:55 PM Tom Rini trini@konsulko.com wrote:
On Tue, Nov 09, 2021 at 11:19:01AM +0100, Heinrich Schuchardt wrote:
The UEFI specification requires for ExitBootServices() that "the boot services watchdog timer is disabled". We already disable the software watchdog. We should additionally disable the hardware watchdogs.
Reported-by: Andre Przywara andre.przywara@arm.com Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com
Let me start by saying thank you for bringing this up with UEFI folks as well. To be clear, for right now I would much rather see U-Boot continue to be non-complaint with UEFI in this regard and assume that a running watchdog will be able to be handled by the running OS (which tends to be the case, but not always as sunxi has just shown) than to attempt to be complaint with the spec as it stands now as I am hopeful that we can get this case handled in a way that matches long standing industry practice.
I’m also not keen to see the watchdogs disabled, even if it does make platforms technically non-compliant with UEFI. Disabling the watchdog does not make sense for throw devices using U-Boot get used.
I’ll put this on the agenda for the next EBBR biweekly
g.
-- Tom