* [edk2-devel] [PATCH v1 0/2] AMD CPU extended topology
@ 2024-01-16 7:01 Abdul Lateef Attar via groups.io
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 1/2] MdePkg: Adds AMD Extended CPU topology CPUID Abdul Lateef Attar via groups.io
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology Abdul Lateef Attar via groups.io
0 siblings, 2 replies; 6+ messages in thread
From: Abdul Lateef Attar via groups.io @ 2024-01-16 7:01 UTC (permalink / raw)
To: devel
Cc: Abdul Lateef Attar, Michael D Kinney, Liming Gao, Zhiguang Liu,
Ray Ni, Rahul Kumar, Gerd Hoffmann
From: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
PR: https://github.com/tianocore/edk2/pull/5263
Implements AMD extended toplogy.
Adds CPUID macro
update APIC library to use new exteded cpuid.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
Abdul Lateef Attar (2):
MdePkg: Adds AMD Extended CPU topology CPUID
UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology
MdePkg/Include/Register/Amd/Cpuid.h | 23 +++-
.../Library/BaseXApicLib/BaseXApicLib.c | 122 +++++++++++++++++-
.../BaseXApicX2ApicLib/BaseXApicX2ApicLib.c | 122 +++++++++++++++++-
3 files changed, 264 insertions(+), 3 deletions(-)
--
2.34.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113867): https://edk2.groups.io/g/devel/message/113867
Mute This Topic: https://groups.io/mt/103757655/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* [edk2-devel] [PATCH v1 1/2] MdePkg: Adds AMD Extended CPU topology CPUID
2024-01-16 7:01 [edk2-devel] [PATCH v1 0/2] AMD CPU extended topology Abdul Lateef Attar via groups.io
@ 2024-01-16 7:01 ` Abdul Lateef Attar via groups.io
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology Abdul Lateef Attar via groups.io
1 sibling, 0 replies; 6+ messages in thread
From: Abdul Lateef Attar via groups.io @ 2024-01-16 7:01 UTC (permalink / raw)
To: devel; +Cc: Abdul Lateef Attar, Michael D Kinney, Liming Gao, Zhiguang Liu
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="y", Size: 366 bytes --]
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113868): https://edk2.groups.io/g/devel/message/113868
Mute This Topic: https://groups.io/mt/103757656/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology
2024-01-16 7:01 [edk2-devel] [PATCH v1 0/2] AMD CPU extended topology Abdul Lateef Attar via groups.io
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 1/2] MdePkg: Adds AMD Extended CPU topology CPUID Abdul Lateef Attar via groups.io
@ 2024-01-16 7:01 ` Abdul Lateef Attar via groups.io
2024-01-16 9:41 ` Gerd Hoffmann
2024-01-17 2:15 ` Ni, Ray
1 sibling, 2 replies; 6+ messages in thread
From: Abdul Lateef Attar via groups.io @ 2024-01-16 7:01 UTC (permalink / raw)
To: devel; +Cc: Abdul Lateef Attar, Ray Ni, Rahul Kumar, Gerd Hoffmann
From: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
This patch adds support for AMD's new extended topology.
If processor supports CPUID 80000026 leaf then obtain
the topology information using new method.
Algorithm:
if CPUID is AMD:
then
check for AMD's extended cpu tology leaf.
if yes
then extract cpu tology based on
AMD programmer manual's instruction.
else
then fallback to existing topology function.
endif
endif
Cc: Ray Ni <ray.ni@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
---
.../Library/BaseXApicLib/BaseXApicLib.c | 122 +++++++++++++++++-
.../BaseXApicX2ApicLib/BaseXApicX2ApicLib.c | 122 +++++++++++++++++-
2 files changed, 242 insertions(+), 2 deletions(-)
diff --git a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c b/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c
index efb9d71ca1..5e941d0dc8 100644
--- a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c
+++ b/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c
@@ -4,7 +4,7 @@
This local APIC library instance supports xAPIC mode only.
Copyright (c) 2010 - 2023, Intel Corporation. All rights reserved.<BR>
- Copyright (c) 2017 - 2020, AMD Inc. All rights reserved.<BR>
+ Copyright (c) 2017 - 2024, AMD Inc. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent
@@ -1157,6 +1157,121 @@ GetProcessorLocationByApicId (
}
}
+/**
+ Get Package ID/Die ID/Module ID/Core ID/Thread ID of a AMD processor family.
+
+ The algorithm assumes the target system has symmetry across physical
+ package boundaries with respect to the number of threads per core, number of
+ cores per module, number of modules per die, number
+ of dies per package.
+
+ @param[in] InitialApicId Initial APIC ID of the target logical processor.
+ @param[out] Package Returns the processor package ID.
+ @param[out] Die Returns the processor die ID.
+ @param[out] Tile Returns zero.
+ @param[out] Module Returns the processor module ID.
+ @param[out] Core Returns the processor core ID.
+ @param[out] Thread Returns the processor thread ID.
+**/
+VOID
+EFIAPI
+AmdGetProcessorLocation2ByApicId (
+ IN UINT32 InitialApicId,
+ OUT UINT32 *Package OPTIONAL,
+ OUT UINT32 *Die OPTIONAL,
+ OUT UINT32 *Tile OPTIONAL,
+ OUT UINT32 *Module OPTIONAL,
+ OUT UINT32 *Core OPTIONAL,
+ OUT UINT32 *Thread OPTIONAL
+ )
+{
+ CPUID_EXTENDED_TOPOLOGY_EAX ExtendedTopologyEax;
+ CPUID_EXTENDED_TOPOLOGY_EBX ExtendedTopologyEbx;
+ CPUID_EXTENDED_TOPOLOGY_ECX ExtendedTopologyEcx;
+ UINT32 MaxExtendedCpuIdIndex;
+ UINT32 SubIndex;
+ UINT32 PreviousLevel;
+ UINT32 Data;
+
+ if (Die != NULL) {
+ *Die = 0;
+ }
+
+ if (Tile != NULL) {
+ *Tile = 0;
+ }
+
+ if (Module != NULL) {
+ *Module = 0;
+ }
+
+ /// Check if extended toplogy supported
+ AsmCpuid (CPUID_EXTENDED_FUNCTION, &MaxExtendedCpuIdIndex, NULL, NULL, NULL);
+ if (MaxExtendedCpuIdIndex < AMD_CPUID_EXTENDED_TOPOLOGY) {
+ GetProcessorLocationByApicId (InitialApicId, Package, Core, Thread);
+ return;
+ }
+
+ PreviousLevel = 0;
+ SubIndex = 0;
+ do {
+ AsmCpuidEx (
+ AMD_CPUID_EXTENDED_TOPOLOGY,
+ SubIndex,
+ &ExtendedTopologyEax.Uint32,
+ &ExtendedTopologyEbx.Uint32,
+ &ExtendedTopologyEcx.Uint32,
+ NULL
+ );
+
+ if (ExtendedTopologyEbx.Bits.LogicalProcessors == CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID) {
+ break;
+ }
+
+ Data = InitialApicId >> PreviousLevel;
+ Data &= (1 << (ExtendedTopologyEax.Bits.ApicIdShift - PreviousLevel)) - 1;
+
+ switch (ExtendedTopologyEcx.Bits.LevelType) {
+ case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_SMT:
+ if (Thread != NULL) {
+ *Thread = Data;
+ }
+
+ break;
+ case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_CORE:
+ if (Core != NULL) {
+ *Core = Data;
+ }
+
+ break;
+ case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_MODULE:
+ if (Module != NULL) {
+ *Module = Data;
+ }
+
+ break;
+ case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_TILE:
+ if (Die != NULL) {
+ *Die = Data;
+ }
+
+ break;
+ default:
+ break;
+ }
+
+ SubIndex++;
+ PreviousLevel = ExtendedTopologyEax.Bits.ApicIdShift;
+ } while (ExtendedTopologyEbx.Bits.LogicalProcessors != CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID);
+
+ /// Package value
+ if ((PreviousLevel != 0) && (Package != NULL)) {
+ *Package = InitialApicId >> PreviousLevel;
+ }
+
+ return;
+}
+
/**
Get Package ID/Die ID/Tile ID/Module ID/Core ID/Thread ID of a processor.
@@ -1194,6 +1309,11 @@ GetProcessorLocation2ByApicId (
UINT32 Bits[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
UINT32 *Location[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
+ if (StandardSignatureIsAuthenticAMD ()) {
+ AmdGetProcessorLocation2ByApicId (InitialApicId, Package, Die, Tile, Module, Core, Thread);
+ return;
+ }
+
for (LevelType = 0; LevelType < ARRAY_SIZE (Bits); LevelType++) {
Bits[LevelType] = 0;
}
diff --git a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c b/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c
index c0a8475833..a7563f6596 100644
--- a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c
+++ b/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c
@@ -5,7 +5,7 @@
which have xAPIC and x2APIC modes.
Copyright (c) 2010 - 2023, Intel Corporation. All rights reserved.<BR>
- Copyright (c) 2017 - 2020, AMD Inc. All rights reserved.<BR>
+ Copyright (c) 2017 - 2024, AMD Inc. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent
@@ -1396,6 +1396,121 @@ GetProcessorLocationByApicId (
}
}
+/**
+ Get Package ID/Die ID/Module ID/Core ID/Thread ID of a AMD processor family.
+
+ The algorithm assumes the target system has symmetry across physical
+ package boundaries with respect to the number of threads per core, number of
+ cores per module, number of modules per die, number
+ of dies per package.
+
+ @param[in] InitialApicId Initial APIC ID of the target logical processor.
+ @param[out] Package Returns the processor package ID.
+ @param[out] Die Returns the processor die ID.
+ @param[out] Tile Returns zero.
+ @param[out] Module Returns the processor module ID.
+ @param[out] Core Returns the processor core ID.
+ @param[out] Thread Returns the processor thread ID.
+**/
+VOID
+EFIAPI
+AmdGetProcessorLocation2ByApicId (
+ IN UINT32 InitialApicId,
+ OUT UINT32 *Package OPTIONAL,
+ OUT UINT32 *Die OPTIONAL,
+ OUT UINT32 *Tile OPTIONAL,
+ OUT UINT32 *Module OPTIONAL,
+ OUT UINT32 *Core OPTIONAL,
+ OUT UINT32 *Thread OPTIONAL
+ )
+{
+ CPUID_EXTENDED_TOPOLOGY_EAX ExtendedTopologyEax;
+ CPUID_EXTENDED_TOPOLOGY_EBX ExtendedTopologyEbx;
+ CPUID_EXTENDED_TOPOLOGY_ECX ExtendedTopologyEcx;
+ UINT32 MaxExtendedCpuIdIndex;
+ UINT32 SubIndex;
+ UINT32 PreviousLevel;
+ UINT32 Data;
+
+ if (Die != NULL) {
+ *Die = 0;
+ }
+
+ if (Tile != NULL) {
+ *Tile = 0;
+ }
+
+ if (Module != NULL) {
+ *Module = 0;
+ }
+
+ /// Check if extended toplogy supported
+ AsmCpuid (CPUID_EXTENDED_FUNCTION, &MaxExtendedCpuIdIndex, NULL, NULL, NULL);
+ if (MaxExtendedCpuIdIndex < AMD_CPUID_EXTENDED_TOPOLOGY) {
+ GetProcessorLocationByApicId (InitialApicId, Package, Core, Thread);
+ return;
+ }
+
+ PreviousLevel = 0;
+ SubIndex = 0;
+ do {
+ AsmCpuidEx (
+ AMD_CPUID_EXTENDED_TOPOLOGY,
+ SubIndex,
+ &ExtendedTopologyEax.Uint32,
+ &ExtendedTopologyEbx.Uint32,
+ &ExtendedTopologyEcx.Uint32,
+ NULL
+ );
+
+ if (ExtendedTopologyEbx.Bits.LogicalProcessors == CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID) {
+ break;
+ }
+
+ Data = InitialApicId >> PreviousLevel;
+ Data &= (1 << (ExtendedTopologyEax.Bits.ApicIdShift - PreviousLevel)) - 1;
+
+ switch (ExtendedTopologyEcx.Bits.LevelType) {
+ case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_SMT:
+ if (Thread != NULL) {
+ *Thread = Data;
+ }
+
+ break;
+ case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_CORE:
+ if (Core != NULL) {
+ *Core = Data;
+ }
+
+ break;
+ case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_MODULE:
+ if (Module != NULL) {
+ *Module = Data;
+ }
+
+ break;
+ case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_TILE:
+ if (Die != NULL) {
+ *Die = Data;
+ }
+
+ break;
+ default:
+ break;
+ }
+
+ SubIndex++;
+ PreviousLevel = ExtendedTopologyEax.Bits.ApicIdShift;
+ } while (ExtendedTopologyEbx.Bits.LogicalProcessors != CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID);
+
+ /// Package value
+ if ((PreviousLevel != 0) && (Package != NULL)) {
+ *Package = InitialApicId >> PreviousLevel;
+ }
+
+ return;
+}
+
/**
Get Package ID/Die ID/Tile ID/Module ID/Core ID/Thread ID of a processor.
@@ -1433,6 +1548,11 @@ GetProcessorLocation2ByApicId (
UINT32 Bits[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
UINT32 *Location[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
+ if (StandardSignatureIsAuthenticAMD ()) {
+ AmdGetProcessorLocation2ByApicId (InitialApicId, Package, Die, Tile, Module, Core, Thread);
+ return;
+ }
+
for (LevelType = 0; LevelType < ARRAY_SIZE (Bits); LevelType++) {
Bits[LevelType] = 0;
}
--
2.34.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113869): https://edk2.groups.io/g/devel/message/113869
Mute This Topic: https://groups.io/mt/103757657/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology Abdul Lateef Attar via groups.io
@ 2024-01-16 9:41 ` Gerd Hoffmann
2024-01-16 13:14 ` Abdul Lateef Attar via groups.io
2024-01-17 2:15 ` Ni, Ray
1 sibling, 1 reply; 6+ messages in thread
From: Gerd Hoffmann @ 2024-01-16 9:41 UTC (permalink / raw)
To: Abdul Lateef Attar; +Cc: devel, Abdul Lateef Attar, Ray Ni, Rahul Kumar
On Tue, Jan 16, 2024 at 12:31:21PM +0530, Abdul Lateef Attar wrote:
> From: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
>
> This patch adds support for AMD's new extended topology.
> If processor supports CPUID 80000026 leaf then obtain
> the topology information using new method.
> + /// Check if extended toplogy supported
> + AsmCpuid (CPUID_EXTENDED_FUNCTION, &MaxExtendedCpuIdIndex, NULL, NULL, NULL);
> + if (MaxExtendedCpuIdIndex < AMD_CPUID_EXTENDED_TOPOLOGY) {
> + GetProcessorLocationByApicId (InitialApicId, Package, Core, Thread);
> + return;
> + }
Sure this is correct?
On Intel processors checking MaxExtendedCpuIdIndex alone is not enough,
see commit 170d4ce8e90a ("UefiCpuPkg/BaseXApicX2ApicLib: fix
CPUID_V2_EXTENDED_TOPOLOGY detection")
Especially in virtual machines it can happen that the vCPU supports
extended cpuid leaf N but does not support N-1, so checking
MaxExtendedCpuIdIndex alone looks rather fragile to me.
take care,
Gerd
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113882): https://edk2.groups.io/g/devel/message/113882
Mute This Topic: https://groups.io/mt/103757657/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology
2024-01-16 9:41 ` Gerd Hoffmann
@ 2024-01-16 13:14 ` Abdul Lateef Attar via groups.io
0 siblings, 0 replies; 6+ messages in thread
From: Abdul Lateef Attar via groups.io @ 2024-01-16 13:14 UTC (permalink / raw)
To: Gerd Hoffmann, Abdul Lateef Attar; +Cc: devel, Ray Ni, Rahul Kumar
Thanks Gerd for providing the inputs.
I'll enhance the logic accordingly and submit the V2 patch.
Thanks
AbduL
On 16-01-2024 15:11, Gerd Hoffmann wrote:
> Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
>
>
> On Tue, Jan 16, 2024 at 12:31:21PM +0530, Abdul Lateef Attar wrote:
>> From: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
>>
>> This patch adds support for AMD's new extended topology.
>> If processor supports CPUID 80000026 leaf then obtain
>> the topology information using new method.
>> + /// Check if extended toplogy supported
>> + AsmCpuid (CPUID_EXTENDED_FUNCTION, &MaxExtendedCpuIdIndex, NULL, NULL, NULL);
>> + if (MaxExtendedCpuIdIndex < AMD_CPUID_EXTENDED_TOPOLOGY) {
>> + GetProcessorLocationByApicId (InitialApicId, Package, Core, Thread);
>> + return;
>> + }
> Sure this is correct?
>
> On Intel processors checking MaxExtendedCpuIdIndex alone is not enough,
> see commit 170d4ce8e90a ("UefiCpuPkg/BaseXApicX2ApicLib: fix
> CPUID_V2_EXTENDED_TOPOLOGY detection")
>
> Especially in virtual machines it can happen that the vCPU supports
> extended cpuid leaf N but does not support N-1, so checking
> MaxExtendedCpuIdIndex alone looks rather fragile to me.
>
> take care,
> Gerd
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113897): https://edk2.groups.io/g/devel/message/113897
Mute This Topic: https://groups.io/mt/103757657/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology Abdul Lateef Attar via groups.io
2024-01-16 9:41 ` Gerd Hoffmann
@ 2024-01-17 2:15 ` Ni, Ray
1 sibling, 0 replies; 6+ messages in thread
From: Ni, Ray @ 2024-01-17 2:15 UTC (permalink / raw)
To: Abdul Lateef Attar, devel@edk2.groups.io
Cc: Abdul Lateef Attar, Kumar, Rahul R, Gerd Hoffmann
Can you remove "EFIAPI" from the two Amd* local functions?
The two are *local* functions called from the accordingly public LIB APIs.
Thanks,
Ray
> -----Original Message-----
> From: Abdul Lateef Attar <abdattar@amd.com>
> Sent: Tuesday, January 16, 2024 3:01 PM
> To: devel@edk2.groups.io
> Cc: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>; Ni, Ray
> <ray.ni@intel.com>; Kumar, Rahul R <rahul.r.kumar@intel.com>; Gerd
> Hoffmann <kraxel@redhat.com>
> Subject: [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD
> extended cpu topology
>
> From: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
>
> This patch adds support for AMD's new extended topology.
> If processor supports CPUID 80000026 leaf then obtain
> the topology information using new method.
>
> Algorithm:
> if CPUID is AMD:
> then
> check for AMD's extended cpu tology leaf.
> if yes
> then extract cpu tology based on
> AMD programmer manual's instruction.
> else
> then fallback to existing topology function.
> endif
> endif
>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Rahul Kumar <rahul1.kumar@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
> ---
> .../Library/BaseXApicLib/BaseXApicLib.c | 122
> +++++++++++++++++-
> .../BaseXApicX2ApicLib/BaseXApicX2ApicLib.c | 122
> +++++++++++++++++-
> 2 files changed, 242 insertions(+), 2 deletions(-)
>
> diff --git a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c
> b/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c
> index efb9d71ca1..5e941d0dc8 100644
> --- a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c
> +++ b/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c
> @@ -4,7 +4,7 @@
> This local APIC library instance supports xAPIC mode only.
>
> Copyright (c) 2010 - 2023, Intel Corporation. All rights reserved.<BR>
> - Copyright (c) 2017 - 2020, AMD Inc. All rights reserved.<BR>
> + Copyright (c) 2017 - 2024, AMD Inc. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -1157,6 +1157,121 @@ GetProcessorLocationByApicId (
> }
> }
>
> +/**
> + Get Package ID/Die ID/Module ID/Core ID/Thread ID of a AMD processor
> family.
> +
> + The algorithm assumes the target system has symmetry across physical
> + package boundaries with respect to the number of threads per core,
> number of
> + cores per module, number of modules per die, number
> + of dies per package.
> +
> + @param[in] InitialApicId Initial APIC ID of the target logical processor.
> + @param[out] Package Returns the processor package ID.
> + @param[out] Die Returns the processor die ID.
> + @param[out] Tile Returns zero.
> + @param[out] Module Returns the processor module ID.
> + @param[out] Core Returns the processor core ID.
> + @param[out] Thread Returns the processor thread ID.
> +**/
> +VOID
> +EFIAPI
> +AmdGetProcessorLocation2ByApicId (
> + IN UINT32 InitialApicId,
> + OUT UINT32 *Package OPTIONAL,
> + OUT UINT32 *Die OPTIONAL,
> + OUT UINT32 *Tile OPTIONAL,
> + OUT UINT32 *Module OPTIONAL,
> + OUT UINT32 *Core OPTIONAL,
> + OUT UINT32 *Thread OPTIONAL
> + )
> +{
> + CPUID_EXTENDED_TOPOLOGY_EAX ExtendedTopologyEax;
> + CPUID_EXTENDED_TOPOLOGY_EBX ExtendedTopologyEbx;
> + CPUID_EXTENDED_TOPOLOGY_ECX ExtendedTopologyEcx;
> + UINT32 MaxExtendedCpuIdIndex;
> + UINT32 SubIndex;
> + UINT32 PreviousLevel;
> + UINT32 Data;
> +
> + if (Die != NULL) {
> + *Die = 0;
> + }
> +
> + if (Tile != NULL) {
> + *Tile = 0;
> + }
> +
> + if (Module != NULL) {
> + *Module = 0;
> + }
> +
> + /// Check if extended toplogy supported
> + AsmCpuid (CPUID_EXTENDED_FUNCTION, &MaxExtendedCpuIdIndex,
> NULL, NULL, NULL);
> + if (MaxExtendedCpuIdIndex < AMD_CPUID_EXTENDED_TOPOLOGY) {
> + GetProcessorLocationByApicId (InitialApicId, Package, Core, Thread);
> + return;
> + }
> +
> + PreviousLevel = 0;
> + SubIndex = 0;
> + do {
> + AsmCpuidEx (
> + AMD_CPUID_EXTENDED_TOPOLOGY,
> + SubIndex,
> + &ExtendedTopologyEax.Uint32,
> + &ExtendedTopologyEbx.Uint32,
> + &ExtendedTopologyEcx.Uint32,
> + NULL
> + );
> +
> + if (ExtendedTopologyEbx.Bits.LogicalProcessors ==
> CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID) {
> + break;
> + }
> +
> + Data = InitialApicId >> PreviousLevel;
> + Data &= (1 << (ExtendedTopologyEax.Bits.ApicIdShift - PreviousLevel)) -
> 1;
> +
> + switch (ExtendedTopologyEcx.Bits.LevelType) {
> + case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_SMT:
> + if (Thread != NULL) {
> + *Thread = Data;
> + }
> +
> + break;
> + case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_CORE:
> + if (Core != NULL) {
> + *Core = Data;
> + }
> +
> + break;
> + case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_MODULE:
> + if (Module != NULL) {
> + *Module = Data;
> + }
> +
> + break;
> + case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_TILE:
> + if (Die != NULL) {
> + *Die = Data;
> + }
> +
> + break;
> + default:
> + break;
> + }
> +
> + SubIndex++;
> + PreviousLevel = ExtendedTopologyEax.Bits.ApicIdShift;
> + } while (ExtendedTopologyEbx.Bits.LogicalProcessors !=
> CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID);
> +
> + /// Package value
> + if ((PreviousLevel != 0) && (Package != NULL)) {
> + *Package = InitialApicId >> PreviousLevel;
> + }
> +
> + return;
> +}
> +
> /**
> Get Package ID/Die ID/Tile ID/Module ID/Core ID/Thread ID of a
> processor.
>
> @@ -1194,6 +1309,11 @@ GetProcessorLocation2ByApicId (
> UINT32
> Bits[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
> UINT32
> *Location[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
>
> + if (StandardSignatureIsAuthenticAMD ()) {
> + AmdGetProcessorLocation2ByApicId (InitialApicId, Package, Die, Tile,
> Module, Core, Thread);
> + return;
> + }
> +
> for (LevelType = 0; LevelType < ARRAY_SIZE (Bits); LevelType++) {
> Bits[LevelType] = 0;
> }
> diff --git a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c
> b/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c
> index c0a8475833..a7563f6596 100644
> --- a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c
> +++ b/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c
> @@ -5,7 +5,7 @@
> which have xAPIC and x2APIC modes.
>
> Copyright (c) 2010 - 2023, Intel Corporation. All rights reserved.<BR>
> - Copyright (c) 2017 - 2020, AMD Inc. All rights reserved.<BR>
> + Copyright (c) 2017 - 2024, AMD Inc. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -1396,6 +1396,121 @@ GetProcessorLocationByApicId (
> }
> }
>
> +/**
> + Get Package ID/Die ID/Module ID/Core ID/Thread ID of a AMD processor
> family.
> +
> + The algorithm assumes the target system has symmetry across physical
> + package boundaries with respect to the number of threads per core,
> number of
> + cores per module, number of modules per die, number
> + of dies per package.
> +
> + @param[in] InitialApicId Initial APIC ID of the target logical processor.
> + @param[out] Package Returns the processor package ID.
> + @param[out] Die Returns the processor die ID.
> + @param[out] Tile Returns zero.
> + @param[out] Module Returns the processor module ID.
> + @param[out] Core Returns the processor core ID.
> + @param[out] Thread Returns the processor thread ID.
> +**/
> +VOID
> +EFIAPI
> +AmdGetProcessorLocation2ByApicId (
> + IN UINT32 InitialApicId,
> + OUT UINT32 *Package OPTIONAL,
> + OUT UINT32 *Die OPTIONAL,
> + OUT UINT32 *Tile OPTIONAL,
> + OUT UINT32 *Module OPTIONAL,
> + OUT UINT32 *Core OPTIONAL,
> + OUT UINT32 *Thread OPTIONAL
> + )
> +{
> + CPUID_EXTENDED_TOPOLOGY_EAX ExtendedTopologyEax;
> + CPUID_EXTENDED_TOPOLOGY_EBX ExtendedTopologyEbx;
> + CPUID_EXTENDED_TOPOLOGY_ECX ExtendedTopologyEcx;
> + UINT32 MaxExtendedCpuIdIndex;
> + UINT32 SubIndex;
> + UINT32 PreviousLevel;
> + UINT32 Data;
> +
> + if (Die != NULL) {
> + *Die = 0;
> + }
> +
> + if (Tile != NULL) {
> + *Tile = 0;
> + }
> +
> + if (Module != NULL) {
> + *Module = 0;
> + }
> +
> + /// Check if extended toplogy supported
> + AsmCpuid (CPUID_EXTENDED_FUNCTION, &MaxExtendedCpuIdIndex,
> NULL, NULL, NULL);
> + if (MaxExtendedCpuIdIndex < AMD_CPUID_EXTENDED_TOPOLOGY) {
> + GetProcessorLocationByApicId (InitialApicId, Package, Core, Thread);
> + return;
> + }
> +
> + PreviousLevel = 0;
> + SubIndex = 0;
> + do {
> + AsmCpuidEx (
> + AMD_CPUID_EXTENDED_TOPOLOGY,
> + SubIndex,
> + &ExtendedTopologyEax.Uint32,
> + &ExtendedTopologyEbx.Uint32,
> + &ExtendedTopologyEcx.Uint32,
> + NULL
> + );
> +
> + if (ExtendedTopologyEbx.Bits.LogicalProcessors ==
> CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID) {
> + break;
> + }
> +
> + Data = InitialApicId >> PreviousLevel;
> + Data &= (1 << (ExtendedTopologyEax.Bits.ApicIdShift - PreviousLevel)) -
> 1;
> +
> + switch (ExtendedTopologyEcx.Bits.LevelType) {
> + case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_SMT:
> + if (Thread != NULL) {
> + *Thread = Data;
> + }
> +
> + break;
> + case CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_CORE:
> + if (Core != NULL) {
> + *Core = Data;
> + }
> +
> + break;
> + case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_MODULE:
> + if (Module != NULL) {
> + *Module = Data;
> + }
> +
> + break;
> + case CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_TILE:
> + if (Die != NULL) {
> + *Die = Data;
> + }
> +
> + break;
> + default:
> + break;
> + }
> +
> + SubIndex++;
> + PreviousLevel = ExtendedTopologyEax.Bits.ApicIdShift;
> + } while (ExtendedTopologyEbx.Bits.LogicalProcessors !=
> CPUID_EXTENDED_TOPOLOGY_LEVEL_TYPE_INVALID);
> +
> + /// Package value
> + if ((PreviousLevel != 0) && (Package != NULL)) {
> + *Package = InitialApicId >> PreviousLevel;
> + }
> +
> + return;
> +}
> +
> /**
> Get Package ID/Die ID/Tile ID/Module ID/Core ID/Thread ID of a
> processor.
>
> @@ -1433,6 +1548,11 @@ GetProcessorLocation2ByApicId (
> UINT32
> Bits[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
> UINT32
> *Location[CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE + 2];
>
> + if (StandardSignatureIsAuthenticAMD ()) {
> + AmdGetProcessorLocation2ByApicId (InitialApicId, Package, Die, Tile,
> Module, Core, Thread);
> + return;
> + }
> +
> for (LevelType = 0; LevelType < ARRAY_SIZE (Bits); LevelType++) {
> Bits[LevelType] = 0;
> }
> --
> 2.34.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113925): https://edk2.groups.io/g/devel/message/113925
Mute This Topic: https://groups.io/mt/103757657/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/12367111/7686176/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-01-17 2:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-16 7:01 [edk2-devel] [PATCH v1 0/2] AMD CPU extended topology Abdul Lateef Attar via groups.io
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 1/2] MdePkg: Adds AMD Extended CPU topology CPUID Abdul Lateef Attar via groups.io
2024-01-16 7:01 ` [edk2-devel] [PATCH v1 2/2] UefiCpuPkg/BaseXApicX2ApicLib: Implements AMD extended cpu topology Abdul Lateef Attar via groups.io
2024-01-16 9:41 ` Gerd Hoffmann
2024-01-16 13:14 ` Abdul Lateef Attar via groups.io
2024-01-17 2:15 ` Ni, Ray
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox