Re: [PATCH 0/2] OvmfPkg, ArmVirtPkg: sync Timeout with PcdPlatformBootTimeOut

["Laszlo Ersek" <lersek@redhat.com>]

On 03/04/20 16:50, Philippe Mathieu-Daudé wrote:
> On 3/4/20 10:44 AM, Laszlo Ersek wrote:
>> Repo:   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 <ard.biesheuvel@linaro.org>
>> Cc: Jordan Justen <jordan.l.justen@intel.com>
>> Cc: Leif Lindholm <leif@nuviainc.com>
>> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
>>
>> Thanks
>> Laszlo
>>
>> Laszlo Ersek (2):
>>    OvmfPkg/PlatformBootManagerLib: sync Timeout with
>>      PcdPlatformBootTimeOut
>>    ArmVirtPkg/PlatformBootManagerLib: sync Timeout with
>>      PcdPlatformBootTimeOut
>>
>>   ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c              
>> | 27 ++++++++++++++++++--
>>   ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
>> |  1 +
>>   OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c                
>> | 26 +++++++++++++++++--
>>   OvmfPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf   
>> |  2 ++
>>   4 files changed, 52 insertions(+), 4 deletions(-)
>>
> 
> Series:
> Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
> 

Thank you!

I plan to push this tomorrow, with your R-b, and Ard's.

Thanks
Laszlo