Re: [edk2-discuss] [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-18 Thread Ni, Ray
...@apple.com > Cc: Wang, Jian J ; Wu, Hao A ; > Gao, Zhichao ; > Kinney, Michael D ; Wang, Sunny (HPS SW) > > Subject: RE: [edk2-discuss] [edk2-devel] [PATCH] Support skipping automatic > BM enumeration > > Sorry for the delay. Somehow I didn't catch the follow-up ema

Re: [edk2-discuss] [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-17 Thread Wang, Sunny (HPS SW)
day, December 11, 2019 4:46 AM To: Ni, Ray ; devel@edk2.groups.io; Laszlo Ersek ; af...@apple.com; disc...@edk2.groups.io Cc: Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Can we discuss

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-17 Thread Ashish Singhal
, December 12, 2019 10:52 AM To: Ni, Ray ; devel@edk2.groups.io ; Laszlo Ersek ; af...@apple.com ; disc...@edk2.groups.io Cc: Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Thanks for the

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-12 Thread Jeff Brasen
devel@edk2.groups.io; Laszlo Ersek ; af...@apple.com; disc...@edk2.groups.io Cc: Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Can we discuss this at the design meeting this week (12/12)? Tha

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-11 Thread Ni, Ray
inghal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Can we discuss this at the design meeting this week (12/12)? Thanks, Jeff From: Jeff Brasen Sent: Thursday, November

Re: [edk2-discuss] [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-11 Thread Wang, Sunny (HPS SW)
J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Yes, I think that would be good. Summarizing everything in this thread Problem: Platform needs to customize the boot options, this can be done for normal boot but

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-10 Thread Jeff Brasen
, Zhichao ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Yes, I think that would be good. Summarizing everything in this thread Problem: Platform needs to customize the boot options, this can be done for normal boot but the UiApp calls

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-14 Thread Jeff Brasen
Sent: Wednesday, November 13, 2019 7:09 PM To: devel@edk2.groups.io; Jeff Brasen ; Laszlo Ersek ; af...@apple.com Cc: Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Jeff, I think it's a

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-13 Thread Ni, Ray
o On Behalf Of Jeff Brasen Sent: Thursday, November 14, 2019 2:43 AM To: Ni, Ray ; devel@edk2.groups.io; Laszlo Ersek ; af...@apple.com Cc: Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Thi

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-13 Thread Jeff Brasen
an J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration I am not sure a PCD would work (unless I am missing something) We do want to do a connect all and re-enumerate in UiApp but we need the platform code to be involve

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-11 Thread Jeff Brasen
@edk2.groups.io ; Jeff Brasen ; Laszlo Ersek ; af...@apple.com Cc: Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Jeff, If adding a PCD to control UiApp can meet the real needs, I

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-11 Thread Ni, Ray
.groups.io; af...@apple.com Cc: Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration If we are concerned about deploying this and breaking builds we could do this via a new protocol instead. In that

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-11 Thread Jeff Brasen
ael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration On 11/07/19 18:46, Jeff Brasen wrote: > Fixing UiApp seems reasonable, I do think we would want a hook to the > platform library in here as the enumeration that occurs in the UiApp is > intended to do a full

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-08 Thread Laszlo Ersek
@apple.com > Cc: Ashish Singhal ; Laszlo Ersek > ; Wang, Jian J ; Wu, Hao A > ; Gao, Zhichao ; Kinney, Michael D > > Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration > > I treat the issue in this way: > > 1. Platform Boot Manager lib

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-07 Thread Jeff Brasen
, November 7, 2019 12:21 AM To: devel@edk2.groups.io; Jeff Brasen ; af...@apple.com Cc: Ashish Singhal ; Laszlo Ersek ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration I treat the issue in this way: 1

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Ni, Ray
Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration The issue is there are some auto created options we do not want on our platform. Get Outlook for Android<https://aka.ms/ghei36> From: Ni, Ray mailto:ray...@intel.com&g

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Jeff Brasen
Ashish Singhal ; Laszlo Ersek ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Jeff, RefreshAllBootOption() only modifies/creates the auto-created boot options. For the boot options created by platform boot m

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Ni, Ray
lto:hao.a...@intel.com>>; Gao, Zhichao mailto:zhichao@intel.com>>; Kinney, Michael D mailto:michael.d.kin...@intel.com>> Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration On Nov 5, 2019, at 7:34 PM, Jeff Brasen mailto:jbra...@nvidia.com>> w

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Ni, Ray
Sent: Thursday, November 7, 2019 12:13 PM To: af...@apple.com; Ni, Ray Cc: devel@edk2.groups.io; Ashish Singhal ; Laszlo Ersek ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration As the suggestions below made

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Jeff Brasen
9:20 AM To: Ni, Ray Cc: devel@edk2.groups.io; Jeff Brasen ; Ashish Singhal ; Laszlo Ersek ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Mike Kinney Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Ray, Is there an obvious hook point we could point Jeff and Ashish at

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Laszlo Ersek
On 11/06/19 17:15, Andrew Fish wrote: > > >> On Nov 6, 2019, at 3:56 AM, Laszlo Ersek wrote: >> >> On 11/06/19 00:19, Jeff Brasen wrote: >>> Wouldn't having a variable that we create and delete on every boot put >>> unnecessary stress on the SPI-NOR that the variable store lives on? >> >> Yes,

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Andrew Fish via Groups.Io
hal ; Laszlo Ersek > ; Ni, Ray ; Wang, Jian J > ; Wu, Hao A ; Gao, Zhichao > ; Kinney, Michael D > Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration > > > > > On Nov 5, 2019, at 7:34 PM, Jeff Brasen <mailto:jbra...@nvidia.com>

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Andrew Fish via Groups.Io
> On Nov 6, 2019, at 3:56 AM, Laszlo Ersek wrote: > > On 11/06/19 00:19, Jeff Brasen wrote: >> Wouldn't having a variable that we create and delete on every boot put >> unnecessary stress on the SPI-NOR that the variable store lives on? > > Yes, it most likely would. Should we write a BZ fo

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-06 Thread Laszlo Ersek
On 11/06/19 00:19, Jeff Brasen wrote: > Wouldn't having a variable that we create and delete on every boot put > unnecessary stress on the SPI-NOR that the variable store lives on? Yes, it most likely would. [...] Thanks Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Ni, Ray
rsek ; Ni, Ray ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao ; Kinney, Michael D Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration On Nov 5, 2019, at 7:34 PM, Jeff Brasen mailto:jbra...@nvidia.com>> wrote: Wouldn't having a variable that we create and del

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Andrew Fish via Groups.Io
> On Nov 5, 2019, at 7:34 PM, Jeff Brasen wrote: > > > Wouldn't having a variable that we create and delete on every boot put > unnecessary stress on the SPI-NOR that the variable store lives on? > What about the alternative approach where we allow the platform code to > modify the attribute

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Jeff Brasen
Wouldn't having a variable that we create and delete on every boot put unnecessary stress on the SPI-NOR that the variable store lives on? What about the alternative approach where we allow the platform code to modify the attributes of the auto created variable to disable it with hidden/!active

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Ashish Singhal
ng, Jian J ; Wu, Hao A ; Gao, Zhichao ; Mike Kinney Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration On 11/05/19 19:00, Ashish Singhal wrote: > I agree that I can use the platform boot manager protocol to modify > the boot variable and give the custom optional data

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Andrew Fish via Groups.Io
> On Nov 5, 2019, at 5:19 PM, Jeff Brasen wrote: > > Wouldn't having a variable that we create and delete on every boot put > unnecessary stress on the SPI-NOR that the variable store lives on? > What about the alternative approach where we allow the platform code to > modify the attributes o

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Jeff Brasen
Wouldn't having a variable that we create and delete on every boot put unnecessary stress on the SPI-NOR that the variable store lives on? What about the alternative approach where we allow the platform code to modify the attributes of the auto created variable to disable it with hidden/!active

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Laszlo Ersek
On 11/05/19 19:00, Ashish Singhal wrote: > I agree that I can use the platform boot manager protocol to modify > the boot variable and give the custom optional data I need, but that > would not stop UefiBootManagerLib from creating another boot option > with mBmAutoCreateBootOptionGuid as the opti

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Ashish Singhal
Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration > On Nov 5, 2019, at 3:54 AM, Laszlo Ersek wrote: > > On 11/05/19 07:15, Andrew Fish via Groups.Io wrote: > >> You could also edit any existing variables that point to your Load File to >> m

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Andrew Fish via Groups.Io
> On Nov 5, 2019, at 3:54 AM, Laszlo Ersek wrote: > > On 11/05/19 07:15, Andrew Fish via Groups.Io wrote: > >> You could also edit any existing variables that point to your Load File to >> make sure they follow the rules you care about? >> >> It is legal from an UEFI spec point of view for

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Laszlo Ersek
On 11/05/19 07:15, Andrew Fish via Groups.Io wrote: > You could also edit any existing variables that point to your Load File to > make sure they follow the rules you care about? > > It is legal from an UEFI spec point of view for a platform to edit the nvram > boot variables based on platform

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-05 Thread Laszlo Ersek
On 11/05/19 04:24, Ashish Singhal wrote: > Hi Ray, > > I did not see any example with which one can tell BDS not to > enumerate a boot option. Can you please point me to an example where > a protocol (Block IO, Simple FS or Load File) does not get enumerated > as a boot option? (I'm going to stat

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Andrew Fish via Groups.Io
szlo Ersek > ; Wang, Jian J ; Wu, Hao A > ; Gao, Zhichao ; Mike Kinney > > Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration > > Ashish, > > Just to be clear on your definition of enumerate. In this case it means the > Boot Manager lib that is just one

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Ashish Singhal
J ; Wu, Hao A ; Gao, Zhichao ; Mike Kinney Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration Ashish, Just to be clear on your definition of enumerate. In this case it means the Boot Manager lib that is just one of may implementations a platform could use to manage

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Andrew Fish via Groups.Io
.@redhat.com>>; > devel@edk2.groups.io <mailto:devel@edk2.groups.io>; Ashish Singhal > mailto:ashishsin...@nvidia.com>>; Wang, Jian J > mailto:jian.j.w...@intel.com>>; Wu, Hao A > mailto:hao.a...@intel.com>>; Gao, Zhichao > mailto:zhichao@intel.com>

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Ashish Singhal
; Wu, Hao A ; Gao, Zhichao ; Mike Kinney Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration On Nov 4, 2019, at 9:24 PM, Ashish Singhal mailto:ashishsin...@nvidia.com>> wrote: Hi Ray, I did not see any example with which one can tell BDS not to enumerate a boot

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Andrew Fish via Groups.Io
...@intel.com>>; Gao, Zhichao > mailto:zhichao....@intel.com>> > Cc: Kinney, Michael D <mailto:michael.d.kin...@intel.com>> > Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration > > + Mike > > With the UEFI driver model, if a d

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Ashish Singhal
: Monday, November 4, 2019 7:42 PM To: Laszlo Ersek ; devel@edk2.groups.io; Ashish Singhal ; Wang, Jian J ; Wu, Hao A ; Gao, Zhichao Cc: Kinney, Michael D Subject: RE: [edk2-devel] [PATCH] Support skipping automatic BM enumeration + Mike With the UEFI driver model, if a device is not needed by

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Ni, Ray
o; ashishsin...@nvidia.com; Wang, Jian J > ; Wu, Hao A ; Ni, Ray > ; Gao, Zhichao > Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM > enumeration > > +Ray, +Zhichao > > (although, the actual patch seems to be missing from the posting) > > Thanks >

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-04 Thread Ashish Singhal
Ersek > Sent: Friday, November 1, 2019 3:42 PM > To: Ashish Singhal ; devel@edk2.groups.io; > jian.j.w...@intel.com; hao.a...@intel.com; Ray Ni ; > Zhichao Gao > Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM > enumeration > > On 10/31/19 13:42, Ash

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-01 Thread Laszlo Ersek
M > To: Ashish Singhal ; devel@edk2.groups.io; > jian.j.w...@intel.com; hao.a...@intel.com; Ray Ni ; Zhichao > Gao > Subject: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration > > On 10/31/19 13:42, Ashish Singhal wrote: >> Hello Laszlo, >> >> This

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-01 Thread Ashish Singhal
: Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration On 10/31/19 13:42, Ashish Singhal wrote: > Hello Laszlo, > > This is the cover letter. The patch was also submitted. Can you please state the groups.io permalink for the posted patch, and/or the Message-Id header for t

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-01 Thread Laszlo Ersek
On 10/31/19 13:42, Ashish Singhal wrote: > Hello Laszlo, > > This is the cover letter. The patch was also submitted. Can you please state the groups.io permalink for the posted patch, and/or the Message-Id header for the posted patch? > Please let me know if you are unable to find it and I'll re

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-10-31 Thread Laszlo Ersek
+Ray, +Zhichao (although, the actual patch seems to be missing from the posting) Thanks Laszlo On 10/30/19 04:47, Ashish Singhal wrote: > Right now, any and every handle with a BlockIO or SimpleFileSystem or > LoadFile protocol installed on the system is used to enumerate BM > automatically. The

[edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-10-29 Thread Ashish Singhal
Right now, any and every handle with a BlockIO or SimpleFileSystem or LoadFile protocol installed on the system is used to enumerate BM automatically. There may be cases where on a platform, some of these are not desirable to be enumerated automatically. This patch adds support for skipping this au