From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [205.139.110.61]) by mx.groups.io with SMTP id smtpd.web11.15481.1583345516409895957 for ; Wed, 04 Mar 2020 10:11:56 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=WVCOwbtL; spf=pass (domain: redhat.com, ip: 205.139.110.61, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1583345515; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=K5JRahopCFbQrLSMEy5PeCBNdKk5NuGJS3XvehAJYyY=; b=WVCOwbtLsbZDoyMNWN4AmVEXQCLvN3aRvlKz3Qr/qYwDFqWiWAyQZQ5mqD7vZvB72C3kFd kZSZBv3SLVeE0pglGU+tlUwENnazTlwkqjg1eL8vp9+WQ6JqAGC1k7qIMGRBUit0ay5q6n f7pE8fa/qh8pU0tipS50H3UC4/9X8Ao= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-110-9feOzvtWPWimW73QFFEEJA-1; Wed, 04 Mar 2020 13:11:50 -0500 X-MC-Unique: 9feOzvtWPWimW73QFFEEJA-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 0508718B5FA0; Wed, 4 Mar 2020 18:11:49 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-117-59.ams2.redhat.com [10.36.117.59]) by smtp.corp.redhat.com (Postfix) with ESMTP id B8E4191D74; Wed, 4 Mar 2020 18:11:47 +0000 (UTC) Subject: Re: [PATCH 0/2] OvmfPkg, ArmVirtPkg: sync Timeout with PcdPlatformBootTimeOut To: =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , edk2-devel-groups-io Cc: Ard Biesheuvel , Jordan Justen , Leif Lindholm References: <20200304094413.19462-1-lersek@redhat.com> <48c79bd9-5961-867e-f081-a12f60b3e095@redhat.com> From: "Laszlo Ersek" Message-ID: <886d69ce-f419-e593-fb7d-31f9df41f7d4@redhat.com> Date: Wed, 4 Mar 2020 19:11:46 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <48c79bd9-5961-867e-f081-a12f60b3e095@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 03/04/20 16:50, Philippe Mathieu-Daud=C3=A9 wrote: > On 3/4/20 10:44 AM, Laszlo Ersek wrote: >> Repo:=C2=A0=C2=A0 https://pagure.io/lersek/edk2.git >> Branch: timeout_var >> >> In the PlatformBootManagerLib instances, set the Timeout global variable >> to the same value as PcdPlatformBootTimeOut. This way the "setvar" >> command in the UEFI shell, and the "efibootmgr" command in a Linux >> guest, can report the front page timeout that was requested on the QEMU >> command line (see GetFrontPageTimeoutFromQemu() in >> "OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.c"). >> >> A DEBUG_VERBOSE message is logged on success too, for our QE team's >> sake. >> >> Cc: Ard Biesheuvel >> Cc: Jordan Justen >> Cc: Leif Lindholm >> Cc: Philippe Mathieu-Daud=C3=A9 >> >> Thanks >> Laszlo >> >> Laszlo Ersek (2): >> =C2=A0=C2=A0 OvmfPkg/PlatformBootManagerLib: sync Timeout with >> =C2=A0=C2=A0=C2=A0=C2=A0 PcdPlatformBootTimeOut >> =C2=A0=C2=A0 ArmVirtPkg/PlatformBootManagerLib: sync Timeout with >> =C2=A0=C2=A0=C2=A0=C2=A0 PcdPlatformBootTimeOut >> >> =C2=A0 ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 >> | 27 ++++++++++++++++++-- >> =C2=A0 ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.= inf >> |=C2=A0 1 + >> =C2=A0 OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0 >> | 26 +++++++++++++++++-- >> =C2=A0 OvmfPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf= =C2=A0=C2=A0=C2=A0 >> |=C2=A0 2 ++ >> =C2=A0 4 files changed, 52 insertions(+), 4 deletions(-) >> >=20 > Series: > Reviewed-by: Philippe Mathieu-Daude >=20 Thank you! I plan to push this tomorrow, with your R-b, and Ard's. Thanks Laszlo