Re: [NDCTL PATCH 0/5] cxl: Add tests for extended linear cache support

Alison Schofield
2025-11-15T01:20:51Z

On Fri, Oct 31, 2025 at 10:39:58AM -0700, Dave Jiang wrote:
> The series adds unit tests to verify the kernel support for extended
> linear cache (ELC). Added a test to check if the ELC region is setup
> correctly, and another test to go through the poison handling flow
> via the poison injection testing.
>
>
> Dave Jiang (5):
> cxl/test: Add test for extended linear cache support
> cxl/test: Fix cxl-poison.sh to detect the correct elc sysfs attrib
> cxl/test: Move cxl-poison.sh to use cxl_test auto region

The next 2 patches may add more complexity than is needed by splitting
the work among a common file and 2 test scripts. Consider keeping the
work in one file and wrapping the test execution in a 'run_poison_tests'
function. Then call it that after each load of the module, like -

run_poison_tests
.
.
.
modprobe -r cxl_test
modprobe cxl_test
run_poison_tests
modprobe -r cxl_test

modprobe cxl_test extended_linear_cache=1
run_poison_tests
modprobe -r cxl_test

> cxl/test: Move common part of poison unit test to common file
> cxl/test: Add support for poison test for ELC
>
> test/common-poison | 202 ++++++++++++++++++++++++++++++++++++++++
> test/cxl-elc.sh | 89 ++++++++++++++++++
> test/cxl-poison-elc.sh | 41 +++++++++
> test/cxl-poison.sh | 203 +----------------------------------------
> test/meson.build | 4 +
> 5 files changed, 338 insertions(+), 201 deletions(-)
> create mode 100644 test/common-poison
> create mode 100755 test/cxl-elc.sh
> create mode 100755 test/cxl-poison-elc.sh
>
>
> base-commit: 01c90830d65b6b331986f5996dcf6ad73c1579f4
> --
> 2.51.0
>