Andrew, could you review this please? Thanks. Rebecca Cran

Acked-by: Ilias Apalodimas <ilias.apalodimas@...>

Acked-by: Ilias Apalodimas <ilias.apalodimas@...>

Acked-by: Ilias Apalodimas <ilias.apalodimas@...>

Acked-by: Ilias Apalodimas <ilias.apalodimas@...>

That looks like the same issue, yes. Yes, I am also a bit surprised that it was not caught before.

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