ACK: [PATCH 1/1] dmicheck: Type 4 Processor Characteristics bit 9 is no longer reserved

Alex Hung alex.hung at canonical.com
Tue Jan 12 00:06:19 UTC 2021


On 2021-01-11 4:22 p.m., Rebecca Cran wrote:
> SMBIOS 3.4.0 updates the Type 4 Processor Characteristics field
> so that bit 9 is the "Arm64 SoC ID".
> 
> Update dmicheck.c to avoid flagging use of that bit as an error.
> 
> Signed-off-by: Rebecca Cran <rebecca at nuviainc.com>
> ---
>  src/dmi/dmicheck/dmicheck.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/dmi/dmicheck/dmicheck.c b/src/dmi/dmicheck/dmicheck.c
> index 86d644574b62..5fa8ae018bc7 100644
> --- a/src/dmi/dmicheck/dmicheck.c
> +++ b/src/dmi/dmicheck/dmicheck.c
> @@ -1365,7 +1365,7 @@ static void dmicheck_entry(fwts_framework *fw,
>  			dmi_str_check(fw, table, addr, "Part Number", hdr, 0x22);
>  			if (hdr->length < 0x28)
>  				break;
> -			dmi_reserved_bits_check(fw, table, addr, "Processor Characteristics", hdr, sizeof(uint16_t), 0x26, 9, 15);
> +			dmi_reserved_bits_check(fw, table, addr, "Processor Characteristics", hdr, sizeof(uint16_t), 0x26, 10, 15);
>  			if (hdr->length < 0x30)
>  				break;
>  			dmi_min_max_uint16_check(fw, table, addr, "Core Count 2", hdr, 0x2a, 0, 0xfffe);
> 

Thanks for the fix.

Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list