ShellPkg/SmbiosView: Update SmbiosView for SMBIOS3.2.0

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1099

Update SmbiosView to parse the new definitions which
are introduced in SMBIOS3.2.0

V2:
1. Add structure length check before dump the fileds in
Type 9 and Type 17 in case some fileds are not organized
and reported by drivers.
2. Dump the InterfaceTypeSpecificData in Type 42.

V3:
1. Correct the structure length in Type17.
2. Remove the redundant check "if (PeerGroupCount > 0)" in Type 9.
3. Use the Uint16 filed instead of Bits field in union
MEMORY_DEVICE_OPERATING_MODE_CAPABILITY to dump data.

Cc: Jaben Carsey <jaben.carsey@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>

diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
index 93c6094df1fffec376986eeb770971fe57d5f7ba..af8551fe2e733f0874c68d443d35db5de0e04586 100644 (file)
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
@@ -543,22 +543,45 @@ SmbiosPrintStructure (
   // System Slots (Type 9)\r
   //\r
   case 9:\r
-    PRINT_PENDING_STRING (Struct, Type9, SlotDesignation);\r
-    DisplaySystemSlotType (Struct->Type9->SlotType, Option);\r
-    DisplaySystemSlotDataBusWidth (Struct->Type9->SlotDataBusWidth, Option);\r
-    DisplaySystemSlotCurrentUsage (Struct->Type9->CurrentUsage, Option);\r
-    DisplaySystemSlotLength (Struct->Type9->SlotLength, Option);\r
-    DisplaySystemSlotId (\r
-      Struct->Type9->SlotID,\r
-      Struct->Type9->SlotType,\r
-      Option\r
-     );\r
-    DisplaySlotCharacteristics1 (*(UINT8 *) &(Struct->Type9->SlotCharacteristics1), Option);\r
-    DisplaySlotCharacteristics2 (*(UINT8 *) &(Struct->Type9->SlotCharacteristics2), Option);\r
-    if (AE_SMBIOS_VERSION (0x2, 0x6) && (Struct->Hdr->Length > 0xD)) {\r
-      PRINT_STRUCT_VALUE_H (Struct, Type9, SegmentGroupNum);\r
-      PRINT_STRUCT_VALUE_H (Struct, Type9, BusNum);\r
-      PRINT_STRUCT_VALUE_H (Struct, Type9, DevFuncNum);\r
+    {\r
+      MISC_SLOT_PEER_GROUP  *PeerGroupPtr;\r
+      UINT8                 PeerGroupCount;\r
+\r
+      PRINT_PENDING_STRING (Struct, Type9, SlotDesignation);\r
+      DisplaySystemSlotType (Struct->Type9->SlotType, Option);\r
+      DisplaySystemSlotDataBusWidth (Struct->Type9->SlotDataBusWidth, Option);\r
+      DisplaySystemSlotCurrentUsage (Struct->Type9->CurrentUsage, Option);\r
+      DisplaySystemSlotLength (Struct->Type9->SlotLength, Option);\r
+      DisplaySystemSlotId (\r
+        Struct->Type9->SlotID,\r
+        Struct->Type9->SlotType,\r
+        Option\r
+       );\r
+      DisplaySlotCharacteristics1 (*(UINT8 *) &(Struct->Type9->SlotCharacteristics1), Option);\r
+      DisplaySlotCharacteristics2 (*(UINT8 *) &(Struct->Type9->SlotCharacteristics2), Option);\r
+      if (AE_SMBIOS_VERSION (0x2, 0x6) && (Struct->Hdr->Length > 0xD)) {\r
+        PRINT_STRUCT_VALUE_H (Struct, Type9, SegmentGroupNum);\r
+        PRINT_STRUCT_VALUE_H (Struct, Type9, BusNum);\r
+        PRINT_STRUCT_VALUE_H (Struct, Type9, DevFuncNum);\r
+      }\r
+      if (AE_SMBIOS_VERSION (0x3, 0x2)) {\r
+        if (Struct->Hdr->Length > 0x11) {\r
+          PRINT_STRUCT_VALUE (Struct, Type9, DataBusWidth);\r
+        }\r
+        if (Struct->Hdr->Length > 0x12) {\r
+          PRINT_STRUCT_VALUE (Struct, Type9, PeerGroupingCount);\r
+\r
+          PeerGroupCount = Struct->Type9->PeerGroupingCount;\r
+          PeerGroupPtr   = Struct->Type9->PeerGroups;\r
+          for (Index = 0; Index < PeerGroupCount; Index++) {\r
+            ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SLOT_PEER_GROUPS), gShellDebug1HiiHandle, Index + 1);\r
+            ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SEGMENT_GROUP_NUM), gShellDebug1HiiHandle, PeerGroupPtr[Index].SegmentGroupNum);\r
+            ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BUS_NUM), gShellDebug1HiiHandle, PeerGroupPtr[Index].BusNum);\r
+            ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DEV_FUNC_NUM), gShellDebug1HiiHandle, PeerGroupPtr[Index].DevFuncNum);\r
+            ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DATA_BUS_WIDTH), gShellDebug1HiiHandle, PeerGroupPtr[Index].DataBusWidth);\r
+          }\r
+        }\r
+      }\r
     }\r
     break;\r
 \r
@@ -755,6 +778,29 @@ SmbiosPrintStructure (
       PRINT_STRUCT_VALUE (Struct, Type17, MaximumVoltage);\r
       PRINT_STRUCT_VALUE (Struct, Type17, ConfiguredVoltage);\r
     }\r
+    if (AE_SMBIOS_VERSION (0x3, 0x2)) {\r
+      if (Struct->Hdr->Length > 0x28) {\r
+        DisplayMemoryDeviceMemoryTechnology (Struct->Type17->MemoryTechnology, Option);\r
+        DisplayMemoryDeviceMemoryOperatingModeCapability (Struct->Type17->MemoryOperatingModeCapability.Uint16, Option);\r
+        PRINT_PENDING_STRING (Struct, Type17, FirwareVersion);\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, ModuleManufacturerID);\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, ModuleProductID);\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, MemorySubsystemControllerManufacturerID);\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, MemorySubsystemControllerProductID);\r
+      }\r
+      if (Struct->Hdr->Length > 0x34) {\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, NonVolatileSize);\r
+      }\r
+      if (Struct->Hdr->Length > 0x3C) {\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, VolatileSize);\r
+      }\r
+      if (Struct->Hdr->Length > 0x44) {\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, CacheSize);\r
+      }\r
+      if (Struct->Hdr->Length > 0x4C) {\r
+        PRINT_STRUCT_VALUE_H (Struct, Type17, LogicalSize);\r
+      }\r
+    }\r
     break;\r
 \r
   //\r
@@ -1108,6 +1154,10 @@ SmbiosPrintStructure (
   //\r
   case 42:\r
     DisplayMCHostInterfaceType (Struct->Type42->InterfaceType, Option);\r
+    if (AE_SMBIOS_VERSION (0x3, 0x2)) {\r
+      PRINT_STRUCT_VALUE_H (Struct, Type42, InterfaceTypeSpecificDataLength);\r
+      PRINT_BIT_FIELD (Struct, Type42, InterfaceTypeSpecificData, Struct->Type42->InterfaceTypeSpecificDataLength);\r
+    }\r
     break;\r
 \r
   //\r
@@ -2111,6 +2161,10 @@ DisplayProcessorFamily (
     Print (L"Intel Core i3 processor\n");\r
     break;\r
 \r
+  case 0xCF:\r
+    Print (L"Intel Core i9 processor\n");\r
+    break;\r
+\r
   case 0xD2:\r
     Print (L"ViaC7M\n");\r
     break;\r

diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
index b56cd61d07fadea98559d1fad46a337e449d5027..9a1c33fbe013d2dd4cb2aac6f3ebf689317c01c1 100644 (file)
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
@@ -579,6 +579,22 @@ TABLE_ITEM  ProcessorUpgradeTable[] = {
   {\r
     0x38,\r
     L"Socket SP3r2"\r
+  },\r
+  {\r
+    0x39,\r
+    L"Socket LGA2066"\r
+  },\r
+  {\r
+    0x3A,\r
+    L"Socket BGA1392"\r
+  },\r
+  {\r
+    0x3B,\r
+    L"Socket BGA1510"\r
+  },\r
+  {\r
+    0x3C,\r
+    L"Socket BGA1528"\r
   }\r
 };\r
 \r
@@ -1088,6 +1104,10 @@ TABLE_ITEM  PortConnectorTypeTable[] = {
     0x22,\r
     L"SAS/SATA Plug Receptacle"\r
   },\r
+  {\r
+    0x23,\r
+    L"USB Type-C Receptacle"\r
+  },\r
   {\r
     0xA0,\r
     L"PC-98"\r
@@ -1251,6 +1271,14 @@ TABLE_ITEM  PortTypeTable[] = {
     0x21,\r
     L"SAS Port"\r
   },\r
+  {\r
+    0x22,\r
+    L"Multi-Function Display Port (MFDP)"\r
+  },\r
+  {\r
+    0x23,\r
+    L"Thunderbolt"\r
+  },\r
   {\r
     0xA0,\r
     L"8251 Compatible"\r
@@ -1576,6 +1604,10 @@ TABLE_ITEM  SystemSlotCurrentUsageTable[] = {
     0x04,\r
     L" In use"\r
   },\r
+  {\r
+    0x05,\r
+    L" Unavailable"\r
+  }\r
 };\r
 \r
 TABLE_ITEM  SystemSlotLengthTable[] = {\r
@@ -1645,6 +1677,10 @@ TABLE_ITEM  SlotCharacteristics2Table[] = {
   {\r
     2,\r
     L" PCI slot supports SMBus signal"\r
+  },\r
+  {\r
+    3,\r
+    L" PCIe slot supports bifurcation"\r
   }\r
 };\r
 \r
@@ -2468,6 +2504,10 @@ TABLE_ITEM  MemoryDeviceTypeTable[] = {
   {\r
     0x1E,\r
     L"  LPDDR4"\r
+  },\r
+  {\r
+    0x1F,\r
+    L"  Logical non-volatile device"\r
   }\r
 };\r
 \r
@@ -2530,6 +2570,61 @@ TABLE_ITEM  MemoryDeviceTypeDetailTable[] = {
   }\r
 };\r
 \r
+TABLE_ITEM  MemoryDeviceMemoryTechnologyTable[] = {\r
+  {\r
+    0x01,\r
+    L" Other"\r
+  },\r
+  {\r
+    0x02,\r
+    L" Unknown"\r
+  },\r
+  {\r
+    0x03,\r
+    L" DRAM"\r
+  },\r
+  {\r
+    0x04,\r
+    L" NVDIMM-N"\r
+  },\r
+  {\r
+    0x05,\r
+    L" NVDIMM-F"\r
+  },\r
+  {\r
+    0x06,\r
+    L" NVDIMM-P"\r
+  },\r
+  {\r
+    0x07,\r
+    L" Intel persistent memory"\r
+  }\r
+};\r
+\r
+TABLE_ITEM  MemoryDeviceMemoryOperatingModeCapabilityTable[] = {\r
+  {\r
+    1,\r
+    L" Other"\r
+  },\r
+  {\r
+    2,\r
+    L" Unknown"\r
+  },\r
+  {\r
+    3,\r
+    L" Volatile memory"\r
+  },\r
+  {\r
+    4,\r
+    L" Byte-accessible persistent memory"\r
+  },\r
+  {\r
+    5,\r
+    L" Block-accessible persistent memory"\r
+  }\r
+};\r
+\r
+\r
 TABLE_ITEM  MemoryErrorTypeTable[] = {\r
   {\r
     0x01,\r
@@ -3155,7 +3250,11 @@ TABLE_ITEM  IPMIDIBMCInterfaceTypeTable[] = {
     L" BT: Block Transfer "\r
   },\r
   {\r
-    0xFF04,\r
+    0x04,\r
+    L" SSIF: SMBus System Interface "\r
+  },\r
+  {\r
+    0xFF05,\r
     L" Reserved for future assignment by this specification "\r
   },\r
 };\r
@@ -4222,6 +4321,40 @@ DisplayMemoryDeviceTypeDetail (
   PRINT_BITS_INFO (MemoryDeviceTypeDetailTable, Para);\r
 }\r
 \r
+/**\r
+  Display Memory Device (Type 17) memory technology.\r
+\r
+  @param[in] Para     The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
+VOID\r
+DisplayMemoryDeviceMemoryTechnology (\r
+  IN UINT8  Para,\r
+  IN UINT8  Option\r
+  )\r
+{\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_MEMORY_TECHNOLOGY), gShellDebug1HiiHandle);\r
+  PRINT_INFO_OPTION (Para, Option);\r
+  PRINT_TABLE_ITEM (MemoryDeviceMemoryTechnologyTable, Para);\r
+}\r
+\r
+/**\r
+  Display Memory Device (Type 17) memory operating mode capability.\r
+\r
+  @param[in] Para     The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
+VOID\r
+DisplayMemoryDeviceMemoryOperatingModeCapability (\r
+  IN UINT16  Para,\r
+  IN UINT8   Option\r
+  )\r
+{\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_MEM_OPER_MODE_CAPA), gShellDebug1HiiHandle);\r
+  PRINT_INFO_OPTION (Para, Option);\r
+  PRINT_BITS_INFO (MemoryDeviceMemoryOperatingModeCapabilityTable, Para);\r
+}\r
+\r
 /**\r
   Display 32-bit Memory Error Information (Type 18) type.\r
 \r

diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.h b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.h
index bd9e6898d40afe95781e5498f9360f91b51e1531..3449da357ee3bf7b0b81ae02310e5ed5315218e9 100644 (file)
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.h
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.h
@@ -2,7 +2,7 @@
   Build a table, each item is (key, info) pair.\r
   and give a interface of query a string out of a table.\r
 \r
-  Copyright (c) 2005 - 2017, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2005 - 2018, 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
@@ -563,6 +563,30 @@ DisplayMemoryDeviceTypeDetail (
   IN UINT8   Option\r
   );\r
 \r
+/**\r
+  Display Memory Device (Type 17) memory technology.\r
+\r
+  @param[in] Para     The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
+VOID\r
+DisplayMemoryDeviceMemoryTechnology (\r
+  IN UINT8  Para,\r
+  IN UINT8  Option\r
+  );\r
+\r
+/**\r
+  Display Memory Device (Type 17) memory operating mode capability.\r
+\r
+  @param[in] Para     The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
+VOID\r
+DisplayMemoryDeviceMemoryOperatingModeCapability (\r
+  IN UINT16  Para,\r
+  IN UINT8   Option\r
+  );\r
+\r
 /**\r
   Display 32-bit Memory Error Information (Type 18) type.\r
 \r

diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosViewStrings.uni b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosViewStrings.uni
index 2cd682630557e3ab5a8ed5d6261194b78a405447..021ed62508b0487d3d377bb76ef2a5a7c47bc5b4 100644 (file)
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosViewStrings.uni
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosViewStrings.uni
@@ -1,6 +1,6 @@
 // /**\r
 //\r
-// Copyright (c) 2005 - 2017, Intel Corporation. All rights reserved.<BR>\r
+// Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR>\r
 // (C) Copyright 2014-2015 Hewlett-Packard Development Company, L.P.<BR>\r
 // (C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP<BR>\r
 // This program and the accompanying materials\r
@@ -104,6 +104,11 @@
 #string STR_SMBIOSVIEW_PRINTINFO_INSTALLED                      #language en-US "Installed "\r
 #string STR_SMBIOSVIEW_PRINTINFO_ENABLED                        #language en-US "Enabled "\r
 #string STR_SMBIOSVIEW_PRINTINFO_EXTERNAL                       #language en-US "External "\r
+#string STR_SMBIOSVIEW_PRINTINFO_SLOT_PEER_GROUPS               #language en-US "Peer Groups %d:\r\n"\r
+#string STR_SMBIOSVIEW_PRINTINFO_SEGMENT_GROUP_NUM              #language en-US "  SegmentGroup Num: 0x%04x\r\n"\r
+#string STR_SMBIOSVIEW_PRINTINFO_BUS_NUM                        #language en-US "  Bus Num:          0x%02x\r\n"\r
+#string STR_SMBIOSVIEW_PRINTINFO_DEV_FUNC_NUM                   #language en-US "  DevFunc Num:      0x%02x\r\n"\r
+#string STR_SMBIOSVIEW_PRINTINFO_DATA_BUS_WIDTH                 #language en-US "  DataBus Width:    %d\r\n"\r
 #string STR_SMBIOSVIEW_PRINTINFO_DESC_STRING                    #language en-US "Description String: "\r
 #string STR_SMBIOSVIEW_PRINTINFO_SUPOPRTED_EVENT                #language en-US "Supported Event Log Type Descriptors %d:\r\n"\r
 #string STR_SMBIOSVIEW_PRINTINFO_ACCESS_METHOD_NOT_SUPOPRTED    #language en-US "Access Method %d has not supported\r\n"\r
@@ -427,6 +432,8 @@
 #string STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_FORM_FACTOR        #language en-US "Memory Device - Form Factor: "\r
 #string STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_TYPE               #language en-US "Memory Device - Type: "\r
 #string STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_TYPE_DETAIL        #language en-US "Memory Device - Type Detail: "\r
+#string STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_MEMORY_TECHNOLOGY  #language en-US "Memory Device - Memory Technology: "\r
+#string STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_MEM_OPER_MODE_CAPA #language en-US "Memory Device - Memory Operating Mode Capability: "\r
 #string STR_SMBIOSVIEW_QUERYTABLE_MEM_ERROR_INFO                #language en-US "32-bit Memory Error Information - Type: "\r
 #string STR_SMBIOSVIEW_QUERYTABLE_MEM_ERROR_GRANULARITY         #language en-US "Memory Error - Error granularity: "\r
 #string STR_SMBIOSVIEW_QUERYTABLE_MEM_ERROR_OP                  #language en-US "Memory Error - Error Operation: "\r