I mean there is already a BaseCpuLib in MdePkg. Is it possible to add it to there? If no usage outside of UefiCpuPkg, I'm ok to move this function to UefiCpuLib. Thanks, Eric From: Ni, Ray Sent: Thursday, June 11, 2020 12:56 PM To: Dong, Eric ; devel@edk2.groups.io; garrett.kirkendall@amd.com Cc: Laszlo Ersek Subject: RE: UefiCpuPkg: Discuss: Move StandardSignatureIsAuthenticAMD function to BaseUefiCpuLib UefiCpuLib is in UefiCpuPkg. All consumers are in UefiCpuPkg. I think we can keep its location unchanged. From: Dong, Eric > Sent: Thursday, June 11, 2020 9:29 AM To: Ni, Ray >; devel@edk2.groups.io; garrett.kirkendall@amd.com Cc: Laszlo Ersek > Subject: RE: UefiCpuPkg: Discuss: Move StandardSignatureIsAuthenticAMD function to BaseUefiCpuLib This proposal is ok to me. I'm not sure whether it can be added to MdePkg, just like add to CpuLib. I found all the definitions and functions used in StandardSignatureIsAuthenticAMD function defined in MdePkg. If it can be added to MdePkg, all code base can use this function, if it been added to UefiCpuPkg, other core packages may can't use it. Thanks, Eric From: Ni, Ray > Sent: Thursday, June 11, 2020 9:12 AM To: devel@edk2.groups.io; garrett.kirkendall@amd.com Cc: Dong, Eric >; Laszlo Ersek > Subject: RE: UefiCpuPkg: Discuss: Move StandardSignatureIsAuthenticAMD function to BaseUefiCpuLib Since all platform DSCs have already listed UefiCpuLib.inf so this change doesn't impact platform DSC. I don't see an issue with this change. Eric, What's your opinion? Thanks, Ray > -----Original Message----- > From: devel@edk2.groups.io > On Behalf Of Kirkendall, Garrett > Sent: Wednesday, June 10, 2020 10:18 PM > To: devel@edk2.groups.io > Cc: Dong, Eric >; Ni, Ray >; Laszlo Ersek > > Subject: [edk2-devel] UefiCpuPkg: Discuss: Move StandardSignatureIsAuthenticAMD function to BaseUefiCpuLib > > [AMD Public Use] > > There are currently three instances of StandardSignatureIsAuthenticAMD Function in the below locations. I need to > propose another change that will require the same function in PiSmmCpuDxeSmm. I would like to see this function move > to a single "LibraryClass" location. I think no one knew where to propose the function, so we ended up with three > instances so far. The best place I can find would be UefiCpuPkg/BaseUefiCpuLib/. Thoughts? Is there a better function > name such that a similar Intel identifier function could be created when needed in the future? > > I'm thinking this will be 3 patches > 1. Add StandardSignatureIsAuthenticAMD to BaseUefiCpuLib > 2. Move current instances to Library function > 3. Propose code change in PiSmmCpuDxeSmm > > UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c: StandardSignatureIsAuthenticAMD ( > UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c: StandardSignatureIsAuthenticAMD ( > UefiCpuPkg/Library/MpInitLib/MpLib.c: StandardSignatureIsAuthenticAMD ( > > GARRETT KIRKENDALL > SMTS Firmware Engineer | CTE > 7171 Southwest Parkway, Austin, TX 78735 USA > AMD facebook | amd.com > >