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

[Alison Schofield <alison.schofield@intel.com>]

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
>