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 (#114197): https://edk2.groups.io/g/devel/message/114197 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] -=-=-=-=-=-=-=-=-=-=-=-