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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to