public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH] MdePkg:Include: Update SmBios header file
@ 2019-09-17  6:24 Abner Chang
  2019-09-17 13:33 ` Leif Lindholm
  0 siblings, 1 reply; 4+ messages in thread
From: Abner Chang @ 2019-09-17  6:24 UTC (permalink / raw)
  To: devel
  Cc: abner.chang, Michael D Kinney, Liming Gao, Leif Lindholm,
	Gilbert Chen

Update SmBios header file to conform with SMBIOS v3.3.0.
The major update is to add definitions of SMBIOS Type 44h record.

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 | 74 +++++++++++++++++++++++++++++++-
 1 file changed, 72 insertions(+), 2 deletions(-)

diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
index f3b6f18..ebf0ceb 100644
--- a/MdePkg/Include/IndustryStandard/SmBios.h
+++ b/MdePkg/Include/IndustryStandard/SmBios.h
@@ -3,6 +3,7 @@
 
 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
 
 **/
@@ -46,7 +47,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 #define SMBIOS_3_0_TABLE_MAX_LENGTH 0xFFFFFFFF
 
 //
-// SMBIOS type macros which is according to SMBIOS 2.7 specification.
+// SMBIOS type macros which is according to SMBIOS 3.3.0 specification.
 //
 #define SMBIOS_TYPE_BIOS_INFORMATION                     0
 #define SMBIOS_TYPE_SYSTEM_INFORMATION                   1
@@ -92,6 +93,7 @@ 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.
@@ -727,7 +729,10 @@ typedef enum {
   ProcessorFamilyMII                   = 0x012E,
   ProcessorFamilyWinChip               = 0x0140,
   ProcessorFamilyDSP                   = 0x015E,
-  ProcessorFamilyVideoProcessor        = 0x01F4
+  ProcessorFamilyVideoProcessor        = 0x01F4,
+  ProcessorFamilyRiscvRV32             = 0x0200,  ///< SMBIOS spec 3.3.0 added
+  ProcessorFamilyRiscVRV64             = 0x0201,  ///< SMBIOS spec 3.3.0 added
+  ProcessorFamilyRiscVRV128            = 0x0202   ///< SMBIOS spec 3.3.0 added
 } PROCESSOR_FAMILY2_DATA;
 
 ///
@@ -857,6 +862,19 @@ 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;
@@ -2508,6 +2526,57 @@ 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).
 ///
@@ -2586,6 +2655,7 @@ 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

* Re: [PATCH] MdePkg:Include: Update SmBios header file
  2019-09-17  6:24 [PATCH] MdePkg:Include: Update SmBios header file Abner Chang
@ 2019-09-17 13:33 ` Leif Lindholm
  2019-09-17 14:42   ` [edk2-devel] " Liming Gao
  0 siblings, 1 reply; 4+ messages in thread
From: Leif Lindholm @ 2019-09-17 13:33 UTC (permalink / raw)
  To: Abner Chang; +Cc: devel, Michael D Kinney, Liming Gao, Gilbert Chen

On Tue, Sep 17, 2019 at 02:24:30PM +0800, Abner Chang wrote:
> Update SmBios header file to conform with SMBIOS v3.3.0.

Ah, I note SMBIOS 3.3 has not yet been released - so this can not be
merged in edk2 master at this point. I did not realise this when I
requested you send the patch.

However, you can carry this in your edk2-staging branch, and once the
specification gets released we can take it into edk2.

(After code review, a couple of minor comments below.)

> The major update is to add definitions of SMBIOS Type 44h record.
> 
> 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 | 74 +++++++++++++++++++++++++++++++-
>  1 file changed, 72 insertions(+), 2 deletions(-)
> 
> diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
> index f3b6f18..ebf0ceb 100644
> --- a/MdePkg/Include/IndustryStandard/SmBios.h
> +++ b/MdePkg/Include/IndustryStandard/SmBios.h
> @@ -3,6 +3,7 @@
>  
>  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
>  
>  **/
> @@ -46,7 +47,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>  #define SMBIOS_3_0_TABLE_MAX_LENGTH 0xFFFFFFFF
>  
>  //
> -// SMBIOS type macros which is according to SMBIOS 2.7 specification.
> +// SMBIOS type macros which is according to SMBIOS 3.3.0 specification.
>  //
>  #define SMBIOS_TYPE_BIOS_INFORMATION                     0
>  #define SMBIOS_TYPE_SYSTEM_INFORMATION                   1
> @@ -92,6 +93,7 @@ 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.
> @@ -727,7 +729,10 @@ typedef enum {
>    ProcessorFamilyMII                   = 0x012E,
>    ProcessorFamilyWinChip               = 0x0140,
>    ProcessorFamilyDSP                   = 0x015E,
> -  ProcessorFamilyVideoProcessor        = 0x01F4
> +  ProcessorFamilyVideoProcessor        = 0x01F4,
> +  ProcessorFamilyRiscvRV32             = 0x0200,  ///< SMBIOS spec 3.3.0 added

Please drop the "///< SMBIOS spec 3.3.0 added" comment. Here and below.

> +  ProcessorFamilyRiscVRV64             = 0x0201,  ///< SMBIOS spec 3.3.0 added
> +  ProcessorFamilyRiscVRV128            = 0x0202   ///< SMBIOS spec 3.3.0 added

No further comments from me (MdePkg maintainers may have some).

/
    Leif

>  } PROCESSOR_FAMILY2_DATA;
>  
>  ///
> @@ -857,6 +862,19 @@ 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;
> @@ -2508,6 +2526,57 @@ 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).
>  ///
> @@ -2586,6 +2655,7 @@ 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	[flat|nested] 4+ messages in thread

* Re: [edk2-devel] [PATCH] MdePkg:Include: Update SmBios header file
  2019-09-17 13:33 ` Leif Lindholm
@ 2019-09-17 14:42   ` Liming Gao
  2019-09-19  3:38     ` Abner Chang
  0 siblings, 1 reply; 4+ messages in thread
From: Liming Gao @ 2019-09-17 14:42 UTC (permalink / raw)
  To: devel@edk2.groups.io, leif.lindholm@linaro.org, Abner Chang
  Cc: Kinney, Michael D, Gilbert Chen

Abner:
  I add my comments. 

> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Leif Lindholm
> Sent: Tuesday, September 17, 2019 9:34 PM
> To: Abner Chang <abner.chang@hpe.com>
> Cc: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>; Gilbert Chen
> <gilbert.chen@hpe.com>
> Subject: Re: [edk2-devel] [PATCH] MdePkg:Include: Update SmBios header file
> 
> On Tue, Sep 17, 2019 at 02:24:30PM +0800, Abner Chang wrote:
> > Update SmBios header file to conform with SMBIOS v3.3.0.
> 
> Ah, I note SMBIOS 3.3 has not yet been released - so this can not be
> merged in edk2 master at this point. I did not realise this when I
> requested you send the patch.

Please submit BZ https://bugzilla.tianocore.org/ for this update. This is a new feature. 
> 
> However, you can carry this in your edk2-staging branch, and once the
> specification gets released we can take it into edk2.
> 
> (After code review, a couple of minor comments below.)
> 
> > The major update is to add definitions of SMBIOS Type 44h record.
> >
> > 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 | 74 +++++++++++++++++++++++++++++++-
> >  1 file changed, 72 insertions(+), 2 deletions(-)
> >
> > diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
> > index f3b6f18..ebf0ceb 100644
> > --- a/MdePkg/Include/IndustryStandard/SmBios.h
> > +++ b/MdePkg/Include/IndustryStandard/SmBios.h
> > @@ -3,6 +3,7 @@
> >
> >  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

File header should be updated.

  Industry Standard Definitions of SMBIOS Table Specification v3.2.0.
==>
  Industry Standard Definitions of SMBIOS Table Specification v3.3.0.

And, please refer to https://bugzilla.tianocore.org/show_bug.cgi?id=1099 for 3.2 update,
prepare all changes for SmBios 3.3 update.

Thanks
Liming
> >
> >  **/
> > @@ -46,7 +47,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> >  #define SMBIOS_3_0_TABLE_MAX_LENGTH 0xFFFFFFFF
> >
> >  //
> > -// SMBIOS type macros which is according to SMBIOS 2.7 specification.
> > +// SMBIOS type macros which is according to SMBIOS 3.3.0 specification.
> >  //
> >  #define SMBIOS_TYPE_BIOS_INFORMATION                     0
> >  #define SMBIOS_TYPE_SYSTEM_INFORMATION                   1
> > @@ -92,6 +93,7 @@ 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.
> > @@ -727,7 +729,10 @@ typedef enum {
> >    ProcessorFamilyMII                   = 0x012E,
> >    ProcessorFamilyWinChip               = 0x0140,
> >    ProcessorFamilyDSP                   = 0x015E,
> > -  ProcessorFamilyVideoProcessor        = 0x01F4
> > +  ProcessorFamilyVideoProcessor        = 0x01F4,
> > +  ProcessorFamilyRiscvRV32             = 0x0200,  ///< SMBIOS spec 3.3.0 added
> 
> Please drop the "///< SMBIOS spec 3.3.0 added" comment. Here and below.
> 
> > +  ProcessorFamilyRiscVRV64             = 0x0201,  ///< SMBIOS spec 3.3.0 added
> > +  ProcessorFamilyRiscVRV128            = 0x0202   ///< SMBIOS spec 3.3.0 added
> 
> No further comments from me (MdePkg maintainers may have some).
> 
> /
>     Leif
> 
> >  } PROCESSOR_FAMILY2_DATA;
> >
> >  ///
> > @@ -857,6 +862,19 @@ 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;
> > @@ -2508,6 +2526,57 @@ 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).
> >  ///
> > @@ -2586,6 +2655,7 @@ 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	[flat|nested] 4+ messages in thread

* Re: [edk2-devel] [PATCH] MdePkg:Include: Update SmBios header file
  2019-09-17 14:42   ` [edk2-devel] " Liming Gao
@ 2019-09-19  3:38     ` Abner Chang
  0 siblings, 0 replies; 4+ messages in thread
From: Abner Chang @ 2019-09-19  3:38 UTC (permalink / raw)
  To: Gao, Liming, devel@edk2.groups.io, leif.lindholm@linaro.org
  Cc: Kinney, Michael D, Chen, Gilbert



> -----Original Message-----
> From: Gao, Liming [mailto:liming.gao@intel.com]
> Sent: Tuesday, September 17, 2019 10:43 PM
> To: devel@edk2.groups.io; leif.lindholm@linaro.org; Chang, Abner (HPS
> SW/FW Technologist) <abner.chang@hpe.com>
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Chen, Gilbert
> <gilbert.chen@hpe.com>
> Subject: RE: [edk2-devel] [PATCH] MdePkg:Include: Update SmBios header
> file
> 
> Abner:
>   I add my comments.
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> > Leif Lindholm
> > Sent: Tuesday, September 17, 2019 9:34 PM
> > To: Abner Chang <abner.chang@hpe.com>
> > Cc: devel@edk2.groups.io; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>;
> > Gilbert Chen <gilbert.chen@hpe.com>
> > Subject: Re: [edk2-devel] [PATCH] MdePkg:Include: Update SmBios header
> > file
> >
> > On Tue, Sep 17, 2019 at 02:24:30PM +0800, Abner Chang wrote:
> > > Update SmBios header file to conform with SMBIOS v3.3.0.
> >
> > Ah, I note SMBIOS 3.3 has not yet been released - so this can not be
> > merged in edk2 master at this point. I did not realise this when I
> > requested you send the patch.
> 
> Please submit BZ INVALID URI REMOVED
> 3A__bugzilla.tianocore.org_&d=DwIFAg&c=C5b8zRQO1miGmBeVZ2LFWg&r
> =_SN6FZBN4Vgi4Ulkskz6qU3NYRO03nHp9P7Z5q59A3E&m=Hg_QWeYnS53nt
> pt5iTV5XORHiCtdmH1WwCEMvog8Umk&s=9i2dIKluz1GytIz4gy4XCxpphhGm
> sO4Rr5vn3dffLGA&e=  for this update. This is a new feature.

Done. ID: 2202
> >
> > However, you can carry this in your edk2-staging branch, and once the
> > specification gets released we can take it into edk2.
> >
> > (After code review, a couple of minor comments below.)
> >
> > > The major update is to add definitions of SMBIOS Type 44h record.
> > >
> > > 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 | 74
> > > +++++++++++++++++++++++++++++++-
> > >  1 file changed, 72 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/MdePkg/Include/IndustryStandard/SmBios.h
> > > b/MdePkg/Include/IndustryStandard/SmBios.h
> > > index f3b6f18..ebf0ceb 100644
> > > --- a/MdePkg/Include/IndustryStandard/SmBios.h
> > > +++ b/MdePkg/Include/IndustryStandard/SmBios.h
> > > @@ -3,6 +3,7 @@
> > >
> > >  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
> 
> File header should be updated.
> 
>   Industry Standard Definitions of SMBIOS Table Specification v3.2.0.
> ==>
>   Industry Standard Definitions of SMBIOS Table Specification v3.3.0.
> 
> And, please refer to INVALID URI REMOVED
> 3A__bugzilla.tianocore.org_show-5Fbug.cgi-3Fid-
> 3D1099&d=DwIFAg&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ulks
> kz6qU3NYRO03nHp9P7Z5q59A3E&m=Hg_QWeYnS53ntpt5iTV5XORHiCtdmH1
> WwCEMvog8Umk&s=cygMqovZS9Qo_maugU8bx1lvpSIt0jn72VTQfVMtZNA
> &e=  for 3.2 update, prepare all changes for SmBios 3.3 update.
> 
PATCH v2 sent.
Thanks
Abner

> Thanks
> Liming
> > >
> > >  **/
> > > @@ -46,7 +47,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> > > #define SMBIOS_3_0_TABLE_MAX_LENGTH 0xFFFFFFFF
> > >
> > >  //
> > > -// SMBIOS type macros which is according to SMBIOS 2.7 specification.
> > > +// SMBIOS type macros which is according to SMBIOS 3.3.0 specification.
> > >  //
> > >  #define SMBIOS_TYPE_BIOS_INFORMATION                     0
> > >  #define SMBIOS_TYPE_SYSTEM_INFORMATION                   1
> > > @@ -92,6 +93,7 @@ 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.
> > > @@ -727,7 +729,10 @@ typedef enum {
> > >    ProcessorFamilyMII                   = 0x012E,
> > >    ProcessorFamilyWinChip               = 0x0140,
> > >    ProcessorFamilyDSP                   = 0x015E,
> > > -  ProcessorFamilyVideoProcessor        = 0x01F4
> > > +  ProcessorFamilyVideoProcessor        = 0x01F4,
> > > +  ProcessorFamilyRiscvRV32             = 0x0200,  ///< SMBIOS spec 3.3.0
> added
> >
> > Please drop the "///< SMBIOS spec 3.3.0 added" comment. Here and
> below.
> >
> > > +  ProcessorFamilyRiscVRV64             = 0x0201,  ///< SMBIOS spec 3.3.0
> added
> > > +  ProcessorFamilyRiscVRV128            = 0x0202   ///< SMBIOS spec 3.3.0
> added
> >
> > No further comments from me (MdePkg maintainers may have some).
> >
> > /
> >     Leif
> >
> > >  } PROCESSOR_FAMILY2_DATA;
> > >
> > >  ///
> > > @@ -857,6 +862,19 @@ 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;
> @@
> > > -2508,6 +2526,57 @@ 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).
> > >  ///
> > > @@ -2586,6 +2655,7 @@ 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	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-09-19  3:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-09-17  6:24 [PATCH] MdePkg:Include: Update SmBios header file Abner Chang
2019-09-17 13:33 ` Leif Lindholm
2019-09-17 14:42   ` [edk2-devel] " Liming Gao
2019-09-19  3:38     ` Abner Chang

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