[+cc: Tobin] Hi Brijesh, Would it make sense to always use EfiACPIMemoryNVS for the injected secret area, even for regular SEV (non-SNP)? -Dov

Update CML microcode from otcshare. Signed-off-by: jinjhuli <jin.jhu.lim@...> Cc: Kathappan Esakkithevar <kathappan.esakkithevar@...> Cc: Sai Chaganty <rangasai.v.chaganty@...> ---

Update CML microcode from otcshare. Signed-off-by: jinjhuli <jin.jhu.lim@...> Cc: Nate DeSimone <nathaniel.l.desimone@...> --- .../Microcode/MicrocodeUpdates.inf | 4 ++--

Acked-by: Jiewen Yao <Jiewen.yao@...> Need ARM expert to comment if it is OK to refer AArch64 for ARM? Thank you Yao Jiewen

Reviewed-by: Jiewen Yao <jiewen.yao@...>

While coding it I am coming to realizing that mapping from PVALIDATE return value to EFI will add more code in assemblym and wanted to make sure that we all are okay with it. The proposed mapping

It's currently used in BaseRngLib, and since there are other Aarch64 functions in BaseLib I thought it should belong here - especially since I don't think MdePkg can depend on ArmPkg? That makes

Hi Rebecca, Please see my response inline marked [SAMI]. Regards, Sami Mujawar On 28/04/2021 09:44 PM, Rebecca Cran wrote: [SAMI] Please add doxygen headers for the

Hi Rebecca, Please find my response inline marked [SAMI]. Regards, Sami Mujawar [SAMI] Should this function be part of ArmLib? [/SAMI] [SAMI] I can see that the X86RdRand64 functions are

Apologies, I missed the fact that you said document the mapping between the PVALIDATE return value and EFI_STATUS. So a caller is responsible to look at the EFI document to know what the error code

Hi Jin Jhu, Please make sure you update Silicon/Intel/CometlakeSiliconBinPkg/Microcode/MicrocodeUpdates.inf with the new filenames. Also, please remove the link to otcshare from the commit

I am perfectly fine if the function return UINTN and then use #define instead of the enum to define the PVALIDATE return code. So that caller can check the error code. Let me know your thought on

https://bugzilla.tianocore.org/show_bug.cgi?id=3178 (At this point, the title of that bug should be updated so it say "edk2-stable202105".) Laszlo

From "qemu-system-arm", it seems like you really mean 32-bit ARM. Yes, there is support for that in edk2. Build: $ git submodule update --init --force $ . edksetup.sh $ nice make -C

Noted. Noted. Noted. Interestingly I did ran the series with edk2 CI and everything passed before I submitted for the review. But, I will go ahead and fix the order. Works with me. Works

Noted. I will add something in comment to clarify that UNMASH and PSMASH are not available in MSR protocol. That macro is replaced is no longer used, I ended up creating a more meaningful macro

Thanks. I guess I was confused by the term "for the requested page size". Yes, that's clear. Thanks! Laszlo

OK, thanks! Indeed, the v1 patches have not reached the list. Thanks, Laszlo

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3314 This patch uses Openssl's EVP API's to perform RSASSA-PSS verification of a binary blob. Cc: Jian J Wang <jian.j.wang@...> Cc: Xiaoyu

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3314 This patch uses Openssl's EVP API's to perform RSASSA-PSS verification of a binary blob. Patch v1 Cover Letter :