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 A8C5CC433EF for ; Tue, 9 Nov 2021 23:46:03 +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 A018E611BF for ; Tue, 9 Nov 2021 23:46:02 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org A018E611BF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=secretlab.ca 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 E033C836CB; Wed, 10 Nov 2021 00:45:59 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=secretlab.ca Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=secretlab.ca header.i=@secretlab.ca header.b="ZfC1zEC1"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 5A88D836CC; Wed, 10 Nov 2021 00:45:57 +0100 (CET) Received: from mail-yb1-xb30.google.com (mail-yb1-xb30.google.com [IPv6:2607:f8b0:4864:20::b30]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 39EA2836BD for ; Wed, 10 Nov 2021 00:45:53 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=secretlab.ca Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=glikely@secretlab.ca Received: by mail-yb1-xb30.google.com with SMTP id s186so1597305yba.12 for ; Tue, 09 Nov 2021 15:45:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=secretlab.ca; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=AYOGdhOE3I6PXDFq0idQduxyGzf7pw3fBBpsS1EkvLI=; b=ZfC1zEC1H3DVr3xXEbPmW6OZYfBZLXw1HJ0Qo759BlB638wYY5LQvvgJlmlQWHVya1 YvdnD7UqAxD3Y1Lne7MpBGwyfYFh6wZt4ZUf0Mhiii2vejolBV/7NQoGdkeArg4h/HGw /khSY5NejqxuojMknquC8YShXA3+0LfxdXXSB/R4+6l9wUoPNLCnYi6/d4eT8BmO2cII Ssfw6kmNBfv8iAwN2YdAo30TgQoX56u1ZMEwyrryy1xdvMYIXiL8dzPWYPY93YGXuy8l 9pQDZnXEsW2abstHRHeH3+uJLMYQmTQ0ubnsuYKEgnh1uj07pXY3Wj7PBMjqqiCE+9WW 7XcA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=AYOGdhOE3I6PXDFq0idQduxyGzf7pw3fBBpsS1EkvLI=; b=dxCSWm+1HP+47y5cFcQLwX1IG+1QzqtCBQslLaOVqnO8NpQIUfUdo/IH/K8n9ugWD1 sFx1258Y/BOVPaA70QD+HdIWSukWtEZ2aSNBXLNledN3ahaEv91tLUxrs0AtfV4TzLtc KclCdDoluWI8XHhrPLYSihxJTFXFY7YKNUhN9SI7of68z8aUk3E66BYAaLj8KSPTD6D7 u6/zEoNaDPJ+oqAQOf8WXRvbmmUAuH0MALNG5MEDcNW2hk8KH2lCTWX2b8G6sd40agC2 uDbVsGzUFQ09hmTw/o8GraiU6GdCF0SSknr+GFwSv1ZcqGIn+IL6c/7a/l10j+s3w+R0 TMqA== X-Gm-Message-State: AOAM533jaVfYrDfluEeBNmdv62Q3nQ/ugsuC7ZbUySBIRWCzkOJHM1Lq QS1E2wmz+MS4q/Ak/KqV23FYQH+e66vyTWvrK3EhQA== X-Google-Smtp-Source: ABdhPJzc2GnJ/PTwf4gMafJ9V2CY0FW+mKEO7IjZOBSU6MmuXzh7CcsKVRBsUUW1TN/luQEh9GW0QzSKBLhMzruWNJU= X-Received: by 2002:a25:48c9:: with SMTP id v192mr12921082yba.50.1636501551794; Tue, 09 Nov 2021 15:45:51 -0800 (PST) MIME-Version: 1.0 References: <20211109101901.24015-1-heinrich.schuchardt@canonical.com> <20211109101901.24015-2-heinrich.schuchardt@canonical.com> <20211109175526.GW24579@bill-the-cat> In-Reply-To: <20211109175526.GW24579@bill-the-cat> From: Grant Likely Date: Tue, 9 Nov 2021 23:45:40 +0000 Message-ID: Subject: Re: [PATCH 1/1] efi_loader: stop watchdogs in ExitBootServices() To: Tom Rini Cc: Heinrich Schuchardt , u-boot@lists.denx.de, Andre Przywara , Heinrich Schuchardt , Alexander Graf Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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, Nov 9, 2021 at 5:55 PM 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. I=E2=80=99m also not keen to see the watchdogs disabled, even if it does ma= ke platforms technically non-compliant with UEFI. Disabling the watchdog does not make sense for throw devices using U-Boot get used. I=E2=80=99ll put this on the agenda for the next EBBR biweekly g. > > -- > Tom