Re: [PATCH v16 04/22] cxl: Move register/capability check to driver

[Dan Williams <dan.j.williams@intel.com>]

Alejandro Lucero Palau wrote:
[..]
> You did not like to add a new capability field to current structs 
> because the information needed was already there in the map. I think it 
> was a fair comment, so the caps, a variable the caller gives, is set 
> during the discovery without any internal struct added.

The objection was not limited to data structure changes, it also
includes sprinkling an @caps argument throughout the stack for an
as yet to be determined benefit.

> Regarding what you suggest below, I have to disagree. This change was 
> introduced for dealing with a driver using CXL, that is a Type2 or 
> future Type1 driver. IMO, most of the innerworkings should be hidden to 
> those clients and therefore working with the map struct is far from 
> ideal, and it is not currently accessible from those drivers.

Checking a couple validity flags in a now public (in include/cxl/pci.h)
data-structure is far from ideal?

> With these new drivers the core does not know what should be there, so
> the check is delayed and left to the driver.

Correct, left to the driver to read from an existing mechanism.

> IMO, from a Type2/Type1 driver perspective, it is better to deal with
> caps expected/found than being aware of those internal CXL register
> discovery and maps.

Not if a maintainer of the CXL register discovery and maps remains
unconvinced to merge a parallel redundant mechanism to achieve the exact
same goal.