[edk2-devel] [edk2][PATCH V3 1/1] ArmPlatformPkg/PrePeiCore: Explicitly invoke constructor for SEC phase

Rohit Mathew rohit.mathew at arm.com
Fri Jul 22 17:12:03 UTC 2022


Hi Sami,

On Thu, Jul 21, 2022 at 05:26 PM, Sami Mujawar wrote:

> 
> Hi Rohit,
> 
> On Wed, Jul 6, 2022 at 06:42 AM, Rohit Mathew wrote:
> 
>> =20
>> @@ -86,6 +86,10 @@ CEntryPoint (
>> ArmEnableVFP ();
>> }
>> =20
>> + // Explicitly invoke the library constructor to resolve any library
>> + // dependency.
>> + ProcessLibraryConstructorList();
>> +
> 
> I missed this in my previous feedback, but ProcessLibraryConstructorList()
> should only be called by the Primary core.
> 
> Regards,
> 
> Sami Mujawar

I have moved the call within -

if (ArmPlatformIsPrimaryCore (MpId)) {

This, and the other feedbacks have been addressed in V4.

Regards,
Rohit


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91727): https://edk2.groups.io/g/devel/message/91727
Mute This Topic: https://groups.io/mt/92206586/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/20220722/a17cfef4/attachment.htm>


More information about the edk2-devel-archive mailing list