From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mx.groups.io with SMTP id smtpd.web10.509.1571186727770893234 for ; Tue, 15 Oct 2019 17:45:28 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.88, mailfrom: zhichao.gao@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 15 Oct 2019 17:45:27 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.67,301,1566889200"; d="scan'208";a="220613749" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by fmsmga004.fm.intel.com with ESMTP; 15 Oct 2019 17:45:27 -0700 Received: from fmsmsx152.amr.corp.intel.com (10.18.125.5) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 15 Oct 2019 17:45:27 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by FMSMSX152.amr.corp.intel.com (10.18.125.5) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 15 Oct 2019 17:45:26 -0700 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.96]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.165]) with mapi id 14.03.0439.000; Wed, 16 Oct 2019 08:45:24 +0800 From: "Gao, Zhichao" To: Abner Chang , "devel@edk2.groups.io" CC: "Carsey, Jaben" , "Ni, Ray" , Leif Lindholm , Gilbert Chen Subject: Re: [PATCH v7 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0 Thread-Topic: [PATCH v7 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0 Thread-Index: AQHVgzD2NJPn91xS70eMN6HHWXQ64qdcaggg Date: Wed, 16 Oct 2019 00:45:23 +0000 Message-ID: <3CE959C139B4C44DBEA1810E3AA6F9000B857018@SHSMSX101.ccr.corp.intel.com> References: <1571125545-32577-1-git-send-email-abner.chang@hpe.com> In-Reply-To: <1571125545-32577-1-git-send-email-abner.chang@hpe.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: zhichao.gao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable > -----Original Message----- > From: Abner Chang [mailto:abner.chang@hpe.com] > Sent: Tuesday, October 15, 2019 3:46 PM > To: devel@edk2.groups.io > Cc: abner.chang@hpe.com; Carsey, Jaben ; Ni, > Ray ; Gao, Zhichao ; Leif > Lindholm ; Gilbert Chen > Subject: [PATCH v7 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to > support SmBios 3.3.0 >=20 > Shell command smbiosview updates for SMBIOS 3.3.0 spec. >=20 > Bugzilla link, > https://bugzilla.tianocore.org/show_bug.cgi?id=3D2202 >=20 > Signed-off-by: Abner Chang >=20 > Cc: Jaben Carsey > Cc: Ray Ni > Cc: Zhichao Gao > Cc: Leif Lindholm > Cc: Gilbert Chen > --- > .../SmbiosView/PrintInfo.c | 21 ++++++- > .../SmbiosView/QueryTable.c | 65 ++++++++++++++++= +++++- > .../SmbiosView/PrintInfo.h | 13 ++++- > .../SmbiosView/SmbiosViewStrings.uni | 3 +- > .../UefiShellDebug1CommandsLib.uni | 3 +- > 5 files changed, 100 insertions(+), 5 deletions(-) >=20 > diff --git > a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c > b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c > index cfd7a2b..a75caff 100644 > --- > a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c > +++ > b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c > @@ -3,7 +3,7 @@ >=20 > Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.
> (C) Copyright 2014 Hewlett-Packard Development Company, L.P.
> - (C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP
> + (C) Copyright 2015-2019 Hewlett Packard Enterprise Development LP
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -1169,6 +1169,13 @@ SmbiosPrintStructure ( > break; >=20 > // > + // Processor Additional Information (Type 44) // case 44: > + DisplayProcessorArchitectureType (Struct->Type44- > >ProcessorSpecificBlock.ProcessorArchType, Option); > + break; > + > + // > // Inactive (Type 126) > // > case 126: > @@ -2350,6 +2357,18 @@ DisplayProcessorFamily2 ( > Print (L"Video Processor\n"); > break; >=20 > + 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/QueryTable. > c > b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable. > c > index bfb7c3b..87f446e 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. >=20 > Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.
> - (C) Copyright 2016-2017 Hewlett Packard Enterprise Development LP
> + (C) Copyright 2016-2019 Hewlett Packard Enterprise Development LP
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -620,6 +620,10 @@ TABLE_ITEM ProcessorCharacteristicsTable[] =3D { > { > 7, > L" Power/Performance Control" > + }, > + { > + 8, > + L" 128-bit Capable" > } > }; >=20 > @@ -3268,6 +3272,44 @@ TABLE_ITEM MCHostInterfaceTypeTable[] =3D { > }, > }; >=20 > +TABLE_ITEM ProcessorArchitectureTypesTable[] =3D { > + { > + 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) " > + } > +}; >=20 > TABLE_ITEM StructureTypeInfoTable[] =3D { > { > @@ -3447,6 +3489,10 @@ TABLE_ITEM StructureTypeInfoTable[] =3D { > L" TPM Device" > }, > { > + 44, > + L" Processor Additional Information" > + }, > + { > 0x7E, > L" Inactive" > }, > @@ -4697,6 +4743,23 @@ DisplayMCHostInterfaceType ( } >=20 > /** > + Display Processor Architecture Type (Type 44). > + > + @param[in] Key The key of the structure. > + @param[in] Option The optional information. > +**/ > +VOID > +DisplayProcessorArchitectureType ( > + IN UINT8 Key, > + IN UINT8 Option > + ) > +{ > + ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN > +(STR_SMBIOSVIEW_QUERYTABLE_PROCESSOR_ARCH_TYPE), > +gShellDebug1HiiHandle); Refer to the CSS spec 5.2.2.6, "Always put space before an open parenthesis= ". Please change to ShellPrintHiiEx (...); > + PRINT_INFO_OPTION (Key, Option); > + PRINT_TABLE_ITEM (ProcessorArchitectureTypesTable, Key); } > + > +/** > Display the structure type information. >=20 > @param[in] Key The key of the structure. > diff --git > a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h > b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h > index a8c32ab..77a86fc 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. >=20 > Copyright (c) 2005 - 2015, Intel Corporation. All rights reserved.
> - (C) Copyright 2017 Hewlett Packard Enterprise Development LP
> + (C) Copyright 2017 - 2019 Hewlett Packard Enterprise Development > + LP
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -427,4 +427,15 @@ DisplayTpmDeviceCharacteristics ( > IN UINT8 Option > ); >=20 > +/** > + Display Processor Architecture Type (Type 44). > + > + @param[in] Key The key of the structure. > + @param[in] Option The optional information. Here it is not force, but better to align the comment. There is no need to send another version patch. Fix them before push. With the mentioned addressed, Reviewed-by: Zhichao Gao Thanks, Zhichao > +**/ > +VOID > +DisplayProcessorArchitectureType ( > + IN UINT8 Key, > + IN UINT8 Option > + ); > #endif > diff --git > a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView > Strings.uni > b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView > Strings.uni > index 94aff91..7fbcc9a 100644 > --- > a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView > Strings.uni > +++ > b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView > S > +++ trings.uni > @@ -2,7 +2,7 @@ > // > // Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved. // (C) > Copyright 2014-2015 Hewlett-Packard Development Company, L.P.
-// > (C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP
> +// (C) Copyright 2015-2019 Hewlett Packard Enterprise Development > +LP
> // SPDX-License-Identifier: BSD-2-Clause-Patent // // Module Name: > @@ -447,6 +447,7 @@ > #string STR_SMBIOSVIEW_QUERYTABLE_BMC_INTERFACE_TYPE > #language en-US "BMC Interface Type:" > #string STR_SMBIOSVIEW_QUERYTABLE_MC_HOST_INTERFACE_TYPE > #language en-US "MC Host Interface Type:" > #string STR_SMBIOSVIEW_QUERYTABLE_STRUCT_TYPE #languag= e > en-US "Structure Type:" > +#string STR_SMBIOSVIEW_QUERYTABLE_PROCESSOR_ARCH_TYPE > #language en-US "Processor Architecture Type:" > #string STR_SMBIOSVIEW_SMBIOSVIEW_ONE_VAR_ARGV > #language en-US "%s " > #string STR_SMBIOSVIEW_SMBIOSVIEW_QUERY_STRUCT_COND > #language en-US "Query Structure, conditions are:\r\n" > #string STR_SMBIOSVIEW_SMBIOSVIEW_QUERYTYPE_RANDOM > #language en-US "QueryType =3D Random \r\n" > diff --git > a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.uni > b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.uni > index 42a9295..6693be2 100644 > --- > a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.uni > +++ > b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > d > +++ sLib.uni > @@ -2,7 +2,7 @@ > // > // Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved. // (C) > Copyright 2013-2015 Hewlett-Packard Development Company, L.P.
-// > (C) Copyright 2016 Hewlett Packard Enterprise Development LP
> +// (C) Copyright 2016 - 2019 Hewlett Packard Enterprise Development > +LP
> // SPDX-License-Identifier: BSD-2-Clause-Patent // // Module Name: > @@ -976,6 +976,7 @@ > " 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