On Sat, Mar 16, 2024 at 08:03:34AM +0900, Wonjae Lee wrote: > alison.schofi...@intel.com wrote: > > From: Alison Schofield <alison.schofi...@intel.com> > > > > Exercise cxl list, libcxl, and driver pieces of the get poison list > > pathway. Inject and clear poison using debugfs and use cxl-cli to > > read the poison list by memdev and by region. > > > > Signed-off-by: Alison Schofield <alison.schofi...@intel.com> > > --- > > test/cxl-poison.sh 137 +++++++++++++++++++++++++++++++++++++++++++++ > > test/meson.build 2 + > > 2 files changed, 139 insertions(+) > > create mode 100644 test/cxl-poison.sh > > > > diff --git a/test/cxl-poison.sh b/test/cxl-poison.sh > > new file mode 100644 > > index 000000000000..af2e9dcd1a11 > > --- /dev/null > > +++ b/test/cxl-poison.sh > > [snip] > > > +# Turn tracing on. Note that 'cxl list --poison' does toggle the tracing. > > Hi, > > I know it's trivial and not sure if I'm understanding the history of > the patch series correctly, but --poison seems to be an option that > was suggested before --media-errors. I'm wondering if it's okay to > leave this comment.
Thanks Wonjae - I appreciate your find. I'll fix it up. Alison > > Thanks, > Wonjae