Re: [PATCH v9 05/13] cxl/region: Separate region parameter setup and region construction

From: Alison Schofield

Date: Tue Jan 13 2026 - 22:17:20 EST


On Sat, Jan 10, 2026 at 12:46:50PM +0100, Robert Richter wrote:
> To construct a region, the region parameters such as address range and
> interleaving config need to be determined. This is done while
> constructing the region by inspecting the endpoint decoder
> configuration. The endpoint decoder is passed as a function argument.
>
> With address translation the endpoint decoder data is no longer
> sufficient to extract the region parameters as some of the information
> is obtained using other methods such as using firmware calls.
>
> In a first step, separate code to determine the region parameters from
> the region construction. Temporarily store all the data to create the
> region in the new struct cxl_region_context. Once the region data is
> determined and struct cxl_region_context is filled, construct the
> region.
>
> Patch is a prerequisite to implement address translation. The code
> separation helps to later extend it to determine region parameters
> using other methods as needed, esp. to support address translation.
>

Reviewed-by: Alison Schofield <alison.schofield@xxxxxxxxx>