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 emai

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

2019-12-17 Thread Wang, Sunny (HPS SW)
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 this at the design meeting this wee

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

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

2019-12-12 Thread Jeff Brasen
.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)? Thanks, Jeff

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

2019-12-11 Thread Ni, Ray
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 14, 2019 10:04 AM To

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

2019-12-11 Thread Wang, Sunny (HPS SW)
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 the UiApp calls

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
: 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 good topic

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

2019-11-13 Thread Ni, Ray
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 Thinking

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

2019-11-13 Thread Jeff Brasen
; 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 involved in that process. Than

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

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

2019-11-11 Thread Jeff Brasen
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 en

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
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>> Sen

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 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

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

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

2019-11-05 Thread Ni, Ray
, 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 delete on every bo

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

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
; 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 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

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 >&g

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

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

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

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
om>>; > 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>&g

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
nt: 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, Ashish Singhal wro

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

2019-11-01 Thread Laszlo Ersek
sh 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 is the cov

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: [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.

[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