From: Jehoon Park <jehoon.park@samsung.com>
To: linux-cxl@vger.kernel.org
Cc: dan.j.williams@intel.com, vishal.l.verma@intel.com,
ira.weiny@intel.com, alison.schofield@intel.com,
bwidawsk@kernel.org, jehoon park <jehoon.park@samsung.com>
Subject: [ndctl patch RFC 0/2] add support for IDENTIFY command
Date: Tue, 7 Mar 2023 17:21:00 +0900 [thread overview]
Message-ID: <20230307082102.27195-1-jehoon.park@samsung.com> (raw)
In-Reply-To: CGME20230307081917epcas2p1accc8f7bf3f31e08525684abb2efa788@epcas2p1.samsung.com
From: jehoon park <jehoon.park@samsung.com>
This patchset supports CXL IDENTIFY mailbox command and corresponding
cxl tool interface command.
CXL 3.0 Spec 8.2.9.8.1 defines IDENTIFY command which retrieves basic
information about CXL memory device. The information consist of device's
firmware version, capacity, LSA size, event log size, poison list size,
inject poison limit, poison handling capabilities and QoS telemetry
capabilities. Firmware version, capacity and LSA size are already supported
and used for partition commands or sysfs attributes while others are not.
Since patches about event log [1] and poison [2] are discussed recently,
support for those information will be helpful.
Example:
# cxl identify mem0
FW Revision : BWFW VERSION 00
Total Capacity : 1.00 GB
Volatile Only Capacity : 1.00 GB
Persistent Only Capacity : 0 B
Partition Alignment : 0 B
Informational Event Log Size : 0
Warning Event Log Size : 0
Failure Event Log Size : 0
Fatal Event Log Size : 0
LSA Size : 0 B
Poison List Maximum Media Error Records : 256
Inject Poison Limit : 0
Poison Handling Capabilities
Injects Persistent Poison : Not Supported
Scans for Poison : Not Supported
QoS Telemetry Capabilities
Egress Port Congestion : Not Supported
Temporary Throughput Reduction : Not Supported
cxl memdev: cmd_identify: identified 1 mem
This patch is RFC because some of the information are already provided by
"list -m <memdev>" command and sysfs attributes.
I think a separate cxl tool interface command for identify is useful to provide
device's information clearly. In case of nvme-cli, there are separate
interface commands for identifying controller and namespace: id-ctrl and id-ns.
[1] https://lore.kernel.org/linux-cxl/20221216-cxl-ev-log-v7-0-2316a5c8f7d8@intel.com/
[2] https://lore.kernel.org/linux-cxl/cover.1676685180.git.alison.schofield@intel.com/
jehoon park (2):
libcxl: add accessors for IDENTIFY command
cxl: add identify command to cxl tool
Documentation/cxl/cxl-identify.txt | 57 ++++++++++++
Documentation/cxl/meson.build | 1 +
cxl/builtin.h | 1 +
cxl/cxl.c | 1 +
cxl/lib/libcxl.c | 73 +++++++++++++++
cxl/lib/libcxl.sym | 11 +++
cxl/lib/private.h | 11 +++
cxl/libcxl.h | 16 ++++
cxl/memdev.c | 141 +++++++++++++++++++++++++++++
9 files changed, 312 insertions(+)
create mode 100644 Documentation/cxl/cxl-identify.txt
base-commit: b830c4af984e72e5849c0705669aad2ffa19db13
--
2.17.1
next parent reply other threads:[~2023-03-07 8:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20230307081917epcas2p1accc8f7bf3f31e08525684abb2efa788@epcas2p1.samsung.com>
2023-03-07 8:21 ` Jehoon Park [this message]
2023-03-07 8:21 ` [ndctl patch RFC 1/2] libcxl: add accessors for IDENTIFY command Jehoon Park
2023-03-07 8:21 ` [ndctl patch RFC 2/2] cxl: add identify command to cxl tool Jehoon Park
2023-03-07 20:18 ` [ndctl patch RFC 0/2] add support for IDENTIFY command Alison Schofield
2023-03-08 9:01 ` Jehoon Park
2023-03-08 18:22 ` Ira Weiny
2023-03-08 21:58 ` Alison Schofield
2023-03-10 6:54 ` Jehoon Park
2023-03-10 9:45 ` Dan Williams
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=20230307082102.27195-1-jehoon.park@samsung.com \
--to=jehoon.park@samsung.com \
--cc=alison.schofield@intel.com \
--cc=bwidawsk@kernel.org \
--cc=dan.j.williams@intel.com \
--cc=ira.weiny@intel.com \
--cc=linux-cxl@vger.kernel.org \
--cc=vishal.l.verma@intel.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox