|
[PATCH v5 0/2] ArmPkg: Implement EFI_MP_SERVICES_PROTOCOL based on PSCI calls for AArch64
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
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
|
By
Kun Qin
· #98972
·
|
|
[PATCH v4 2/3] ArmPkg: implement EFI_MP_SERVICES_PROTOCOL based on PSCI calls
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 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
|
By
Kun Qin
· #98413
·
|
|
[PATCH v4 3/3] MdeModulePkg: Add new Application/MpServicesTest application
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
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
|
By
Kun Qin
· #98143
·
|
|
[PATCH v4 2/3] ArmPkg: implement EFI_MP_SERVICES_PROTOCOL based on PSCI calls
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
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
|
By
Kun Qin
· #98142
·
|
|
[PATCH v4 2/3] ArmPkg: implement EFI_MP_SERVICES_PROTOCOL based on PSCI calls
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, 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
|
By
Kun Qin
· #98141
·
|
|
[PATCH v4 0/3] ArmPkg,MdeModulePkg: Implement EFI_MP_SERVICES_PROTOCOL for AArch64 and add an MpServicesTest application to exercise it
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 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
|
By
Kun Qin
· #98135
·
|
|
[PATCH v4 0/3] ArmPkg,MdeModulePkg: Implement EFI_MP_SERVICES_PROTOCOL for AArch64 and add an MpServicesTest application to exercise it
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
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
|
By
Kun Qin
· #98061
·
|
|
[PATCH 1/2] ArmPkg: implement EFI_MP_SERVICES_PROTOCOL based on PSCI calls
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
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
|
By
Kun Qin
· #96718
·
|
|
[PATCH 1/2] ArmPkg: implement EFI_MP_SERVICES_PROTOCOL based on PSCI calls
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/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
|
By
Kun Qin
· #96660
·
|
|
[PATCH 1/2] ArmPkg: implement EFI_MP_SERVICES_PROTOCOL based on PSCI calls
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
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
|
By
Kun Qin
· #94529
·
|
|
[PATCH v1 1/1] BaseTools: Edk2ToolsBuild: Fixing pipeline build due to path too long
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
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
|
By
Kun Qin
· #94162
·
|
|
[PATCH v1 1/1] BaseTools: Edk2ToolsBuild: Fixing pipeline build due to path too long
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
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
|
By
Kun Qin
· #94088
·
|
|
[PATCH v1 0/1] Fixing BaseTool build break
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
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
|
By
Kun Qin
· #94087
·
|
|
[PATCH v1 0/2] MM communicate functionality in variable policy
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 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
|
By
Kun Qin
· #93453
·
|
|
[PATCH v4 0/6] Enhance DynamicTablesPkg modules
Hi Sami, Thank you so much the test and help! Regards, Kun
Hi Sami, Thank you so much the test and help! Regards, Kun
|
By
Kun Qin
· #93063
·
|
|
[PATCH v4 6/6] DynamicTablesPkg: AcpiSsdtPcieLibArm: Added case handling for PCI config
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, 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
|
By
Kun Qin
· #92984
·
|
|
[PATCH v4 6/6] DynamicTablesPkg: AcpiSsdtPcieLibArm: Added case handling for PCI config
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 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
|
By
Kun Qin
· #92529
·
|
|
[PATCH v4 6/6] DynamicTablesPkg: AcpiSsdtPcieLibArm: Added case handling for PCI config
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
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
|
By
Kun Qin
· #92495
·
|
|
[edk2-platforms][PATCH v5 0/4] Resolving SecureBootVariableLib dependency
Thank you so much for the help!
Thank you so much for the help!
|
By
Kun Qin
· #92480
·
|
|
[edk2-platforms][PATCH v5 4/4] PhytiumCommonPkg: Platform Build: Resolving newly introduced dependency
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
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
|
By
Kun Qin
· #92449
·
|