From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from foss.arm.com (foss.arm.com [217.140.110.172])
 by mx.groups.io with SMTP id smtpd.web11.5413.1592393563393132655
 for <devel@edk2.groups.io>;
 Wed, 17 Jun 2020 04:32:43 -0700
Authentication-Results: mx.groups.io;
 dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@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 E21FB1042;
	Wed, 17 Jun 2020 04:32:41 -0700 (PDT)
Received: from [192.168.1.69] (unknown [172.31.20.19])
	by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id E5E133F71F;
	Wed, 17 Jun 2020 04:32:40 -0700 (PDT)
Subject: Re: [PATCH] ArmPkg/PlatformBootManagerLib: regenerate boot options on boot failure
To: Leif Lindholm <leif@nuviainc.com>
Cc: devel@edk2.groups.io, Pete Batard <pete@akeo.ie>,
 Andrei Warkentin <awarkentin@vmware.com>,
 Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
References: <20200616174834.1110310-1-ard.biesheuvel@arm.com>
 <20200617111235.GJ6739@vanye>
From: "Ard Biesheuvel" <ard.biesheuvel@arm.com>
Message-ID: <6a089a72-00cb-f8ce-16d7-50cefed9b47d@arm.com>
Date: Wed, 17 Jun 2020 13:32:36 +0200
User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:68.0) Gecko/20100101
 Thunderbird/68.9.0
MIME-Version: 1.0
In-Reply-To: <20200617111235.GJ6739@vanye>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit

On 6/17/20 1:12 PM, Leif Lindholm wrote:
> On Tue, Jun 16, 2020 at 19:48:34 +0200, Ard Biesheuvel wrote:
>> One of the side effects of the recent changes to PlatformBootManagerLib
>> changes to avoid connecting all devices on every boot is that we no
>> longer default to network boot on a virgin boot, but end up in the
>> UiApp menu. At this point, the autogenerated boot options that we used
>> to rely on will be instantiated too,
> 
> The passive voice is confusing me a bit here - who does the updating,
> and when specifically?
> 

Originally, the ArmPkg PlatformBmLib would always refresh all boot 
options, but now, only the UiApp does that upon entry, at which point 
your sitting in the menu idly, and so automated network boot no longer 
works.