Re: [edk2-devel] [PATCH v2 1/6] SecurityPkg: Create library for setting Secure Boot variables.

2021-06-02 Thread Min Xu
On Tuesday, June 1, 2021 9:12 PM, Grzegorz Bernacki wrote: > This commits add library, which consist functions related creation/removal > Secure Boot variables. Some of the functions was moved from > SecureBootConfigImpl.c file. > > Signed-off-by: Grzegorz Bernacki > --- > SecurityPkg/SecurityPk

Re: [edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload: Add definition for extra info in payload

2021-06-02 Thread Wu, Hao A
> -Original Message- > From: devel@edk2.groups.io On Behalf Of Ni, Ray > Sent: Thursday, June 3, 2021 2:23 PM > To: devel@edk2.groups.io > Cc: Kinney, Michael D ; Liming Gao > ; Liu, Zhiguang ; Ma, > Maurice ; Dong, Guo > Subject: [edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload:

[edk2-devel] [PATCH v2 2/3] UefiPayloadPkg: Add PayloadLoaderPeim which can load ELF payload

2021-06-02 Thread Ni, Ray
Per universal payload spec, the payload is in ELF format. The patch adds a payload loader that supports to load ELF image. The location of extra data sections whose names start with "upld." is stored in PLD_EXTRA_DATA HOB. Signed-off-by: Maurice Ma Signed-off-by: Ray Ni Cc: Maurice Ma Cc: Guo

[edk2-devel] [PATCH v2 0/3] Add PayloadLoaderPeim which can load ELF payload

2021-06-02 Thread Ni, Ray
v2: Separate the patch set from the patch that adds CLANGDWARF toolchain. Add missing function header commments. Change DEBUG_ERROR to DEBUG_INFO for information debug message. Ray Ni (3): MdeModulePkg/UniversalPayload: Add definition for extra info in payload UefiPayloadPkg: Add Pa

[edk2-devel] [PATCH v2 3/3] PeiCore: Remove assertion when failing to load PE image

2021-06-02 Thread Ni, Ray
EFI_PEI_LOAD_FILE_PPI is invoked by DxeIpl for loading DxeCore. It's possible that the instance produced by PeiCore fails to load but other instances of EFI_PEI_LOAD_FILE_PPI can load. Signed-off-by: Ray Ni Cc: Jian J Wang Cc: Hao A Wu Cc: Dandan Bi Cc: Liming Gao --- MdeModulePkg/Core/Pei/I

[edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload: Add definition for extra info in payload

2021-06-02 Thread Ni, Ray
The payload is in ELF format per the universal payload spec. PLD_INFO_HEADER is stored in the ELF payload as a separate section named ".upld_info". Extra data needed by payload is stored in sections whose name starts with ".upld.". Signed-off-by: Ray Ni Cc: Michael D Kinney Cc: Liming Gao Cc:

Re: [edk2-devel] [PATCH 0/6] Add CLANGDWARF toolchain for universal payload needs

2021-06-02 Thread Ni, Ray
V2 is at https://edk2.groups.io/g/devel/message/75955?p=,,,20,0,0,0::Created,,CLANGDWARF,20,2,0,83253912 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#76007): https://edk2.groups.io/g/devel/message/76007 Mute This Topic: https://groups.

Re: [edk2-devel] [PATCH 0/6] Add CLANGDWARF toolchain for universal payload needs

2021-06-02 Thread Ni, Ray
Liming, I resent the V2 that removed the conflict messages and corrected your mail address. I am using CLANG V11. https://bugzilla.tianocore.org/show_bug.cgi?id=3431 was submitted. Thanks, Ray > -Original Message- > From: gaoliming > Sent: Thursday, June 3, 2021 9:18 AM > To: devel@edk2

[edk2-devel] [Patch V2] UefiPayloadPkg: Use DynamicEx instead of Dynamic to pass PCD across binary

2021-06-02 Thread Zhiguang Liu
V1: When passing PCD database from Edk2 boot loader to Universal Payload, the local token number in boot loader PCD database can be different with that in Payload PCD database. Dynamic PCD directly use local token number, while DynamicEx will search token number by Guid and ExTokenNumber, which ar

Re: [edk2-devel] [PATCH] UefiPayloadPkg: Use DynamicEx instead of Dynamic to pass PCD across binary

2021-06-02 Thread Guo Dong
Reviewed-by: Guo Dong > -Original Message- > From: Liu, Zhiguang > Sent: Wednesday, June 2, 2021 7:39 AM > To: devel@edk2.groups.io > Cc: Ma, Maurice ; Dong, Guo > ; You, Benjamin > Subject: [PATCH] UefiPayloadPkg: Use DynamicEx instead of Dynamic to pass > PCD across binary > > When

[edk2-devel] [edk2-platforms][PATCH v1 2/2] MinPlatformPkg/TpmPlatformHierarchyLib: Disable TPM platform hierarchy

2021-06-02 Thread Michael Kubacki
From: Jeremiah Cox REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3411 Updates the current ConfigureTpmPlatformHierarchy() implementation to instruct the TPM to disable the platform hierarchy to prevent later boot/OS code from accessing TPM platform features. This modifies the current behavi

[edk2-devel] [edk2-platforms][PATCH v1 1/2] MinPlatformPkg: Add TpmPlatformHierarchyLib to Components in DSC

2021-06-02 Thread Michael Kubacki
From: Michael Kubacki Adds TpmPlatformHierarchyLib to the [Components] section in MinPlatformPkg.dsc so it is always built in the package build. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Liming Gao Cc: Eric Dong Signed-off-by: Michael Kubacki --- Platform/Intel/MinPlatformPkg/MinPlatformPkg.ds

[edk2-devel] [edk2-platforms][PATCH v1 0/2] MinPlatformPkg: Disable TPM platform hierarchy

2021-06-02 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3411 This patch series mostly focuses on disabling the TPM platform hierarchy (patch 2/2) in ConfigureTpmPlatformHierarchy() with a preliminary change to add the library being modified to the [Components] section in the Min

回复: [edk2-devel] [PATCH 0/6] Add CLANGDWARF toolchain for universal payload needs

2021-06-02 Thread gaoliming
Ray: Thanks for your effort to integrate CLANGDWARF tool chain. Previous work bases on CLANG8. Now, which version CLANG compiler is used? And, can you submit one BZ for this new feature? Then, I can add it to the feature planning. Besides, please remove conflict message in the commit messa

回复: [edk2-devel] [PATCH] MdePkg/Include: Smbios Specification 3.4.0 changes

2021-06-02 Thread gaoliming
Gopi: Please use the command ‘git send-email xxx.patch --to=devel@edk2.groups.io ’ to send this patch. Thanks Liming 发件人: devel@edk2.groups.io 代表 Thotala, Gopi 发送时间: 2021年6月3日 0:24 收件人: devel@edk2.groups.io 主题: [edk2-devel] [PATCH] MdePkg/Include: Smbios

Re: [EXTERNAL] Re: [edk2-devel] [PATCH v2 4/6] SecurityPkg: Add EnrollFromDefaultKeys application.

2021-06-02 Thread Bret Barkelew via groups.io
> +#define FAIL(fmt...) AsciiPrint("EnrollFromDefaultKeysApp: " fmt) I don’t think this sort of implied concatenation works on all compilers. - Bret From: Pete Batard via groups.io Sent: Wednesday, June 2, 2021 10:40 AM To: devel@edk2.groups.io

Re: [edk2-devel] [edk2-platforms][PATCH V1 0/3] Platform/Sgi: enable support for UEFI secure boot

2021-06-02 Thread Sami Mujawar
Pushed as d4fe6d9defc2..1d23831b5f07 Thanks. Regards, Sami Mujawar On 24/05/2021 06:22 PM, Sayanta Pattanayak wrote: This patch series adds secure boot support for Arm's reference design platforms. The first patch refactors the existing StandaloneMM platform description file and splits into

Re: [edk2-devel] [PATCH v1 0/8] Measured SEV boot with kernel/initrd/cmdline

2021-06-02 Thread James Bottomley
On Tue, 2021-06-01 at 14:11 +0200, Laszlo Ersek wrote: > Ard, > > I'll have a specific question for you below; please feel free to jump > forward (search for your name). Thanks. > > Dov, my comments below: > > On 05/25/21 07:31, Dov Murik wrote: > > Booting with SEV prevented the loading of kern

Re: [edk2-devel] [PATCH v2 1/1] Add missing EFIAPI to VirtioMmioSetQueueAddress

2021-06-02 Thread Laszlo Ersek
On 06/02/21 06:59, Gerd Hoffmann wrote: > This error was found while compiling VirtioMmioDeviceLib for X64 > with the GCC5 toolchain, where EFIAPI makes a difference. > > Signed-off-by: Gerd Hoffmann > --- > OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h | 1 + > OvmfPkg/Library/

Re: [edk2-devel] [edk2-platforms PATCH v2] Platform/RaspberryPi: Enable default Secure Boot variables initialization

2021-06-02 Thread Pete Batard
This whole patch series looks fine to me. I have tested it on Raspberry Pi 4, and I have some changes lined up to ensure that the next Pi 4 firmware we produce, after this series has been integrated, can use the new feature. For the record, since we are using an automated build system (and th

Re: [edk2-devel] [PATCH v2 6/6] SecurityPkg: Add option to reset secure boot keys.

2021-06-02 Thread Pete Batard
On 2021.06.01 14:12, Grzegorz Bernacki wrote: This commit add option which allows reset content of Secure Boot keys and databases to default variables. Signed-off-by: Grzegorz Bernacki --- SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigDxe.inf | 1 + SecurityPk

Re: [edk2-devel] [PATCH v2 5/6] SecurityPkg: Add new modules to Security package.

2021-06-02 Thread Pete Batard
One very minor remark: On 2021.06.01 14:12, Grzegorz Bernacki wrote: This commits adds modules related to initialization and usage of default Secure Boot key variables to SecurityPkg. Signed-off-by: Grzegorz Bernacki --- SecurityPkg/SecurityPkg.dec | 14 ++ SecurityPkg/SecurityP

Re: [edk2-devel] [PATCH v2 4/6] SecurityPkg: Add EnrollFromDefaultKeys application.

2021-06-02 Thread Pete Batard
On 2021.06.01 14:12, Grzegorz Bernacki wrote: This application allows user to force key enrollment from Secure Boot default variables. Signed-off-by: Grzegorz Bernacki --- SecurityPkg/EnrollFromDefaultKeysApp/EnrollFromDefaultKeysApp.inf | 47 + SecurityPkg/EnrollFromDefaultKeysApp

Re: [edk2-devel] [PATCH v2 3/6] SecurityPkg: Add SecureBootDefaultKeysDxe driver

2021-06-02 Thread Pete Batard
3 very minor remarks below: On 2021.06.01 14:12, Grzegorz Bernacki wrote: This driver initializes default Secure Boot keys and databases based on keys embedded in flash. Signed-off-by: Grzegorz Bernacki --- SecurityPkg/VariableAuthenticated/SecureBootDefaultKeysDxe/SecureBootDefaultKeysDxe.

Re: [edk2-devel] [PATCH v2 2/6] SecurityPkg: Create include file for default key content.

2021-06-02 Thread Pete Batard
On 2021.06.01 14:12, Grzegorz Bernacki wrote: This commits add file which can be included by platform Flash Description File. It allows to specify certificate files, which will be embedded into binary file. The content of these files can be used to initialize Secure Boot default keys and database

Re: [edk2-devel] [PATCH v2 1/6] SecurityPkg: Create library for setting Secure Boot variables.

2021-06-02 Thread Pete Batard
On 2021.06.01 14:12, Grzegorz Bernacki wrote: This commits add library, which consist functions related creation/removal Secure Boot variables. Some of the functions was moved from SecureBootConfigImpl.c file. Signed-off-by: Grzegorz Bernacki --- SecurityPkg/SecurityPkg.dsc

Re: [edk2-devel] [RESEND PATCH RFC v3 00/22] Add AMD Secure Nested Paging (SEV-SNP) support

2021-06-02 Thread Laszlo Ersek
On 05/27/21 11:42, Laszlo Ersek wrote: > On 05/27/21 01:10, Brijesh Singh wrote: >> Changes since v2: >> * Add support for the AP creation. >> * Use the module-scoping override to make AmdSevDxe use the IO port for PCI >> reads. >> * Use the reserved memory type for CPUID and Secrets page. >

[edk2-devel] [PATCH] MdePkg/Include: Smbios Specification 3.4.0 changes

2021-06-02 Thread Thotala, Gopi
Initial patch submitted for review. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#75985): https://edk2.groups.io/g/devel/message/75985 Mute This Topic: https://groups.io/mt/83263426/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscri

[edk2-devel] [edk2-platforms][PATCH V1 4/4] Platform/ARM/N1Sdp: Configuration Manager for N1Sdp

2021-06-02 Thread Khasim Mohammed
The dynamic tables framework utilizes the configuration manager protocol to get the platform specific information required for building the firmware tables. The configuration manager is a platform specific component that collates the platform hardware information and builds an abstract platform co

[edk2-devel] [edk2-platforms][PATCH V1 1/4] Silicon/ARM/NeoverseN1Soc: Add mem regions to support multi-chip usecase

2021-06-02 Thread Khasim Mohammed
This patch adds resource descriptor for multi-chip usecase and introduces corresponding PCD definitions. Signed-off-by: Chandni Cherukuri Signed-off-by: Khasim Syed Mohammed --- Silicon/ARM/NeoverseN1Soc/NeoverseN1Soc.dec | 30 ++- Silicon/ARM/NeoverseN1Soc/Librar

[edk2-devel] [edk2-platforms][PATCH V1 0/4] N1Sdp ACPI table and configuration manager support

2021-06-02 Thread Khasim Mohammed
From: Khasim Mohammed Resending the patches as the previous try didn't make it mailing list. This patch series implements the configuration manager for N1Sdp platform. It enables support for generating the following ACPI tables: 1. FACP 2. DSDT 3. GTDT

[edk2-devel] [edk2-platforms][PATCH V1 3/4] Platform/ARM/N1Sdp: Enable N1Sdp platform specific configurations

2021-06-02 Thread Khasim Mohammed
This patch updates the fdf file and introduces component description and configuration files for N1Sdp. Signed-off-by: Deepak Pandey Signed-off-by: Khasim Syed Mohammed --- Platform/ARM/N1Sdp/N1SdpPlatform.dec | 90 Platform/ARM/N1Sdp/N1SdpPlatform.dsc | 28 +- Platform

[edk2-devel] [edk2-platforms][PATCH V1 0/4]N1Sdp ACPI table and configuration manager support

2021-06-02 Thread Khasim Mohammed
Resending the patches again as the previous two sets didn't make it to the mailing list. This patch series implements the configuration manager for N1Sdp platform. It enables support for generating the following ACPI tables: 1. FACP 2. DSDT 3. GTDT

[edk2-devel] [edk2-platforms][PATCH V1 4/4] Platform/ARM/N1Sdp: Configuration Manager for N1Sdp

2021-06-02 Thread Khasim Mohammed
The dynamic tables framework utilizes the configuration manager protocol to get the platform specific information required for building the firmware tables. The configuration manager is a platform specific component that collates the platform hardware information and builds an abstract platform co

[edk2-devel] [edk2-platforms][PATCH V1 2/4] Platform/ARM/N1Sdp: Introduce platform DXE driver

2021-06-02 Thread Khasim Mohammed
Add an initial platform DXE driver and support for ramdisk devices. Signed-off-by: Deepak Pandey Signed-off-by: Khasim Syed Mohammed --- Platform/ARM/N1Sdp/Drivers/PlatformDxe/PlatformDxe.inf | 44 + Platform/ARM/N1Sdp/Drivers/PlatformDxe/PlatformDxe.c | 51 ++

[edk2-devel] [edk2-platforms][PATCH V1 3/4] Platform/ARM/N1Sdp: Enable N1Sdp platform specific configurations

2021-06-02 Thread Khasim Mohammed
This patch updates the fdf file and introduces component description and configuration files for N1Sdp. Signed-off-by: Deepak Pandey Signed-off-by: Khasim Syed Mohammed --- Platform/ARM/N1Sdp/N1SdpPlatform.dec | 90 Platform/ARM/N1Sdp/N1SdpPlatform.dsc | 28 +- Platform

[edk2-devel] [edk2-platforms][PATCH V1 2/4] Platform/ARM/N1Sdp: Introduce platform DXE driver

2021-06-02 Thread Khasim Mohammed
Add an initial platform DXE driver and support for ramdisk devices. Signed-off-by: Deepak Pandey Signed-off-by: Khasim Syed Mohammed --- Platform/ARM/N1Sdp/Drivers/PlatformDxe/PlatformDxe.inf | 44 + Platform/ARM/N1Sdp/Drivers/PlatformDxe/PlatformDxe.c | 51 ++

[edk2-devel] [edk2-platforms][PATCH V1 4/4] Platform/ARM/N1Sdp: Configuration Manager for N1Sdp

2021-06-02 Thread Khasim Mohammed
From: Khasim Mohammed The dynamic tables framework utilizes the configuration manager protocol to get the platform specific information required for building the firmware tables. The configuration manager is a platform specific component that collates the platform hardware information and builds

[edk2-devel] [edk2-platforms][PATCH V1 0/4]N1Sdp ACPI table and configuration manager support

2021-06-02 Thread Khasim Mohammed
Resending the patches again as the previous two sets didn't make it to the mailing list. This patch series implements the configuration manager for N1Sdp platform. It enables support for generating the following ACPI tables: 1. FACP 2. DSDT 3. GTDT

[edk2-devel] [edk2-platforms][PATCH V1 1/4] Silicon/ARM/NeoverseN1Soc: Add mem regions to support multi-chip usecase

2021-06-02 Thread Khasim Mohammed
This patch adds resource descriptor for multi-chip usecase and introduces corresponding PCD definitions. Signed-off-by: Chandni Cherukuri Signed-off-by: Khasim Syed Mohammed --- Silicon/ARM/NeoverseN1Soc/NeoverseN1Soc.dec | 30 ++- Silicon/ARM/NeoverseN1Soc/Librar

[edk2-devel] [edk2-platforms][PATCH V1 3/4] Platform/ARM/N1Sdp: Enable N1Sdp platform specific configurations

2021-06-02 Thread Khasim Mohammed
From: Khasim Mohammed This patch updates the fdf file and introduces component description and configuration files for N1Sdp. Signed-off-by: Deepak Pandey Signed-off-by: Khasim Syed Mohammed --- Platform/ARM/N1Sdp/N1SdpPlatform.dec | 90 Platform/ARM/N1Sdp/N1SdpPlatform.d

[edk2-devel] [edk2-platforms][PATCH V1 1/4] Silicon/ARM/NeoverseN1Soc: Add mem regions to support multi-chip usecase

2021-06-02 Thread Khasim Mohammed
From: Khasim Mohammed This patch adds resource descriptor for multi-chip usecase and introduces corresponding PCD definitions. Signed-off-by: Chandni Cherukuri Signed-off-by: Khasim Syed Mohammed --- Silicon/ARM/NeoverseN1Soc/NeoverseN1Soc.dec | 30 ++- Silicon/

[edk2-devel] [edk2-platforms][PATCH V1 2/4] Platform/ARM/N1Sdp: Introduce platform DXE driver

2021-06-02 Thread Khasim Mohammed
From: Khasim Mohammed Add an initial platform DXE driver and support for ramdisk devices. Signed-off-by: Deepak Pandey Signed-off-by: Khasim Syed Mohammed --- Platform/ARM/N1Sdp/Drivers/PlatformDxe/PlatformDxe.inf | 44 + Platform/ARM/N1Sdp/Drivers/PlatformDxe/PlatformDxe.c

[edk2-devel] [PATCH] UefiPayloadPkg: Use DynamicEx instead of Dynamic to pass PCD across binary

2021-06-02 Thread Zhiguang Liu
When passing PCD database from Edk2 boot loader to Universal Payload, the local token number in boot loader PCD database can be different with that in Payload PCD database. Dynamic PCD directly use local token number, while DynamicEx will search token number by Guid and ExTokenNumber, which are un

[edk2-devel] [Patch V2] UefiPayloadPkg: Get platform specific logic from protocol for BDS driver

2021-06-02 Thread Zhiguang Liu
V1: Currently, BDS driver will link a PlatformBootManagerLib, which contains platform sepcific logic. This patch get the platform specific logic from a protocol, so that platform logic for Boot manager can be in another binary. V2: Add function comments in PlatformBootManagerOverride.h Cc: Maur

Re: [edk2-devel] [PATCH v3 0/2] Dynamically build UARTs info in ACPI

2021-06-02 Thread Samer El-Haj-Mahmoud
CCing the maintainers and reviewers > -Original Message- > From: devel@edk2.groups.io On Behalf Of Sunny > Wang via groups.io > Sent: Monday, May 31, 2021 4:23 AM > To: devel@edk2.groups.io > Cc: Sunny Wang > Subject: [edk2-devel] [PATCH v3 0/2] Dynamically build UARTs info in ACPI > > I

Re: [edk2-devel] [PATCH v1 0/2] Define AML_NAME_SEG_SIZE

2021-06-02 Thread Sami Mujawar
Pushed as b5379899b38e..1f515342d8d8 Thanks. Regards, Sami Mujawar From: pierre.gond...@arm.com Date: Tuesday, 1 June 2021 at 23:05 To: devel@edk2.groups.io , dandan...@intel.com , gaolim...@byosoft.com.cn , michael.d.kin...@intel.com , Sami Mujawar , Alexei Fedorov Subject: [PATCH v1 0/2]

Re: [edk2-devel] [edk2-platforms PATCH 1/1] Readme.md: Refer users to the Arm GNU-A Downloads page for toolchains

2021-06-02 Thread Sami Mujawar
Hi Leif, If there are no further comments on this patch, may I have your RB and permission to merge this change, please? Regards, Sami Mujawar On 01/06/2021 02:50 PM, Sami Mujawar wrote: Hi Rebecca, Thank you for this patch. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 28/05/20

Re: [edk2-devel] [PATCH 00/43] OvmfPkg: remove Xen support from OvmfPkg*.dsc, in favor of OvmfXen.dsc

2021-06-02 Thread Laszlo Ersek
Anthony, Julien, (or anyone else subscribed to xen-devel -- CC'd now), On 05/26/21 22:14, Laszlo Ersek wrote: > Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=2122 > Repo: https://pagure.io/lersek/edk2.git > Branch: xen_split_bz_2122 can you please build the OvmfXen platform on t

Re: [edk2-devel] [PATCH v1 0/2] Define AML_NAME_SEG_SIZE

2021-06-02 Thread Sami Mujawar
Hi Pierre, Thank you for this patch series. Reviewed-by: Sami Mujawar Regards, Sami Mujawar From: pierre.gond...@arm.com Date: Tuesday, 1 June 2021 at 23:05 To: devel@edk2.groups.io , dandan...@intel.com , gaolim...@byosoft.com.cn , michael.d.kin...@intel.com , Sami Mujawar , Alexei Fedor

Re: [edk2-devel] [edk2-platforms][PATCH V1 0/4]N1Sdp ACPI table and configuration manager support

2021-06-02 Thread Sami Mujawar
Hi Khasim, When exporting for the first time, you should check if you're subscribed to the EDK II development mailing list. Otherwise, your patch email will get rejected. You can subscribe here: https://edk2.groups.io/g/devel. Make sure that you reply to the email with subscription confirmation

Re: [edk2-devel] [PATCH v4 1/1] UefiCpuPkg/CpuCommonFeaturesLib: Correct the CPU location check

2021-06-02 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: Li, Daoxiang > Sent: Wednesday, June 2, 2021 11:01 AM > To: devel@edk2.groups.io > Cc: Li, Daoxiang ; Dong, Eric ; > Ni, Ray ; Laszlo Ersek ; Kumar, > Rahul1 > Subject: [PATCH v4 1/1] UefiCpuPkg/CpuCommonFeaturesLib: Correct the > CPU loc

[edk2-devel] [PATCH v2 6/6] BaseTools: Change CLANG8ELF to CLANGDWARF

2021-06-02 Thread Ni, Ray
CLANGDWARF is more proper because it's similar to CLANGPDB that generates PE images but with DWARF debug symbols. This toolchain is needed for creating ELF format universal payload that follows https://universalpayload.github.io/documentation/. Signed-off-by: Ray Ni Cc: Bob Feng Cc: Liming Gao

[edk2-devel] [PATCH v2 5/6] BaseTools: Update ClangBase.lds to keep dynamic section

2021-06-02 Thread Ni, Ray
The .dynamic section is needed for ELF runtime relocation. Signed-off-by: Ray Ni Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/Scripts/ClangBase.lds | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/BaseTools/Scripts/ClangBase.lds b/BaseTools/Scripts/ClangBas

[edk2-devel] [PATCH v2 1/6] BaseTools: Add ClangBase.lds for CLANG8 tool chain with max-page-size

2021-06-02 Thread Ni, Ray
From: Liming Gao LLVM LLD linker doesn't support common-page-size option. So, max-page-size is used. To not impact GCC tool chain, new ClangBase.lds is added. Signed-off-by: Liming Gao Reviewed-by: Feng Bob C --- BaseTools/Scripts/ClangBase.lds | 79 + 1 file c

[edk2-devel] [PATCH v2 3/6] BaseTools: Update build_rule to skip CLANG resource section generation

2021-06-02 Thread Ni, Ray
From: Liming Gao LLVM/CLANG doesn't support resource section generation when ELF image generated. Signed-off-by: Liming Gao Reviewed-by: Feng Bob C --- BaseTools/Conf/build_rule.template | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/BaseTools/Conf/build_rule.templat

[edk2-devel] [PATCH v2 2/6] BaseTools GenFw: Support CLANG8ELF with conversion ELF to PE/COFF image

2021-06-02 Thread Ni, Ray
From: Liming Gao CLANG8ELF tool chain generated ELF image with the different attributes in section. Update GenFw to handle them. 1. .text section with writable attribute (support) 2. .reloc section has the symbol for *ABS* (skip) Signed-off-by: Liming Gao Reviewed-by: Feng Bob C --- BaseTools

[edk2-devel] [PATCH v2 4/6] BaseTools: Add new CLANG8ELF tool chain for new LLVM/CLANG8

2021-06-02 Thread Ni, Ray
From: Liming Gao BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1603 LLVM/CLANG8 formal release http://releases.llvm.org/download.html#8.0.0 It can be downloaded and installed in Windows/Linux/Mac OS. CLANG8ELF tool chain is added to generate ELF image, and convert to PE/COFF. On Windows OS, s

[edk2-devel] [PATCH v2 0/6] Add CLANGDWARF toolchain for universal payload needs

2021-06-02 Thread Ni, Ray
Universal payload requires its format in ELF format while today's EDKII doesn't contain a cross OS toolchain that generates ELF images. The patch set is based on Liming's work in year 2019 and some very minor modifications are made: 1. Update toolchain name from CLANG8ELF to CLANGDWARF. 2. Update

Re: [edk2-devel] [PATCH EDK2 v1 1/1] MdeModulePkg/Xhci: Fix TRT when data length is 0

2021-06-02 Thread Wu, Hao A
Patch pushed via: PR - https://github.com/tianocore/edk2/pull/1683 Commit - https://github.com/tianocore/edk2/commit/b5379899b38ed84561db6dc07dc4641a049ae238 Best Regards, Hao Wu > -Original Message- > From: devel@edk2.groups.io On Behalf Of Wu, Hao > A > Sent: Monday, May 31, 2021 11:2

[edk2-devel] [PATCH 6/6] BaseTools: Change CLANG8ELF to CLANGDWARF

2021-06-02 Thread Ni, Ray
CLANGDWARF is more proper because it's similar to CLANGPDB that generates PE images but with DWARF debug symbols. This toolchain is needed for creating ELF format universal payload that follows https://universalpayload.github.io/documentation/. Signed-off-by: Ray Ni Cc: Bob Feng Cc: Liming Gao

[edk2-devel] [PATCH 5/6] BaseTools: Update ClangBase.lds to keep dynamic section

2021-06-02 Thread Ni, Ray
The .dynamic section is needed for ELF runtime relocation. Signed-off-by: Ray Ni Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/Scripts/ClangBase.lds | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/BaseTools/Scripts/ClangBase.lds b/BaseTools/Scripts/ClangBas

[edk2-devel] [PATCH 4/6] BaseTools: Add new CLANG8ELF tool chain for new LLVM/CLANG8

2021-06-02 Thread Ni, Ray
From: Liming Gao BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1603 LLVM/CLANG8 formal release http://releases.llvm.org/download.html#8.0.0 It can be downloaded and installed in Windows/Linux/Mac OS. CLANG8ELF tool chain is added to generate ELF image, and convert to PE/COFF. On Windows OS, s

[edk2-devel] [PATCH 3/6] BaseTools: Update build_rule to skip CLANG resource section generation

2021-06-02 Thread Ni, Ray
From: Liming Gao LLVM/CLANG doesn't support resource section generation when ELF image generated. Signed-off-by: Liming Gao Reviewed-by: Feng Bob C # Conflicts: # BaseTools/Conf/build_rule.template --- BaseTools/Conf/build_rule.template | 5 ++--- 1 file changed, 2 insertions(+), 3 del

[edk2-devel] [PATCH 2/6] BaseTools GenFw: Support CLANG8ELF with conversion ELF to PE/COFF image

2021-06-02 Thread Ni, Ray
From: Liming Gao CLANG8ELF tool chain generated ELF image with the different attributes in section. Update GenFw to handle them. 1. .text section with writable attribute (support) 2. .reloc section has the symbol for *ABS* (skip) Signed-off-by: Liming Gao Reviewed-by: Feng Bob C # Conflicts:

[edk2-devel] [PATCH 0/6] Add CLANGDWARF toolchain for universal payload needs

2021-06-02 Thread Ni, Ray
Universal payload requires its format in ELF format while today's EDKII doesn't contain a cross OS toolchain that generates ELF images. The patch set is based on Liming's work in year 2019 and some very minor modifications are made: 1. Update toolchain name from CLANG8ELF to CLANGDWARF. 2. Update

[edk2-devel] [PATCH 1/6] BaseTools: Add ClangBase.lds for CLANG8 tool chain with max-page-size

2021-06-02 Thread Ni, Ray
From: Liming Gao LLVM LLD linker doesn't support common-page-size option. So, max-page-size is used. To not impact GCC tool chain, new ClangBase.lds is added. Signed-off-by: Liming Gao Reviewed-by: Feng Bob C --- BaseTools/Scripts/ClangBase.lds | 79 + 1 file c

Re: [edk2-devel] [PATCH] EmbeddedPkg/RealTimeClockRuntimeDxe: Improve GetWakeupTime

2021-06-02 Thread Ard Biesheuvel
On Mon, 24 May 2021 at 05:55, Marcin Wojtas wrote: > > GetWakeupTime should return full time information, including > the daylight/timezone. Make use of the existing non-volatile > variables for that purpose. Moreover add an error checking > of possibly invalid parameters. > This partially fixes F

Re: [edk2-devel] [edk2-platforms PATCH 0/6] Marvell ACS fixes

2021-06-02 Thread Ard Biesheuvel
On Mon, 24 May 2021 at 07:29, Marcin Wojtas wrote: > > Hi, > > This series addresses a couple of issues found during > ACS3.0 test suit runs. The SMBIOS Type17 table > extension and RTC fixes/improvements. > Moreover the SPCR console subtype is switched to > a different value and Cn913xDb ACPI tab