Re: [PATCH 1/4] Documentation/driver-api/cxl: Add a testing page

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

On Sat, Oct 17, 2026 at 07:52:29AM +0900, Anisa Su wrote:
> On Tue, Oct 13, 2026 at 09:17:52PM -0700, Alison Schofield wrote:
> > The CXL documentation does not describe the subsystem's testing
> > infrastructure or provide guidance on choosing between cxl_test
> > and KUnit.
> > 
> > Add a testing page that describes when to use each framework, how
> > to run the tests, and how to add a CXL KUnit test.
> > 
> > Suggested-by: Dave Jiang <dave.jiang@intel.com>
> > Signed-off-by: Alison Schofield <alison.schofield@intel.com>
> > ---
> >  Documentation/driver-api/cxl/index.rst   |   1 +
> >  Documentation/driver-api/cxl/testing.rst | 117 +++++++++++++++++++++++
> >  2 files changed, 118 insertions(+)

snip

> > +KUnit
> > +=====
> > +
> > +KUnit tests run self-contained logic in the kernel without any CXL topology.
> > +KUnit is suited to logic that can be exercised with controlled test inputs,
> > +including encode/decode helpers, address translation and interleave calculations,
> > +parsers of device- or firmware-supplied data, and mailbox command handling
> > +through a stub transport.
> > +
> > +To run all of the CXL KUnit tests, from the top of the kernel tree::
> > +
> > +  ./tools/testing/kunit/kunit.py run --kunitconfig=drivers/cxl
> > +
> > +This builds a test kernel in .kunit/ using drivers/cxl/.kunitconfig and runs it
> > +under User Mode Linux (UML). The running kernel is not modified, and no
> > +installation is required. The test kernel runs the tests, reports the results,
> > +and exits.
> > +
> > +Add --arch=x86_64 to run the tests under QEMU instead.
> > +
> Add --arch=<arch> ...
> Maybe also point at "--arch=help"
> 
> Reviewed-by: Anisa Su <anisa.su@samsung.com>

Thanks for the review Anisa.

I have this prep'd for v2:

-Add --arch=x86_64 to run the tests under QEMU instead.
+Use --arch=<arch> to run the tests under QEMU instead.
+Use --arch=help to list supported architectures.

-- Alison