[edk2-devel] [PATCH 2/2] ArmPkg: Add Library/MpInitLib to support EFI_MP_SERVICES_PROTOCOL
Rebecca Cran
rebecca at nuviainc.com
Wed Dec 15 17:18:53 UTC 2021
Hi Sami,
I've been looking at this again to make sure I've addressed all the
issues before sending out a v2 patch series. I'm stuck on one point
though: I'm not sure how it can be split into 2-3 patches without
breaking bisecting. The CpuDxe and MpInitLib changes introduce a
dependency on MpInitLib, so I don't see how I can put the ArmVirtPkg
change into a separate commit without breaking it?
--
Rebecca Cran
On 10/22/21 11:51 AM, Sami Mujawar wrote:
>
> Hi Rebecca,
>
> Thanks you for this patch.
>
> I think this patch could be split in 2 - 3 patches (CpuDxe & MpInitLib
> and the ArmVirtPkg change should be a separate patch).
>
> Please find my feedback inline marked [SAMI].
>
> I think the usage of MPIDR and CpuInfo->ProcessorId needs to be
> revisited in this patch series. I have kept my feedback on usage of
> MPIDR and the MT bit [24] to give the context. But, it will be good to
> only retain the affinity bits and mask all other bits of the MPIDR.
>
> Please let me know if you have any queries.
>
> Regards,
>
> Sami Mujawar
>
>
> On 18/10/2021 04:39 PM, Rebecca Cran wrote:
>> Add support for EFI_MP_SERVICES_PROTOCOL during the DXE phase under
>> AArch64.
>>
>> PSCI_CPU_ON is called to power on the core, the supplied procedure is
>> executed and PSCI_CPU_OFF is called to power off the core.
>>
>> Minimal setup is done before calling the supplied procedure: for example
>> the MMU and caches are not enabled.
>>
>> Signed-off-by: Rebecca Cran<rebecca at nuviainc.com>
>> ---
>> ArmPkg/ArmPkg.dec | 4 +
>> ArmPkg/ArmPkg.dsc | 4 +
>> ArmPkg/Drivers/CpuDxe/AArch64/Arch.c | 22 +
>> ArmPkg/Drivers/CpuDxe/Arm/Arch.c | 22 +
>> ArmPkg/Drivers/CpuDxe/CpuDxe.c | 2 +
>> ArmPkg/Drivers/CpuDxe/CpuDxe.h | 10 +
>> ArmPkg/Drivers/CpuDxe/CpuDxe.inf | 6 +
>> ArmPkg/Drivers/CpuDxe/CpuMpInit.c | 604 ++++++++
>> ArmPkg/Include/Library/MpInitLib.h | 366 +++++
>> ArmPkg/Library/MpInitLib/AArch64/MpFuncs.S | 61 +
>> ArmPkg/Library/MpInitLib/DxeMpInitLib.inf | 53 +
>> ArmPkg/Library/MpInitLib/DxeMpLib.c | 1498 ++++++++++++++++++++
>> ArmPkg/Library/MpInitLib/InternalMpInitLib.h | 358 +++++
>> ArmVirtPkg/ArmVirt.dsc.inc | 3 +
>> 14 files changed, 3013 insertions(+)
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84912): https://edk2.groups.io/g/devel/message/84912
Mute This Topic: https://groups.io/mt/86417186/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20211215/c70c0978/attachment.htm>
More information about the edk2-devel-archive
mailing list