Re: [edk2-devel] [PATCH v6 00/14] Add the VariablePolicy feature

2020-09-17 Thread Bret Barkelew via groups.io
Hmmm…. no response from Jordan. Maybe he’s still trying dishes to see what he doesn’t get sick of. It’s good to be thorough. So, I’ll turn to you, Mr. Fish – if that is your real name and not your supervillain pseudonym. While you’ve always struck me as a gentle soul, and likely not resistant to

Re: [edk2-devel] more development process failure [was: UefiPayloadPkg: Runtime MMCONF]

2020-09-17 Thread Ni, Ray
Laszlo, I support your idea of having a meaningful description for BZ, for commit message, for code comments. Thinking from 1 or 2 years from now, the simple message we created may help nothing to remind me or others why the changes were made. We cannot reply on people's memories and even the p

[edk2-devel] TianoCore Community Design Meeting Minutes - Sep 18, 2020

2020-09-17 Thread Ni, Ray
Topic: Sanitization Protocols for Option Cards (Sunny Wang / HPE) Slides: https://edk2.groups.io/g/devel/files/Designs/2020/0918/Sanitization%20Protocols%20for%20Option%20Cards_200918.pdf More materials @ https://edk2.groups.io/g/devel/files/Designs/2020/0918/ 1. Overview Sunny presented HPE's

Re: [edk2-devel] [PATCH] MdeModulePkg/NonDiscoverablePciDeviceDxe: Add NULL pointer check

2020-09-17 Thread Wu, Hao A
Pushed via commit 698d3d7726232694018d437279dd4166e462deb7. Best Regards, Hao Wu > -Original Message- > From: devel@edk2.groups.io On Behalf Of Wu, Hao A > Sent: Tuesday, September 15, 2020 9:48 AM > To: Jeff Brasen ; devel@edk2.groups.io > Cc: Ni, Ray > Subject: Re: [edk2-devel] [PATCH

回复: 回复: [edk2-devel] [PATCH v10 0/5] Use RngLib instead of TimerLib for OpensslLib

2020-09-17 Thread gaoliming
Matthew: Here is pull merge https://github.com/tianocore/edk2/pull/938 for this patch set. Thanks Liming 发件人: Matthew Carlson 发送时间: 2020年9月16日 11:17 收件人: edk2-devel-groups-io ; Matthew Carlson ; gaoliming 主题: Re: 回复: [edk2-devel] [PATCH v10 0/5] Use RngLib instead of TimerLib for Ope

Re: [edk2-devel] TianoCore Design Meeting - APAC/NAMO - Fri, 09/18/2020 9:30am-10:30am #cal-reminder

2020-09-17 Thread Ni, Ray
Please be aware that the meeting is now using Webex, NOT Zoom. From: devel@edk2.groups.io Sent: Friday, September 18, 2020 9:15 AM To: devel@edk2.groups.io Subject: [edk2-devel] TianoCore Design Meeting - APAC/NAMO - Fri, 09/18/2020 9:30am-10:30am #cal-reminder Reminder: TianoCore Design Meeti

Re: [edk2-devel] [PATCH v1 1/1] MdePkg: Correct EFI_BLOCK_IO_PROTOCOL_REVISION3 value

2020-09-17 Thread Ni, Ray
Mike, Have you evaluated the impact to the already-released module that relies on the macro value? Basically, you changed to a smaller value that may cause a revision3 check fail: a released module expects the revision is bigger than 0x31, but the value is 0x1f. Thanks, Ray > -Original Mes

[edk2-devel] TianoCore Design Meeting - APAC/NAMO - Fri, 09/18/2020 9:30am-10:30am #cal-reminder

2020-09-17 Thread devel@edk2.groups.io Calendar
*Reminder:* TianoCore Design Meeting - APAC/NAMO *When:* Friday, 18 September 2020, 9:30am to 10:30am, (GMT+08:00) Asia/Chongqing *Where:* https://intel.webex.com/intel/j.php?MTID=m6bf2875d89c1ee88ca37eda8fd41a47b View Event ( https://edk2.groups.io/g/devel/viewevent?eventid=915765 ) *Organize

回复: 回复: [edk2-devel] [PATCH v1 1/1] MdePkg: Correct EFI_BLOCK_IO_PROTOCOL_REVISION3 value

2020-09-17 Thread gaoliming
Michael: Here is pull request https://github.com/tianocore/edk2/pull/936 Thanks Liming > -邮件原件- > 发件人: bounce+27952+65387+4905953+8761...@groups.io > 代表 Michael > Kubacki > 发送时间: 2020年9月18日 0:49 > 收件人: devel@edk2.groups.io; gaolim...@byosoft.com.cn > 抄送: 'Michael D Kinney' ; 'Zhiguang L

Re: [edk2-devel] [PATCH v12 1/1] ShellPkg/DynamicCommand: add HttpDynamicCommand

2020-09-17 Thread Vladimir Olovyannikov via groups.io
Hi Zhichao, Thank you for pointing to these. > -Original Message- > From: Gao, Zhichao > Sent: Wednesday, September 16, 2020 7:28 PM > To: devel@edk2.groups.io; vladimir.olovyanni...@broadcom.com > Cc: Samer El-Haj-Mahmoud ; Laszlo > Ersek ; Maciej Rabeda > ; Wu, Jiaxin ; Fu, > Siyuan ; N

Re: [edk2-devel] [PATCH 15/22] .mailmap: add entry for Wei6 Xu

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/17/20 5:51 PM, Laszlo Ersek wrote: > On 09/17/20 16:53, Philippe Mathieu-Daudé wrote: >> Hi Laszlo, >> >> On 9/8/20 3:50 AM, Xu, Wei6 wrote: >>> Hi Laszlo, >>> >>> Thanks a lot for helping solve this issue. >>> >>> Reviewed-by: Wei6 Xu >>> >>> BR, >>> Wei >> >> Wei signs as 'Wei', so maybe we

Re: 回复: [edk2-devel] [PATCH v1 1/1] MdePkg: Correct EFI_BLOCK_IO_PROTOCOL_REVISION3 value

2020-09-17 Thread Michael Kubacki
Hi Liming, I have rebased this commit and added all Reviewed-bys on: https://github.com/makubacki/edk2/commits/fix_block_io_protocol_rev3_value Please PR it when you are ready. Thanks, Michael On 9/14/2020 5:54 PM, gaoliming wrote: Reviewed-by: Liming Gao -邮件原件- 发件人: bounce+27952+6

Re: [edk2-devel] [PATCH 07/22] .mailmap: add entry for Eric Jin

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/17/20 5:34 PM, Laszlo Ersek wrote: > On 09/17/20 16:49, Philippe Mathieu-Daudé wrote: >> On 9/7/20 9:30 PM, Laszlo Ersek wrote: >>> ... for git-shortlog purposes. >>> >>> Cc: Eric Jin >>> Cc: Philippe Mathieu-Daudé >>> Signed-off-by: Laszlo Ersek >>> --- >>> .mailmap | 1 + >>> 1 file chan

Re: [edk2-devel] [PATCH 02/22] .mailmap: add entry for Ching JenX Cheng

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/17/20 5:28 PM, Laszlo Ersek wrote: > On 09/17/20 16:47, Philippe Mathieu-Daudé wrote: >> On 9/7/20 9:30 PM, Laszlo Ersek wrote: >>> ... for git-shortlog purposes. >>> >>> Cc: Ching JenX Cheng >>> Cc: Philippe Mathieu-Daudé >>> Signed-off-by: Laszlo Ersek >>> --- >>> .mailmap | 1 + >>> 1 f

Re: [edk2-devel] [PATCH 22/22] .mailmap: add entries for Rebecca Cran

2020-09-17 Thread Laszlo Ersek
On 09/17/20 16:56, Philippe Mathieu-Daudé wrote: > On 9/7/20 9:31 PM, Laszlo Ersek wrote: >> ... for git-shortlog purposes. >> >> NOTE: this patch does not introduce a cross-domain mapping; it only maps >> both email addresses of Rebecca to the full name "Rebecca Cran". >> >> Cc: Philippe Mathieu-D

Re: [edk2-devel] [PATCH 15/22] .mailmap: add entry for Wei6 Xu

2020-09-17 Thread Laszlo Ersek
On 09/17/20 16:53, Philippe Mathieu-Daudé wrote: > Hi Laszlo, > > On 9/8/20 3:50 AM, Xu, Wei6 wrote: >> Hi Laszlo, >> >> Thanks a lot for helping solve this issue. >> >> Reviewed-by: Wei6 Xu >> >> BR, >> Wei > > Wei signs as 'Wei', so maybe we should simply use: > > Wei Xu In this case I disa

Re: [edk2-devel] [PATCH 12/22] .mailmap: add entry for XiaoyuX Lu

2020-09-17 Thread Laszlo Ersek
On 09/17/20 16:51, Philippe Mathieu-Daudé wrote: > On 9/7/20 9:30 PM, Laszlo Ersek wrote: >> ... for git-shortlog purposes. >> >> Cc: Philippe Mathieu-Daudé >> Cc: XiaoyuX Lu >> Signed-off-by: Laszlo Ersek >> --- >> .mailmap | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/.mailmap b

Re: [edk2-devel] [PATCH 08/22] .mailmap: add entry for Michael D Kinney

2020-09-17 Thread Laszlo Ersek
On 09/17/20 16:50, Philippe Mathieu-Daudé wrote: > On 9/7/20 9:30 PM, Laszlo Ersek wrote: >> ... for git-shortlog purposes. >> >> Cc: Michael D Kinney >> Cc: Philippe Mathieu-Daudé >> Signed-off-by: Laszlo Ersek >> --- >> .mailmap | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/.mai

Re: [edk2-devel] [PATCH 07/22] .mailmap: add entry for Eric Jin

2020-09-17 Thread Laszlo Ersek
On 09/17/20 16:49, Philippe Mathieu-Daudé wrote: > On 9/7/20 9:30 PM, Laszlo Ersek wrote: >> ... for git-shortlog purposes. >> >> Cc: Eric Jin >> Cc: Philippe Mathieu-Daudé >> Signed-off-by: Laszlo Ersek >> --- >> .mailmap | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/.mailmap b/.

Re: [edk2-devel] [PATCH 02/22] .mailmap: add entry for Ching JenX Cheng

2020-09-17 Thread Laszlo Ersek
On 09/17/20 16:47, Philippe Mathieu-Daudé wrote: > On 9/7/20 9:30 PM, Laszlo Ersek wrote: >> ... for git-shortlog purposes. >> >> Cc: Ching JenX Cheng >> Cc: Philippe Mathieu-Daudé >> Signed-off-by: Laszlo Ersek >> --- >> .mailmap | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/.mai

Re: [edk2-devel] [PATCH 04/22] .mailmap: add entries for Guo Dong

2020-09-17 Thread Laszlo Ersek
On 09/17/20 16:48, Philippe Mathieu-Daudé wrote: > On 9/7/20 9:30 PM, Laszlo Ersek wrote: >> ... for git-shortlog purposes. >> >> Cc: Guo Dong >> Cc: Philippe Mathieu-Daudé >> Signed-off-by: Laszlo Ersek >> --- >> .mailmap | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/.mailmap b

Re: [edk2-devel] [PATCH 22/22] .mailmap: add entries for Rebecca Cran

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:31 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > NOTE: this patch does not introduce a cross-domain mapping; it only maps > both email addresses of Rebecca to the full name "Rebecca Cran". > > Cc: Philippe Mathieu-Daudé > Cc: Rebecca Cran > Signed-off-by: Laszlo Ersek

Re: [edk2-devel] [PATCH 19/22] .mailmap: add (another) entry for Vladimir Olovyannikov

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Philippe Mathieu-Daudé > Cc: Vladimir Olovyannikov > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 93431e6cd975..6c523a1bb79d 100

Re: [edk2-devel] [PATCH 18/22] .mailmap: add entry for Paul Grimes

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Paul Grimes > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 33652b8f0b46..93431e6cd975 100644 > ---

Re: [edk2-devel] [PATCH 15/22] .mailmap: add entry for Wei6 Xu

2020-09-17 Thread Philippe Mathieu-Daudé
Hi Laszlo, On 9/8/20 3:50 AM, Xu, Wei6 wrote: > Hi Laszlo, > > Thanks a lot for helping solve this issue. > > Reviewed-by: Wei6 Xu > > BR, > Wei Wei signs as 'Wei', so maybe we should simply use: Wei Xu > > -Original Message- > From: Laszlo Ersek > Sent: Tuesday, September 8, 20

Re: [edk2-devel] [RFC] Request for the new package "RedfishPkg" under edk2 repo

2020-09-17 Thread Abner Chang
Hi Felix, Yes, Redfish (EFI Redfish Discover Protocol) relies on the EFI REST EX protocol instance which provides EFI_REST_EX_SERVICE_REDFISH type service as defined in UEFI spec 29.7.2.2 GetService(). The REST EX Driver we will contribute is the REST EX Redfish instance, so that is the reason w

Re: [edk2-devel] [PATCH 14/22] .mailmap: add entry for Jiaxin Wu

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Jiaxin Wu > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 05887373154a..0c731e74b811 100644 > --- a/

Re: [edk2-devel] [PATCH 13/22] .mailmap: add entry for Steven Shi

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Philippe Mathieu-Daudé > Cc: Steven Shi > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 407fe4559b4c..05887373154a 100644 > --- a

Re: [edk2-devel] [PATCH 12/22] .mailmap: add entry for XiaoyuX Lu

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Philippe Mathieu-Daudé > Cc: XiaoyuX Lu > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 3866642d4dbe..407fe4559b4c 100644 > --- a

Re: [edk2-devel] [PATCH 10/22] .mailmap: add entry for Derek Lin

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Derek Lin > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 0d2be4255ff8..33ecbffa5191 100644 > --- a/

Re: [edk2-devel] [PATCH 08/22] .mailmap: add entry for Michael D Kinney

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Michael D Kinney > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 47608acbf931..2e23a0f43814 100644 >

Re: [edk2-devel] [PATCH 07/22] .mailmap: add entry for Eric Jin

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Eric Jin > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index df05a29ed3bf..47608acbf931 100644 > --- a/.

Re: [edk2-devel] [PATCH 04/22] .mailmap: add entries for Guo Dong

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Guo Dong > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/.mailmap b/.mailmap > index 38cfec35c70e..8ed1d77c9220 100644 > --- a

Re: [edk2-devel] [PATCH 03/22] .mailmap: add entries for Maggie Chu

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Maggie Chu > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/.mailmap b/.mailmap > index c22834e2..38cfec35c70e 100644 > ---

Re: [edk2-devel] [PATCH 02/22] .mailmap: add entry for Ching JenX Cheng

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Ching JenX Cheng > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index f173754d7ee8..c22834e2 100644 >

Re: [edk2-devel] [PATCH 01/22] .mailmap: add entry for Marc W Chen

2020-09-17 Thread Philippe Mathieu-Daudé
On 9/7/20 9:30 PM, Laszlo Ersek wrote: > ... for git-shortlog purposes. > > Cc: Marc W Chen > Cc: Philippe Mathieu-Daudé > Signed-off-by: Laszlo Ersek > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index ba246ff6cd80..f173754d7ee8 100644 > ---

Re: [edk2-devel] [PATCH 1/1] OvmfPkg: drop redundant VendorID check in VirtioMmioDeviceLib

2020-09-17 Thread Leif Lindholm
On Mon, Sep 14, 2020 at 17:35:13 +0200, Laszlo Ersek wrote: > Hi Leif, > > On 09/12/20 18:40, Leif Lindholm wrote: > > There is a DEBUG warning printout in VirtioMmioDeviceLib if the current > > device's VendorID does not match the traditional 16-bit Red Hat PCIe > > vendor ID used with virtio-pci

Re: [edk2-devel] more development process failure [was: UefiPayloadPkg: Runtime MMCONF]

2020-09-17 Thread Laszlo Ersek
On 09/17/20 09:31, Yao, Jiewen wrote: > Laszlo > I like your description to compare the process with the programming language > and software design. We have to clean up the resource. > > Please allow me to point out, this is the exactly the issue we are having > today. > > 1) Take buffer overfl

Re: [edk2-devel] Open Design Meeting to discuss storage/configuration purge design

2020-09-17 Thread Wang, Sunny (HPS SW)
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, Sunny (HPS

[edk2-devel] Open Design Meeting to discuss storage/configuration purge design

2020-09-17 Thread Ni, Ray
All, There will be one topic in this week's open design meeting discussing about storage/configuration purge design. (In case you think the meeting is cancelled as usually.) Sunny will present and I invited Rothman Michael to help review the proposal. Michael is the expert of storage and HII. D

Re: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure Boot

2020-09-17 Thread Ni, Ray
Reviewed-by: Ray Ni From: Wadhawan, Divneil R Sent: Thursday, September 17, 2020 3:43 PM To: Ni, Ray ; devel@edk2.groups.io Cc: gaoliming ; 'Andrew Fish' ; Justen, Jordan L ; Kinney, Michael D ; Wadhawan, Divneil R Subject: RE: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure Bo

Re: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure Boot

2020-09-17 Thread Wadhawan, Divneil R
Hi Ray, Yes, I have tested the following: 1. SECURE_BOOT_ENABLE=true * Key Enrollment (PK, KEK, db) via custom mode * Execution of unit test shell application (signed one works okay, unsigned gives an Access denied) 1. SECURE_BOOT_ENABLE=false (default case) * Secure Boot

Re: [edk2-devel] more development process failure [was: UefiPayloadPkg: Runtime MMCONF]

2020-09-17 Thread Yao, Jiewen
Laszlo I like your description to compare the process with the programming language and software design. We have to clean up the resource. Please allow me to point out, this is the exactly the issue we are having today. 1) Take buffer overflow as an example. It has been there for 30 years. We ha

Re: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure Boot

2020-09-17 Thread Ni, Ray
Divneil, Just want to double confirm: did you test the secure boot and non-secure boot? Thanks, Ray From: Wadhawan, Divneil R Sent: Wednesday, September 16, 2020 11:53 PM To: devel@edk2.groups.io Cc: Ni, Ray ; gaoliming ; 'Andrew Fish' ; Justen, Jordan L ; Kinney, Michael D ; Wadhawan, Divneil