Hi Laszlo, Thanks for the feedback. Sorry I missed this email this morning.
I will prepare a 2nd patch with these additional updates. Mike > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Laszlo > Ersek > Sent: Tuesday, January 23, 2024 5:57 AM > To: devel@edk2.groups.io; Huang, Qing <qing.hu...@intel.com> > Subject: Re: [edk2-devel] [PATCH v1] MdePkg/BaseCpuLibNull: Add stub > function of StandardSignatureIsAuthenticAMD() in CpuLibNull instance > > On 1/23/24 12:36, Huang, Qing wrote: > > CpuLib.h exposes StandardSignatureIsAuthenticAMD() API and we require > stub function in its BaseCpuLibNull library > > instance to avoid potential link issue. > > > > Signed-off-by: Qing Huang <qing.hu...@intel.com> > > --- > > MdePkg/Library/BaseCpuLibNull/BaseCpuLibNull.c | 17 ++++++++++++++++- > > 1 file changed, 16 insertions(+), 1 deletion(-) > > > > diff --git a/MdePkg/Library/BaseCpuLibNull/BaseCpuLibNull.c > b/MdePkg/Library/BaseCpuLibNull/BaseCpuLibNull.c > > index 3ba7a35096..ba7981551d 100644 > > --- a/MdePkg/Library/BaseCpuLibNull/BaseCpuLibNull.c > > +++ b/MdePkg/Library/BaseCpuLibNull/BaseCpuLibNull.c > > @@ -1,7 +1,7 @@ > > /** @file > > Null instance of CPU Library. > > > > - Copyright (c) 2020, Intel Corporation. All rights reserved.<BR> > > + Copyright (c) 2020 - 2024, Intel Corporation. All rights > reserved.<BR> > > SPDX-License-Identifier: BSD-2-Clause-Patent > > > > **/ > > @@ -35,3 +35,18 @@ CpuFlushTlb ( > > ) > > { > > } > > + > > +/** > > + Determine if the standard CPU signature is "AuthenticAMD". > > + > > + @retval TRUE The CPU signature matches. > > + @retval FALSE The CPU signature does not match. > > +**/ > > +BOOLEAN > > +EFIAPI > > +StandardSignatureIsAuthenticAMD ( > > + VOID > > + ) > > +{ > > + return FALSE; > > +} > > (1) Could we complete the Null instance with all the missing functions, > in one go? Such as: InitializeFloatingPointUnits, > StandardSignatureIsAuthenticAMD, GetCpuFamilyModel, GetCpuSteppingId? > > (2) All four of the mentioned APIs are only declared for IA32 and X64, > by the lib class header. Therefore their stub implementations, including > that of StandardSignatureIsAuthenticAMD(), should be restricted to IA32 > and X64 too. > > Thanks > Laszlo > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#114235): https://edk2.groups.io/g/devel/message/114235 Mute This Topic: https://groups.io/mt/103907626/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-