From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D9B21C433EF for ; Tue, 9 Nov 2021 21:48:46 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id DE7BD61178 for ; Tue, 9 Nov 2021 21:48:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org DE7BD61178 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 5F3D1836C0; Tue, 9 Nov 2021 22:48:43 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: by phobos.denx.de (Postfix, from userid 109) id 64F4A836BE; Tue, 9 Nov 2021 22:48:41 +0100 (CET) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by phobos.denx.de (Postfix) with ESMTP id 47A7B836BE for ; Tue, 9 Nov 2021 22:48:37 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=andre.przywara@arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 7F6C72B; Tue, 9 Nov 2021 13:48:36 -0800 (PST) Received: from slackpad.fritz.box (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id AEC6B3F7F5; Tue, 9 Nov 2021 13:48:35 -0800 (PST) Date: Tue, 9 Nov 2021 21:47:06 +0000 From: Andre Przywara To: Tom Rini Cc: Heinrich Schuchardt , u-boot@lists.denx.de, Heinrich Schuchardt , Alexander Graf Subject: Re: [PATCH 1/1] efi_loader: stop watchdogs in ExitBootServices() Message-ID: <20211109214706.579ddb1e@slackpad.fritz.box> In-Reply-To: <20211109181828.GX24579@bill-the-cat> References: <20211109101901.24015-1-heinrich.schuchardt@canonical.com> <20211109101901.24015-2-heinrich.schuchardt@canonical.com> <20211109175526.GW24579@bill-the-cat> <20211109181828.GX24579@bill-the-cat> Organization: Arm Ltd. X-Mailer: Claws Mail 3.17.1 (GTK+ 2.24.31; x86_64-slackware-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean On Tue, 9 Nov 2021 13:18:28 -0500 Tom Rini wrote: > On Tue, Nov 09, 2021 at 07:15:10PM +0100, Heinrich Schuchardt wrote: > > On 11/9/21 18:55, Tom Rini 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 > > > > Signed-off-by: Heinrich Schuchardt > > > > > > 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. > > > > > > > We have either merge this patch or > > > > [1/1] watchdog: don't autostart watchdog on Sunxi boards > > https://patchwork.ozlabs.org/project/uboot/patch/20211105183431.117221-1-heinrich.schuchardt@canonical.com/ > > > > or we will be breaking boot processes that have been running up to now. > > > > As Sunxi watchdogs were only enabled by a recent patch disabling autostart > > should not cause any harm. > > Yes, I was expecting a PR from Andre in my inbox this morning with your This was my plan as well, until I opened up the UEFI spec ;-) Also I didn't see any problems so far in my - admittedly limited - casual testing (because I boot quickly and have the WDT compiled in). But I see it now when disabling the watchdog in the kernel, so will take the "don't autostart" patch anyway. If people have an embedded deployment where they rely on the watchdog, they can surely enable it before building U-Boot. Cheers, Andre > patch, as he had said he would apply it. I'm just assuming now I got a > bit ahead of myself with expecting it so quickly. >