Re: [PATCH v5 08/27] cxl/mem: Read dynamic capacity configuration from the device

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Davidlohr Bueso wrote:
> On Tue, 29 Oct 2024, ira.weiny@xxxxxxxxx wrote:
> 
> >Linux has no use for the trailing fields of the Get Dynamic Capacity
> >Configuration Output Payload (Total number of supported extents, number
> >of available extents, total number of supported tags, and number of
> >available tags).  Avoid defining those fields to use the more useful
> >dynamic C array.
> 
> More of a general question, if anyone knows: why does the spec define
> the fields in that order, and not put the region config structure(s)
> at the end, as with all other cmds, afaik.

Because the 3.0 spec only left 7 bytes of reserved space before the region
config structure array.  One might argue that no one would ever implement
the 3.0 spec for DCD but at the time it was seen as an unbreakable
structure.

While it is not impossible to utilize those fields they are not required
right now.  So in this implementation it was better to use the dynamic C
array's and leave the use of those fields for another day.

Ira




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux