Thanks for sharing the updated patches. I tested the patches on both QEMU and FVP, they work for all my use cases. Although my vote probably does not mean much, thanks again for sharing this update: T

Hi Rebecca, Thanks for working through the feedback as you mentioned on the other thread. But hopefully this email finds you before the v5 gets published. I ran through some more tests and found a few

Hi Rebecca, Thanks for sharing this patch. I found a few minor issues when running this test app. Please see comments with [KQ] below. Regards, Kun [KQ] The NumCpu and NumEnabledCpu probably should be

Sorry, hit the send button too soon... One more comment under [KQ-2]. Thanks, Kun [KQ-2] This will make the core never being able to accept the subsequent start up calls due to this CPU data state wil

Hi Rebecca, I have tried to use this for a few different scenarios and below is a few minor issues I have found and potential changes (marked with [KQ]). Please let me know if I have any misunderstand

Hi Rebecca, Thanks for the information. Somehow these patches (i.e. https://edk2.groups.io/g/devel/message/98106) just landed on groups.io last night. I will test the latest version of your patches an

Hi Ard/Rebecca, Thanks for bringing this to the mailing list. But somehow I cannot find the patches sent along with this v4 cover letter. Could you please point me to them? I have been running the pre

Thanks for the update, Rebecca. I will also keep an eye on the cache issue when I have a chance to run this on FVP models. Regards, Kun

Hi Rebecca/Ard, I was trying to reach out regarding the original patches earlier (see below) but it might fell off your stacks due to high traffic on the mailing list. Could you please kindly review t

Hi Rebecca, Thanks for sending this patch. I have a few questions inline "[KQ]". Could you please help me to understand the patch better? Thanks in advance. Regards, Kun [KQ]: Is there any reason why

Thanks for the help, Bob. It seems that one of the "Reviewed-by" tags are breaking the pipeline patchcheck: https://dev.azure.com/tianocore/edk2-ci/_build/results?buildId=62531&view=logs&j=12f1170f-54

From: Sean Brogan <sean.brogan@...> Current implementation of looking up toolchain will _insert_ the findings from vsvarsall.bat to existing path and potentially stuff the variable to exceed

Currently the BaseTool build step is failing and caused the pipeline to malfunciton. The issue is due to the environment path too long during the build process and adding VC toolchain path to the envi

Hi Ard, No worries. Liming has helped to merge these patches here: MM communicate functionality in variable policy by lgao4 · Pull Request #2443 · tianocore/edk2 (github.com). Thanks for checking back

Hi Sami, Thank you so much the test and help! Regards, Kun

Hi Sami, Now that the merge window opens, could you please help us to merge in these patch series (after dropping this specific patch) when you have a chance? Please let me know if you ran into issues

Hi Sami, Thank you for the help! I agree that we can drop this patch and merge the rest when the window is open. Pierre, Thanks for your input on the usage as well! Regards, Kun

Hi Pierre, You are correct that if CM_ARM_PCI_ADDRESS_MAP_INFO.PCI_SS_CONFIG is no longer being used, this patch is not needed. Thanks for catching this. On the other hand, just for my learning purpos

Thank you so much for the help!

The new changes in SecureBootVariableLib brought in a new dependency of PlatformPKProtectionLib. This change added the new library instance from SecurityPkg to resolve PhytiumCommonPkg platforms build