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/[email protected]/ 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
