Hi Ard Can it be resolved in patch - https://edk2.groups.io/g/devel/message/75135?p=,,,20,0,0,0::Created,,cryptopkg,20,2,0,82822574 ? I really hope to improve CI to catch this earlier... Hi

This patch is now merged, and is breaking the build on AARCH64/CLANG38: CryptoPkg/Library/BaseCryptLib/Pk/CryptRsaPss.c:117:7: error: variable 'Result' is used uninitialized whenever 'if' condition

Yes it does.

Etienne: Thanks for your reminder. I try VS compiler and meet with the compiler error on this line. Here, does if ((SecCoreEntryAddress & 1) != 0) mean the lowest bit of this address is

This change allows to build StandaloneMmPkg components for 32bit Arm StandaloneMm firmware. This change mainly moves AArch64/ source files to Arm/ side directory for several components:

Use intermediate (UINTN) cast when casting int from/to pointer. This is needed as UINT64 values cast from/to 32bit pointer for 32bit architectures. Cc: Ard Biesheuvel <ardb+tianocore@...> Cc:

Change GenFv for Arm architecture to generate a specific jump instruction as image entry instruction, when the target entry label is assembled with Thumb instruction set. This is possible

Changes in ArmPkg to prepare building StandaloneMm firmware for 32bit Arm architectures. Adds MmCommunicationDxe driver and ArmMmuPeiLib and ArmmmuStandaloneMmLib libraries to the list of the

Defines ARM_SVC_ID_FFA_* and ARM_SVC_ID_SP_* identifiers for 32bit function IDs as per SMCCC specification. Defines also generic ARM SVC identifier macros to wrap 32bit or 64bit identifiers upon

Laszlo: Thanks for your detail review. I have no comments for the changes in this patch set. Reviewed-by: Liming Gao <gaoliming@...> Thanks Liming

Sorry, I missed this one. This needs extra parantheses. I'll sent a v3. My apologies... etienne

Acked-by: Liming Gao <gaoliming@...> from would

Create new entry for Cloud Hypervisor and assign reviewer to Sami Mujawar. Cc: Sami Mujawar <sami.mujawar@...> Signed-off-by: Jianyong Wu <jianyong.wu@...> --- Maintainers.txt | 7 +++++++ 1

Cloud Hypervisor is kvm based VMM and is implemented in rust. Just like other VMMs it need UEFI support to let ACPI work. That's why Cloud Hypervisor is introduced here. Cc: Laszlo Ersek

The current implementation of PlatformHasAcpiDt is not a common library and is on behalf of qemu. So give a specific version for Cloud Hypervisor here. There is no device like Fw-cfg in qemu in Cloud

As there is lack of a machnism in Cloud Hypervisor to pass the base address of Rsdp in Acpi, so a PCD varialbe is introduced here to feed it. Cc: Hao A Wu <hao.a.wu@...> Cc: Jian J Wang

Cloud Hypervisor is kvm based VMM implemented in rust. This library populates the system memory map for the Cloud Hypervisor virtual platform. Cc: Laszlo Ersek <lersek@...> Cc: Ard Biesheuvel

Cloud Hypervisor is an open source Virtual Machine Monitor (VMM) that runs on top of KVM. Cloud Hypervisor is implemented in Rust and is based on the rust-vmm crates. See [1] to find more. To support

Dear all, For info, Tested on Qemu/arm and stm32mp1 (both armv7-a) and on 64bit Qemu/AArch64. I used OP-TEE 3.13.0 to host the StMM instance. Regards, Etienne <etienne.carriere@...> wrote:

Reviewed-by: Chasel Chiu <chasel.chiu@...>