Re: [edk2-devel] [PATCH] MdeModulePkg SmbiosMeasurementDxe: Add Type4 CurrentSpeed to filter table

["Yao, Jiewen" <jiewen.yao@intel.com>]

Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Heng
> Luo
> Sent: Thursday, September 8, 2022 1:41 PM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; Bi,
> Dandan <dandan.bi@intel.com>; Zeng, Star <star.zeng@intel.com>; Gao,
> Zhichao <zhichao.gao@intel.com>
> Subject: [edk2-devel] [PATCH] MdeModulePkg SmbiosMeasurementDxe:
> Add Type4 CurrentSpeed to filter table
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4051
> 
> The Type4 CurrentSpeed field may be various.
> So this patch adds it into the filter table.
> 
> Signed-off-by: Heng Luo <heng.luo@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> ---
> 
> MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurement
> Dxe.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git
> a/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasuremen
> tDxe.c
> b/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasureme
> ntDxe.c
> index 348082ff86..d61edc846b 100644
> ---
> a/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasuremen
> tDxe.c
> +++
> b/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasureme
> ntDxe.c
> @@ -1,7 +1,7 @@
>  /** @file
>    This driver measures SMBIOS table to TPM.
> 
> -Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2015 - 2022, Intel Corporation. All rights reserved.<BR>
>  SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>  **/
> @@ -66,6 +66,7 @@ SMBIOS_FILTER_TABLE  mSmbiosFilterType4BlackList[]
> = {
>    { 0x04, OFFSET_OF (SMBIOS_TABLE_TYPE4, EnabledCoreCount2),
> FIELD_SIZE_OF (SMBIOS_TABLE_TYPE4, EnabledCoreCount2),
> 0                                  },
>    { 0x04, OFFSET_OF (SMBIOS_TABLE_TYPE4, ThreadCount2),
> FIELD_SIZE_OF (SMBIOS_TABLE_TYPE4, ThreadCount2),
> 0                                  },
>    { 0x04, OFFSET_OF (SMBIOS_TABLE_TYPE4, Voltage),           FIELD_SIZE_OF
> (SMBIOS_TABLE_TYPE4, Voltage),           0                                  },
> +  { 0x04, OFFSET_OF (SMBIOS_TABLE_TYPE4, CurrentSpeed),
> FIELD_SIZE_OF (SMBIOS_TABLE_TYPE4, CurrentSpeed),
> 0                                  },
>  };
>  SMBIOS_FILTER_TABLE  mSmbiosFilterType17BlackList[] = {
>    { 0x11, OFFSET_OF (SMBIOS_TABLE_TYPE17, SerialNumber),
> FIELD_SIZE_OF (SMBIOS_TABLE_TYPE17, SerialNumber),
> SMBIOS_FILTER_TABLE_FLAG_IS_STRING },
> --
> 2.31.1.windows.1
> 
> 
> 
> 
>