Re: [PATCH v6 2/3] cxl/acpi: Support CXL XOR Interleave Math (CXIMS)

[Alison Schofield <alison.schofield@intel.com>]

On Mon, Nov 14, 2022 at 10:57:29AM -0800, Davidlohr Bueso wrote:
> On Mon, 14 Nov 2022, Alison Schofield wrote:
> 
> > The reasoning behind removing the check, versus expanding it, is that the
> > check shouldn't have been there in the first place. The ACPI driver is
> > responsible for parsing the CFMWS's but it is the responsibility of the
> > region driver to decide which CFMWS's it can use.
> > 
> > If you believe that premise, then we let all interleave arithmetics
> > 'get through' to the region driver.
> 
> The way I see it, acpi should be setting the callback because it is a _direct_
> consequence of parsing the cfmws, and that's really the same logic as we have
> now, in that it sets it when allocating the rp decoder. And that doesn't take
> away from the region driver responsibility of choosing the actual host bridge.

With a little help from my friends ;) I'm hopeful the next version
meets your expectation by setting the calc_hb only once when allocating
that root port decoder.

> 
> Thanks,
> Davidlohr