Re: [edk2-devel] [PATCH 10/11] CryptoPkg/OpensslLib: use RngLib to get high quality random entropy

2019-11-13 Thread Ard Biesheuvel
On Thu, 14 Nov 2019 at 02:18, Jian J Wang wrote: > > Per BZ1871, OpensslLib should use RngLib to get high quality of random > entropy. This patch remove all code depending on TimerLib for this job, > and add RngLib interface to implement the rand interface required by > openssl. > > Ref: https://b

Re: [edk2-devel] [PATCH 09/11] ArmVirtPkg/ArmVirt.dsc.inc: specify RngLib instances in dsc files

2019-11-13 Thread Ard Biesheuvel
On Thu, 14 Nov 2019 at 02:18, Wang, Jian J wrote: > > Per BZ1871, OpensslLib will depend on RngLib instead of TimerLib. Update > ArmVirt.dsc.inc file to accommodate the coming changes. It's supposed > that only TlsDxe needs random number. The RngDxeLib is added for it. For > all other drivers, Rng

Re: [edk2-devel] [[PATCH 1/1]] EmulatorPkg using CpuBreakPoint

2019-11-13 Thread Ni, Ray
Kilian, Thank you very much. Comments below. From: Kilian Kegel Sent: Thursday, November 14, 2019 1:24 PM To: Ni, Ray ; devel@edk2.groups.io; Gao, Liming Cc: Richardson, Brian ; Kinney, Michael D Subject: RE: [edk2-devel] [[PATCH 1/1]] EmulatorPkg using CpuBreakPoint I have no preference at

[edk2-devel] [edk2-platforms] [PATCH V1 13/13] MinPlatformPkg: Remove BoardInitLib dependency from PlatformSecLib

2019-11-13 Thread Nate DeSimone
SecFspWrapperPlatformSecLib contains the implementation of SecPlatformDisableTemporaryMemory(), which SecMain in UefiCpuPkg will call as part of its implementation of EFI_PEI_TEMPORARY_RAM_DONE_PPI. For platforms that use FSP, the implementation of SecPlatformDisableTemporaryMemory() can be made ge

[edk2-devel] [edk2-platforms] [PATCH V1 09/13] MinPlatformPkg: FSP Dispatch Mode Support for PlatformSecLib

2019-11-13 Thread Nate DeSimone
Cc: Michael Kubacki Cc: Chasel Chiu Cc: Liming Gao Signed-off-by: Nate DeSimone --- .../FspWrapperPlatformSecLib.c| 34 --- .../SecFspWrapperPlatformSecLib.inf | 7 +++- .../SecTempRamDone.c | 42 +++ .../Intel/Min

[edk2-devel] [edk2-platforms] [PATCH V1 11/13] KabylakeOpenBoardPkg: Add support for PcdFspDispatchModeUseFspPeiMain

2019-11-13 Thread Nate DeSimone
If PcdFspDispatchModeUseFspPeiMain is FALSE, then the BoardPkg should include PeiMain in FvPreMemory. Cc: Chasel Chiu Cc: Michael Kubacki Signed-off-by: Nate DeSimone --- .../KabylakeRvp3/OpenBoardPkg.fdf | 5 +++-- .../KabylakeRvp3/OpenBoardPkgPcd.dsc | 20 ++

[edk2-devel] [edk2-platforms] [PATCH V1 08/13] WhiskeylakeOpenBoardPkg: Remove SecFspWrapperPlatformSecLib override

2019-11-13 Thread Nate DeSimone
Cc: Chasel Chiu Cc: Michael Kubacki Signed-off-by: Nate DeSimone --- .../FspWrapperPlatformSecLib.c| 163 .../SecFspWrapperPlatformSecLib/FsptCoreUpd.h | 40 -- .../SecFspWrapperPlatformSecLib/Ia32/Fsp.h| 43 --- .../Ia32/PeiCoreEntry.nasm| 130

[edk2-devel] [edk2-platforms] [PATCH V1 10/13] MinPlatformPkg: Coding style cleanups in MinPlatformPkg.dec

2019-11-13 Thread Nate DeSimone
Cc: Michael Kubacki Cc: Chasel Chiu Cc: Liming Gao Signed-off-by: Nate DeSimone --- .../Intel/MinPlatformPkg/MinPlatformPkg.dec | 281 +- 1 file changed, 139 insertions(+), 142 deletions(-) diff --git a/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dec b/Platform/Intel/MinPl

[edk2-devel] [edk2-platforms] [PATCH V1 02/13] KabylakeOpenBoardPkg: Update location of SiliconInitLib

2019-11-13 Thread Nate DeSimone
Cc: Chasel Chiu Cc: Michael Kubacki Cc: Jeremy Soller Signed-off-by: Nate DeSimone --- Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | 2 +- .../Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/

[edk2-devel] [edk2-platforms] [PATCH V1 01/13] KabylakeSiliconPkg: Change MODULE_TYPE of SiliconInitLib to PEIM

2019-11-13 Thread Nate DeSimone
This library contains PEI phase only code. For example, ClearIchSmiAndWake() calls PeiServicesGetBootMode(). Accordingly, this library should have never been marked as a BASE library. Cc: Chasel Chiu Cc: Michael Kubacki Cc: Sai Chaganty Signed-off-by: Nate DeSimone --- .../PeiSiliconInitLib.i

[edk2-devel] [edk2-platforms] [PATCH V1 12/13] KabylakeOpenBoardPkg: Remove SecFspWrapperPlatformSecLib override

2019-11-13 Thread Nate DeSimone
Cc: Chasel Chiu Cc: Michael Kubacki Cc: Jeremy Soller Signed-off-by: Nate DeSimone --- .../FspWrapperPlatformSecLib.c| 186 - .../SecFspWrapperPlatformSecLib/FsptCoreUpd.h | 40 -- .../SecFspWrapperPlatformSecLib/Ia32/Fsp.h| 42 -- .../Ia32/PeiCoreEntry.nasm

[edk2-devel] [edk2-platforms] [PATCH V1 00/13] SecFspWrapperPlatformSecLib Cleanup

2019-11-13 Thread Nate DeSimone
This patch series cleans up two outstanding problems with SecFspWrapperPlatformSecLib: 1. There are several copies of SecFspWrapperPlatformSecLib thoughout the codebase, this patch series consolidates the various implementations down to a single implementation contained in MinPlat

[edk2-devel] [edk2-platforms] [PATCH V1 07/13] WhiskeylakeOpenBoardPkg: Whitespace cleanup in BoardInitLib

2019-11-13 Thread Nate DeSimone
Cc: Chasel Chiu Cc: Michael Kubacki Signed-off-by: Nate DeSimone --- .../PeiWhiskeylakeURvpInitPostMemLib.c| 185 ++--- .../PeiWhiskeylakeURvpInitPreMemLib.c | 257 +- 2 files changed, 216 insertions(+), 226 deletions(-) diff --git a/Platform/Intel/Whis

[edk2-devel] [edk2-platforms] [PATCH V1 03/13] KabylakeSiliconPkg: Cleanup old comments

2019-11-13 Thread Nate DeSimone
Cc: Chasel Chiu Cc: Michael Kubacki Cc: Sai Chaganty Signed-off-by: Nate DeSimone --- .../Library/PeiSiliconInitLib/SiliconInitPreMem.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/SiliconInitPreM

[edk2-devel] [edk2-platforms] [PATCH V1 04/13] CoffeeLakeSiliconPkg: Move TcoWdtHob.h

2019-11-13 Thread Nate DeSimone
Moves TcoWdtHob.h from WhiskeylakeOpenBoardPkg to CoffeelakeSiliconPkg. Oddly, gTcoWdtHobGuid was defined in CoffeelakeSiliconPkg/SiPkg.dec while the HOB definition itself was in WhiskeylakeOpenBoardPkg, this change cleans up the disparity. Cc: Chasel Chiu Cc: Michael Kubacki Cc: Sai Chaganty S

[edk2-devel] [edk2-platforms] [PATCH V1 06/13] WhiskeylakeOpenBoardPkg: Add SiliconInitLib APIs to BoardInitLib

2019-11-13 Thread Nate DeSimone
Cc: Chasel Chiu Cc: Michael Kubacki Signed-off-by: Nate DeSimone --- .../BoardInitLib/PeiBoardInitPostMemLib.inf | 1 + .../BoardInitLib/PeiBoardInitPreMemLib.inf| 1 + .../PeiMultiBoardInitPostMemLib.inf | 1 + .../PeiMultiBoardInitPreMemLib.inf| 2 +- .../PeiWh

[edk2-devel] [edk2-platforms] [PATCH V1 05/13] CoffeelakeSiliconPkg: Add SiliconInitLib

2019-11-13 Thread Nate DeSimone
SiliconInitLib contains Silicon Init APIs that can be reused by BoardInitLib. It is expected that several implementations of BoardInitLib exist for a given SOC, these APIs allow the various BoardInitLib implementations to reuse common silicon initialization code. This matches the implementation alr

Re: [edk2-devel] [[PATCH 1/1]] EmulatorPkg using CpuBreakPoint

2019-11-13 Thread Kilian Kegel
Hi Ray, [Ray] 1. Can you remove “if(1)”?: DONE Please define it in the begin of the function, and use edk2 data type and coding style, such as UINTN Index. DONE [Ray] 2. strncmp assumes user may input “/debugx”. Is that a valid assumption? I thought we only accept “/debug” th

Re: [edk2-devel] [PATCH 00/11] Use proper entropy sources

2019-11-13 Thread Wang, Jian J
Hi Mike, Here're some suggestions: - If OpensslLib library is used, RngLib instance must be specified along with it. - For package dsc, BaseLibRngNull can be used because their main purpose is just for build. - For platforms which don't need random number at all, BaseLibRngNull i

[edk2-devel] [PATCH v1] MdeModulePkg/NvmExpressDxe: Fix wrong queue size for async IO queues

2019-11-13 Thread Wu, Hao A
From: Sean Brogan REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2118 When a packet is queued/completed for the asynchronous IO queue, the logic to roll over to the front of the queue doesn't account for actual size of the IO Submission/Completion queue. This causes a device to hang due to

Re: [edk2-devel] [PATCH 04/11] MdePkg/BaseLib: add interface to wrap rdseed IA instruction

2019-11-13 Thread Wang, Jian J
Mike, Thanks for finding it. I'll update the code in v2. Regards, Jian > -Original Message- > From: Kinney, Michael D > Sent: Thursday, November 14, 2019 12:17 PM > To: devel@edk2.groups.io; Wang, Jian J ; Kinney, > Michael D > Cc: Gao, Liming ; Ni, Ray > Subject: RE: [edk2-devel] [PA

Re: [edk2-devel] [PATCH 05/11] SecurityPkg/RngLibRdSeed: add an instance of RngLib to make use rdseed

2019-11-13 Thread Wang, Jian J
Mike, I figured that rdseed is only needed in cases demanding highest entropy, like seeding other pseudo-RNG. It's not for general purpose randomness. Then I put it in SecurityPkg. But I'm ok to put it into MdePkg. I have no strong opinion for this. Regards, Jian > -Original Message- > F

Re: [edk2-devel] [PATCH 05/11] SecurityPkg/RngLibRdSeed: add an instance of RngLib to make use rdseed

2019-11-13 Thread Michael D Kinney
Jian, Why is this lib instance in the SecurityPkg? It only depends on the MdePkg. Can't non security feature related modules that want to a random number use this lib without using the SecurityPkg? Could this lib instance be added to MdePkg? Thanks, Mike > -Original Message- > From:

Re: [edk2-devel] [PATCH 00/11] Use proper entropy sources

2019-11-13 Thread Michael D Kinney
Jian, In this patch series I see mixed use of different RngLib instances. How does a platform developer working on their DSC file know if the BaseCryptLib services they are using require a Null or a complete implementation of the RngLib? How does a platform developer know if they made the wrong

Re: [edk2-devel] [PATCH 04/11] MdePkg/BaseLib: add interface to wrap rdseed IA instruction

2019-11-13 Thread Michael D Kinney
Jian, According to the NASM docuimentation, the rdseed instruction has been supported since 2.10.02. https://www.nasm.us/xdoc/2.10.09/html/nasmdocc.html Please use rdseed instead of db bytes. Thanks, Mike > -Original Message- > From: devel@edk2.groups.io On > Behalf Of Wang, Jian J >

Re: [edk2-devel] [PATCH V2] BaseTools:Add [packages] section in dsc file

2019-11-13 Thread Bob Feng
This patch looks good. Reviewed-by: Bob Feng -Original Message- From: Fan, ZhijuX Sent: Thursday, November 14, 2019 9:28 AM To: devel@edk2.groups.io Cc: Gao, Liming ; Feng, Bob C Subject: [PATCH V2] BaseTools:Add [packages] section in dsc file BZ:https://bugzilla.tianocore.org/show_bu

Re: [edk2-devel] [PATCH 00/11] Use proper entropy sources

2019-11-13 Thread Wang, Jian J
The patches can also be found at in my fork in github https://github.com/jwang36/edk2/tree/fix-bz1871-use-proper-entropy-sources-v1 Regards, Jian > -Original Message- > From: devel@edk2.groups.io On Behalf Of Wang, Jian > J > Sent: Thursday, November 14, 2019 10:18 AM > To: devel@edk2.g

[edk2-devel] [PATCH 04/11] MdePkg/BaseLib: add interface to wrap rdseed IA instruction

2019-11-13 Thread Wang, Jian J
BaseRngLib uses rdrand instruction to get random number from DRNG in the processor, which is not a good candidate for seeding openssl rand interface. rdseed in x86 cpu provides non-deterministic random number (NRBG) which meets NIST SP 800-90B and NIST SP800-90C standards and can be used as seed f

[edk2-devel] [PATCH 01/11] NetworkPkg/NetworkPkg.dsc: specify RngLib instance for build

2019-11-13 Thread Wang, Jian J
Per BZ1871, OpensslLib will depend on RngLib instead of TimerLib. RngLibNull is used by default. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Cc: Liming Gao Cc: Ray Ni Signed-off-by: Jian J Wang --- NetworkPkg/NetworkPkg.dsc | 1 + 1

[edk2-devel] [PATCH 06/11] SecurityPkg/DxeRngLibRngProtocol: add RNG protocol version of RngLib

2019-11-13 Thread Wang, Jian J
From: Matthew Carlson This version of RngLib makes use of EFI_RNG_PROTOCOL to provide random number. According to UEFI spec, EFI_RNG_PROTOCOL should meet NIST SP 800-90 and/or ANSI X9.31 standards. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Jiewen Yao Cc: Chao Zhang Cc: Laszl

[edk2-devel] [PATCH 03/11] FmpDevicePkg/FmpDevicePkg.dsc: specify RngLib instances in dsc files

2019-11-13 Thread Wang, Jian J
Per BZ1871, OpensslLib will depend on RngLib instead of TimerLib. RngLibNull is used by default. TimerLib will be removed once its dependency is removed. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Liming Gao Cc: Michael D Kinney Cc: Liming Gao Cc: Ray Ni Signed-off-by: Jian J

[edk2-devel] [PATCH 10/11] CryptoPkg/OpensslLib: use RngLib to get high quality random entropy

2019-11-13 Thread Wang, Jian J
Per BZ1871, OpensslLib should use RngLib to get high quality of random entropy. This patch remove all code depending on TimerLib for this job, and add RngLib interface to implement the rand interface required by openssl. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Xiaoyu Lu Cc: L

[edk2-devel] [PATCH 07/11] SecurityPkg/SecurityPkg.dsc: add new RngLib instances for build

2019-11-13 Thread Wang, Jian J
Add new libraries DxeRngLibRngProtocol and RngLibRdSeed into SecurityPkg.dsc. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Jiewen Yao Cc: Chao Zhang Cc: Liming Gao Cc: Ray Ni Signed-off-by: Jian J Wang --- SecurityPkg/SecurityPkg.dsc | 6 ++ 1 file changed, 6 insertions(+

[edk2-devel] [PATCH 11/11] FmpDevicePkg/FmpDevicePkg.dsc: remove TimerLib instance

2019-11-13 Thread Wang, Jian J
TimerLib instance was added for openssl1.1.1b upgrade due to rand interface implementation based on TimerLib. Since RngLib is now used instead, TimerLib is not needed any more. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Liming Gao Cc: Michael D Kinney Cc: Liming Gao Cc: Ray Ni

[edk2-devel] [PATCH 08/11] OvmfPkg: specify RngLib instances in dsc files

2019-11-13 Thread Wang, Jian J
Per BZ1871, OpensslLib will depend on RngLib instead of TimerLib. Update OVMF dsc files to accommodate the coming changes. It's supposed that only TlsDxe needs random number. The DxeRngLibRngProtocol is added for it. For all other drivers, BaseRngLibNull is used by default. Ref: https://bugzilla.t

[edk2-devel] [PATCH 00/11] Use proper entropy sources

2019-11-13 Thread Wang, Jian J
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Patch series summary: - Add BaseRngLibNull to package dsc - Add DxeRngLibRngProtocol to make use EFI_RNG_PROTOCOL - Add RdSeed interface and RngLibRdSeed for IA32/X64 arch - Remove following files rand_pool_noise.h rand_pool_nois

[edk2-devel] [PATCH 02/11] SignedCapsulePkg/SignedCapsulePkg.dsc: specify RngLib instances

2019-11-13 Thread Wang, Jian J
Per BZ1871, OpensslLib will depend on RngLib instead of TimerLib. RngLibNull is used by default just for build. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Liming Gao Cc: Michael D Kinney Cc: Liming Gao Cc: Ray Ni Signed-off-by: Jian J Wang --- SignedCapsulePkg/SignedCapsule

[edk2-devel] [PATCH 05/11] SecurityPkg/RngLibRdSeed: add an instance of RngLib to make use rdseed

2019-11-13 Thread Wang, Jian J
This version of RngLib makes use of AsmRdSeed to get non-deterministic random number, which can be used for seeding other software DRNG like rand interface in openssl. It can be used only on IA32/X64 processors which supports rdseed instruction. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=

[edk2-devel] [PATCH 09/11] ArmVirtPkg/ArmVirt.dsc.inc: specify RngLib instances in dsc files

2019-11-13 Thread Wang, Jian J
Per BZ1871, OpensslLib will depend on RngLib instead of TimerLib. Update ArmVirt.dsc.inc file to accommodate the coming changes. It's supposed that only TlsDxe needs random number. The RngDxeLib is added for it. For all other drivers, RngLibNull is used by default. Ref: https://bugzilla.tianocore.

Re: [edk2-devel] [PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

2019-11-13 Thread Wang, Jian J
It's pushed at c9af866cdd6214d65c1539e7008f6a7255316c80 Regards, Jian > -Original Message- > From: Kinney, Michael D > Sent: Thursday, November 14, 2019 2:48 AM > To: devel@edk2.groups.io; leif.lindh...@linaro.org; Wang, Jian J > ; Kinney, Michael D > Cc: Steele, Kelly ; Bi, Dandan ; >

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

2019-11-13 Thread Ni, Ray
Jeff, I think it's a good topic that we could discuss in the open design meeting. Are you ok to present the problem you have and discuss the potential solutions in that meeting? https://github.com/tianocore/tianocore.github.io/wiki/Design-Meeting Thanks, Ray From: devel@edk2.groups.io On Behalf

Re: [edk2-devel] [edk2-platforms][PATCH V1 01/49] AdvancedFeaturePkg: Package DSC style cleanup

2019-11-13 Thread Chaganty, Rangasai V
Reviewed-by: Sai Chaganty -Original Message- From: Kubacki, Michael A Sent: Tuesday, November 12, 2019 7:27 PM To: devel@edk2.groups.io Cc: Chaganty, Rangasai V ; Gao, Liming Subject: [edk2-platforms][PATCH V1 01/49] AdvancedFeaturePkg: Package DSC style cleanup This change refactors

[edk2-devel] [PATCH V2] BaseTools:Add [packages] section in dsc file

2019-11-13 Thread Fan, ZhijuX
BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=2270 Currently a PCD (e.g. FeaturePCD) cannot be used in a conditional statement in a DSC/FDF file without a module in the build referencing the PCD package DEC file. An example implementation that to support this is to allow a [Packages] section

Re: [edk2-devel] [PATCH] MdeModulePkg: Unify the definitions of size_t

2019-11-13 Thread Liming Gao
Reviewed-by: Liming Gao >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Zhiguang Liu >Sent: Tuesday, November 12, 2019 4:48 PM >To: devel@edk2.groups.io >Cc: Wang, Jian J ; Wu, Hao A >Subject: [edk2-devel] [PATCH] MdeModulePkg: Unify the defini

Re: [edk2-devel][edk2-staging/EdkRepo][PATCH] EdkRepo: Support uninstalling packages with '_' in their name.

2019-11-13 Thread Nate DeSimone
Reviewed-by: Nate DeSimone On 11/13/19, 3:10 PM, "Desimone, Ashley E" wrote: pip does not support '_' in package names and converts them to '-' after installing the wheel. In order to uninstall affected packages the '_' needs to be converted to a '-' when calculating the packag

[edk2-devel][edk2-staging/EdkRepo][PATCH] EdkRepo: Support uninstalling packages with '_' in their name.

2019-11-13 Thread Desimone, Ashley E
pip does not support '_' in package names and converts them to '-' after installing the wheel. In order to uninstall affected packages the '_' needs to be converted to a '-' when calculating the package name. Signed-off-by: Ashley E Desimone Cc: Nate DeSimone Cc: Puja Pandya --- edkrepo_instal

[edk2-devel] [edk2-platforms][Patch V3 1/1] MinPlatformPkg: Library for customizing TPM platform hierarchy

2019-11-13 Thread Rodrigo Gonzalez del Cueto
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2331 In V3: + Kept callback function and registration in Tcg2PlatformDxe module. + Renamed new library with a more precise and descriptive name TpmPlatformHierarchyLib. Defines an API for configuring the TPM's Platform Hierachy. The AP

Re: [edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt

2019-11-13 Thread Leif Lindholm
Thanks Mike (and other reviewers). Pushed as cee28ab1c8da..f7271a06170b. / Leif On Wed, Nov 13, 2019 at 06:42:16PM +, Kinney, Michael D wrote: > Series > > Reviewed-by: Michael D Kinney > > Mike > > > -Original Message- > > From: devel@edk2.groups.io On > > Behalf Of Leif L

Re: [edk2-devel] [PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

2019-11-13 Thread Michael D Kinney
Leif, If it helps for testing purposes, the PR for edk2 has been run through EDK II CI. https://github.com/tianocore/edk2/pull/168 The public branch with the new lib is at: https://github.com/jwang36/edk2/tree/add-rnglibnull-instance The lib is added in a single commit: https://

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

2019-11-13 Thread Jeff Brasen
Thinking about this more I think we could do this with a PCD and a new group event without having to define any new function interfaces. We could change UiApp and BootManagerMenuApp (and any others that are relevant) from EfiBootManagerRefreshAllBootOption (); to if (FeaturePcdGet (PcdEventBa

Re: [edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt

2019-11-13 Thread Michael D Kinney
Series Reviewed-by: Michael D Kinney Mike > -Original Message- > From: devel@edk2.groups.io On > Behalf Of Leif Lindholm > Sent: Wednesday, November 13, 2019 10:00 AM > To: devel@edk2.groups.io > Cc: Ard Biesheuvel ; Marcin > Wojtas ; Kinney, Michael D > ; Wang, Jian J > > Subject: [

Re: [edk2-devel] [PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

2019-11-13 Thread Leif Lindholm
Hi Jian, Can MdePkg/Library/BaseRngLibNull/ be pushed to edk2, so that we can run some build tests? Best Regards, Leif On Wed, Nov 13, 2019 at 02:11:40PM +0800, Jian J Wang wrote: > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 > > It's a prerequisite of removing dependency of Timer

[edk2-devel] [PATCH edk2-platforms v2 5/5] Maintainers.txt: drop Ard from default section

2019-11-13 Thread Leif Lindholm
Ard will still be looking after specific platforms and drivers, but he was mainly added to the "default" maintainers group to offload me until edk2-platforms got off the ground, so drop him from that target now we add the GetMaintainers.py. Cc: Ard Biesheuvel Cc: Michael D Kinney Signed-off-by:

[edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt

2019-11-13 Thread Leif Lindholm
This set does a few things: - Imports the documentation header from edk2/Maintainers.txt - Reformats some maintainer entries to get rid of ',' characters in the names. - Adds F: entries for all sections, and a default fallback section for those where there is no match. - Merges the Marvell sect

[edk2-devel] [PATCH edk2-platforms v2 4/5] Maintainers.txt: create explicit sections for remaining modules

2019-11-13 Thread Leif Lindholm
Many of the original (ARM) platforms, and drivers brought in for them, had no explicit maintainership defined. Add entries for these, roughly corresponding to how they have already been handled in the past. Cc: Ard Biesheuvel Cc: Michael D Kinney Signed-off-by: Leif Lindholm Acked-by: Ard Biesh

[edk2-devel] [PATCH edk2-platforms v2 3/5] Maintainers.txt: merge Marvell platforms and silicon sections

2019-11-13 Thread Leif Lindholm
Cc: Marcin Wojtas Signed-off-by: Leif Lindholm Acked-by: Ard Biesheuvel Reviewed-by: Marcin Wojtas Reviewed-by: Philippe Mathieu-Daude --- Maintainers.txt | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/Maintainers.txt b/Maintainers.txt index 17613936c37e..

[edk2-devel] [PATCH edk2-platforms v2 1/5] Maintainers.txt: Update documentation for new format

2019-11-13 Thread Leif Lindholm
Synchronize header with edk2/Maintainers.txt and change references from "EDK II to EDK II Platforms". Cc: Ard Biesheuvel Cc: Michael D Kinney Signed-off-by: Leif Lindholm Acked-by: Ard Biesheuvel Reviewed-by: Philippe Mathieu-Daude --- Maintainers.txt | 40 ---

[edk2-devel] [PATCH edk2-platforms v2 2/5] Maintainers.txt: revamp file to machine-readable format

2019-11-13 Thread Leif Lindholm
Convert existing package maintainer/reviewer descriptions with adding the new path definitions. Delete the "Platform" and "Silicon" section maintainership descriptions. The EDK II Platforms maintainers section will now match that. Cc: Ard Biesheuvel Cc: Michael D Kinney Signed-off-by: Leif Lind

Re: [edk2-devel] EDK II Maintainers - EDK II CI is now active on edk2/master

2019-11-13 Thread Laszlo Ersek
On 11/13/19 17:23, Kinney, Michael D wrote: > Hi Laszlo, > > I am also reviewing a number of the PRs. > > If a PR is composed of a single commit, and the commit > message includes the reference to the Tianocore BZ, then > the web view of the PR looks good and includes the link > to the BZ. A cou

Re: [edk2-devel] EDK II Maintainers - EDK II CI is now active on edk2/master

2019-11-13 Thread Michael D Kinney
Hi Laszlo, I am also reviewing a number of the PRs. If a PR is composed of a single commit, and the commit message includes the reference to the Tianocore BZ, then the web view of the PR looks good and includes the link to the BZ. A couple examples: https://github.com/tianocore/edk2/pull/161 ht

Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib

2019-11-13 Thread Wang, Jian J
Laszlo, > -Original Message- > From: Laszlo Ersek > Sent: Wednesday, November 13, 2019 5:28 PM > To: Wang, Jian J ; devel@edk2.groups.io > Cc: Kinney, Michael D ; Gao, Liming > ; Ard Biesheuvel ; Ni, Ray > > Subject: Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib > > On

Re: [edk2-devel] [edk2] INF file : use conditional sentence

2019-11-13 Thread Tiger Liu(BJ-RD)
Based on INF Spec, maybe FeatureFlagExpress is a choice. When present, the feature flag expression determines whether the entry line is valid. If the feature flag expression evaluates to FALSE, this entry will be ignored by the EDK II build tools. For below example, is MSFT/GCC FeatureFlag ? [S

Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib

2019-11-13 Thread Laszlo Ersek
On 11/13/19 09:43, Wang, Jian J wrote: > As requested by Ray, I'd like to mention more usage about this library > instance. > > To be more specific, if following components or functionalities are used in a > platform, the BaseRngLibNull should *not* be used. Instead, a non-Null version > of RngLib

Re: [edk2-devel] EDK II Maintainers - EDK II CI is now active on edk2/master

2019-11-13 Thread Laszlo Ersek
Hi again, (+Ray) On 11/12/19 03:55, Michael D Kinney wrote: > EDK II Maintainers, > > EDK II CI Phase 1 feature is now active on edk2/master. > > Please use a GitHub pull request from a branch in a personal > fork of the edk2 repository with a 'push' label to request > a set of patches to be pu

[edk2-devel] [edk2] INF file : use conditional sentence

2019-11-13 Thread Tiger Liu(BJ-RD)
Dear All: I have a question about INF file written syntax. Could I use conditional sentence in INF file? For example: [Defines] .. [Sources] !ifdef ABC a.c !else b.c !endif .. Thanks 保密声明: 本邮件含有保密或专有信息,仅供指定收件人使用。严禁对本邮件或其内容做任何未经授权的查阅、使用、复制或转发。 CONFIDENTIAL NOTE: This email contains c

Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib

2019-11-13 Thread Wang, Jian J
As requested by Ray, I'd like to mention more usage about this library instance. To be more specific, if following components or functionalities are used in a platform, the BaseRngLibNull should *not* be used. Instead, a non-Null version of RngLib must be used (like BaseRngLib for IA32/X64, or com

Re: [edk2-devel] [PATCH V2 1/1] MdeModulePkg/Variable: Fix volatile variable RT cache update logic

2019-11-13 Thread Laszlo Ersek
On 11/13/19 00:40, Kubacki, Michael A wrote: > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2333 > > During a SetVariable () invocation, UpdateVariable () is called. > UpdateVariable () contains logic to determine whether a volatile or > non-volatile UEFI variable was set so the correspondin

Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib

2019-11-13 Thread Wang, Jian J
Laszlo, Thanks for the comments. I agree all of them. Since there's still EBC listed in SUPPORTED_ARCHITECTURES in dsc file, I don't see any reason not to add it. But likewise, I'll leave the final decision to package maintainers. Regards, Jian > -Original Message- > From: Laszlo Ersek

Re: [edk2-devel] [edk2-wiki PATCH 0/3] Release Planning updates for edk2-stable201911

2019-11-13 Thread Laszlo Ersek
On 11/13/19 09:24, Gao, Liming wrote: > Good catch. > > Reviewed-by: Liming Gao Thanks! Commit range 037b5adb5890..7ad574e68f9f. Laszlo >> -Original Message- >> From: Laszlo Ersek [mailto:ler...@redhat.com] >> Sent: Tuesday, November 12, 2019 6:01 PM >> To: edk2-devel-groups-io >> C

Re: [edk2-devel] [edk2-wiki PATCH 0/3] Release Planning updates for edk2-stable201911

2019-11-13 Thread Liming Gao
Good catch. Reviewed-by: Liming Gao >-Original Message- >From: Laszlo Ersek [mailto:ler...@redhat.com] >Sent: Tuesday, November 12, 2019 6:01 PM >To: edk2-devel-groups-io >Cc: Gao, Liming >Subject: [edk2-wiki PATCH 0/3] Release Planning updates for edk2- >stable201911 > >This short se

Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib

2019-11-13 Thread Laszlo Ersek
Hi Jian, thanks for the update. Comments below: On 11/13/19 06:35, Jian J Wang wrote: >> v2: >> - Change the name from RngLibNull to BaseRngLibNull according to Laszlo's >>comments >> - Move the module from SecurityPkg to MdePkg according to Laszlo's comments >> - Update commit message acc

Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib

2019-11-13 Thread Liming Gao
The change is good. Reviewed-by: Liming Gao >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Wang, Jian J >Sent: Wednesday, November 13, 2019 1:35 PM >To: devel@edk2.groups.io >Cc: Kinney, Michael D ; Gao, Liming >; Laszlo Ersek ; Ard Biesheuvel

[edk2-devel] test message, please ignore

2019-11-13 Thread Laszlo Ersek
sending this manually, to test an update in the RH email system, for the DMARC issue ("... via groups.io") thanks for the patience -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#50572): https://edk2.groups.io/g/devel/message/50572 Mute