public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH v3 0/3] Update to support SmBios 3.3.0
@ 2019-09-20  3:55 Abner Chang
  2019-09-20  3:55 ` [PATCH v3 1/3] MdeModulePkg: " Abner Chang
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Abner Chang @ 2019-09-20  3:55 UTC (permalink / raw)
  To: devel
  Cc: abner.chang, Michael D Kinney, Liming Gao, Hao A Wu, Jian J Wang,
	Jaben Carsey, Ray Ni, Zhichao Gao, Leif Lindholm, Gilbert Chen

Codes changes for SMBIOS spec 3.3.0 (Not public yet)
Bugzilla link,
https://bugzilla.tianocore.org/show_bug.cgi?id=2202

Abner Chang (3):
  MdeModulePkg: Update to support SmBios 3.3.0
  MdePkg/Include: Update to support SmBios 3.3.0
  ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0

 MdeModulePkg/MdeModulePkg.dec                      |  6 +-
 MdePkg/Include/IndustryStandard/SmBios.h           | 76 +---------------------
 .../SmbiosView/PrintInfo.c                         | 23 +------
 .../SmbiosView/PrintInfo.h                         | 13 +---
 .../SmbiosView/QueryTable.c                        | 63 +-----------------
 .../UefiShellDebug1CommandsLib.uni                 |  3 +-
 6 files changed, 10 insertions(+), 174 deletions(-)

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Jaben Carsey <jaben.carsey@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Zhichao Gao <zhichao.gao@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Gilbert Chen <gilbert.chen@hpe.com>
-- 
2.7.4


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH v3 1/3] MdeModulePkg: Update to support SmBios 3.3.0
  2019-09-20  3:55 [PATCH v3 0/3] Update to support SmBios 3.3.0 Abner Chang
@ 2019-09-20  3:55 ` Abner Chang
  2019-09-20  3:55 ` [PATCH v3 2/3] MdePkg/Include: " Abner Chang
  2019-09-20  3:55 ` [PATCH v3 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support Abner Chang
  2 siblings, 0 replies; 4+ messages in thread
From: Abner Chang @ 2019-09-20  3:55 UTC (permalink / raw)
  To: devel; +Cc: abner.chang, Hao A Wu, Jian J Wang, Leif Lindholm, Gilbert Chen

This commit update the revision of SMBIOS spec reported in PCDs.

Signed-off-by: Abner Chang <abner.chang@hpe.com>

Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Gilbert Chen <gilbert.chen@hpe.com>
---
 MdeModulePkg/MdeModulePkg.dec | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec
index e3a65ab..19935c8 100644
--- a/MdeModulePkg/MdeModulePkg.dec
+++ b/MdeModulePkg/MdeModulePkg.dec
@@ -1792,10 +1792,10 @@
 
   ## SMBIOS version.
   # @Prompt SMBIOS version.
-  gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosVersion|0x0303|UINT16|0x00010055
+  gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosVersion|0x0302|UINT16|0x00010055
 
-  ## SMBIOS Docrev field in SMBIOS 3.3 (64-bit) Entry Point Structure.
-  # @Prompt SMBIOS Docrev field in SMBIOS 3.3 (64-bit) Entry Point Structure.
+  ## SMBIOS Docrev field in SMBIOS 3.0 (64-bit) Entry Point Structure.
+  # @Prompt SMBIOS Docrev field in SMBIOS 3.0 (64-bit) Entry Point Structure.
   gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosDocRev|0x0|UINT8|0x0001006A
 
   ## SMBIOS produce method.
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH v3 2/3] MdePkg/Include: Update to support SmBios 3.3.0
  2019-09-20  3:55 [PATCH v3 0/3] Update to support SmBios 3.3.0 Abner Chang
  2019-09-20  3:55 ` [PATCH v3 1/3] MdeModulePkg: " Abner Chang
@ 2019-09-20  3:55 ` Abner Chang
  2019-09-20  3:55 ` [PATCH v3 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support Abner Chang
  2 siblings, 0 replies; 4+ messages in thread
From: Abner Chang @ 2019-09-20  3:55 UTC (permalink / raw)
  To: devel
  Cc: abner.chang, Michael D Kinney, Liming Gao, Leif Lindholm,
	Gilbert Chen

Update SmBios.h to support SMBIOS 3.3.0 spec.

Signed-off-by: Abner Chang <abner.chang@hpe.com>

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Gilbert Chen <gilbert.chen@hpe.com>
---
 MdePkg/Include/IndustryStandard/SmBios.h | 76 ++------------------------------
 1 file changed, 3 insertions(+), 73 deletions(-)

diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
index f504cc8..f3b6f18 100644
--- a/MdePkg/Include/IndustryStandard/SmBios.h
+++ b/MdePkg/Include/IndustryStandard/SmBios.h
@@ -1,9 +1,8 @@
 /** @file
-  Industry Standard Definitions of SMBIOS Table Specification v3.3.0.
+  Industry Standard Definitions of SMBIOS Table Specification v3.2.0.
 
 Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>
 (C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP<BR>
-(C) Copyright 2015 - 2019 Hewlett Packard Enterprise Development LP<BR>
 SPDX-License-Identifier: BSD-2-Clause-Patent
 
 **/
@@ -47,7 +46,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 #define SMBIOS_3_0_TABLE_MAX_LENGTH 0xFFFFFFFF
 
 //
-// SMBIOS type macros which is according to SMBIOS 3.3.0 specification.
+// SMBIOS type macros which is according to SMBIOS 2.7 specification.
 //
 #define SMBIOS_TYPE_BIOS_INFORMATION                     0
 #define SMBIOS_TYPE_SYSTEM_INFORMATION                   1
@@ -93,7 +92,6 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 #define SMBIOS_TYPE_ONBOARD_DEVICES_EXTENDED_INFORMATION 41
 #define SMBIOS_TYPE_MANAGEMENT_CONTROLLER_HOST_INTERFACE 42
 #define SMBIOS_TYPE_TPM_DEVICE                           43
-#define SMBIOS_TYPE_PROCESSOR_ADDITIONAL_INFORMATION     44
 
 ///
 /// Inactive type is added from SMBIOS 2.2. Reference SMBIOS 2.6, chapter 3.3.43.
@@ -729,10 +727,7 @@ typedef enum {
   ProcessorFamilyMII                   = 0x012E,
   ProcessorFamilyWinChip               = 0x0140,
   ProcessorFamilyDSP                   = 0x015E,
-  ProcessorFamilyVideoProcessor        = 0x01F4,
-  ProcessorFamilyRiscvRV32             = 0x0200,
-  ProcessorFamilyRiscVRV64             = 0x0201,
-  ProcessorFamilyRiscVRV128            = 0x0202
+  ProcessorFamilyVideoProcessor        = 0x01F4
 } PROCESSOR_FAMILY2_DATA;
 
 ///
@@ -862,19 +857,6 @@ typedef struct {
 } PROCESSOR_FEATURE_FLAGS;
 
 typedef struct {
-  UINT32  ProcessorReserved1             :1;
-  UINT32  ProcessorUnknown               :1;
-  UINT32  Processor64BitCapble           :1;
-  UINT32  ProcessorMultiCore             :1;
-  UINT32  ProcessorHardwareThread        :1;
-  UINT32  ProcessorExecuteProtection     :1;
-  UINT32  ProcessorEnhancedVirtulization :1;
-  UINT32  ProcessorPowerPerformanceCtrl  :1;
-  UINT32  Processor128bitCapble          :1;
-  UINT32  ProcessorReserved2             :7;
-} PROCESSOR_CHARACTERISTIC_FLAGS;
-
-typedef struct {
   PROCESSOR_SIGNATURE     Signature;
   PROCESSOR_FEATURE_FLAGS FeatureFlags;
 } PROCESSOR_ID_DATA;
@@ -2526,57 +2508,6 @@ typedef struct {
   UINT8                             InterfaceTypeSpecificData[4];   ///< This field has a minimum of four bytes
 } SMBIOS_TABLE_TYPE42;
 
-
-///
-/// Processor Specific Block - Processor Architecture Type
-///
-typedef enum{
-  ProcessorSpecificBlockArchTypeReserved   = 0x00,
-  ProcessorSpecificBlockArchTypeIa32       = 0x01,
-  ProcessorSpecificBlockArchTypeX64        = 0x02,
-  ProcessorSpecificBlockArchTypeItanium    = 0x03,
-  ProcessorSpecificBlockArchTypeAarch32    = 0x04,
-  ProcessorSpecificBlockArchTypeAarch64    = 0x05,
-  ProcessorSpecificBlockArchTypeRiscVRV32  = 0x06,
-  ProcessorSpecificBlockArchTypeRiscVRV64  = 0x07,
-  ProcessorSpecificBlockArchTypeRiscVRV128 = 0x08
-} PROCESSOR_SPECIFIC_BLOCK_ARCH_TYPE;
-
-///
-/// Processor Specific Block is the standard container of processor-specific data.
-///
-typedef struct {
-  UINT8                              Length;
-  UINT8                              ProcessorArchType;
-  ///
-  /// Below followed by Processor-specific data
-  ///
-  ///
-} PROCESSOR_SPECIFIC_BLOCK;
-
-///
-/// Processor Additional Information(Type 44).
-///
-/// The information in this structure defines the processor additional information in case
-/// SMBIOS type 4 is not sufficient to describe processor characteristics.
-/// The SMBIOS type 44 structure has a reference handle field to link back to the related
-/// SMBIOS type 4 structure. There may be multiple SMBIOS type 44 structures linked to the
-/// same SMBIOS type 4 structure. For example, when cores are not identical in a processor,
-/// SMBIOS type 44 structures describe different core-specific information.
-///
-/// SMBIOS type 44 defines the standard header for the processor-specific block, while the
-/// contents of processor-specific data are maintained by processor
-/// architecture workgroups or vendors in separate documents.
-///
-typedef struct {
-  SMBIOS_STRUCTURE                  Hdr;
-  SMBIOS_HANDLE                     RefHandle;                 ///< This field refer to associated SMBIOS type 4
-  ///
-  /// Below followed by Processor-specific block
-  ///
-  PROCESSOR_SPECIFIC_BLOCK          ProcessorSpecificBlock;
-} SMBIOS_TABLE_TYPE44;
-
 ///
 /// TPM Device (Type 43).
 ///
@@ -2655,7 +2586,6 @@ typedef union {
   SMBIOS_TABLE_TYPE41   *Type41;
   SMBIOS_TABLE_TYPE42   *Type42;
   SMBIOS_TABLE_TYPE43   *Type43;
-  SMBIOS_TABLE_TYPE44   *Type44;
   SMBIOS_TABLE_TYPE126  *Type126;
   SMBIOS_TABLE_TYPE127  *Type127;
   UINT8                 *Raw;
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH v3 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support
  2019-09-20  3:55 [PATCH v3 0/3] Update to support SmBios 3.3.0 Abner Chang
  2019-09-20  3:55 ` [PATCH v3 1/3] MdeModulePkg: " Abner Chang
  2019-09-20  3:55 ` [PATCH v3 2/3] MdePkg/Include: " Abner Chang
@ 2019-09-20  3:55 ` Abner Chang
  2 siblings, 0 replies; 4+ messages in thread
From: Abner Chang @ 2019-09-20  3:55 UTC (permalink / raw)
  To: devel
  Cc: abner.chang, Jaben Carsey, Ray Ni, Zhichao Gao, Leif Lindholm,
	Gilbert Chen

Shell command smbiosview updates for SMBIOS 3.3.0 spec.

Signed-off-by: Abner Chang <abner.chang@hpe.com>

Cc: Jaben Carsey <jaben.carsey@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Zhichao Gao <zhichao.gao@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Gilbert Chen <gilbert.chen@hpe.com>
---
 .../SmbiosView/PrintInfo.c                         | 23 +-------
 .../SmbiosView/PrintInfo.h                         | 13 +----
 .../SmbiosView/QueryTable.c                        | 63 +---------------------
 .../UefiShellDebug1CommandsLib.uni                 |  3 +-
 4 files changed, 4 insertions(+), 98 deletions(-)

diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
index ff0c1db..cfd7a2b 100644
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
@@ -3,7 +3,7 @@
 
   Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR>
   (C) Copyright 2014 Hewlett-Packard Development Company, L.P.<BR>
-  (C) Copyright 2015-2019 Hewlett Packard Enterprise Development LP<BR>
+  (C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP<BR>
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
 **/
@@ -1169,15 +1169,6 @@ SmbiosPrintStructure (
     break;
 
   //
-  // Processor Additional Information (Type 44)
-  //
-  case 44:
-    PRINT_STRUCT_VALUE_H (Struct, Type44, RefHandle);
-    PRINT_STRUCT_VALUE_H (Struct, Type44, ProcessorSpecificBlock.ProcessorArchType);
-    ProcessorArchitectureType (Struct->Type44->ProcessorSpecificBlock.ProcessorArchType, Option);
-    break;
-
-  //
   // Inactive (Type 126)
   //
   case 126:
@@ -2359,18 +2350,6 @@ DisplayProcessorFamily2 (
       Print (L"Video Processor\n");
       break;
 
-    case 0x200:
-      Print (L"RISC-V RV32\n");
-      break;
-
-    case 0x201:
-      Print (L"RISC-V RV64\n");
-      break;
-
-    case 0x202:
-      Print (L"RISC-V RV128\n");
-      break;
-
     default:
      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNDEFINED_PROC_FAMILY), gShellDebug1HiiHandle);
   }
diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h
index 22f59d6..a8c32ab 100644
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h
@@ -2,7 +2,7 @@
   Module to clarify the element info of the smbios structure.
 
   Copyright (c) 2005 - 2015, Intel Corporation. All rights reserved.<BR>
-  (C) Copyright 2017 - 2019 Hewlett Packard Enterprise Development LP<BR>
+  (C) Copyright 2017 Hewlett Packard Enterprise Development LP<BR>
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
 **/
@@ -427,15 +427,4 @@ DisplayTpmDeviceCharacteristics (
   IN UINT8   Option
   );
 
-/**
-  Display Processor Architecture Type (Type 44).
-
-  @param[in] Type           The key of the structure.
-  @param[in] Option         The optional information.
-**/
-VOID
-ProcessorArchitectureType (
-  IN UINT8 Type,
-  IN UINT8 Option
-  );
 #endif
diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
index 54e9a52..bfb7c3b 100644
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
@@ -3,7 +3,7 @@
   And give a interface of query a string out of a table.
 
   Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR>
-  (C) Copyright 2016-2019 Hewlett Packard Enterprise Development LP<BR>
+  (C) Copyright 2016-2017 Hewlett Packard Enterprise Development LP<BR>
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
 **/
@@ -620,10 +620,6 @@ TABLE_ITEM  ProcessorCharacteristicsTable[] = {
   {
     7,
     L" Power/Performance Control"
-  },
-  {
-    8,
-    L" 128-bit Capable"
   }
 };
 
@@ -3272,44 +3268,6 @@ TABLE_ITEM  MCHostInterfaceTypeTable[] = {
   },
 };
 
-TABLE_ITEM  ProcessorArchitectureTypesTable[] = {
-  {
-    0,
-    L" Reserved "
-  },
-  {
-    1,
-    L" IA32 (x86) "
-  },
-  {
-    2,
-    L" x64 (x86-64, intel64, AMD64, EM64T) "
-  },
-  {
-    3,
-    L" Intel Itanium architecture "
-  },
-  {
-    4,
-    L" 32-bit ARM (Aarch32) "
-  },
-  {
-    5,
-    L" 64-bit ARM (Aarch64) "
-  },
-  {
-    6,
-    L" 32-bit RISC-V (RV32) "
-  },
-  {
-    7,
-    L" 64-bit RISC-V (RV64) "
-  },
-  {
-    8,
-    L" 128-bit RISC-V (RV128) "
-  }
-};
 
 TABLE_ITEM  StructureTypeInfoTable[] = {
   {
@@ -3489,10 +3447,6 @@ TABLE_ITEM  StructureTypeInfoTable[] = {
     L" TPM Device"
   },
   {
-    44,
-    L" Processor Additional Information"
-  },
-  {
     0x7E,
     L" Inactive"
   },
@@ -4743,21 +4697,6 @@ DisplayMCHostInterfaceType (
 }
 
 /**
-  Display Processor Architecture Type (Type 44).
-
-  @param[in] Type           The key of the structure.
-  @param[in] Option         The optional information.
-**/
-VOID
-ProcessorArchitectureType (
-  IN UINT8 Type,
-  IN UINT8 Option
-  )
-{
-  PRINT_TABLE_ITEM (ProcessorArchitectureTypesTable, Type);
-}
-
-/**
   Display the structure type information.
 
   @param[in] Key      The key of the structure.
diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.uni b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.uni
index 6693be2..42a9295 100644
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.uni
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.uni
@@ -2,7 +2,7 @@
 //
 // Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved.<BR>
 // (C) Copyright 2013-2015 Hewlett-Packard Development Company, L.P.<BR>
-// (C) Copyright 2016 - 2019 Hewlett Packard Enterprise Development LP<BR>
+// (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
 // SPDX-License-Identifier: BSD-2-Clause-Patent
 //
 // Module Name:
@@ -976,7 +976,6 @@
 "       41 - Onboard Devices Extended Information\r\n"
 "       42 - Management Controller Host Interface\r\n"
 "       43 - TPM Device\r\n"
-"       44 - Processor Additional Information\r\n"
 "  2. Enter the SmbiosHandle parameter in hexadecimal format.\r\n"
 "     Do not use the '0x' prefix format for hexadecimal values.\r\n"
 "  3. Internal commands:\r\n"
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-09-20  4:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-09-20  3:55 [PATCH v3 0/3] Update to support SmBios 3.3.0 Abner Chang
2019-09-20  3:55 ` [PATCH v3 1/3] MdeModulePkg: " Abner Chang
2019-09-20  3:55 ` [PATCH v3 2/3] MdePkg/Include: " Abner Chang
2019-09-20  3:55 ` [PATCH v3 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support Abner Chang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox