Re: [edk2-devel] [tianocore.github.io.wiki PATCH 1/1] Xcode.md: Update instructions to work on modern macOS and Xcode versions

2021-07-20 Thread Andrew Fish via groups.io
These Xcode instructions look good to me in general. Thanks for doing this I usually do things following a non public path. I think to make these instructions work you need to update *_XCODE5_*_MTOC_PATH By default, this will install mtoc at /opt/local/bin/mtoc.

Re: [edk2-devel] [PATCH] EmulatorPkg: Update lldbefi.py to work with current lldb which uses python3

2021-07-20 Thread Andrew Fish via groups.io
Reviewed-by: Andrew Fish > On May 9, 2021, at 12:26 PM, Rebecca Cran wrote: > > The version of lldb shipping with macOS Big Sur is lldb-1205.0.27.3, and > it uses python3. Update lldbefi.py to work with it, including removing > the unused 'commands' import and fixing the print statements. > >

Re: [edk2-devel] [PATCH v2 1/1] MdeModulePkg: Add BootDiscoveryPolicyUiLib.

2021-07-20 Thread Gao, Zhichao
OK. I am not familiar with PCD, it is new usage for me. And now I got to know the reason. But seems the behavior would be different base on the initialization on DSC file. Whatever, this patch is OK to me. Reviewed-by: Zhichao Gao Thanks, Zhichao > -Original Message- > From: Grzegorz

Re: [edk2-devel] [edk2-platforms][PATCH V1] KabylakeOpenBoardPkg: Implement the PEI Reset Services

2021-07-20 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Desimone, Nathaniel L > Sent: Tuesday, July 13, 2021 5:27 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Jeremy Soller > ; Michael Kubacki ; > Benjamin Doron > Subject: [edk2-platforms][PATCH V1] KabylakeOpenBoardPkg:

Re: [edk2-devel] [edk2-platforms] [PATCH V1] WhitleySiliconPkg: Improve comments for silicon policy structures

2021-07-20 Thread Chiu, Chasel
Hi Nate, Just one small feedbacks inline, please check them. With that resolved: Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Desimone, Nathaniel L > Sent: Wednesday, July 21, 2021 4:22 AM > To: devel@edk2.groups.io > Cc: Oram, Isaac W ; Chiu, Chasel

[edk2-devel] Whitley FSP Released

2021-07-20 Thread Nate DeSimone
Hi All, Intel is pleased to announce that Whitley FSP is now available at https://github.com/intel/FSP. With Best Regards, Nate -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#78002): https://edk2.groups.io/g/devel/message/78002

Re: [edk2-devel] [PATCH v6 00/11] Secure Boot default keys

2021-07-20 Thread Sunny Wang
Ard, Liming, Ray, Thanks for your review for ArmVirtPkg, ArmPlatformPkg, and EmulatorPkg patches. As for the patch for Intel Platforms below, it is in another series for edk2-platforms. - [edk2-platforms PATCH v6 1/4] Intel Platforms: add SecureBootVariableLib class resolution

Re: [edk2-devel] [PATCH v6 05/11] EmulatorPkg: add SecureBootVariableLib class resolution

2021-07-20 Thread Ni, Ray
Reviewed-by: Ray Ni -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#78000): https://edk2.groups.io/g/devel/message/78000 Mute This Topic: https://groups.io/mt/84200219/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe:

Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Add Jiewen Yao as OvmfPkg Maintainer

2021-07-20 Thread Yao, Jiewen
Reviewed-by: Jiewen Yao > -Original Message- > From: Kinney, Michael D > Sent: Wednesday, July 21, 2021 8:44 AM > To: devel@edk2.groups.io > Cc: Yao, Jiewen ; Ard Biesheuvel > ; Justen, Jordan L ; > Andrew Fish ; Leif Lindholm > Subject: [Patch 1/1] Maintainers.txt: Add Jiewen Yao as

回复: [edk2-devel] 回复: [PATCH v7 1/3] BaseTools: Remove COMMON section from the GCC discard list

2021-07-20 Thread gaoliming
Christopher: Based on current information, I am OK for this change. Reviewed-by: Liming Gao Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Christopher > Zurcher > 发送时间: 2021年7月21日 9:15 > 收件人: devel@edk2.groups.io; gaolim...@byosoft.com.cn > 抄送: 'Ard Biesheuvel' ; 'Bob Feng' >

Re: [edk2-devel] [PATCH v2 1/1] MdeModulePkg: Print which PCD was unable to be found

2021-07-20 Thread Wu, Hao A
When merging, change the patch subject a little to: MdeModulePkg PCD: Print which PCD was unable to be found Pushed via: PR - https://github.com/tianocore/edk2/pull/1825 Commit - https://github.com/tianocore/edk2/commit/83b43c4cb17ec8c9fd2bb5843bf66e5961f28c82 Best Regards, Hao Wu >

Re: [edk2-devel] 回复: [PATCH v7 1/3] BaseTools: Remove COMMON section from the GCC discard list

2021-07-20 Thread Christopher Zurcher
Yes, this was verified with OVMF. Thanks, Christopher Zurcher -Original Message- From: devel@edk2.groups.io On Behalf Of gaoliming Sent: Tuesday, July 20, 2021 18:12 To: christopher.zurc...@outlook.com; devel@edk2.groups.io Cc: 'Ard Biesheuvel' ; 'Bob Feng' Subject: [edk2-devel] 回复:

[edk2-devel] 回复: [PATCH v7 1/3] BaseTools: Remove COMMON section from the GCC discard list

2021-07-20 Thread gaoliming
Christopher: Thanks for your update. Can you let me know which platform is verified with this change by GCC tool chain? Ovmf? Thanks Liming > -邮件原件- > 发件人: christopher.zurc...@outlook.com > > 发送时间: 2021年7月21日 6:07 > 收件人: devel@edk2.groups.io > 抄送: Ard Biesheuvel ; Bob Feng ; > Liming

[edk2-devel] [Patch 1/1] Maintainers.txt: Add Jiewen Yao as OvmfPkg Maintainer

2021-07-20 Thread Michael D Kinney
Cc: Jiewen Yao Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Andrew Fish Cc: Leif Lindholm Signed-off-by: Michael D Kinney --- Maintainers.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Maintainers.txt b/Maintainers.txt index e9dda5c5ca0c..dd9647327774 100644 --- a/Maintainers.txt +++

[edk2-devel] [PATCH 2/2] IntelSiliconPkg: Add IgdOpRegion30.h to support IGD OpRegion v3.0

2021-07-20 Thread Digant H Solanki
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3426 - There are many OpRegion fields obsoleted in MBOX1 - MBOX2 is re-purposed for Backlight related fields for dual LFP. - Backlight related fields moved to MBOX2 from MBOX3 and some fields are obsoleted in MBOX3. Signed-off-by: Digant H

[edk2-devel] Buildtools error

2021-07-20 Thread Matthew Carlson via groups.io
Hey all, I got this error when doing a build: build.py... : error C0DE: Unknown fatal error when processing [/edk2/MdePkg/Library/BaseMemoryLibOptPei/BaseMemoryLibOptPei.inf] (Please send email to edk2-de...@lists.01.org for help, attaching following call stack trace!) (Python 2.7.16 on

[edk2-devel] [PATCH v7 2/3] CryptoPkg/OpensslLib: Add native instruction support for X64

2021-07-20 Thread Christopher Zurcher
From: Christopher Zurcher BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2507 Adding OpensslLibX64.inf and modifying process_files.pl to process this file and generate the necessary assembly files. Adding OpensslLibX64Gcc.inf to allow building with GCC toolchain. ApiHooks.c contains a stub

[edk2-devel] [PATCH v7 0/3] CryptoPkg/OpensslLib: Add native instruction support for X64

2021-07-20 Thread Christopher Zurcher
From: Christopher Zurcher V7 Changes: Move BaseTools change to a separate commit. V6 Changes: Add GCC-compatible version of these modifications. Supporting GCC build of native OpenSSL .S files requires removal of *(COMMON) from the /DISCARD/ section of the GCC linker script. The

[edk2-devel] [PATCH v7 1/3] BaseTools: Remove COMMON section from the GCC discard list

2021-07-20 Thread Christopher Zurcher
From: Christopher Zurcher BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2507 The COMMON section is used by OpenSSL assembly-optimized crypto functions. OpenSSL assembly code is auto-generated from the submodule and cannot be modified to remove dependence on the COMMON section. The default

[edk2-devel] [edk2-platforms] [PATCH V1] WhitleySiliconPkg: Improve comments for silicon policy structures

2021-07-20 Thread Nate DeSimone
Signed-off-by: Nate DeSimone Cc: Isaac Oram Cc: Chasel Chiu --- .../Include/Ppi/RasImcS3Data.h| 6 + .../Include/Ppi/UpiPolicyPpi.h| 5 +- .../WhitleySiliconPkg/Include/Upi/KtiHost.h | 250 +- .../SouthClusterLbg/Include/PchPolicyCommon.h |

Re: [edk2-devel] [edk2-libc PATCH 1/1] Update LibC to use safe string functions

2021-07-20 Thread Rebecca Cran
This hasn't had any reviews yet. Could someone take a look at it please? Thanks. Rebecca Cran On 6/7/21 10:21 AM, Rebecca Cran via groups.io wrote: The insecure string functions such as StrCpy were removed a while ago, breaking the StdLib build. Migrate StdLib/LibC to the safe string

Re: [edk2-devel] Gdb and lldb support for OVMF and more

2021-07-20 Thread Rebecca Cran
Could you commit the fixes I submitted to EmulatorPkg/Unix/lldbefi.py too, please? See "[edk2-devel] [PATCH] EmulatorPkg: Update lldbefi.py to work with current lldb which uses python3" Also, could you take another look at the updated Xcode wiki instructions I provided in "[edk2-devel]

Re: [edk2-devel] [PATCH v3 10/11] OvmfPkg: add BlobVerifierLibSevHashes

2021-07-20 Thread Brijesh Singh via groups.io
On 7/20/21 3:04 AM, Dov Murik wrote: Add an implementation for BlobVerifierLib that locates the SEV hashes table and verifies that the calculated hashes of the kernel, initrd, and cmdline blobs indeed match the expected hashes stated in the hashes table. If there's a missing hash or a hash

Re: [edk2-devel] [PATCH v3 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-20 Thread Ard Biesheuvel
On Tue, 20 Jul 2021 at 19:22, Tom Lendacky wrote: > > On 7/20/21 3:03 AM, Dov Murik wrote: > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3457 > > I believe the convention is that this line be in the individual patch > commit messages just like this (i.e. with the BZ: tag and the first

Re: [edk2-devel] USB isochronous transfers

2021-07-20 Thread Andrew Fish via groups.io
> On Jul 20, 2021, at 8:32 AM, Ethin Probst wrote: > > Hello all, > > A few days ago Leif determined that the USB DXE does not implement > isochronous transfers. This will make USB Audio quite impossible as > the audio data endpoint requires isochronous transfers. > > I was looking at the

Re: [edk2-devel] [PATCH v3 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-20 Thread Lendacky, Thomas via groups.io
On 7/20/21 3:03 AM, Dov Murik wrote: > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3457 I believe the convention is that this line be in the individual patch commit messages just like this (i.e. with the BZ: tag and the first line), not as a Ref: tag at the end of the commit message. I'll

Re: [edk2-devel] [PATCH v3 11/11] OvmfPkg/AmdSev: Enforce hash verification of kernel blobs

2021-07-20 Thread Lendacky, Thomas via groups.io
On 7/20/21 3:04 AM, Dov Murik wrote: > In the AmdSevX64 build, use BlobVerifierLibSevHashes to enforce > verification of hashes of the kernel/initrd/cmdline blobs fetched from > firmware config. > > This allows for secure (measured) boot of SEV guests with QEMU's > -kernel/-initrd/-append

Re: [edk2-devel] [PATCH v3 10/11] OvmfPkg: add BlobVerifierLibSevHashes

2021-07-20 Thread Lendacky, Thomas via groups.io
On 7/20/21 3:04 AM, Dov Murik wrote: > Add an implementation for BlobVerifierLib that locates the SEV hashes > table and verifies that the calculated hashes of the kernel, initrd, and > cmdline blobs indeed match the expected hashes stated in the hashes > table. > > If there's a missing hash or a

Re: [edk2-devel] USB isochronous transfers

2021-07-20 Thread Michael D Kinney
PEI use of USB is usually limited to access to recovery images from media devices. I would recommend focusing on UEFI components, Mike > -Original Message- > From: devel@edk2.groups.io On Behalf Of Ethin Probst > Sent: Tuesday, July 20, 2021 8:33 AM > To: devel@edk2.groups.io > Cc:

Re: [edk2-devel] [PATCH 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID

2021-07-20 Thread Jeff Brasen via groups.io
In my opinion MdePkg is where this should be as it is meant to be used by multiple software entities (linux kernel, grub, edk2, coreboot w/ uefi binding) and probably should be documented in some spec (Although, I am not sure which one would make sense) I am fine with MdeModulePkg as well

Re: [edk2-devel] [PATCH v3 09/11] OvmfPkg/AmdSev: reserve MEMFD space for for firmware config hashes

2021-07-20 Thread Brijesh Singh via groups.io
On 7/20/21 3:03 AM, Dov Murik wrote: From: James Bottomley Split the existing 4KB page reserved for SEV launch secrets into two parts: first 3KB for SEV launch secrets and last 1KB for firmware config hashes. The area of the firmware config hashes will be attested (measured) by the PSP and

Re: [edk2-devel] [PATCH v3 08/11] OvmfPkg/AmdSev/SecretPei: build hob for full page

2021-07-20 Thread Brijesh Singh via groups.io
On 7/20/21 3:03 AM, Dov Murik wrote: Round up the size of the SEV launch secret area to a whole page, as required by BuildMemoryAllocationHob. This will allow the secret area defined in the MEMFD to take less than a whole 4KB page. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Ashish Kalra

Re: [edk2-devel] [PATCH v3 08/11] OvmfPkg/AmdSev/SecretPei: build hob for full page

2021-07-20 Thread Lendacky, Thomas via groups.io
On 7/20/21 3:03 AM, Dov Murik wrote: > Round up the size of the SEV launch secret area to a whole page, as > required by BuildMemoryAllocationHob. This will allow the secret > area defined in the MEMFD to take less than a whole 4KB page. > > Cc: Ard Biesheuvel > Cc: Jordan Justen > Cc: Ashish

Re: [edk2-devel] [PATCH v3 07/11] OvmfPkg/QemuKernelLoaderFsDxe: call VerifyBlob after fetch from fw_cfg

2021-07-20 Thread Lendacky, Thomas via groups.io
On 7/20/21 3:03 AM, Dov Murik wrote: > In QemuKernelLoaderFsDxeEntrypoint we use FetchBlob to read the content > of the kernel/initrd/cmdline from the QEMU fw_cfg interface. Insert a > call to VerifyBlob after fetching to allow BlobVerifierLib > implementations to add a verification step for

Re: [edk2-devel] [PATCH v3 05/11] OvmfPkg: add BlobVerifierLibNull to DSC

2021-07-20 Thread Lendacky, Thomas via groups.io
On 7/20/21 3:03 AM, Dov Murik wrote: > This prepares the ground for calling VerifyBlob() in > QemuKernelLoaderFsDxe. > > Cc: Ard Biesheuvel > Cc: Jordan Justen > Cc: Ashish Kalra > Cc: Brijesh Singh > Cc: Erdem Aktas > Cc: James Bottomley > Cc: Jiewen Yao > Cc: Min Xu > Cc: Tom Lendacky

Re: [edk2-devel] [PATCH v3 04/11] OvmfPkg: add library class BlobVerifierLib with null implementation

2021-07-20 Thread Lendacky, Thomas via groups.io
On 7/20/21 3:03 AM, Dov Murik wrote: > BlobVerifierLib will be used to verify blobs fetching them from QEMU's > firmware config (fw_cfg) in platforms that enable such verification. > > The null implementation BlobVerifierLibNull treats all blobs as valid. > > Cc: Ard Biesheuvel > Cc: Jordan

[edk2-devel] USB isochronous transfers

2021-07-20 Thread Ethin Probst
Hello all, A few days ago Leif determined that the USB DXE does not implement isochronous transfers. This will make USB Audio quite impossible as the audio data endpoint requires isochronous transfers. I was looking at the EHCI driver but I can't determine a couple things: - Isochronous

[edk2-devel] Gdb and lldb support for OVMF and more

2021-07-20 Thread Andrew Fish via groups.io
I’ve got a branch on a private repo (BZ3500-gdb @ https://github.com/ajfish/edk2.git ) that has gdb and lldb scripts that can attach to QEMU and symbolicate EFI. These scripts also provide other useful commands (symbols, hob, guid, device path, system

回复: [edk2-devel] [PATCH v4 5/5] StandaloneMmPkg: build for 32bit arm machines

2021-07-20 Thread gaoliming
Ard: Thanks! I have added this feature into https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Ard > Biesheuvel > 发送时间: 2021年7月20日 15:46 > 收件人: edk2-devel-groups-io ; Liming Gao (Byosoft > address) >

Re: [edk2-devel] [edk2-platform][PATCH v1 1/1] Platform/RaspberryPi/RPi4: Fix non-standard ACPI HIDs

2021-07-20 Thread Sunny Wang
Just saw two minor issues with the patch subject and messages. Maybe Ard can directly help fix them while pushing the change.  - For the subject, we should change "HIDs" to either "CIDs" or "IDs". - For the messages, there are two windows drivers' commits. The first commit number is

[edk2-devel] [PATCH v3 08/11] OvmfPkg/AmdSev/SecretPei: build hob for full page

2021-07-20 Thread Dov Murik
Round up the size of the SEV launch secret area to a whole page, as required by BuildMemoryAllocationHob. This will allow the secret area defined in the MEMFD to take less than a whole 4KB page. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Ashish Kalra Cc: Brijesh Singh Cc: Erdem Aktas Cc:

[edk2-devel] [PATCH v3 02/11] OvmfPkg/AmdSev: use GenericQemuLoadImageLib in AmdSev builds

2021-07-20 Thread Dov Murik
Newer kernels support efistub and therefore don't need all the legacy stuff in X86QemuLoadImageLib, which are harder to secure. Specifically the verification of kernel/initrd/cmdline blobs will be added only to the GenericQemuLoadImageLib implementation, so use that for SEV builds. Cc: Ard

[edk2-devel] [PATCH v3 03/11] OvmfPkg: PlatformBootManagerLibGrub: Allow executing kernel via fw_cfg

2021-07-20 Thread Dov Murik
From: James Bottomley Support QEMU's -kernel option. Create a QemuKernel.c for PlatformBootManagerLibGrub which is an exact copy of the file PlatformBootManagerLib/QemuKernel.c . Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Ashish Kalra Cc: Brijesh Singh Cc: Erdem Aktas Cc: James Bottomley

[edk2-devel] [PATCH v3 07/11] OvmfPkg/QemuKernelLoaderFsDxe: call VerifyBlob after fetch from fw_cfg

2021-07-20 Thread Dov Murik
In QemuKernelLoaderFsDxeEntrypoint we use FetchBlob to read the content of the kernel/initrd/cmdline from the QEMU fw_cfg interface. Insert a call to VerifyBlob after fetching to allow BlobVerifierLib implementations to add a verification step for these blobs. This will allow confidential

[edk2-devel] [PATCH v3 05/11] OvmfPkg: add BlobVerifierLibNull to DSC

2021-07-20 Thread Dov Murik
This prepares the ground for calling VerifyBlob() in QemuKernelLoaderFsDxe. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Ashish Kalra Cc: Brijesh Singh Cc: Erdem Aktas Cc: James Bottomley Cc: Jiewen Yao Cc: Min Xu Cc: Tom Lendacky Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3457

[edk2-devel] [PATCH v3 11/11] OvmfPkg/AmdSev: Enforce hash verification of kernel blobs

2021-07-20 Thread Dov Murik
In the AmdSevX64 build, use BlobVerifierLibSevHashes to enforce verification of hashes of the kernel/initrd/cmdline blobs fetched from firmware config. This allows for secure (measured) boot of SEV guests with QEMU's -kernel/-initrd/-append switches (with the corresponding QEMU support for

[edk2-devel] [PATCH v3 10/11] OvmfPkg: add BlobVerifierLibSevHashes

2021-07-20 Thread Dov Murik
Add an implementation for BlobVerifierLib that locates the SEV hashes table and verifies that the calculated hashes of the kernel, initrd, and cmdline blobs indeed match the expected hashes stated in the hashes table. If there's a missing hash or a hash mismatch then EFI_ACCESS_DENIED is returned

[edk2-devel] [PATCH v3 09/11] OvmfPkg/AmdSev: reserve MEMFD space for for firmware config hashes

2021-07-20 Thread Dov Murik
From: James Bottomley Split the existing 4KB page reserved for SEV launch secrets into two parts: first 3KB for SEV launch secrets and last 1KB for firmware config hashes. The area of the firmware config hashes will be attested (measured) by the PSP and thus the untrusted VMM can't pass in

[edk2-devel] [PATCH v3 06/11] ArmVirtPkg: add BlobVerifierLibNull to DSC

2021-07-20 Thread Dov Murik
This prepares the ground for calling VerifyBlob() in QemuKernelLoaderFsDxe. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Sami Mujawar Cc: Jordan Justen Cc: Ashish Kalra Cc: Brijesh Singh Cc: Erdem Aktas Cc: James Bottomley Cc: Jiewen Yao Cc: Min Xu Cc: Tom Lendacky Ref:

[edk2-devel] [PATCH v3 04/11] OvmfPkg: add library class BlobVerifierLib with null implementation

2021-07-20 Thread Dov Murik
BlobVerifierLib will be used to verify blobs fetching them from QEMU's firmware config (fw_cfg) in platforms that enable such verification. The null implementation BlobVerifierLibNull treats all blobs as valid. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Ashish Kalra Cc: Brijesh Singh Cc: Erdem

[edk2-devel] [PATCH v3 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-20 Thread Dov Murik
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3457 Booting with SEV prevented the loading of kernel, initrd, and kernel command-line via QEMU fw_cfg interface because they arrive from the VMM which is untrusted in SEV. However, in some cases the kernel, initrd, and cmdline are not secret

[edk2-devel] [PATCH v3 01/11] OvmfPkg/AmdSev/SecretDxe: fix header comment to generic naming

2021-07-20 Thread Dov Murik
From: James Bottomley Commit 96201ae7bf97 ("OvmfPkg/AmdSev/SecretDxe: make secret location naming generic", 2020-12-15) replaced references to SEV with the generic term Confidential Computing, but missed the file header comment. Fix the naming in that header. Cc: Ard Biesheuvel Cc: Jordan

Re: [edk2-devel] [PATCH v4 5/5] StandaloneMmPkg: build for 32bit arm machines

2021-07-20 Thread Ard Biesheuvel
On Tue, 20 Jul 2021 at 04:01, gaoliming wrote: > > Hi, all > This patch set has passed code review. How about merge it for this stable > tag edk2 202108? > OK, I will pick these up. Would you mind creating the entry for the release notes? > Thanks > Liming > > -邮件原件- > > 发件人:

Re: [edk2-devel] [PATCH v2 03/11] OvmfPkg: PlatformBootManagerLibGrub: Allow executing kernel via fw_cfg

2021-07-20 Thread Ard Biesheuvel
On Tue, 20 Jul 2021 at 09:33, Dov Murik wrote: > > > > On 19/07/2021 22:14, Dov Murik wrote: > > > > > > On 19/07/2021 18:21, Tom Lendacky wrote: > >> On 7/6/21 3:54 AM, Dov Murik wrote: > >>> From: James Bottomley > >>> > >>> Support QEMU's -kernel option. > >>> > >>>

Re: [edk2-devel] [edk2-platform][PATCH v1 1/1] Platform/RaspberryPi/RPi4: Fix non-standard ACPI HIDs

2021-07-20 Thread Ard Biesheuvel
On Mon, 19 Jul 2021 at 22:45, Samer El-Haj-Mahmoud wrote: > > Remove non-standard RPI ACPI _CIDs that are not needed. > This also fixes the FWTS failure reported in > https://github.com/pftf/RPi4/issues/67 > > The windows drivers at https://github.com/raspberrypi/windows-drivers > are still able

Re: [edk2-devel] [PATCH v2 03/11] OvmfPkg: PlatformBootManagerLibGrub: Allow executing kernel via fw_cfg

2021-07-20 Thread Dov Murik
On 19/07/2021 22:14, Dov Murik wrote: > > > On 19/07/2021 18:21, Tom Lendacky wrote: >> On 7/6/21 3:54 AM, Dov Murik wrote: >>> From: James Bottomley >>> >>> Support QEMU's -kernel option. >>> >>> OvmfPkg/Library/PlatformBootManagerLibGrub/QemuKernel.c is an exact copy >>> of

Re: [edk2-devel] [edk2-platforms: PATCH] Features/Intel/IpmiFeaturePkg: Use MdePkg macros instead of redefining.

2021-07-20 Thread Chaganty, Rangasai V
Reviewed-by: Sai Chaganty -Original Message- From: Chiu, Chasel Sent: Thursday, July 15, 2021 7:38 AM To: devel@edk2.groups.io Cc: Chiu, Chasel ; Desimone, Nathaniel L ; Chaganty, Rangasai V ; Liming Gao ; Oram, Isaac W Subject: [edk2-platforms: PATCH]

Re: [edk2-devel] [RFC PATCH] OvmfPkg/OvmfXen: set PcdAcpiS3Enable at initialization

2021-07-20 Thread Gary Lin via groups.io
On Mon, Jul 19, 2021 at 05:07:21PM +0100, Anthony PERARD wrote: > It would have been nice to have this patch in a patch series with > "OvmfPkg/OvmfXen: add QemuKernelLoaderFsDxe", mostly to make it simpler > to understand the problem needed to be fixed. > To be honest, I don't have Xen

Re: [edk2-devel] [staging/edk2-redfish-client PATCH v2 1/2] RedfishClientPkg: Initial commit

2021-07-20 Thread Nickle Wang
Reviewed-by: Nickle Wang Thanks, Nickle -Original Message- From: Chang, Abner (HPS SW/FW Technologist) Sent: Tuesday, July 20, 2021 11:54 AM To: devel@edk2.groups.io Cc: Wang, Nickle (HPS SW) ; Liming Gao ; Andrew Fish ; Leif Lindholm ; Michael D Kinney Subject: