From: Ben Cheatham <Benjamin.Cheatham@amd.com>
To: <nvdimm@lists.linux.dev>, <alison.schofield@intel.com>,
<dave.jiang@intel.com>
Cc: <linux-cxl@vger.kernel.org>, <benjamin.cheatham@amd.com>
Subject: [ndctl PATCH v7 0/7] Add error injection support
Date: Thu, 22 Jan 2026 14:37:21 -0600 [thread overview]
Message-ID: <20260122203728.622-1-Benjamin.Cheatham@amd.com> (raw)
v7 Changes:
- Use PATH_MAX instead of strlen(ctx->cxl_debugfs) for debugfs paths (Dave)
- Add goto for error paths to memdev poison functions in patch 3/7 (Dave)
- Add Dave's Reviewed-by tags
v6 Changes:
- Rebase to pending branch (Alison)
- Drop const for ctx->debugfs (Alison)
- Rename get_debugfs_dir() to get_cxl_debugfs_dir() and return cxl directory in debugfs
(i.e. "/sys/kernel/debug" -> "/sys/kernel/debug/cxl")
- Rename ctx->debugfs to ctx->cxl_debugfs
- Fix missing free of einj path (Alison)
- Add protocol errors in order to perrors list (Alison)
- Use hex constants instead of BIT() for protocol errors (Alison)
- Add symbols to LIBCXL_11 instead of LIBCXL_10 (Alison)
- Update commit message to reflect util_cxl_dport_filter() behavior (Alison)
- Remove EINJ_TYPES_BUF_SIZE #ifdef (Alison)
- Fix type mismatch of addr in poison_action() (Alison)
- Fix inject_action() to catch missing 'type' option (Alison)
- Remove '-N' option and show the information behind that option by default when
CXL debugfs is present (Alison)
- Add 'protocol_injectable' attribute for dports (Alison)
- Update inject-error man page with port injection example (Alison)
- Add warning to inject-error man page (Alison)
v5 Changes:
- Use setmntent()/getmntent() instead of open-coding getting the
debugfs path (Dave)
- Use correct return code for sysfs_read_attr() (Dave)
v4 Changes:
- Variable renames for clarity (Dave)
- Use errno instead of rc for access() calls (Dave)
- Check returns for snprintf() (Dave)
- Add util_cxl_dport_filter() (Dave)
- Replace printf() calls with log_info() (Dave)
- Write correct value to debugfs during protocol error injection
(BIT(error) vs. error)
v3 Changes:
- Rebase on v83 release
- Fix whitespace errors (Alison)
This series adds support for injecting CXL protocol (CXL.cache/mem)
errors[1] into CXL RCH Downstream ports and VH root ports[2] and
poison into CXL memory devices through the CXL debugfs. Errors are
injected using a new 'inject-error' command. Device poison can be
cleared using the 'clear-error' command. The 'inject-error' and
'clear-error' commands require access to the CXL driver's debugfs.
The documentation for the new cxl-inject-error command shows both usage
and the possible device/error types, as well as how to retrieve them
using cxl-list. cxl-list has been updated to include the possible error
types for protocol error injection (under the "bus" object) and which CXL
dports and memory devices support injection.
[1]: ACPI v6.5 spec, section 18.6.4
[2]: ACPI v6.5 spec, table 18.31
Ben Cheatham (7):
libcxl: Add debugfs path to CXL context
libcxl: Add CXL protocol errors
libcxl: Add poison injection support
cxl: Add inject-error command
cxl: Add clear-error command
cxl/list: Add injectable errors in output
Documentation: Add docs for inject/clear-error commands
Documentation/cxl/cxl-clear-error.txt | 69 ++++++
Documentation/cxl/cxl-inject-error.txt | 161 ++++++++++++
Documentation/cxl/meson.build | 2 +
cxl/builtin.h | 2 +
cxl/cxl.c | 2 +
cxl/filter.c | 26 ++
cxl/filter.h | 2 +
cxl/inject-error.c | 248 +++++++++++++++++++
cxl/json.c | 38 +++
cxl/lib/libcxl.c | 327 +++++++++++++++++++++++++
cxl/lib/libcxl.sym | 10 +
cxl/lib/private.h | 14 ++
cxl/libcxl.h | 18 ++
cxl/meson.build | 1 +
14 files changed, 920 insertions(+)
create mode 100644 Documentation/cxl/cxl-clear-error.txt
create mode 100644 Documentation/cxl/cxl-inject-error.txt
create mode 100644 cxl/inject-error.c
--
2.52.0
next reply other threads:[~2026-01-22 20:37 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-22 20:37 Ben Cheatham [this message]
2026-01-22 20:37 ` [PATCH 1/7] libcxl: Add debugfs path to CXL context Ben Cheatham
2026-01-22 20:37 ` [PATCH 2/7] libcxl: Add CXL protocol errors Ben Cheatham
2026-01-22 20:37 ` [PATCH 3/7] libcxl: Add poison injection support Ben Cheatham
2026-01-22 20:37 ` [PATCH 4/7] cxl: Add inject-error command Ben Cheatham
2026-01-22 20:37 ` [PATCH 5/7] cxl: Add clear-error command Ben Cheatham
2026-01-22 20:37 ` [PATCH 6/7] cxl/list: Add injectable errors in output Ben Cheatham
2026-01-22 20:37 ` [PATCH 7/7] Documentation: Add docs for inject/clear-error commands Ben Cheatham
2026-01-29 19:45 ` Verma, Vishal L
2026-01-30 19:59 ` Cheatham, Benjamin
2026-01-30 20:58 ` Verma, Vishal L
2026-01-30 21:45 ` Cheatham, Benjamin
2026-02-03 1:31 ` Alison Schofield
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260122203728.622-1-Benjamin.Cheatham@amd.com \
--to=benjamin.cheatham@amd.com \
--cc=alison.schofield@intel.com \
--cc=dave.jiang@intel.com \
--cc=linux-cxl@vger.kernel.org \
--cc=nvdimm@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.