From mboxrd@z Thu Jan  1 00:00:00 1970
Subject: Re: [edk2-devel] [edk2][PATCH V3 1/1] ArmPlatformPkg/PrePeiCore: Explicitly invoke constructor for SEC phase
To: Rohit Mathew <rohit.mathew@arm.com>,devel@edk2.groups.io
From: "Sami Mujawar" <sami.mujawar@arm.com>
X-Originating-Location: Cambridge, England, GB (217.140.106.51)
X-Originating-Platform: Windows Firefox 102
User-Agent: GROUPS.IO Web Poster
MIME-Version: 1.0
Date: Thu, 21 Jul 2022 09:26:38 -0700
References: <20220706134234.17847-1-rohit.mathew@arm.com>
In-Reply-To: <20220706134234.17847-1-rohit.mathew@arm.com>
Message-ID: <15026.1658420798225177209@groups.io>
Content-Type: multipart/alternative; boundary="XhO5UymttnFeQbtLZfs6"

--XhO5UymttnFeQbtLZfs6
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hi Rohit,

On Wed, Jul 6, 2022 at 06:42 AM, Rohit Mathew wrote:

>=20
> =3D20
> @@ -86,6 +86,10 @@ CEntryPoint (
> ArmEnableVFP ();
> }
> =3D20
> + // 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

--XhO5UymttnFeQbtLZfs6
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

--XhO5UymttnFeQbtLZfs6--