y and emails.
Reviewed-by: Sunny Wang
-Original Message-
From: Li, Walon
Sent: Friday, February 19, 2021 5:40 PM
To: devel@edk2.groups.io
Cc: Li, Walon ; Wang, Sunny (HPS SW) ;
ler...@redhat.com; ray...@intel.com; hao.a...@intel.com;
gaolim...@byosoft.com.cn
Subject: [PATCH v2] MdeModu
f Lindholm ;
devel@edk2.groups.io
Cc: Samer El-Haj-Mahmoud ; Andrei Warkentin
(awarken...@vmware.com) ; Wang, Sunny (HPS SW)
; zhichao@intel.com; ray...@intel.com; Ard Biesheuvel
; Andrew Fish ; Michael D Kinney
; Jian J Wang ; Hao A Wu
Subject: Re: [EXTERNAL] Re: [edk2-devel] [edk2
re, I also think it's fine to have this change.
Reviewed-by: Sunny Wang < sunnyw...@hpe.com>
Regards,
Sunny Wang
-Original Message-
From: Dong, Guo
Sent: Saturday, February 6, 2021 7:16 AM
To: Patrick Rudolph ; Wang, Sunny (HPS SW)
Cc: devel@edk2.groups.io; Park, Aiden
Hi Patrick,
I'm not familiar with UefiPayloadPkg. However, since we may want to enable the
trusted console as earlier as possible, you may still need to keep the
PlatformConsoleInit() call at the beginning of
PlatformBootManagerBeforeConsole() to support the platform that has
trusted/on-board
Thanks, Ray.
Hi All,
If you're working on option cards' UEFI driver (especially storage card UEFI
driver), welcome to join the meeting and provide your valuable thoughts.
Regards,
Sunny Wang
From: Ni, Ray
Sent: Thursday, September 17, 2020 5:25 PM
To: devel@edk2.groups.io
Cc: Wang,
Looks good to me. Not sure if there was a reason to call BmDestroyRamDisk
before ReportStatusCode.
By the way, it is an interesting case that there is a custom boot option or
application that needs the memory that was occupied by the RAM disk. It looks
to me like the custom boot option or appli
h I mentioned works for you, I think that
would be an easier solution.
Regards,
Sunny Wang
-Original Message-----
From: Pete Batard [mailto:p...@akeo.ie]
Sent: Wednesday, June 17, 2020 6:59 PM
To: Wang, Sunny (HPS SW) ; devel@edk2.groups.io
Cc: zhichao@intel.com; ray...@int
Hi Pete.
Since the EfiBootManagerProcessLoadOption is called by ProcessLoadOptions as
well, your change would also cause some unexpected behavior like:
1. Signal one more ReadyToBoot for the PlatformRecovery option which is an
application that calls EfiBootManagerBoot() to launch its recovered
Prep instead and show
the message to users.
Regards,
Sunny Wang
-Original Message-
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Wang,
Sunny (HPS SW)
Sent: Wednesday, March 4, 2020 3:53 PM
To: devel@edk2.groups.io; ray...@intel.com; Sean Brogan
; Kinney, Mich
Sure!
Regards,
Sunny Wang
-Original Message-
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ni, Ray
Sent: Wednesday, March 4, 2020 3:26 PM
To: devel@edk2.groups.io; Wang, Sunny (HPS SW) ; Sean Brogan
; Kinney, Michael D
Cc: Wei, Kent (HPS SW) ; Spottswood, Jason
our offline
discussion, Ray. :)
Regards,
Sunny Wang
-Original Message-
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ni, Ray
Sent: Wednesday, March 4, 2020 11:46 AM
To: Sean Brogan ; Kinney, Michael D
; Wang, Sunny (HPS SW)
Cc: Ni, Ray ; devel@edk2.groups.io
Hi Bret,
Your proposal looks good to me, and most of my questions/concerns were already
answered/solved by the GOOD discussion between you and Jiewen.
For now, I just have one remaining question. Can we also make
DumpVariablePolicy as a platform choice?
DumpVariablePolicy would also expose th
tions look workable to me. I will give them a try. Thanks!
Regards,
Sunny Wang
-Original Message-
From: Ashish Singhal [mailto:ashishsin...@nvidia.com]
Sent: Tuesday, December 24, 2019 12:48 PM
To: Wang, Sunny (HPS SW) ; Ni, Ray ;
devel@edk2.groups.io; Wang, Jian J ; Wu, Hao A
Looks good enough to me. Thanks for addressing my comments, Ashish.
Reviewed-by: Sunny Wang
-Original Message-
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ashish
Singhal
Sent: Tuesday, December 24, 2019 10:58 AM
To: devel@edk2.groups.io; ray...@intel.com; jian.j
--Original Message-
From: Ni, Ray [mailto:ray...@intel.com]
Sent: Tuesday, December 24, 2019 10:40 AM
To: Wang, Sunny (HPS SW) ; devel@edk2.groups.io;
ashishsin...@nvidia.com; Wang, Jian J ; Wu, Hao A
; Gao, Zhichao ; Kinney, Michael D
; 'Andrew Fish (af...@apple.com)'
Cc: Spottswood
ns in
EfiBootManagerRefreshAllBootOption), the original function name
"RefreshAllBootOptions" may cause some confusion.
Regards,
Sunny Wang
-Original Message-
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ashish
Singhal
Sent: Friday, December 20, 2019 12:38 AM
To: Wang, Sunny (
1. Is it a proper solution to the problem?
Yes, it already solved my concern discussed in the other email.
2. Is the new protocol/function name proper?
Yes, but I'm not good at naming. We may need others' feedback. :)
3. Are the parameters in the function proper?
How about we only have
Sorry for the delay. Somehow I didn't catch the follow-up email. Thanks for
checking my comments, Ray and Ashish.
Yeah, agree. 2.b is better. I will review the code change.
Regards,
Sunny Wang
-Original Message-
From: disc...@edk2.groups.io [mailto:disc...@edk2.groups.io] On Behalf Of
Hi all,
Since I can't attend the design meeting (EMEA), I would like to add some use
cases and a suggestion for your guys' reference before this topic being
discussed:
We had similar needs (use cases) as Ashish like the following, so making this
code change would be definitely good for system
Thanks for the clarification, Ray.
Regards,
Sunny Wang
-Original Message-
From: Ni, Ray [mailto:ray...@intel.com]
Sent: Thursday, November 21, 2019 7:51 PM
To: devel@edk2.groups.io; Wang, Sunny (HPS SW) ; Gao,
Zhichao ; Jeff Brasen ;
edk2-de...@lists.01.org
Cc: Gao, Liming ; Kinney
Hi Ray,
May I know why we need to put this PCD to [PcdsFixedAtBuild,
PcdsPatchableInModule] section only? If the reason is the security concern,
Locking the variable (value of PCD) at the EndOfDxe should be secure enough.
For the platforms that want to make it more secure (don't want the PCD t
e task below, there will be no OS or
OS tool supporting it. :)
- https://github.com/rhboot/efibootmgr/issues/41
Regards,
Sunny Wang
-Original Message-
From: Ni, Ray [mailto:ray...@intel.com]
Sent: Wednesday, October 16, 2019 4:43 PM
To: Wang, Sunny (HPS SW) ; devel@edk2.groups.io; G
-
From: Gao, Zhichao [mailto:zhichao@intel.com]
Sent: Wednesday, October 16, 2019 4:36 PM
To: devel@edk2.groups.io; ler...@redhat.com; Wang, Sunny (HPS SW)
Cc: Ni, Ray ; Li, Walon
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/BdsDxe: Make PlatformRecovery
work regardless of OsI
.groups.io] On Behalf Of Gao,
Zhichao
Sent: Wednesday, October 16, 2019 1:57 PM
To: Wang, Sunny (HPS SW) ; devel@edk2.groups.io; Ni, Ray
Cc: Wang, Jian J ; Wu, Hao A ; Zeng,
Star ; Gao, Liming ; Sean Brogan
; Michael Turner ;
Bret Barkelew ; Li, Walon ; Wei,
Kent (HPS SW)
Subject: Re: [edk2-de
According to the UEFI specification section 3.4.1 and 3.4.2, the OsRecovery and
PlatformRecovery options should still be processed regardless of the value of
OsIndications variable if processing of BootOrder does not result in success.
Therefore, update the code to check PcdPlatformRecoverySuppo
Hi Zhichao and Ray,
I have some questions about this code change. Sorry for being late to bring my
questions here.
For now, the code block for iterating the PlatformRecovery variables is
controlled by OsIndications variable. However, it looks to me like that the
PlatformRecovery shoul
26 matches
Mail list logo
