From: Gregory Price <gourry@gourry.net>
To: Alison Schofield <alison.schofield@intel.com>
Cc: dan.j.williams@intel.com, Davidlohr Bueso <dave@stgolabs.net>,
Jonathan Cameron <jonathan.cameron@huawei.com>,
Dave Jiang <dave.jiang@intel.com>,
Vishal Verma <vishal.l.verma@intel.com>,
Ira Weiny <ira.weiny@intel.com>,
linux-cxl@vger.kernel.org
Subject: Re: [PATCH 2/2] cxl/region: Unregister auto-created region when assembly fails
Date: Wed, 4 Feb 2026 23:22:34 -0500 [thread overview]
Message-ID: <aYQbCsOiFzyZHI6R@gourry-fedora-PF4VCD3F> (raw)
In-Reply-To: <aYPlJ2PRv-FaFKFT@aschofie-mobl2.lan>
On Wed, Feb 04, 2026 at 04:32:39PM -0800, Alison Schofield wrote:
> On Sat, Jan 31, 2026 at 10:49:11AM -0500, Gregory Price wrote:
> > i.e. auto-decoders should be intended for trivial SysRAM regions only
>
> It has not been my impression that we have that much control over what
> BIOS may present. ie. If it is CXL Spec legal they may build it. I
> recall you wrote a doc of Linux Expectations of BIOS. Did you actually
> try to limit what BIOS does?
>
>
I should rephrase - barring any new specification updates that actually
allow the BIOS to say what a region may be used for, the only two
reasonable endpoints are sysram and dax - and realistically only sysram
unless you build kmem out of the dax driver (kmem is the auto
destination).
But for non-trivial setups, recovery might not actually get you
anything. If a device fails to actually come up after having been
programmed by bios, it may not even be feasible to tear-down and recreate
what the BIOS tried to do because of platform specifics (Zen5).
So while you can time out and clean up, I'm not sure you can actually do
anything after that reliably in the general case - and i'm not sure how
much effort we should put into fighting those fires.
I suppose you could leave the driver in a state that allows a user to
see it's broken.
~Gregory
next prev parent reply other threads:[~2026-02-05 4:22 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-30 4:23 [PATCH 1/2] cxl/region: Timeout auto region assembly waiting for endpoints Alison Schofield
2026-01-30 4:23 ` [PATCH 2/2] cxl/region: Unregister auto-created region when assembly fails Alison Schofield
2026-01-30 17:45 ` dan.j.williams
2026-01-31 1:04 ` Alison Schofield
2026-01-31 15:49 ` Gregory Price
2026-02-05 0:32 ` Alison Schofield
2026-02-05 4:22 ` Gregory Price [this message]
2026-02-03 3:07 ` dan.j.williams
2026-02-05 0:20 ` Alison Schofield
2026-02-05 1:03 ` dan.j.williams
2026-01-30 4:58 ` [PATCH 1/2] cxl/region: Timeout auto region assembly waiting for endpoints dan.j.williams
2026-01-30 17:42 ` Gregory Price
2026-01-30 18:26 ` dan.j.williams
2026-01-30 19:03 ` Gregory Price
2026-01-30 22:46 ` dan.j.williams
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aYQbCsOiFzyZHI6R@gourry-fedora-PF4VCD3F \
--to=gourry@gourry.net \
--cc=alison.schofield@intel.com \
--cc=dan.j.williams@intel.com \
--cc=dave.jiang@intel.com \
--cc=dave@stgolabs.net \
--cc=ira.weiny@intel.com \
--cc=jonathan.cameron@huawei.com \
--cc=linux-cxl@vger.kernel.org \
--cc=vishal.l.verma@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox