UefiCpuPkg/Cpuid.h: Remove duplicated struct definition for leaf 1FH
authorRay Ni <ray.ni@intel.com>
Thu, 4 Apr 2019 05:34:21 +0000 (13:34 +0800)
committerRay Ni <ray.ni@intel.com>
Thu, 4 Apr 2019 07:36:00 +0000 (15:36 +0800)
Per SDM CPUID.0BH and CPUID.1FH outputs the same format of data in
EAX/EBX/ECX/EDX except CPUID.1FH reports more level types such as
module, tile, die.

The patch removes the unnecessary duplicated structure definitions
for CPUID.1FH because when the structure definitions for CPUID.0BH
can be used for CPUID.1FH.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ray Ni <ray.ni@intel.com>
Reviewed-by: Eric Dong <eric.dong@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Cc: Zhiqiang Qin <zhiqiang.qin@intel.com>
UefiCpuPkg/Application/Cpuid/Cpuid.c
UefiCpuPkg/Include/Register/Cpuid.h

index 67cacf2..d229ac8 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   UEFI Application to display CPUID leaf information.\r
 \r
-  Copyright (c) 2016 - 2018, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2016 - 2019, Intel Corporation. All rights reserved.<BR>\r
   This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
   which accompanies this distribution.  The full text of the license may be found at\r
@@ -1394,26 +1394,26 @@ CpuidV2ExtendedTopologyEnumeration (
   VOID\r
   )\r
 {\r
-  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EAX  Eax;\r
-  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EBX  Ebx;\r
-  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_ECX  Ecx;\r
-  UINT32                                      Edx;\r
+  CPUID_EXTENDED_TOPOLOGY_EAX  Eax;\r
+  CPUID_EXTENDED_TOPOLOGY_EBX  Ebx;\r
+  CPUID_EXTENDED_TOPOLOGY_ECX  Ecx;\r
+  UINT32                       Edx;\r
 \r
-  if (CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION > gMaximumBasicFunction) {\r
+  if (CPUID_V2_EXTENDED_TOPOLOGY > gMaximumBasicFunction) {\r
     return;\r
   }\r
 \r
   AsmCpuidEx (\r
-    CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION,\r
-    CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_MAIN_LEAF,\r
+    CPUID_V2_EXTENDED_TOPOLOGY,\r
+    0,\r
     &Eax.Uint32, &Ebx.Uint32, &Ecx.Uint32, &Edx\r
     );\r
-  Print (L"CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION (Leaf %08x, Sub-Leaf %08x)\n", CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION, CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_MAIN_LEAF);\r
+  Print (L"CPUID_V2_EXTENDED_TOPOLOGY (Leaf %08x, Sub-Leaf %08x)\n", CPUID_V2_EXTENDED_TOPOLOGY, 0);\r
   Print (L"  EAX:%08x  EBX:%08x  ECX:%08x  EDX:%08x\n", Eax.Uint32, Ebx.Uint32, Ecx.Uint32, Edx);\r
 \r
-  PRINT_BIT_FIELD (Eax, BitsNum);\r
-  PRINT_BIT_FIELD (Ebx, ProcessorsNum);\r
-  PRINT_BIT_FIELD (Ecx, LevelNum);\r
+  PRINT_BIT_FIELD (Eax, ApicIdShift);\r
+  PRINT_BIT_FIELD (Ebx, LogicalProcessors);\r
+  PRINT_BIT_FIELD (Ecx, LevelNumber);\r
   PRINT_BIT_FIELD (Ecx, LevelType);\r
   PRINT_VALUE     (Edx, x2APICID);\r
 }\r
index fab199b..e0f4f96 100644 (file)
@@ -6,7 +6,7 @@
   If a register returned is a single 32-bit value, then a data structure is\r
   not provided for that register.\r
 \r
-  Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>\r
   This program and the accompanying materials are licensed and made available under\r
   the terms and conditions of the BSD License which accompanies this distribution.\r
   The full text of the license may be found at\r
@@ -3620,130 +3620,20 @@ typedef union {
   number of logical processors available to BIOS/OS/Applications may be different from the\r
   value of EBX[15:0], depending on software and platform hardware configurations.\r
 \r
-  @param   EAX  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION            (0x1F)\r
-  @param   ECX  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_MAIN_LEAF  (0x0)\r
-\r
-**/\r
-#define CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION                         0x1F\r
-\r
-/**\r
-  CPUID V2 Extended Topology Enumeration Leaf\r
-\r
-  @param   EAX  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION           (0x1F)\r
-  @param   ECX  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_MAIN_LEAF (0x00)\r
-\r
-  @retval  EAX  Returns V2 Extended Topology Enumeration Leaf described by\r
-                the type CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EAX.\r
-  @retval  EBX  Returns V2 Extended Topology Enumeration Leaf described by\r
-                the type CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EBX.\r
-  @retval  ECX  Returns V2 Extended Topology Enumeration Leaf described by\r
-                the type CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_ECX.\r
-  @retval  EDX  Returns x2APIC ID the current logical processor.\r
-\r
-  <b>Example usage</b>\r
-  @code\r
-  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EAX         Eax;\r
-  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EBX         Ebx;\r
-  CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_ECX         Ecx;\r
-  UINT32                                             Edx;\r
-\r
-  AsmCpuidEx (\r
-    CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION,\r
-    CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_MAIN_LEAF,\r
-    &Eax.Uint32, &Ebx.Uint32, &Ecx.Uint32, &Edx\r
-    );\r
-  @endcode\r
-**/\r
-#define CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_MAIN_LEAF               0x00\r
-\r
-/**\r
-  CPUID V2 Extended Topology Enumeration Leaf EAX for CPUID leafs.\r
-**/\r
-typedef union {\r
-  ///\r
-  /// Individual bit fields\r
-  ///\r
-  struct {\r
-    ///\r
-    /// [Bits 4:0] Number of bits to shift right on x2APIC ID to get a unique\r
-    /// topology ID of the next level type. All logical processors with the\r
-    /// same next level ID share current level.\r
-    ///\r
-    UINT32  BitsNum:5;\r
-    ///\r
-    /// [Bits 31:5] Reserved.\r
-    ///\r
-    UINT32  Reserved:27;\r
-  } Bits;\r
-  ///\r
-  /// All bit fields as a 32-bit value\r
-  ///\r
-  UINT32  Uint32;\r
-} CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EAX;\r
-\r
-/**\r
-  CPUID V2 Extended Topology Enumeration Leaf EBX for CPUID leafs.\r
-**/\r
-typedef union {\r
-  ///\r
-  /// Individual bit fields\r
-  ///\r
-  struct {\r
-    ///\r
-    /// [Bits 15:0] Number of logical processors at this level type. The number\r
-    /// reflects configuration as shipped by Intel.\r
-    ///\r
-    UINT32  ProcessorsNum:16;\r
-    ///\r
-    /// [Bits 31:5] Reserved.\r
-    ///\r
-    UINT32  Reserved:16;\r
-  } Bits;\r
-  ///\r
-  /// All bit fields as a 32-bit value\r
-  ///\r
-  UINT32  Uint32;\r
-} CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_EBX;\r
+  @param   EAX  CPUID_V2_EXTENDED_TOPOLOGY                        (0x1F)\r
+  @param   ECX  Level number\r
 \r
-/**\r
-  CPUID V2 Extended Topology Enumeration Leaf ECX for CPUID leafs.\r
 **/\r
-typedef union {\r
-  ///\r
-  /// Individual bit fields\r
-  ///\r
-  struct {\r
-    ///\r
-    /// [Bits 7:0] Level number. Same value in ECX input.\r
-    ///\r
-    UINT32  LevelNum:8;\r
-    ///\r
-    /// [Bits 7:0] Level type.\r
-    ///\r
-    UINT32  LevelType:8;\r
-\r
-    ///\r
-    /// [Bits 31:5] Reserved.\r
-    ///\r
-    UINT32  Reserved:16;\r
-  } Bits;\r
-  ///\r
-  /// All bit fields as a 32-bit value\r
-  ///\r
-  UINT32  Uint32;\r
-} CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_ECX;\r
+#define CPUID_V2_EXTENDED_TOPOLOGY                                     0x1F\r
 \r
 ///\r
-/// @{ Define value for CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_ECX.LevelType\r
+/// @{ Define value for CPUID_EXTENDED_TOPOLOGY_ECX.LevelType\r
 /// The value of the "level type" field is not related to level numbers in\r
 /// any way, higher "level type" values do not mean higher levels.\r
 ///\r
-#define   CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_LEVEL_TYPE_INVALID     0x00\r
-#define   CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_LEVEL_TYPE_SMT         0x01\r
-#define   CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_LEVEL_TYPE_CORE        0x02\r
-#define   CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_LEVEL_TYPE_MODULE      0x03\r
-#define   CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_LEVEL_TYPE_TILE        0x04\r
-#define   CPUID_V2_EXTENDED_TOPOLOGY_ENUMERATION_LEVEL_TYPE_DIE         0x05\r
+#define   CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_MODULE                  0x03\r
+#define   CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_TILE                    0x04\r
+#define   CPUID_V2_EXTENDED_TOPOLOGY_LEVEL_TYPE_DIE                     0x05\r
 ///\r
 /// @}\r
 ///\r