ACK: [PATCH] dmi: dmicheck: fix reserved bits mask for Processor Information
Alex Hung
alex.hung at canonical.com
Thu Jan 7 03:52:24 UTC 2016
On 2016-01-07 01:00 AM, Fan Wu wrote:
> From: Fan Wu <wufan at codeaurora.org>
> Date: Tue, 5 Jan 2016 18:52:54 -0700
> Subject: [PATCH] dmi: dmicheck: fix reserved bits mask for Processor
> Information
>
> This patch addresses following false alarm in FWTS dmicheck test:
>
> dmicheck: Reserved bits 0x009c was usedbits 8..15 would be reserved
> while accessing entry 'Processor Information (Type 4)' @ 0xf97f01d9,
> field 'Processor Characteristics', offset 0x26
>
> Signed-off-by: Fan Wu <wufan at codeaurora.org>
> ---
> 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 eeb227a..74805d2 100644
> --- a/src/dmi/dmicheck/dmicheck.c
> +++ b/src/dmi/dmicheck/dmicheck.c
> @@ -1171,7 +1171,7 @@ static void dmicheck_entry(fwts_framework *fw,
> dmi_str_check(fw, table, addr, "Part Number", hdr,
> 0x22);
> if (hdr->length < 0x28)
> break;
> - if (GET_UINT16(data + 0x26) & 0xf0)
> + if (GET_UINT16(data + 0x26) & 0xff00)
> fwts_failed(fw, LOG_LEVEL_MEDIUM,
> DMI_RESERVED_VALUE_USED,
> "Reserved bits 0x%4.4" PRIx16 " was
> used"
> "bits 8..15 would be reserved while
> accessing entry '%s' @ "
>
Acked-by: Alex Hung <alex.hung at canonical.com>
More information about the fwts-devel
mailing list