Re: [edk2-devel] [PATCH v3 2/6] OvmfPkg/FdtPciHostBridgeLib: io range is not mandatory

2022-04-12 Thread Abner Chang
> -Original Message- > From: Gerd Hoffmann > Sent: Thursday, April 7, 2022 5:33 PM > To: devel@edk2.groups.io > Cc: Leif Lindholm ; Ard Biesheuvel > ; Jordan Justen ; > Jian J Wang ; Ray Ni ; Hao A Wu > ; Gerd Hoffmann ; Jiewen Yao > ; Liming Gao ; Pawel > Polawski ; Chang, Abner (HPS S

Re: [edk2-devel] [PATCH v3 2/6] OvmfPkg/FdtPciHostBridgeLib: io range is not mandatory

2022-04-12 Thread Gerd Hoffmann
Hi, > > - Io.Base = IoBase; > > - Io.Limit = IoBase + IoSize - 1; > > + if (IoSize) { > > +Io.Base = IoBase; > > +Io.Limit = IoBase + IoSize - 1; > > + } else { > > +Io.Base = MAX_UINT64; > > +Io.Limit = 0; > HI Gerd, > Does the consumer of Io (in PCI_ROOT_BRIDGE structu

Re: [edk2-devel] [staging/LoongArch RESEND PATCH v1 19/33] MdePkg: Add LoongArch LOONGARCH64 binding

2022-04-12 Thread Chao Li
Hi Abner, Do I have to remove a "_" at the head and tail? All I found in MdePkg is that *.h files have two "_" at the head and tail. If you are sure that EDK II coding style must contain only one "_" at the head and tail, I will modify it. -- Thanks, Chao On 4月 7 2022,

Re: [edk2-devel] [staging/LoongArch RESEND PATCH v1 20/33] MdePkg/Include: LoongArch definitions.

2022-04-12 Thread Chao Li
Hi Abner, Yes, the PE/COFF has been adapted to LoongArch32 and LoongArch64, please refer to: https://docs.microsoft.com/en-us/windows/win32/debug/pe-format (https://link.getmailspring.com/link/9a61f0c8-a7fa-41ad-b0f6-769ec7687...@getmailspring.com/0?redirect=https%3A%2F%2Fdocs.microsoft.com%2Fen-

Re: [edk2-devel] [staging/LoongArch RESEND PATCH v1 13/33] BaseTools: BaseTools changes for LoongArch platform.

2022-04-12 Thread Chao Li
Hi Anber, Please check my reply in the mail. -- Thanks, Chao On 4月 8 2022, at 2:25 下午, "Chang, Abner (HPS SW/FW Technologist)" wrote: > > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Chao Li > > Sent: Wednesday, February 9, 2022 2:55 PM >

Re: [edk2-devel] [staging/LoongArch RESEND PATCH v1 24/33] MdePkg/BasePeCoff: Add LoongArch PE/Coff related code.

2022-04-12 Thread Chao Li
-- Thanks, Chao On 4月 8 2022, at 5:38 下午, "Chang, Abner (HPS SW/FW Technologist)" wrote: > > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Chao Li > > Sent: Wednesday, February 9, 2022 2:56 PM > > To: devel@edk2.groups.io > > Cc: Michael D

Re: [edk2-devel] [staging/LoongArch RESEND PATCH v1 26/33] MdePkg/BaseSynchronizationLib: LoongArch cache related code.

2022-04-12 Thread Chao Li
Hi Abner, All of the feedback that you pointed out I will fix it in the next version, thank you! -- Thanks, Chao On 4月 8 2022, at 7:02 晚上, "Chang, Abner (HPS SW/FW Technologist)" wrote: > > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Ch

[edk2-devel] [PATCH v2 1/1] MdePkg/Include: Use DEBUG_FILE_PATH to specify debug file path.

2022-04-12 Thread Guomin Jiang
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3840 Use DEBUG_FILE_PATH to control ASSERT path Motivation and Goal: 1. Make replication build more easy and less toolchain dependency 2. Consume the ASSERT string easy for downstream 3. Make code more clear Cc: Michael D Kinney Cc: Liming Gao

Re: [edk2-devel] [staging/LoongArch RESEND PATCH v1 30/33] MdeModulePkg/DxeIplPeim : LoongArch DxeIPL implementation.

2022-04-12 Thread Chao Li
Hi Abner, Yes, I think it might be a coincidence, in fact, we have been using the UDK2018 since 2018, and this code are not been modified after adding it. I think maybe be our are all refer to the ARM architecture. Looking at this code, the function HandOffToDxeCore just switches from the PEI st

Re: [edk2-devel] [edk2-libc Patch 1/1] AppPkg\Applications\Python\Python-3.6.8\Lib: uuid.py module port for UEFI environment

2022-04-12 Thread Jayaprakash, N
Hi Mike, This patch is yet to be merged. Could you look into this? Regards, JP -Original Message- From: Kinney, Michael D Sent: 08 April 2022 23:30 To: Jayaprakash, N ; devel@edk2.groups.io; Kinney, Michael D Cc: Frinzell, Aaron Subject: RE: [edk2-devel] [edk2-libc Patch 1/1] AppPk

[edk2-devel] [PATCH v2 04/10] MdePkg/CompilerIntrinsicsLib: move ia32 intrinsics and strcmp

2022-04-12 Thread Gerd Hoffmann
Move intrinsics for ia32 from CryptoPkg/Library/IntrinsicLib to MdePkg/Library/CompilerIntrinsicsLib. Also move strcmp. Remove CryptoPkg/Library/IntrinsicLib. Use MdePkg/Library/CompilerIntrinsicsLib instead. Update all references. Signed-off-by: Gerd Hoffmann Acked-by: Jiewen Yao Acked-by: Ar

[edk2-devel] [PATCH v2 00/10] consolidate compiler intrinsics

2022-04-12 Thread Gerd Hoffmann
This is an attept to start cleaning up the messy compiler intrinsics situation. Today we don't have a core intrinsics library, resulting in everybody creating their own. ArmPkg has one, CryptoPkg has one. I'm sure there are many more. This doesn't make sense. Given we can't avoid compiler intri

[edk2-devel] [PATCH v2 01/10] MdePkg: promote CompilerIntrinsicsLib from ArmPkg to MdePkg

2022-04-12 Thread Gerd Hoffmann
First step in creating a single, common compiler intrinsics library for everybody who needs it. Move the ArmPkg library to MdePkg, update all references. No code changes. Signed-off-by: Gerd Hoffmann Acked-by: Jiewen Yao Acked-by: Ard Biesheuvel --- ArmVirtPkg/ArmVirt.dsc.inc

[edk2-devel] [PATCH v2 03/10] MdePkg/CompilerIntrinsicsLib: move size_t typedef to header file

2022-04-12 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- .../{memset_ms.c => types_ms.h} | 29 +-- .../Library/CompilerIntrinsicsLib/memcpy_ms.c | 6 +--- .../CompilerIntrinsicsLib/memmove_ms.c| 6 +--- .../Library/CompilerIntrinsicsLib/memset_ms.c | 6 +--- 4 files changed, 4 in

[edk2-devel] [PATCH v2 05/10] MdePkg/CompilerIntrinsicsLib: copy over intrinsics from edk2-libc repo

2022-04-12 Thread Gerd Hoffmann
Copy over unmodified (except for running through uncrustify), from https://github.com/tianocore/edk2-libc/tree/master/StdLib/LibC/CRT Signed-off-by: Gerd Hoffmann --- .../Library/CompilerIntrinsicsLib/Ia32/Gcc.c | 283 ++ .../CompilerIntrinsicsLib/Ia32/lldiv.c| 98 +

[edk2-devel] [PATCH v2 07/10] MdePkg/CompilerIntrinsicsLib: drop include from Gcc.c

2022-04-12 Thread Gerd Hoffmann
Does not exist and is not needed. Signed-off-by: Gerd Hoffmann --- MdePkg/Library/CompilerIntrinsicsLib/Ia32/Gcc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/MdePkg/Library/CompilerIntrinsicsLib/Ia32/Gcc.c b/MdePkg/Library/CompilerIntrinsicsLib/Ia32/Gcc.c index bb211022c0e4..d0996d6276b

[edk2-devel] [PATCH v2 02/10] MdePkg/CompilerIntrinsicsLib: fix msft sources for x64

2022-04-12 Thread Gerd Hoffmann
Update #if conditions so the 64bit version is picked for X64 too. Signed-off-by: Gerd Hoffmann Acked-by: Jiewen Yao Acked-by: Ard Biesheuvel --- MdePkg/Library/CompilerIntrinsicsLib/memcmp_ms.c | 2 +- MdePkg/Library/CompilerIntrinsicsLib/memcpy_ms.c | 2 +- MdePkg/Library/CompilerIntrinsics

[edk2-devel] [PATCH v2 08/10] MdePkg/CompilerIntrinsicsLib: drop debug logging from Gcc.c

2022-04-12 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- .../Library/CompilerIntrinsicsLib/Ia32/Gcc.c | 22 --- 1 file changed, 22 deletions(-) diff --git a/MdePkg/Library/CompilerIntrinsicsLib/Ia32/Gcc.c b/MdePkg/Library/CompilerIntrinsicsLib/Ia32/Gcc.c index d0996d6276bc..3c47bef63c4d 100644 --- a/M

[edk2-devel] [PATCH v2 10/10] MdePkg/CompilerIntrinsicsLib: add new sources to CompilerIntrinsicsLib.inf

2022-04-12 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- .../CompilerIntrinsicsLib.inf | 20 +++ 1 file changed, 20 insertions(+) diff --git a/MdePkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf b/MdePkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf index 1604f1ee80

[edk2-devel] [PATCH v2 06/10] MdePkg/CompilerIntrinsicsLib: add SPDX License tags

2022-04-12 Thread Gerd Hoffmann
It's not an exact match, edk2 expects "BSD-2-Clause-Patent" but http://opensource.org/licenses/bsd-license.php references in comments is "BSD-2-Clause". Guess I need an explicit Intel explicitly agreeing to this even though that is a rather minor change ... Signed-off-by: Gerd Hoffmann --- MdeP

[edk2-devel] [PATCH v2 09/10] MdePkg/CompilerIntrinsicsLib: remove duplicate functions from Gcc.c

2022-04-12 Thread Gerd Hoffmann
Implementations for those functions already exist, so remove them to avoid duplicate symbols. Signed-off-by: Gerd Hoffmann --- .../Library/CompilerIntrinsicsLib/Ia32/Gcc.c | 47 --- 1 file changed, 47 deletions(-) diff --git a/MdePkg/Library/CompilerIntrinsicsLib/Ia32/Gcc.c b/

[edk2-devel] [Patch] pip-requirements.txt: Update basetools version to 0.1.17

2022-04-12 Thread Bob Feng
Synced the basetools patch from edk2 repo to edk2-basetools repo. Update the basetools pip module version to the latest. Signed-off-by: Bob Feng Cc: Sean Brogan Cc: Bret Barkelew Cc: Michael D Kinney Cc: Liming Gao --- pip-requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(

Re: [edk2-devel] [PATCH v3 0/8] CryptoPkg updates for openssl 3.0

2022-04-12 Thread Yao, Jiewen
Thanks for the contribution. Merged https://github.com/tianocore/edk2/pull/2766 > -Original Message- > From: Gerd Hoffmann > Sent: Monday, April 11, 2022 7:25 PM > To: devel@edk2.groups.io > Cc: Oliver Steffen ; Pawel Polawski > ; Wang, Jian J ; Xiaoyu Lu > ; Yao, Jiewen ; Jiang, Guomin >

[edk2-devel] ArmPlatformPkg: understanding PrePi vs PrePeiCore

2022-04-12 Thread Rebecca Cran
I've been working on cleaning up some of the code in ArmPlatformPkg and have been trying to understand the differences between PrePi and PrePeiCore. It looks like PrePi was added to save space over PrePeiCore, but I'm wondering if we still need it? I'm guessing most of the platforms below could

[edk2-devel] [PATCH v4 1/8] MdeModulePkg: Add Variable Flash Info HOB

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Adds a new GUID that is used to identify a HOB that passes variable flash information to UEFI variable drivers in HOB consumption phases such as DXE, Traditional MM, and Standalone MM. This information was previously

[edk2-devel] [PATCH v4 0/8] Add Variable Flash Info HOB

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 The UEFI variable drivers such as VariableRuntimeDxe, VariableSmm, VariableStandaloneMm, etc. (and their dependent protocol/library stack), typically acquire UEFI variable store flash information with PCDs declared in

[edk2-devel] [PATCH v4 2/8] MdeModulePkg/VariableFlashInfoLib: Add initial library

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Adds a new library class VariableFlashInfoLib that abstracts access to variable flash information. The instance provided first attempts to retrieve information from the Variable Flash Info HOB. If that HOB is not prese

[edk2-devel] [PATCH v4 3/8] MdeModulePkg/Variable: Consume Variable Flash Info

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Updates VariableRuntimeDxe, VariableSmm, and VariableStandaloneMm to acquire variable flash information from the Variable Flash Information library. Cc: Jian J Wang Cc: Hao A Wu Cc: Liming Gao Signed-off-by: Michae

[edk2-devel] [PATCH v4 4/8] MdeModulePkg/FaultTolerantWrite: Consume Variable Flash Info

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Adds support to the UEFI variable fault tolerant write (FTW) drivers to receive FTW base and size information dynamically via the Variable Flash Information library. Cc: Jian J Wang Cc: Hao A Wu Cc: Liming Gao Sign

[edk2-devel] [PATCH v4 5/8] ArmVirtPkg/ArmVirt.dsc.inc: Add VariableFlashInfoLib

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Adds an instance of VariableFlashInfoLib to the platform build as it is a new library class introduced in MdeModulePkg. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Sami Mujawar Cc: Gerd Hoffmann Cc: Julien Grall Sig

[edk2-devel] [PATCH v4 6/8] EmulatorPkg: Add VariableFlashInfoLib

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Adds an instance of VariableFlashInfoLib to the platform build as it is a new library class introduced in MdeModulePkg. Cc: Andrew Fish Cc: Ray Ni Cc: Abner Chang Cc: Nickle Wang Signed-off-by: Michael Kubacki --

[edk2-devel] [PATCH v4 7/8] OvmfPkg: Add VariableFlashInfoLib

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Adds an instance of VariableFlashInfoLib to the platform build as it is a new library class introduced in MdeModulePkg. Cc: Anthony Perard Cc: Ard Biesheuvel Cc: Brijesh Singh Cc: Erdem Aktas Cc: Gerd Hoffmann Cc

[edk2-devel] [PATCH v4 8/8] UefiPayloadPkg: Add VariableFlashInfoLib

2022-04-12 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 Adds an instance of VariableFlashInfoLib to the platform build as it is a new library class introduced in MdeModulePkg. Cc: Guo Dong Cc: Ray Ni Cc: Maurice Ma Cc: Benjamin You Cc: Sean Rhodes Signed-off-by: Micha

Re: [edk2-devel] 回复: [PATCH v3 0/8] Add Variable Flash Info HOB

2022-04-12 Thread Michael Kubacki
Hi Liming, To help move reviews along, I went ahead and sent a v4 patch series with the following information: 1. More details about when the HOB is required. 2. More details about when the HOB should be installed in the boot flow. 3. A 4 byte reserved field after the "Version" field. The det

Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: Use SmmWaitForAllProcessor() in VariableSmm driver.

2022-04-12 Thread Li, Zhihao
1. Although SmmCpuRendezvousLib in UefiCpuPkg, add SmmRendezvousLib.h into MdeModulePkg/Include/Library folder bypass the check. 2. The SmmRendezvousLib is a standalone MM library and doesn’t have any DXE service dependency. It can be used by SMM variable module and MM variable module. As the p

Re: [edk2-devel] Intel NUC platform firmware -- no serial I/O support?

2022-04-12 Thread manickavasakam karpagavinayagam via groups.io
Laszlo : Checked internally and came to know that Aptio BIOS for this platform doesn't support the serial redirection support during POST. Thank you -Manic -Original Message- From: Laszlo Ersek Sent: Friday, April 8, 2022 5:20 AM To: Manickavasakam Karpagavinayagam ; Gerd Hoffmann ;

[edk2-devel] TDX patches have broken edk2 bisectability in OVMF

2022-04-12 Thread James Bottomley
I've identified a serious performance regression in recent edk2, so I've been trying to identify it by bisection, but it seems that the TDX patches have broken bisection in edk2. You can see this by trying to checkout b6b2de884864 and build it. It will give you Active Platform = /home/j

[edk2-devel] Regression: 100x I/O performance slowdown in SEC phase caused by TDX

2022-04-12 Thread James Bottomley
I'm using a SEC phase which has a TPM driver to experiment with sorting out measured boot, which is how I noticed (usually SEC doesn't do MMIO) . What I'm seeing is after commit b6b2de884864 ("MdePkg: Support mmio for Tdx guest in BaseIoLibIntrinsic") we get a massive slowdown of about 100x in TPM

Re: [edk2-devel] [Patch] pip-requirements.txt: Update basetools version to 0.1.17

2022-04-12 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: Feng, Bob C > Sent: Tuesday, April 12, 2022 7:23 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Sean Brogan > ; Bret Barkelew ; > Kinney, Michael D ; Gao, Liming > > Subject: [Patch] pip-requirements.txt: Update basetools

Re: [edk2-devel] [Patch] pip-requirements.txt: Update basetools version to 0.1.17

2022-04-12 Thread Bob Feng
Thank you Mike, I created a PR https://github.com/tianocore/edk2/pull/2770. Thanks, Bob -Original Message- From: Kinney, Michael D Sent: Wednesday, April 13, 2022 7:54 AM To: Feng, Bob C ; devel@edk2.groups.io; Kinney, Michael D Cc: Sean Brogan ; Bret Barkelew ; Gao, Liming Subject:

Re: [edk2-devel] [PATCH v2 02/14] IntelFsp2Pkg: Add CpuLib to module INFs that depend on UefiCpuLib

2022-04-12 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Tuesday, April 12, 2022 10:52 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v2 02/14] IntelFsp2Pkg: Add CpuLib to module INFs th

Re: [edk2-devel] [PATCH v2 03/14] IntelFsp2WrapperPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-12 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Tuesday, April 12, 2022 10:52 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v2 03/14] IntelFsp2WrapperPkg: Add CpuLib to module >

Re: [edk2-devel] [PATCH v2 08/14] IntelFsp2Pkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Tuesday, April 12, 2022 10:52 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v2 08/14] IntelFsp2Pkg: Remove UefiCpuLib from module

[edk2-devel] Cancelled Event: TianoCore Bug Triage - APAC / NAMO - Tuesday, April 12, 2022 #cal-cancelled

2022-04-12 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:CANCELLED REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:America/Los_Angeles LAST-MODIFIED:20220317T223602Z TZURL:http://tzurl.org/zoneinfo-outlook/America/Los_Angel

[edk2-devel] [PATCH v2 00/14] merge UefiCpuPkg/UefiCpuLib to MdePkg/CpuLib

2022-04-12 Thread Yu Pu
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3394 Today lots of duplicated code call CPUID and calculates the address mask. Adding an API named GetPhysicalAddressBits in UefiCpuLib can solve this problem, but at the same time cause MdeModulePkg depend on UefiCpuPkg which does not meet the

[edk2-devel] [PATCH v2 01/14] MdePkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-12 Thread Yu Pu
There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib and UefiCpuPkg/UefiCpuLib will be merged to MdePkg/CpuLib. To avoid build failure, add CpuLib dependency to all modules that depend on UefiCpuLib. Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Signed-off-by: Yu Pu --- Mde

[edk2-devel] [PATCH v2 03/14] IntelFsp2WrapperPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-12 Thread Yu Pu
There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib and UefiCpuPkg/UefiCpuLib will be merged to MdePkg/CpuLib. To avoid build failure, add CpuLib dependency to all modules that depend on UefiCpuLib. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Signed-off-by: Yu Pu --- IntelFsp

[edk2-devel] [PATCH v2 02/14] IntelFsp2Pkg: Add CpuLib to module INFs that depend on UefiCpuLib

2022-04-12 Thread Yu Pu
There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib and UefiCpuPkg/UefiCpuLib will be merged to MdePkg/CpuLib. To avoid build failure, add CpuLib dependency to all modules that depend on UefiCpuLib. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Signed-off-by: Yu Pu --- IntelFsp

[edk2-devel] [PATCH v2 04/14] OvmfPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-12 Thread Yu Pu
There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib and UefiCpuPkg/UefiCpuLib will be merged to MdePkg/CpuLib. To avoid build failure, add CpuLib dependency to all modules that depend on UefiCpuLib. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-b

[edk2-devel] [PATCH v2 08/14] IntelFsp2Pkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Yu Pu
Because UefiCpuPkg/UefiCpuLib is merged to MdePkg/CpuLib, remove the dependency of UefiCpuLib. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Signed-off-by: Yu Pu --- IntelFsp2Pkg/FspSecCore/FspSecCoreM.inf | 1 - IntelFsp2Pkg/FspSecCore/SecMain.h | 1 -

[edk2-devel] [PATCH v2 05/14] UefiCpuPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-12 Thread Yu Pu
There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib and UefiCpuPkg/UefiCpuLib will be merged to MdePkg/CpuLib. To avoid build failure, add CpuLib dependency to all modules that depend on UefiCpuLib. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Signed-off-by: Yu Pu --- UefiCpuPkg/Libr

[edk2-devel] [PATCH v2 11/14] SourceLevelDebugPkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Yu Pu
Because UefiCpuPkg/UefiCpuLib is merged to MdePkg/CpuLib, remove the dependency of UefiCpuLib. Cc: Hao A Wu Signed-off-by: Yu Pu --- SourceLevelDebugPkg/SourceLevelDebugPkg.dsc | 1 - 1 file changed, 1 deletion(-) diff --git a/SourceLevelDebugPkg/SourceLevelDebugPkg.dsc b/SourceLevelDebugPkg/

[edk2-devel] [PATCH v2 09/14] OvmfPkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Yu Pu
Because UefiCpuPkg/UefiCpuLib is merged to MdePkg/CpuLib, remove the dependency of UefiCpuLib. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Yu Pu --- OvmfPkg/Sec/SecMain.c | 1 - OvmfPkg/AmdSev/AmdSevX64.dsc | 1 - OvmfPkg/Bhyve/BhyveX64.dsc

[edk2-devel] [PATCH v2 06/14] UefiPayloadPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-12 Thread Yu Pu
There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib and UefiCpuPkg/UefiCpuLib will be merged to MdePkg/CpuLib. To avoid build failure, add CpuLib dependency to all modules that depend on UefiCpuLib. Cc: Guo Dong Cc: Ray Ni Cc: Maurice Ma Cc: Benjamin You Cc: Sean Rhodes Signed-of

[edk2-devel] [PATCH v2 13/14] UefiPayloadPkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Yu Pu
Because UefiCpuPkg/UefiCpuLib is merged to MdePkg/CpuLib, remove the dependency of UefiCpuLib. Cc: Guo Dong Cc: Ray Ni Cc: Maurice Ma Cc: Benjamin You Cc: Sean Rhodes Signed-off-by: Yu Pu --- UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.h| 1 - UefiPayloadPkg/UefiPayloadEntry/Ue

[edk2-devel] [PATCH v2 12/14] UefiCpuPkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Yu Pu
Because UefiCpuPkg/UefiCpuLib is merged to MdePkg/CpuLib, remove the dependency of UefiCpuLib. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Signed-off-by: Yu Pu --- UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c | 1 - UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c | 1

[edk2-devel] [PATCH v2 10/14] PcAtChipsetPkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Yu Pu
Because UefiCpuPkg/UefiCpuLib is merged to MdePkg/CpuLib, remove the dependency of UefiCpuLib. Cc: Ray Ni Signed-off-by: Yu Pu --- PcAtChipsetPkg/PcAtChipsetPkg.dsc | 1 - 1 file changed, 1 deletion(-) diff --git a/PcAtChipsetPkg/PcAtChipsetPkg.dsc b/PcAtChipsetPkg/PcAtChipsetPkg.dsc index 45

[edk2-devel] [PATCH v2 07/14] MdePkg: Move API and implementation from UefiCpuLib to CpuLib

2022-04-12 Thread Yu Pu
There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib. This patch merges UefiCpuPkg/UefiCpuLib to MdePkg/CpuLib. Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Signed-off-by: Yu Pu --- UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLi

[edk2-devel] [PATCH v2 14/14] UefiCpuLib: Remove UefiCpuLib.

2022-04-12 Thread Yu Pu
Because UefiCpuPkg/UefiCpuLib is merged to MdePkg/CpuLib and all modules are updated to not depend on this library, remove it completely. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Signed-off-by: Yu Pu --- UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLibNull.c | 16 - UefiCpuPkg/Include

[edk2-devel] 回复: 回复: [PATCH v3 0/8] Add Variable Flash Info HOB

2022-04-12 Thread gaoliming
Michael: Thanks for your update. Reviewed-by: Liming Gao for this patch set. Thanks Liming > -邮件原件- > 发件人: Michael Kubacki > 发送时间: 2022年4月13日 0:32 > 收件人: gaoliming ; devel@edk2.groups.io > 抄送: 'Abner Chang' ; 'Andrew Fish' > ; 'Anthony Perard' ; 'Ard > Biesheuvel' ; 'Benjamin You' > ;

[edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use DEBUG_FILE_PATH to specify debug file path.

2022-04-12 Thread gaoliming
Guomin: Can you introduce DEBUG_FILE_PATH usage? If the developer wants to enable this feature, how configure DEBUG_FILE_PATH? Thanks Liming > -邮件原件- > 发件人: Guomin Jiang > 发送时间: 2022年4月12日 18:25 > 收件人: devel@edk2.groups.io > 抄送: Michael D Kinney ; Liming Gao > ; Zhiguang Liu > 主题: [PAT

回复: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: Use SmmWaitForAllProcessor() in VariableSmm driver.

2022-04-12 Thread gaoliming
Zhihao: I remember CI has the check for the package dependency. If this patch passes CI, seemly this checker doesn't do. You can see DependencyCheck in MdeModulePkg\MdeModulePkg.ci.yaml. And, this patch introduces new dependency in VariableStandaloneMm. It has been used in edk2 platform ARM

[edk2-devel][PATCH v4 1/8] IntelFsp2Pkg: X64 compatible changes to support PEI in 64bit

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 1.Added EFIAPI to FspNotifyPhasePeimEntryPoint, SwapStack and PEI_CORE_ENTRY. 2.Cast FspData from pointer to UINTN and then from UINTN to UINT32. 3.Changed AsmReadEsp to AsmReadStackPointer. 4.Changed the type of the return value of AsmReadS

[edk2-devel][PATCH v4 0/8] Support PEI 64bit in IntelFsp2Pkg and IntelFsp2WrapperPkg

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 Currently PEI drivers/libraries only support 32bit in IntelFsp2Pkg and IntelFsp2WrapperPkg. The patches below are needed to support these drivers/libraries in 64bit. V2 changes: 1.Change FSPM_UPD_COMMON2 to FSPM_UPD_COMMON_FSP24 in FspApiEnt

[edk2-devel][PATCH v4 2/8] IntelFsp2Pkg: Add FSPx_ARCH2_UPD support for X64

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 1.Added FSPx_ARCH2_UPD structures which support both IA32 and X64. 2.Added FSPx_UPD_COMMON_FSP24 structures. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Cc: Ashraf Ali S Signed-off-by: Ted Kuo --- IntelFsp2Pkg/FspSecCore/Ia32/FspApi

[edk2-devel][PATCH v4 3/8] IntelFsp2Pkg: Update FSP_GLOBAL_DATA and FSP_PLAT_DATA for X64

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 Updated FSP_GLOBAL_DATA and FSP_PLAT_DATA structures to support both IA32 and X64. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Cc: Ashraf Ali S Signed-off-by: Ted Kuo --- IntelFsp2Pkg/FspSecCore/SecFsp.c | 2 +- IntelFsp2Pkg/In

[edk2-devel][PATCH v4 4/8] IntelFsp2Pkg: FspSecCore support for X64

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 1.Added FspSecCore support for X64. 2.Bumped FSP header revision to 7 to indicate FSP 64bit is supported. 3.Corrected few typos. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Cc: Ashraf Ali S Signed-off-by: Ted Kuo --- IntelFsp2Pkg/Fs

[edk2-devel][PATCH v4 5/8] IntelFsp2Pkg: SecFspSecPlatformLibNull support for X64

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 1.Added SecFspSecPlatformLibNull support for X64. 2.Added X64 support to IntelFsp2Pkg.dsc. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Cc: Ashraf Ali S Signed-off-by: Ted Kuo --- IntelFsp2Pkg/IntelFsp2Pkg.dsc |

[edk2-devel][PATCH v4 6/8] IntelFsp2WrapperPkg: Adopt FSPM_UPD_COMMON_FSP24 for X64

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 Adopt FSPM_UPD_COMMON_FSP24 in FspmWrapperPeim to support X64. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Cc: Ashraf Ali S Signed-off-by: Ted Kuo --- .../FspmWrapperPeim/FspmWrapperPeim.c | 25 ++ 1

[edk2-devel][PATCH v4 7/8] IntelFsp2WrapperPkg: BaseFspWrapperApiLib support for X64

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 Add Execute64BitCode to execute 64bit code from long mode directly in PEI 64bit. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Cc: Ashraf Ali S Signed-off-by: Ted Kuo --- .../BaseFspWrapperApiLib/FspWrapperApiLib.c| 42 +++

[edk2-devel][PATCH v4 8/8] IntelFsp2WrapperPkg: SecFspWrapperPlatformSecLibSample support for X64

2022-04-12 Thread Kuo, Ted
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3893 1.Added SecFspWrapperPlatformSecLibSample support for X64. 2.Adopted FSPT_ARCH2_UPD in SecFspWrapperPlatformSecLibSample. 3.Moved Fsp.h up one level to be shared across IA32 and X64. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Star Zeng Cc: Ashr

[edk2-devel] [PATCH v2] ShellPkg: Update smbiosview type 9 with SMBIOS 3.5 fields

2022-04-12 Thread Bo Chang Ke
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3896 update smbiosview type 9 related fileds. Signed-off-by: Bo Chang Ke Cc: Ray Ni Cc: Zhichao Gao --- .../SmbiosView/PrintInfo.c| 7 +- .../SmbiosView/QueryTable.c | 237 ++ .../Smbios

[edk2-devel] [PATCH v2] MdePkg: Update smbiosview type 9 with SMBIOS 3.5 fields

2022-04-12 Thread Bo Chang Ke
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3896 update smbiosview type 9 related fileds. Signed-off-by: Bo Chang Ke Cc: Zhiguang Liu Cc: Zhichao Gao Cc: Dandan Bi Cc: Star Zeng --- MdePkg/Include/IndustryStandard/SmBios.h | 49 +++- 1 file changed, 48 insertions(

Re: [edk2-devel] [staging/LoongArch RESEND PATCH v1 25/33] MdePkg/BaseCpuLib: LoongArch Base CPU library implementation.

2022-04-12 Thread Chao Li
Hi Abner, You pointed out that you attached a patch to your last email, but I didn't find it, do you forgot to put it? In the current EDK II code repo, I can not find the patch which you mentioned. -- Thanks, Chao On 4月 8 2022, at 7:26 晚上, "Chang, Abner (HPS SW/FW Tech

Re: [edk2-devel] Regression: 100x I/O performance slowdown in SEC phase caused by TDX

2022-04-12 Thread Min Xu
On April 13, 2022 5:22 AM, James Bottom wrote: > > I'm using a SEC phase which has a TPM driver to experiment with sorting out > measured boot, which is how I noticed (usually SEC doesn't do MMIO) . > What I'm seeing is after commit b6b2de884864 ("MdePkg: Support mmio for > Tdx guest in BaseIoLibI

Re: [edk2-devel] [PATCH v3 1/6] MdeModulePkg/PciHostBridge: io range is not mandatory

2022-04-12 Thread Abner Chang
> -Original Message- > From: Gerd Hoffmann > Sent: Thursday, April 7, 2022 5:33 PM > To: devel@edk2.groups.io > Cc: Leif Lindholm ; Ard Biesheuvel > ; Jordan Justen ; > Jian J Wang ; Ray Ni ; Hao A Wu > ; Gerd Hoffmann ; Jiewen Yao > ; Liming Gao ; Pawel > Polawski ; Chang, Abner (HPS S

Re: [edk2-devel] [PATCH v3 2/6] OvmfPkg/FdtPciHostBridgeLib: io range is not mandatory

2022-04-12 Thread Abner Chang
> -Original Message- > From: devel@edk2.groups.io On Behalf Of Gerd > Hoffmann > Sent: Tuesday, April 12, 2022 5:04 PM > To: devel@edk2.groups.io; Chang, Abner (HPS SW/FW Technologist) > > Cc: Leif Lindholm ; Ard Biesheuvel > ; Jordan Justen ; > Jian J Wang ; Ray Ni ; Hao A Wu > ; Jiew

Re: [edk2-devel][PATCH v4 1/8] IntelFsp2Pkg: X64 compatible changes to support PEI in 64bit

2022-04-12 Thread Chiu, Chasel
Hi Ted, Please see my comments below inline. Thanks, Chasel > -Original Message- > From: Kuo, Ted > Sent: Wednesday, April 13, 2022 10:43 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [edk2-devel][PATCH v4 1/8] I

Re: [edk2-devel][PATCH v4 0/8] Support PEI 64bit in IntelFsp2Pkg and IntelFsp2WrapperPkg

2022-04-12 Thread Chiu, Chasel
Thanks Ted for updating patch! For this patch series, Reviewed-by: Chasel Chiu > -Original Message- > From: devel@edk2.groups.io On Behalf Of Kuo, > Ted > Sent: Wednesday, April 13, 2022 10:43 AM > To: devel@edk2.groups.io > Subject: [edk2-devel][PATCH v4 0/8] Support PEI 64bit in Int

Re: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use DEBUG_FILE_PATH to specify debug file path.

2022-04-12 Thread Guomin Jiang
Hi Liming, Below is the detail why need this change. If you only care usage. Please jump to "How to use it" directly. Why need: 1. Replication build is not new, you can refer https://reproducible-builds.org for detail 2. The benefit is that 1) make sure the same commit generate same binary so w

Re: [edk2-devel] Intel NUC platform firmware -- no serial I/O support?

2022-04-12 Thread Laszlo Ersek
Hi Manic, thanks a lot for spending time on this! Laszlo On 04/12/22 20:40, Manickavasakam Karpagavinayagam wrote: > Laszlo : > > Checked internally and came to know that Aptio BIOS for this platform doesn't > support the serial redirection support during POST. > > Thank you > > -Manic > >

[edk2-devel] [PATCH 0/3] CryptoPkg: Enable ECC in openssllib by a customize-able way

2022-04-12 Thread yi1 li
EDK2 has Enabled a feature which allow SOURCE section in INF files to use Pcd to customize source files list. If Pcd equal to FALSE, this file will not be compiled. https://github.com/tianocore/edk2/commit/ bf9230a9f3dde065c3c8b4175ccd32e44e8f0362 The patches add a customizable EC feature to Crypt

[edk2-devel] [PATCH 1/3] CryptoPkg: Add instrinsics to support building ECC on IA32 windows

2022-04-12 Thread yi1 li
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3679 This dependency is needed to build openssl lib with ECC ciphers under IA32 Windows and adds implementation for _allmul and _allshr instrinsics. It is taken from Project Mu: microsoft/mu_basecore@b55b341 Cc: Jiewen Yao Cc: Jian J Wang Sig

[edk2-devel] [PATCH 2/3] CryptoPkg: Reconfigure OpensslLib to add EC algorithms

2022-04-12 Thread yi1 li
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3679 REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3828 Reconfigure OpensslLib to add elliptic curve cipher algorithms. Cc: Jiewen Yao Cc: Jian J Wang Signed-off-by: yi1 li --- .../Library/Include/openssl/opensslconf.h | 3 -- Cryp

[edk2-devel] [PATCH 3/3] CryptoPkg: Make EC source file config-able

2022-04-12 Thread yi1 li
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3679 REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3828 Use PCD gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled to config-able source files list in OpensslLib.inf and OpensslLibCrypto.inf. If PcdEcEnabled equals to FALSE, this file will not be compi

Re: [edk2-devel] [PATCH 3/3] CryptoPkg: Make EC source file config-able

2022-04-12 Thread Yao, Jiewen
Since this PcdEcEnabled is static, you don't need add PcdLib in the INF. Thank you Yao Jiewen > -Original Message- > From: Li, Yi1 > Sent: Wednesday, April 13, 2022 2:25 PM > To: devel@edk2.groups.io > Cc: Li, Yi1 ; Yao, Jiewen ; Wang, Jian > J > Subject: [PATCH 3/3] CryptoPkg: Make EC

Re: [edk2-devel] [PATCH 1/3] CryptoPkg: Add instrinsics to support building ECC on IA32 windows

2022-04-12 Thread Yao, Jiewen
Reviewed-by: Jiewen Yao > -Original Message- > From: Li, Yi1 > Sent: Wednesday, April 13, 2022 2:25 PM > To: devel@edk2.groups.io > Cc: Li, Yi1 ; Yao, Jiewen ; Wang, Jian > J > Subject: [PATCH 1/3] CryptoPkg: Add instrinsics to support building ECC on > IA32 > windows > > REF:https://

Re: [edk2-devel] [PATCH 2/3] CryptoPkg: Reconfigure OpensslLib to add EC algorithms

2022-04-12 Thread Yao, Jiewen
I think you can remove the no-ec directly. No need to comment it. -"no-ec", +# "no-ec", Thank you Yao Jiewen > -Original Message- > From: Li, Yi1 > Sent: Wednesday, April 13, 2022 2:25 PM > To: devel@edk2.groups.io > Cc: Li, Yi1 ; Yao, Jiewen ; Wang, Jian

Re: [edk2-devel] [PATCH 0/3] CryptoPkg: Enable ECC in openssllib by a customize-able way

2022-04-12 Thread Yao, Jiewen
Thanks for the contribution. Great progress on adding ECC. With PcdLib removed and "no-ec" removed, the series is reviewed-by: Jiewen Yao > -Original Message- > From: Li, Yi1 > Sent: Wednesday, April 13, 2022 2:24 PM > To: devel@edk2.groups.io > Cc: Li, Yi1 ; Yao, Jiewen ; Wang, Jian

Re: [edk2-devel] [edk2-discuss] GSoC Proposal

2022-04-12 Thread Marvin Häuser
CC Nate (GSoC admin) CC Steven (task mentor) CC edk2-devel (you picked the logically correct list, but it’s pretty dead and barely anyone reads it) Hey Ada, Out of mere curiosity, why did you pick this item? :) Hey Steven, I feel like there is more to your proposal than is given on the task pa