From: "Abner Chang" <abner.chang@hpe.com>
To: "Gao, Liming" <liming.gao@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>,
"leif.lindholm@linaro.org" <leif.lindholm@linaro.org>
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
Date: Thu, 19 Sep 2019 03:38:34 +0000 [thread overview]
Message-ID: <CS1PR8401MB119204825BAF01FB7EF88285FF890@CS1PR8401MB1192.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4FD801@SHSMSX104.ccr.corp.intel.com>
> -----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
> > >
> >
> >
prev parent reply other threads:[~2019-09-19 3:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
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 message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CS1PR8401MB119204825BAF01FB7EF88285FF890@CS1PR8401MB1192.NAMPRD84.PROD.OUTLOOK.COM \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox