From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Wed, 17 Jul 2019 05:59:41 -0700 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 5154A356F8; Wed, 17 Jul 2019 12:59:41 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-117-5.ams2.redhat.com [10.36.117.5]) by smtp.corp.redhat.com (Postfix) with ESMTP id AE8465C21A; Wed, 17 Jul 2019 12:59:38 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH 0/4] Add a pcd PcdBootManagerInBootOrder to control whether BootManager is in BootOrder To: devel@edk2.groups.io, ray.ni@intel.com, "Gao, Zhichao" Cc: "Wang, Jian J" , "Wu, Hao A" , "Zeng, Star" , "Gao, Liming" , Sean Brogan , Michael Turner , Bret Barkelew References: <20190717075008.22716-1-zhichao.gao@intel.com> <734D49CCEBEEF84792F5B80ED585239D5C23325A@SHSMSX104.ccr.corp.intel.com> From: "Laszlo Ersek" Message-ID: Date: Wed, 17 Jul 2019 14:59:32 +0200 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: <734D49CCEBEEF84792F5B80ED585239D5C23325A@SHSMSX104.ccr.corp.intel.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.30]); Wed, 17 Jul 2019 12:59:41 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 07/17/19 10:07, Ni, Ray wrote: > Zhichao, > > Firstly, patch 4/4 is a fix to patch 3/4, why not merge them into one patch? Do we have a policy to strictly cherry-pick the patches from MU? > > Secondly, the change that skips to add the #### to BootOrder may cause the Boot#### pointing to the BootManagerMenu be overwritten later. Reason is BmGetFreeOptionNumber() replies on BootOrder/BootNext to get a free option number. > > If the requirement is to let BootManagerMenuApp hide the BootManagerMenu, we could change the BootManagerMenuApp using a PCD to control. This would be safer, in my opinion, because those platforms that don't consume BootManagerMenuApp, would not depend on the new PCD at all. (Not sure if the proposal satisfies the requirement though.) Thanks! Laszlo > > Thanks, > Ray > >> -----Original Message----- >> From: Gao, Zhichao >> Sent: Wednesday, July 17, 2019 3:50 PM >> To: devel@edk2.groups.io >> Cc: Wang, Jian J ; Wu, Hao A ; >> Ni, Ray ; Zeng, Star ; Gao, Liming >> ; Sean Brogan ; >> Michael Turner ; Bret Barkelew >> >> Subject: [PATCH 0/4] Add a pcd PcdBootManagerInBootOrder to control >> whether BootManager is in BootOrder >> >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1979 >> >> Add a pcd to control whether the BootManager is in BootOrder variable. >> From a UI perspective, it never want to see a Front Page UI in the boot order >> list. So add a pcd PcdBootManagerInBootOrder to control it. If it is FALSE, the >> BootManager App wouldn't be added to BootOrder. And it wouldn't be >> shown in the BootManagerMenuApp. >> >> Notes: >> BootManager means the application with file guid PcdBootManagerMenuFile. >> It's default value is EEC25BDC-67F2-4D95-B1D5-F81B2039D11D, and users >> usually change the pcd to UiApp.inf's file guid. BootManagerMenuApp >> means uefi application with file guid EEC25BDC-67F2-4D95-B1D5- >> F81B2039D11D. >> >> Cc: Jian J Wang >> Cc: Hao A Wu >> Cc: Ray Ni >> Cc: Star Zeng >> Cc: Liming gao >> Cc: Sean Brogan >> Cc: Michael Turner >> Cc: Bret Barkelew >> Signed-off-by: Zhichao Gao >> >> Bret Barkelew (2): >> MdeModulePkg: Add PcdBootManagerInBootOrder to dec file >> MdeModulePkg: Use pcd to control if BootManager in BootOrder >> >> Zhichao Gao (2): >> MdeModulePkg: Add prompt and help string for >> PcdBootManagerInBootOrder >> MdeModulePkg: PcdBootManagerInBootOrder should only affect >> BootOrder >> >> .../Library/UefiBootManagerLib/BmBoot.c | 29 +++++++++++++++---- >> .../UefiBootManagerLib/UefiBootManagerLib.inf | 1 + >> MdeModulePkg/MdeModulePkg.dec | 6 ++++ >> MdeModulePkg/MdeModulePkg.uni | 6 ++++ >> 4 files changed, 36 insertions(+), 6 deletions(-) >> >> -- >> 2.21.0.windows.1 > > > >