[PATCH] dmicheck: fix handling of type 16 capacities > 2 TiB

[Elliott, Robert (Persistent Memory)]

> -----Original Message-----
> From: Alex Hung <alex.hung at canonical.com>
> Sent: Tuesday, October 02, 2018 7:33 PM
> To: Elliott, Robert (Persistent Memory) <elliott at hpe.com>
> Cc: fwts-devel <fwts-devel at lists.ubuntu.com>
> Subject: Re: [PATCH] dmicheck: fix handling of type 16 capacities > 2
> TiB
> 
> Hi Robert,
> 
> Thanks for the patch, and this seems to be similar to
> http://patchwork.ozlabs.org/patch/974876/.
> 
> Do you have comments on the other one, ex. yours is more complete?

I think they're functionally the same; that patch is simpler.

An if clause might better facilitate ensuring the structure length
includes the Extended Maximum Capacity field in the 0x8000000 case,
and verifying this rule from the conformance list in annex A:
"Either Maximum Capacity or Extended Maximum Capacity must be set 
to a known, non-zero value."