Re: [edk2-devel] [edk2-platforms][PATCH V2 2/3] BoardModulePkg/FirmwareBootMediaInfoLib: Add library

2019-10-08 Thread Dong, Eric
Reviewed-by: Eric Dong > -Original Message- > From: Kubacki, Michael A > Sent: Saturday, October 5, 2019 1:29 AM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Gao, Liming > Subject: [edk2-platforms][PATCH V2 2/3] > BoardModulePkg/FirmwareBootMediaInfoLib: Add library > >

Re: [edk2-devel] [edk2-platforms][PATCH V2 3/3] BoardModulePkg/FirmwareBootMediaInfoPei: Add module

2019-10-08 Thread Dong, Eric
Reviewed-by: Eric Dong > -Original Message- > From: Kubacki, Michael A > Sent: Saturday, October 5, 2019 1:29 AM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Gao, Liming > Subject: [edk2-platforms][PATCH V2 3/3] > BoardModulePkg/FirmwareBootMediaInfoPei: Add module > >

Re: [edk2-devel] [edk2-platforms][PATCH V1 2/3] BoardModulePkg/FirmwareBootMediaInfoLib: Add library

2019-10-08 Thread Dong, Eric
Reviewed-by: Eric Dong > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 1, 2019 9:16 AM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Gao, Liming > Subject: [edk2-platforms][PATCH V1 2/3] > BoardModulePkg/FirmwareBootMediaInfoLib: Add library > >

Re: [edk2-devel] [PATCH v1 0/4] Support HTTPS HostName validation feature(CVE-2019-14553)

2019-10-08 Thread Wu, Jiaxin
Hi David, I just realized you have the comments on Bugzilla 960: >"...given that testing is failing and code inspection shows it would never >have been expected to work." Do you mean you didn't pass the verification if URLs with IPv6 literals (https://[2001:8b0:10b:1236::1]/)? Can you also

Re: [edk2-devel] [PATCH V2 1/9] MdeModulePkg/Variable: Consolidate common parsing functions

2019-10-08 Thread Wang, Jian J
Michael, One comment below. > -Original Message- > From: Kubacki, Michael A > Sent: Saturday, September 28, 2019 9:47 AM > To: devel@edk2.groups.io > Cc: Bi, Dandan ; Ard Biesheuvel > ; Dong, Eric ; Laszlo Ersek > ; Gao, Liming ; Kinney, Michael D > ; Ni, Ray ; Wang, Jian J > ; Wu, Hao

[edk2-devel] [PATCH 0/3] Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
Codes changes for SMBIOS spec 3.3.0 (Already public on DMTF). Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Abner Chang (3): MdeModulePkg: Update to support SmBios 3.3.0 MdePkg/Include: Update to support SmBios 3.3.0 ShellPkg/UefiShellDebug1CommandsLib: Update to

[edk2-devel] [PATCH v5 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
Shell command smbiosview updates for SMBIOS 3.3.0 spec. Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Signed-off-by: Abner Chang Cc: Jaben Carsey Cc: Ray Ni Cc: Zhichao Gao Cc: Leif Lindholm Cc: Gilbert Chen --- .../SmbiosView/PrintInfo.c | 23

[edk2-devel] [PATCH v5 1/3] MdeModulePkg: Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
This commit update the revision of SMBIOS spec reported in PCDs. Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Signed-off-by: Abner Chang Cc: Hao A Wu Cc: Jian J Wang Cc: Leif Lindholm Cc: Gilbert Chen --- MdeModulePkg/MdeModulePkg.dec | 4 ++-- 1 file changed, 2

[edk2-devel] [PATCH v5 2/3] MdePkg/Include: Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
Update SmBios.h to support SMBIOS 3.3.0 spec. Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Signed-off-by: Abner Chang Cc: Michael D Kinney Cc: Liming Gao Cc: Leif Lindholm Cc: Gilbert Chen --- MdePkg/Include/IndustryStandard/SmBios.h | 76

[edk2-devel] [PATCH v5 0/3] Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
Codes changes for SMBIOS spec 3.3.0 (Already public on DMTF). Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Abner Chang (3): MdeModulePkg: Update to support SmBios 3.3.0 MdePkg/Include: Update to support SmBios 3.3.0 ShellPkg/UefiShellDebug1CommandsLib: Update to

[edk2-devel] [PATCH 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
Shell command smbiosview updates for SMBIOS 3.3.0 spec. Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Signed-off-by: Abner Chang Cc: Jaben Carsey Cc: Ray Ni Cc: Zhichao Gao Cc: Leif Lindholm Cc: Gilbert Chen --- .../SmbiosView/PrintInfo.c | 23

[edk2-devel] [PATCH 2/3] MdePkg/Include: Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
Update SmBios.h to support SMBIOS 3.3.0 spec. Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Signed-off-by: Abner Chang Cc: Michael D Kinney Cc: Liming Gao Cc: Leif Lindholm Cc: Gilbert Chen --- MdePkg/Include/IndustryStandard/SmBios.h | 76

[edk2-devel] [PATCH 1/3] MdeModulePkg: Update to support SmBios 3.3.0

2019-10-08 Thread Abner Chang
This commit update the revision of SMBIOS spec reported in PCDs. Bugzilla link, https://bugzilla.tianocore.org/show_bug.cgi?id=2202 Signed-off-by: Abner Chang Cc: Hao A Wu Cc: Jian J Wang Cc: Leif Lindholm Cc: Gilbert Chen --- MdeModulePkg/MdeModulePkg.dec | 4 ++-- 1 file changed, 2

Re: [edk2-devel][edk2-test][PATCH 1/1] uefi-sct/SctPkg: SNP ReceiveFilters may be unsupported

2019-10-08 Thread Eric Jin
Push at 3d98de052fcd47f187190802e6201ac6aaabfda7 -Original Message- From: devel@edk2.groups.io On Behalf Of Eric Jin Sent: Monday, September 30, 2019 2:56 PM To: Heinrich Schuchardt ; EDK II Development Cc: Supreeth Venkatesh ; Stephano Cetola Subject: Re:

Re: [edk2-devel] [PATCH v2] MdePkg:Include: Update SmBios header file

2019-10-08 Thread Abner Chang
Thanks Liming, [PATCH v5] was sent. I added BZ link in commit message in each patch and reverted the changes on PcdSmbiosDocRev. Abner > -Original Message- > From: Gao, Liming [mailto:liming@intel.com] > Sent: Tuesday, October 8, 2019 9:15 AM > To: devel@edk2.groups.io; Chang,

Re: [edk2-devel] [PATCH v2] MdePkg:Include: Update SmBios header file

2019-10-08 Thread Leif Lindholm
Hi Liming, If you push this set (v5), could you possibly also push it to https://github.com/tianocore/edk2-staging/tree/RISC-V-V2 ? The three patches apply cleanly on this revision, and are a prerequisite for the code destined to go in there. Best Regards, Leif On Tue, Oct 08, 2019 at

[edk2-devel] [PATCH 0/4] UefiCpuPkg, OvmfPkg: separate PCDs for boot CPU count vs. max CPU count

2019-10-08 Thread Laszlo Ersek
Repo: https://github.com/lersek/edk2.git Branch: max_cpus_bz_1515 Ref:https://bugzilla.tianocore.org/show_bug.cgi?id=1515 UefiCpuPkg/MpInitLib currently lacks support for the following use case: - time-limited AP enumeration is not reliable on the platform (individual AP check-in may

[edk2-devel] [PATCH 2/4] OvmfPkg/OvmfXen.dsc: remove PcdCpu* dynamic defaults

2019-10-08 Thread Laszlo Ersek
PcdCpuMaxLogicalProcessorNumber and PcdCpuApInitTimeOutInMicroSeconds are only referenced in "OvmfPkg/PlatformPei/PlatformPei.inf", and OvmfXen does not include that module. Remove the unnecessary dynamic PCD defaults from "OvmfXen.dsc". Cc: Anthony Perard Cc: Ard Biesheuvel Cc: Igor Mammedov

[edk2-devel] [PATCH 4/4] OvmfPkg/PlatformPei: rewrite MaxCpuCountInitialization() for CPU hotplug

2019-10-08 Thread Laszlo Ersek
MaxCpuCountInitialization() currently handles the following options: (1) QEMU does not report the boot CPU count. In this case, PlatformPei makes MpInitLib enumerate APs up to the default PcdCpuMaxLogicalProcessorNumber value (64) minus 1, or until the default

[edk2-devel] [PATCH 3/4] OvmfPkg/IndustryStandard: define FW_CFG_X86_TOPOLOGY structure type

2019-10-08 Thread Laszlo Ersek
pc-q35-4.2+ and pc-i440fx-4.2+ machine types report CPU topology info in the "etc/x86-smp-topology" fw_cfg file. Add a structure type for parsing this blob. Cc: Ard Biesheuvel Cc: Igor Mammedov Cc: Jordan Justen Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1515 Signed-off-by: Laszlo

[edk2-devel] [PATCH 1/4] UefiCpuPkg/MpInitLib: honor the platform's boot CPU count in AP detection

2019-10-08 Thread Laszlo Ersek
If a platform boots with a CPU topology that is not fully populated (that is, the boot CPU count is smaller than PcdCpuMaxLogicalProcessorNumber), then the platform cannot use the "fast AP detection" logic added in commit 6e1987f19af7. Said logic depends on the boot CPU count being equal to

Re: [edk2-devel] [PATCH 0/4] UefiCpuPkg, OvmfPkg: separate PCDs for boot CPU count vs. max CPU count

2019-10-08 Thread Ard Biesheuvel
On Tue, 8 Oct 2019 at 13:27, Laszlo Ersek wrote: > > Repo: https://github.com/lersek/edk2.git > Branch: max_cpus_bz_1515 > Ref:https://bugzilla.tianocore.org/show_bug.cgi?id=1515 > > UefiCpuPkg/MpInitLib currently lacks support for the following use case: > > - time-limited AP enumeration

Re: [edk2-devel] [PATCH 2/4] OvmfPkg/OvmfXen.dsc: remove PcdCpu* dynamic defaults

2019-10-08 Thread Philippe Mathieu-Daudé
On 10/8/19 1:27 PM, Laszlo Ersek wrote: PcdCpuMaxLogicalProcessorNumber and PcdCpuApInitTimeOutInMicroSeconds are only referenced in "OvmfPkg/PlatformPei/PlatformPei.inf", and OvmfXen does not include that module. Remove the unnecessary dynamic PCD defaults from "OvmfXen.dsc". Cc: Anthony

Re: [edk2-devel] [PATCH v5 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0

2019-10-08 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey Thanks -Jaben > -Original Message- > From: Abner Chang [mailto:abner.ch...@hpe.com] > Sent: Tuesday, October 08, 2019 1:04 AM > To: devel@edk2.groups.io > Cc: abner.ch...@hpe.com; Carsey, Jaben ; Ni, > Ray ; Gao, Zhichao ; Leif > Lindholm ; Gilbert Chen >

Re: [edk2-devel] [Patch 11/12] OvmfPkg: Enable CLANG9 tool chain

2019-10-08 Thread Liming Gao
Laszlo: > -Original Message- > From: Laszlo Ersek > Sent: Tuesday, October 1, 2019 4:42 AM > To: devel@edk2.groups.io; Gao, Liming > Subject: Re: [edk2-devel] [Patch 11/12] OvmfPkg: Enable CLANG9 tool chain > > Hi Liming, > > On 09/27/19 09:46, Liming Gao wrote: > > 1. Apply CLANG9

Re: [edk2-devel] [edk2-platforms: PATCH] Marvell/Drivers: MvPhyDxe: Execute proper initialization callback

2019-10-08 Thread Leif Lindholm
On Mon, Oct 07, 2019 at 02:02:26AM +0200, Marcin Wojtas wrote: > Hitherto code was executing by mistake the first > possible initialization routine (valid for 88E1512 PHY), > regardless the setting in gMarvellTokenSpaceGuid.PcdPhyDeviceIds. > Fix this. > > Signed-off-by: Marcin Wojtas

Re: [edk2-devel] [edk2-platforms][PATCH V1 05/17] KabylakeOpenBoardPkg/GalagoPro3: Remove OpenBoardPkgConfig.dsc

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Jeremy Soller > Subject: [edk2-platforms][PATCH V1 05/17] > KabylakeOpenBoardPkg/GalagoPro3:

Re: [edk2-devel] [edk2-platforms][PATCH V1 04/17] KabylakeOpenBoardPkg/GalagoPro3: Relocate PCD DSC include

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Jeremy Soller > Subject: [edk2-platforms][PATCH V1 04/17] > KabylakeOpenBoardPkg/GalagoPro3:

Re: [edk2-devel] [edk2-platforms][PATCH V1 10/17] CoffeelakeSiliconPkg: Add DSC include file section tags

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chaganty, Rangasai V ; Chiu, Chasel > > Subject: [edk2-platforms][PATCH V1 10/17] CoffeelakeSiliconPkg: Add DSC > include file section

Re: [edk2-devel] Problem with decompression on EDK2

2019-10-08 Thread Liming Gao
BaseTools VolInfo tool. Its input is the binary FvImage file. In the same directory, GuidedSectionTools.txt (built in the FV directory) should be here. Thanks Liming > -Original Message- > From: Loh, Tien Hock > Sent: Tuesday, October 1, 2019 10:34 AM > To: Gao, Liming ;

Re: [edk2-devel] [edk2-platforms][PATCH V1 03/17] KabylakeOpenBoardPkg/KabylakeRvp3: DSC cleanup

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V1 03/17] > KabylakeOpenBoardPkg/KabylakeRvp3: DSC cleanup > >

Re: [edk2-devel] [Patch 05/12] MdePkg BaseIoLibIntrinsic: Remove __inline__ attribute for IO functions

2019-10-08 Thread Liming Gao
From: devel@edk2.groups.io On Behalf Of Andrew Fish via Groups.Io Sent: Tuesday, October 1, 2019 5:12 AM To: devel@edk2.groups.io; ler...@redhat.com Cc: Gao, Liming Subject: Re: [edk2-devel] [Patch 05/12] MdePkg BaseIoLibIntrinsic: Remove __inline__ attribute for IO functions On Sep 30,

Re: [edk2-devel] [PATCH 4/4] OvmfPkg/PlatformPei: rewrite MaxCpuCountInitialization() for CPU hotplug

2019-10-08 Thread Igor Mammedov
On Tue, 8 Oct 2019 13:27:14 +0200 Laszlo Ersek wrote: > MaxCpuCountInitialization() currently handles the following options: > > (1) QEMU does not report the boot CPU count. > > In this case, PlatformPei makes MpInitLib enumerate APs up to the > default PcdCpuMaxLogicalProcessorNumber

Re: [edk2-devel] [edk2-platforms][PATCH V1 11/17] WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: Update FSP base PCDs

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V1 11/17] > WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: Update FSP

Re: [edk2-devel] [PATCH 1/1] MdeModulePkg/BdsDxe: Also call PlatformBootManagerWaitCallback on 0

2019-10-08 Thread Liming Gao
Push @2de1f611be06ded3a59726a4052a9039be7d459b > -Original Message- > From: devel@edk2.groups.io On Behalf Of Laszlo Ersek > Sent: Tuesday, October 1, 2019 7:12 AM > To: devel@edk2.groups.io; Ni, Ray ; p...@akeo.ie > Cc: Gao, Zhichao > Subject: Re: [edk2-devel] [PATCH 1/1]

[edk2-devel] [PATCH v1 0/1] Remove UNDI calls from SNP during ExitBootServices

2019-10-08 Thread Rabeda, Maciej
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1974 Removing calls to UNDI->Shutdown() and Stop() functions in ExitBootServices event notification function. Since SNP event reacting to ExitBootServices is only calling those two functions, I am removing that as well. Cc: Siyuan Fu Cc:

[edk2-devel] [PATCH v1 1/1] NetworkPkg/SnpDxe: Remove ExitBootServices event

2019-10-08 Thread Rabeda, Maciej
Patch addresses Bugzilla #1972. During ExitBootServices stage, drivers should not call any functions known to use Memory Allocation Services. One of such functions (as per UEFI spec) is UNDI->Shutdown(). Since UNDI drivers during ExitBootServices phase are expected to put the adapter to such a

Re: [edk2-devel] [edk2-platforms][PATCH V1 09/17] WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: Remove OpenBoardPkgConfig.dsc

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V1 09/17] > WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: Remove >

Re: [edk2-devel] [edk2-platforms][PATCH V1 08/17] WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: Relocate PCD DSC include

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V1 08/17] > WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: Relocate

Re: [edk2-devel] [PATCH v5 0/3] Update to support SmBios 3.3.0

2019-10-08 Thread Liming Gao
Reviewed-by: Liming Gao > -Original Message- > From: devel@edk2.groups.io On Behalf Of Abner Chang > Sent: Tuesday, October 8, 2019 4:04 PM > To: devel@edk2.groups.io > Cc: abner.ch...@hpe.com > Subject: [edk2-devel] [PATCH v5 0/3] Update to support SmBios 3.3.0 > > Codes changes for

Re: [edk2-devel] [PATCH v2] MdePkg:Include: Update SmBios header file

2019-10-08 Thread Liming Gao
OK. I will push them to edk2-staging/tree/RISC-V-V2. Thanks Liming > -Original Message- > From: Leif Lindholm > Sent: Tuesday, October 8, 2019 6:11 PM > To: Gao, Liming > Cc: devel@edk2.groups.io; abner.ch...@hpe.com; Kinney, Michael D > ; Chen, Gilbert > > Subject: Re: [edk2-devel]

Re: [edk2-devel] [edk2-platforms][PATCH V1 02/17] KabylakeSiliconPkg: Add DSC include file section tags

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chaganty, Rangasai V ; Chiu, Chasel > > Subject: [edk2-platforms][PATCH V1 02/17] KabylakeSiliconPkg: Add DSC > include file section tags

Re: [edk2-devel] [edk2-platforms][PATCH V1 12/17] WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: DSC cleanup

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V1 12/17] > WhiskeylakeOpenBoardPkg/WhiskeylakeURvp: DSC cleanup

Re: [edk2-devel] [Patch 12/12] OvmfPkg SecMain: Add build option "-fno-omit-frame-pointer" for CLANG9 X64

2019-10-08 Thread Liming Gao
Laszlo: I will verify your option (a) and (b). The problem is described in https://bugzilla.tianocore.org/show_bug.cgi?id=2024. I know there are some discussion in Jordan patch for PeiCore change. Before the conclusion for Jordan change is made, I expect to find the alternate option to

Re: [edk2-devel] [edk2-platforms][PATCH V1 01/17] KabylakeOpenBoardPkg/KabylakeRvp3: Remove OpenBoardPkgConfig.dsc

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:16 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V1 01/17] > KabylakeOpenBoardPkg/KabylakeRvp3: Remove

Re: [edk2-devel] Problem with decompression on EDK2

2019-10-08 Thread Loh, Tien Hock
Ok, noted with thanks Liming. I'll try the tools out. Thanks Tien Hock > -Original Message- > From: Gao, Liming > Sent: Tuesday, October 8, 2019 10:04 PM > To: Loh, Tien Hock ; devel@edk2.groups.io; > ler...@redhat.com > Cc: leif.lindh...@linaro.org; ard.biesheu...@linaro.org >

Re: [edk2-devel] [edk2-platforms][PATCH V1 06/17] KabylakeOpenBoardPkg/GalagoPro3: DSC cleanup

2019-10-08 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Jeremy Soller > Subject: [edk2-platforms][PATCH V1 06/17] > KabylakeOpenBoardPkg/GalagoPro3: DSC

Re: [edk2-devel] [PATCH 2/4] OvmfPkg/OvmfXen.dsc: remove PcdCpu* dynamic defaults

2019-10-08 Thread Laszlo Ersek
On 10/08/19 15:11, Philippe Mathieu-Daudé wrote: > On 10/8/19 1:27 PM, Laszlo Ersek wrote: >> PcdCpuMaxLogicalProcessorNumber and PcdCpuApInitTimeOutInMicroSeconds are >> only referenced in "OvmfPkg/PlatformPei/PlatformPei.inf", and OvmfXen >> does >> not include that module. Remove the

Re: [edk2-devel] [PATCH V2 3/9] MdeModulePkg/Variable: Parameterize VARIABLE_INFO_ENTRY buffer

2019-10-08 Thread Kubacki, Michael A
I will clarify that aspect in the commit message. Thanks, Michael > -Original Message- > From: Wu, Hao A > Sent: Monday, October 7, 2019 7:12 PM > To: devel@edk2.groups.io; Kubacki, Michael A > Cc: Bi, Dandan ; Ard Biesheuvel > ; Dong, Eric ; Laszlo Ersek > ; Gao, Liming ; Kinney,

Re: [edk2-devel] [PATCH V2 1/9] MdeModulePkg/Variable: Consolidate common parsing functions

2019-10-08 Thread Kubacki, Michael A
Reply inline. > -Original Message- > From: Wang, Jian J > Sent: Monday, October 7, 2019 11:08 PM > To: Kubacki, Michael A ; > devel@edk2.groups.io > Cc: Bi, Dandan ; Ard Biesheuvel > ; Dong, Eric ; Laszlo Ersek > ; Gao, Liming ; Kinney, Michael > D ; Ni, Ray ; Wu, Hao A > ; Yao, Jiewen

Re: [edk2-devel] [Patch 05/12] MdePkg BaseIoLibIntrinsic: Remove __inline__ attribute for IO functions

2019-10-08 Thread Laszlo Ersek
On 10/08/19 16:47, Gao, Liming wrote: > [Liming] I verify GCC5 tool chain. I will verify GCC48/GCC49 and > XCODE5. > > I don’t know the specific reason about __inline__. If there is no > impact on > > other GCC tool chain, I prefer to remove them. > [Liming] This seems the

Re: [edk2-devel] [PATCH 4/4] OvmfPkg/PlatformPei: rewrite MaxCpuCountInitialization() for CPU hotplug

2019-10-08 Thread Laszlo Ersek
On 10/08/19 17:06, Igor Mammedov wrote: > On Tue, 8 Oct 2019 13:27:14 +0200 > Laszlo Ersek wrote: > >> MaxCpuCountInitialization() currently handles the following options: >> >> (1) QEMU does not report the boot CPU count. >> >> In this case, PlatformPei makes MpInitLib enumerate APs up to

[edk2-devel] [staging/edk2-ci]: Create branch to test EDK II CI concepts

2019-10-08 Thread Michael D Kinney
Hello, I am creating the edk2-ci branch in the edk2-staging repo to test out some of the concepts from the RFCs to add CI to the edk2 repo. https://edk2.groups.io/g/rfc/message/93 https://edk2.groups.io/g/rfc/message/96 Best regards, Mike -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive

Re: [edk2-devel] [Patch 11/12] OvmfPkg: Enable CLANG9 tool chain

2019-10-08 Thread Laszlo Ersek
On 10/08/19 17:02, Gao, Liming wrote: > Laszlo: > >> -Original Message- >> From: Laszlo Ersek >> Sent: Tuesday, October 1, 2019 4:42 AM >> To: devel@edk2.groups.io; Gao, Liming >> Subject: Re: [edk2-devel] [Patch 11/12] OvmfPkg: Enable CLANG9 tool chain >> >> Hi Liming, >> >> On

Re: [edk2-devel] [PATCH 0/4] UefiCpuPkg, OvmfPkg: separate PCDs for boot CPU count vs. max CPU count

2019-10-08 Thread Laszlo Ersek
On 10/08/19 13:35, Ard Biesheuvel wrote: > On Tue, 8 Oct 2019 at 13:27, Laszlo Ersek wrote: >> >> Repo: https://github.com/lersek/edk2.git >> Branch: max_cpus_bz_1515 >> Ref:https://bugzilla.tianocore.org/show_bug.cgi?id=1515 >> >> UefiCpuPkg/MpInitLib currently lacks support for the

Re: [edk2-devel] [PATCH V2 1/9] MdeModulePkg/Variable: Consolidate common parsing functions

2019-10-08 Thread Kubacki, Michael A
The rationale for moving the generic parsing functions out would be better to capture in the commit message. That will be added to V3. Thanks, Michael > -Original Message- > From: Wu, Hao A > Sent: Monday, October 7, 2019 7:12 PM > To: Kubacki, Michael A ; > devel@edk2.groups.io > Cc:

[edk2-devel] Cancelled Event: TianoCore Design / Bug Triage - EMEA #cal-cancelled

2019-10-08 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:CANCEL CALSCALE:GREGORIAN BEGIN:VEVENT STATUS:CANCELLED UID:calendar.15...@groups.io DTSTAMP:20191009T000616Z ORGANIZER;CN=Stephano Cetola:mailto:stephano.cet...@linux.intel.com DTSTART:20190529T15Z

Re: [edk2-devel] [PATCH 1/4] UefiCpuPkg/MpInitLib: honor the platform's boot CPU count in AP detection

2019-10-08 Thread Dong, Eric
Hi Laszlo, > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Laszlo Ersek > Sent: Tuesday, October 8, 2019 7:27 PM > To: edk2-devel-groups-io > Cc: Dong, Eric ; Igor Mammedov > ; Ni, Ray > Subject: [edk2-devel] [PATCH 1/4]

Re: [edk2-devel] [edk2-platforms][PATCH V1 14/17] SimicsOpenBoardPkg/BoardX58Ich10: Relocate DSC includes

2019-10-08 Thread Agyeman, Prince
Reviewed-by: Prince Agyeman -Original Message- From: Kubacki, Michael A Sent: Monday, October 7, 2019 10:17 PM To: devel@edk2.groups.io Cc: Agyeman, Prince ; Wei, David Y Subject: [edk2-platforms][PATCH V1 14/17] SimicsOpenBoardPkg/BoardX58Ich10: Relocate DSC includes This change

Re: [edk2-devel] [PATCH v1 1/1] NetworkPkg/SnpDxe: Remove ExitBootServices event

2019-10-08 Thread Siyuan, Fu
Hi, Maciej Have you tested what will happen if this SNP co-work with those UNDI drivers which don't have an ExitBootService event callback to shut down its DMA activity? And what's the impact to OS if UNDI's DMA is not shut down? Best Regards Siyuan > -Original Message- > From:

Re: [edk2-devel] [PATCH 00/35] edk2: clean up the usage of standardized (VOID*) typedefs

2019-10-08 Thread Laszlo Ersek
On 09/17/19 21:49, Laszlo Ersek wrote: > Repository: https://github.com/lersek/edk2.git > Branch: voidptr > > The UEFI / PI / Shell specifications define a number of standard types > as pointers to VOID. This is arguably a design mistake; those types > should have been pointers to distinct

Re: [edk2-devel] [edk2-platforms][PATCH V1 16/17] SimicsOpenBoardPkg/BoardX58Ich10: DSC cleanup

2019-10-08 Thread Agyeman, Prince
Reviewed-by: Prince Agyeman -Original Message- From: Kubacki, Michael A Sent: Monday, October 7, 2019 10:17 PM To: devel@edk2.groups.io Cc: Agyeman, Prince ; Wei, David Y Subject: [edk2-platforms][PATCH V1 16/17] SimicsOpenBoardPkg/BoardX58Ich10: DSC cleanup

Re: [edk2-devel] [edk2-platforms][PATCH V1 17/17] SimicsOpenBoardPkg: Assign unique token namespace

2019-10-08 Thread Agyeman, Prince
Reviewed-by: Prince Agyeman -Original Message- From: Kubacki, Michael A Sent: Monday, October 7, 2019 10:17 PM To: devel@edk2.groups.io Cc: Agyeman, Prince ; Wei, David Y Subject: [edk2-platforms][PATCH V1 17/17] SimicsOpenBoardPkg: Assign unique token namespace

Re: [edk2-devel] [edk2-platforms][PATCH V1 15/17] SimicsOpenBoardPkg/BoardX58Ich10: Remove OpenBoardPkgConfig.dsc

2019-10-08 Thread Agyeman, Prince
Reviewed-by: Prince Agyeman -Original Message- From: Kubacki, Michael A Sent: Monday, October 7, 2019 10:17 PM To: devel@edk2.groups.io Cc: Agyeman, Prince ; Wei, David Y Subject: [edk2-platforms][PATCH V1 15/17] SimicsOpenBoardPkg/BoardX58Ich10: Remove OpenBoardPkgConfig.dsc

Re: [edk2-devel] [PATCH v5 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0

2019-10-08 Thread Gao, Zhichao
How about change the function name from ProcessorArchitectureType to DisplayProcessorArchitectureType? And why separate the processor arch type display into two section: i.e. first display " ProcessorSpecificBlock.ProcessorArchType: 0xXX" thru PRINT_STRUCT_VALUE_H, second display the value's

Re: [edk2-devel] [PATCH 18/35] NetworkPkg/DxeNetLib: fix type typo in NetLibGetMacAddress()

2019-10-08 Thread Laszlo Ersek
On 10/08/19 02:32, Fu, Siyuan wrote: > Just return from vacation. Sorry for late response. > > Reviewed-by: Siyuan Fu Thank you and Mike both! Laszlo >> -Original Message- >> From: devel@edk2.groups.io On Behalf Of Philippe >> Mathieu-Daudé >> Sent: 2019年9月24日 19:00 >> To:

Re: [edk2-devel] Cancelled Event: TianoCore Design / Bug Triage - EMEA #cal-cancelled

2019-10-08 Thread Stephano Cetola
I'm going to cancel these meetings going forward as we have not been receiving regular designs to review. If I do get any design review requests, I will schedule this meeting on a "as needed" basis. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group.

[edk2-devel] [edk2-platforms][PATCH v2 4/5] Platform/RPi3/PlatformSmbiosDxe: Populate BIOS major/minor from PCD

2019-10-08 Thread Pete Batard
String parsing code is added to BIOSInfoUpdateSmbiosType0() so that any numeric "x.y" value being passed in PcdFirmwareVersionString is now used to populate the BIOS major and minor. Signed-off-by: Pete Batard --- Platform/RaspberryPi/RPi3/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c | 44

[edk2-devel] [edk2-platforms][PATCH v2 2/5] Platform/RPi3/RpiFirmwareDxe: Improve serial number population

2019-10-08 Thread Pete Batard
Improve RpiFirmwareGetSerial() to derive a serial number from the MAC address, in case the platform returns 0 for the serial number. Also fix a typo where "%s" was used instead of "%a". Signed-off-by: Pete Batard --- Platform/RaspberryPi/RPi3/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c | 11

[edk2-devel] [edk2-platforms][PATCH v2 0/5] Platform/RPi3: Various SMBIOS improvements

2019-10-08 Thread Pete Batard
v1 can be found at https://edk2.groups.io/g/devel/message/46806 - 46808 Changes for v2: - Split *functional* changes into multipe commits - Spell out serial number - *NEW* Derive Type 19 addresses from board revision (RPi4 improvement) Also, with regards to further splitting that was requested

[edk2-devel] [edk2-platforms][PATCH v2 3/5] Platform/RPi3/PlatformSmbiosDxe: Improve population of SMBIOS entries

2019-10-08 Thread Pete Batard
This patch cleans up the population SMBIOS entries by removing elements that we don't have data for, as well as properly filling the ones for which we do, through the newly added queries from RpiFirmwareDxe. Additional minor improvements are also applied, such as consistent use of uppercase

[edk2-devel] [edk2-platforms][PATCH v2 5/5] Platform/RPi3/PlatformSmbiosDxe: Derive RAM size from board revision

2019-10-08 Thread Pete Batard
The board revision is the proper channel to use to detect the amount of RAM available as bits [20-22] report the effective RAM size for the board starting with 256 MB (000b) and doubling in size for each value. Signed-off-by: Pete Batard ---

[edk2-devel] [edk2-platforms][PATCH v2 1/5] Platform/RPi3/RpiFirmwareDxe: Add more query functions

2019-10-08 Thread Pete Batard
This patch introduces the capability to also query the Model Name/ Manufacturer Name/CPU Name/Firmware Revision using the RpiFirmware protocol. This is aims at making the driver more suitable to cater for platforms other than the Raspberry Pi 3 as well as simplifying the population of entries in