Skip to content

cxl/test: mock fixups and new topology support

Alison Schofield linux-cxl 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest3 patches 1 replyIn review
  2. v23 patches 5 repliesSuperseded
  3. v13 patches 3 repliesSuperseded

Changes in v3 from the cover letter

- Rebase onto 7.4-rc1

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3tools/testing/cxl: Preserve DPA skip across mock decoder resettools/testing/cxl/test/cxl.c+3 -60 replies
2/3tools/testing/cxl: Skip the auto region when no volatile window is activetools/testing/cxl/test/cxl.c+28 -80 replies
3/3tools/testing/cxl: Add CFMWS windows for region selection teststools/testing/cxl/test/cxl.c+56 -150 replies

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by.

PatchDave JiangRichard ChengJonathan Cameron
1/3R-b R-b
2/3R-b
3/3R-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v3 thread

Cover letter [PATCH v3 0/3] cxl/test: mock fixups and new topology support

Changes in v3:
- Rebase onto 7.4-rc1

Changes in v2:
- Patch 2: Avoid duplicate look up of auto region window (RichardC)
- CoverLetter: Add link to the ndctl companion series

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. Developing tests for those fixes exposed two bugs and a topology gap in the cxl_test mock module.

This cxl_test series addresses those issues:
1/3 fixes a DPA skip leak across mock decoder reset.
2/3 skips auto-region setup when no volatile window is active. 3/3 adds two x1 CFMWS windows needed to exercise root decoder selection.

A companion ndctl series[1] fixes the existing cxl-create-region issues in cxl-cli, clearing the way for out-of-order or automatically generated target lists.

These changes preserve the existing cxl_test unit test behavior while enabling new create-region test cases.

[1] https://lore.kernel.org/nvdimm/[email protected]/

Alison Schofield (3):
tools/testing/cxl: Preserve DPA skip across mock decoder reset tools/testing/cxl: Skip the auto region when no volatile window is active tools/testing/cxl: Add CFMWS windows for region selection tests

 tools/testing/cxl/test/cxl.c | 116 ++++++++++++++++++++++++++---------
 1 file changed, 87 insertions(+), 29 deletions(-)

base-commit: 6628e7d4c6b25c8d18f1345e3a20d21829aae7e4

Signature · 1 line
2.37.3