Hi Guo, Sorry for the late response. Just as we discussed today, and also from the suggestions by Ray, in the new patch I will 1. changed "UNIVERSAL_PAYLOAD_ DEBUG_PRINT_ERROR_LEVEL" to

REF： https://bugzilla.tianocore.org/show_bug.cgi?id=3882 UefiCpuPkg define a new Protocol with the new services SmmWaitForAllProcessor(), which can be used by SMI handler to optionally wait for

REF： https://bugzilla.tianocore.org/show_bug.cgi?id=3882 UefiCpuPkg define a new Protocol with the new services SmmWaitForAllProcessor(), which can be used by SMI handler to optionally wait for

REF： https://bugzilla.tianocore.org/show_bug.cgi?id=3861 UefiCpuPkg define a new Protocol with the new services SmmWaitForAllProcessor(), which can be used by SMI handler to optionally wait for

Agree with Jian. I don’t understand this patch set and the feature. I request a design review in EDKII. Thank you Yao Jiewen

Hi Judah, You could update the readme in edk2-staging or your own forked branch. Here's an example in edk2-staging repo: https://github.com/tianocore/edk2-staging/tree/ProtectedVariable/libs BTW,

Hi Judah, The commit message is too simple to be useful for reviewers and developers. Other patches have the same issue. Please add more descriptions to explain the changes made in each patch (why

REF： https://bugzilla.tianocore.org/show_bug.cgi?id=3882 UefiCpuPkg define a new Protocol with the new services SmmWaitForAllProcessor(), which can be used by SMI handler to optionally wait for

REF： https://bugzilla.tianocore.org/show_bug.cgi?id=3882 UefiCpuPkg define a new Protocol with the new services SmmWaitForAllProcessor(), which can be used by SMI handler to optionally wait for

REF： https://bugzilla.tianocore.org/show_bug.cgi?id=3882 UefiCpuPkg define a new Protocol with the new services SmmWaitForAllProcessor(), which can be used by SMI handler to optionally wait for

I replied in another email. It looks the comments are not addressed in this patch, especially on this comment: In the DebugPrintErrorLevel.h, ErrorLevel is defined as UINT32, but its usage is not

Hi Gerd, Could you provide a patch to fix this issue or revert the original commit? Thanks, Bob

Reviewed-by: Jian J Wang <jian.j.wang@...> Regards, Jian

Thanks, Leif. I'm happy to hear that. It seems like you have not reviewed the v5 of patch 30 yet. Best regards, Nhi

Pushed via: PR - https://github.com/tianocore/edk2/pull/2701 Commit - https://github.com/tianocore/edk2/commit/7456990e8eebe3b935447253bb6d1d3129839122 Best Regards, Hao Wu

Reminder: TianoCore Bug Triage - APAC / NAMO When: 03/29/2022 6:30pm to 7:30pm (UTC-07:00) America/Los

Tools, CI, Code base construction meeting series When: 03/28/2022 4:30pm to 5:30pm (UTC-07:00) America/Los Angeles Where: https://github.com/tianocore/edk2/discussions/2614 View

Thanks, Liming. SMM owners/authors, Could you please also review the original issue and this patch to provide feedback? Thanks, Kun

This breaks building BaseTools with clang 13.1.6 on macOS: /Applications/Xcode.app/Contents/Developer/usr/bin/make -C DevicePath gcc  -c  -I .. -I ../Include/Common -I ../Include/ -I

Reorder the pages in the MEMFD section of AmdSevX64.fdf so that it matches the same order used in OvmfPkgX64.fdf. After this change, this is the difference in the MEMFD of the two targets: $ diff -u