[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aWV-H_YbSrmgAZYd@gourry-fedora-PF4VCD3F>
Date: Mon, 12 Jan 2026 18:05:03 -0500
From: Gregory Price <gourry@...rry.net>
To: "Cheatham, Benjamin" <benjamin.cheatham@....com>
Cc: linux-cxl@...r.kernel.org, linux-kernel@...r.kernel.org,
kernel-team@...a.com, dave@...olabs.net,
jonathan.cameron@...wei.com, dave.jiang@...el.com,
alison.schofield@...el.com, vishal.l.verma@...el.com,
ira.weiny@...el.com, dan.j.williams@...el.com
Subject: Re: [PATCH 4/6] cxl: add CONFIG_CXL_REGION_CTRL_AUTO_* build config
options
On Mon, Jan 12, 2026 at 03:10:55PM -0600, Cheatham, Benjamin wrote:
> On 1/12/2026 10:35 AM, Gregory Price wrote:
> > +choice
> > + prompt "CXL Region Auto Control Mode"
> > + depends on CXL_REGION
> > + default CXL_REGION_CTRL_AUTO_DAX
> > + help
> > + Select the default controller for CXL regions when ctrl mode is
> > + set to 'auto'. This determines how CXL memory regions are exposed
> > + to the system when no explicit control mode is specified.
> > +
> > +config CXL_REGION_CTRL_AUTO_DAX
>
> This should probably be renamed to CXL_REGION_CTRL_DAX since only DAX is mentioned.
>
> > + bool "DAX"
> > + help
> > + When a CXL region's control mode is 'auto', create a DAX region
> > + controller. This allows fine-grained control over the memory region
> > + through the DAX subsystem, and the region can later be converted to
> > + System RAM via daxctl.
> > +
> > + This is the default and recommended option for most use cases.
>
> If you remove the 'auto' mode earlier on, then you can just drop the first sentence here.
> I'd also add a note about when a DAX region can be failed to be created (i.e. BIOS already
> set up and onlined the memory).
>
I think I'm just going to drop this entirely, probably this was just too
ambitious trying to create an easy transition from dax to sysram for
auto regions.
The reality is BIOS-configured decoders "is NOT the way" (TM). If BIOS
configures it - it's DAX, otherwise the user gets a choice (or they can
tear it down and rebuild).
~Gregory
Powered by blists - more mailing lists