From: Alison Schofield <alison.schofield@intel.com>
To: nvdimm@lists.linux.dev, linux-cxl@vger.kernel.org
Cc: Alison Schofield <alison.schofield@intel.com>
Subject: [ndctl PATCH 0/7] cxl/region: create-region command and unit
Date: Mon, 28 Sep 2026 22:36:52 -0700 [thread overview]
Message-ID: <cover.1790658549.git.alison.schofield@intel.com> (raw)
While working to enhance the cxl-cli cxl-create-region command to handle
out-of-order target lists or no user-supplied target list, I ran into some
basic issues in the existing command that need to be addressed first.
Patches 1-2 consolidate and clean up the existing create-region tests.
Patches 3-6 fix target-list parsing and validation, release DPA after a
failed create, and move granularity validation ahead of region creation.
Each fix adds a test case for the problem it addresses.
Patch 7 corrects the documentation to reflect that --decoder is currently
required. Automatic root decoder selection will be considered separately
with the upcoming create-region usability work.
Developing tests for these fixes exposed two bugs and a topology gap in
the cxl_test mock module. A companion cxl_test series is here [1].
[1] https://lore.kernel.org/linux-cxl/cover.1790641236.git.alison.schofield@intel.com/
Alison Schofield (7):
test/cxl-create-region.sh: fold in XOR coverage
test/cxl-create-region.sh: deduplicate topology lookups
cxl/region: parse comma-separated target lists
cxl/region: reject duplicate or unknown target memdevs
cxl/region: release reserved DPA when region creation fails
cxl/region: validate granularity before region creation
cxl/region: document --decoder as required
Documentation/cxl/cxl-create-region.txt | 7 +-
cxl/region.c | 139 ++++++++--
test/cxl-create-region.sh | 338 ++++++++++++++++++++++--
test/cxl-xor-region.sh | 129 ---------
test/meson.build | 2 -
5 files changed, 442 insertions(+), 173 deletions(-)
delete mode 100644 test/cxl-xor-region.sh
--
2.37.3
next reply other threads:[~2026-09-29 5:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-29 5:36 Alison Schofield [this message]
2026-09-29 5:36 ` [ndctl PATCH 1/7] test/cxl-create-region.sh: fold in XOR coverage Alison Schofield
2026-09-29 5:36 ` [ndctl PATCH 2/7] test/cxl-create-region.sh: deduplicate topology lookups Alison Schofield
2026-09-29 5:36 ` [ndctl PATCH 3/7] cxl/region: parse comma-separated target lists Alison Schofield
2026-09-29 5:36 ` [ndctl PATCH 4/7] cxl/region: reject duplicate or unknown target memdevs Alison Schofield
2026-09-29 5:36 ` [ndctl PATCH 5/7] cxl/region: release reserved DPA when region creation fails Alison Schofield
2026-09-29 5:36 ` [ndctl PATCH 6/7] cxl/region: validate granularity before region creation Alison Schofield
2026-09-29 5:36 ` [ndctl PATCH 7/7] cxl/region: document --decoder as required Alison Schofield
2026-09-29 21:39 ` [ndctl PATCH 0/7] cxl/region: create-region command and unit Alison Schofield
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=cover.1790658549.git.alison.schofield@intel.com \
--to=alison.schofield@intel.com \
--cc=linux-cxl@vger.kernel.org \
--cc=nvdimm@lists.linux.dev \
/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