From: Gregory Price <gourry@gourry.net>
To: Dave Jiang <dave.jiang@intel.com>
Cc: linux-cxl@vger.kernel.org, dave@stgolabs.net,
jonathan.cameron@huawei.com, alison.schofield@intel.com,
vishal.l.verma@intel.com, ira.weiny@intel.com,
dan.j.williams@intel.com
Subject: Re: [PATCH v2 1/3] cxl: docs/platform/cdat reference documentation
Date: Wed, 14 May 2025 23:41:51 -0400 [thread overview]
Message-ID: <aCVifxz7t62uOfmn@gourry-fedora-PF4VCD3F> (raw)
In-Reply-To: <20250515000923.2590820-2-dave.jiang@intel.com>
On Wed, May 14, 2025 at 05:09:21PM -0700, Dave Jiang wrote:
> Add documentation for CDAT structures for CXL usages.
>
> Signed-off-by: Dave Jiang <dave.jiang@intel.com>
>
Reviewed-by: Gregory Price <gourry@gourry.net>
> ---
> v2:
> - Move everything into cdat.rst (Gregory)
> - Update various languages in the doc for clarification (Gregory)
> - Add a term definition section (Gregory)
> ---
> Documentation/driver-api/cxl/index.rst | 1 +
> .../driver-api/cxl/platform/cdat.rst | 118 ++++++++++++++++++
> 2 files changed, 119 insertions(+)
> create mode 100644 Documentation/driver-api/cxl/platform/cdat.rst
>
> diff --git a/Documentation/driver-api/cxl/index.rst b/Documentation/driver-api/cxl/index.rst
> index 366faf851fc7..9e1414ad3357 100644
> --- a/Documentation/driver-api/cxl/index.rst
> +++ b/Documentation/driver-api/cxl/index.rst
> @@ -27,6 +27,7 @@ that have impacts on each other. The docs here break up configurations steps.
>
> platform/bios-and-efi
> platform/acpi
> + platform/cdat
> platform/example-configs
>
> .. toctree::
> diff --git a/Documentation/driver-api/cxl/platform/cdat.rst b/Documentation/driver-api/cxl/platform/cdat.rst
> new file mode 100644
> index 000000000000..34bbe7264d71
> --- /dev/null
> +++ b/Documentation/driver-api/cxl/platform/cdat.rst
> @@ -0,0 +1,118 @@
> +.. SPDX-License-Identifier: GPL-2.0
> +
> +======================================
> +Coherent Device Attribute Table (CDAT)
> +======================================
> +
> +The CDAT provides functional and performance attributes of devices such
> +as CXL accelerators, switches, or endpoints. The table formatting is
> +similar to ACPI tables. CDAT data may be parsed by BIOS at boot or may
> +be enumerated at runtime (after device hotplug, for example).
> +
> +Terminology:
> +DPA - Device Physical Address, used by the CXL device to denote the address
> +it supports for that device.
> +
> +DSMADHandle - A device unique handle that is associated with a DPA range
> +defined by the DSMAS table.
> +
> +
> +===============================================
> +Device Scoped Memory Affinity Structure (DSMAS)
> +===============================================
> +
> +The DSMAS contains information such as DSMADHandle, the DPA Base, and DPA
> +Length.
> +
> +This table is used by Linux in conjunction with the Device Scoped Latency and
> +Bandwidth Information Structure (DSLBIS) to determine the performance
> +attributes of the CXL device itself.
> +
> +Example ::
> +
> + Structure Type : 00 [DSMAS]
> + Reserved : 00
> + Length : 0018 <- 24d, size of structure
> + DSMADHandle : 01
> + Flags : 00
> + Reserved : 0000
> + DPA Base : 0000000040000000 <- 1GiB base
> + DPA Length : 0000000080000000 <- 2GiB size
> +
> +
> +==================================================================
> +Device Scoped Latency and Bandwidth Information Structure (DSLBIS)
> +==================================================================
> +
> +This table is used by Linux in conjunction with DSMAS to determine the
> +performance attributes of a CXL device. The DSLBIS contains latency
> +and bandwidth information based on DSMADHandle matching.
> +
> +Example ::
> +
> + Structure Type : 01 [DSLBIS]
> + Reserved : 00
> + Length : 18 <- 24d, size of structure
> + Handle : 0001 <- DSMAS handle
> + Flags : 00 <- Matches flag field for HMAT SLLBIS
> + Data Type : 00 <- Latency
> + Entry Basee Unit : 0000000000001000 <- Entry Base Unit field in HMAT SSLBIS
> + Entry : 010000000000 <- First byte used here, CXL LTC
> + Reserved : 0000
> +
> + Structure Type : 01 [DSLBIS]
> + Reserved : 00
> + Length : 18 <- 24d, size of structure
> + Handle : 0001 <- DSMAS handle
> + Flags : 00 <- Matches flag field for HMAT SLLBIS
> + Data Type : 03 <- Bandwidth
> + Entry Basee Unit : 0000000000001000 <- Entry Base Unit field in HMAT SSLBIS
> + Entry : 020000000000 <- First byte used here, CXL BW
> + Reserved : 0000
> +
> +
> +==================================================================
> +Switch Scoped Latency and Bandwidth Information Structure (SSLBIS)
> +==================================================================
> +
> +The SSLBIS contains information about the latency and bandwidth of a switch.
> +
> +The table is used by Linux to compute the performance coordinates of a CXL path
> +from the device to the root port where a switch is part of the path.
> +
> +Example ::
> +
> + Structure Type : 05 [SSLBIS]
> + Reserved : 00
> + Length : 20 <- 32d, length of record, including SSLB entries
> + Data Type : 00 <- Latency
> + Reserved : 000000
> + Entry Base Unit : 00000000000000001000 <- Matches Entry Base Unit in HMAT SSLBIS
> +
> + <- SSLB Entry 0
> + Port X ID : 0100 <- First port, 0100h represents an upstream port
> + Port Y ID : 0000 <- Second port, downstream port 0
> + Latency : 0100 <- Port latency
> + Reserved : 0000
> + <- SSLB Entry 1
> + Port X ID : 0100
> + Port Y ID : 0001
> + Latency : 0100
> + Reserved : 0000
> +
> +
> + Structure Type : 05 [SSLBIS]
> + Reserved : 00
> + Length : 18 <- 24d, length of record, including SSLB entry
> + Data Type : 03 <- Bandwidth
> + Reserved : 000000
> + Entry Base Unit : 00000000000000001000 <- Matches Entry Base Unit in HMAT SSLBIS
> +
> + <- SSLB Entry 0
> + Port X ID : 0100 <- First port, 0100h represents an upstream port
> + Port Y ID : FFFF <- Second port, FFFFh indicates any port
> + Bandwidth : 1200 <- Port bandwidth
> + Reserved : 0000
> +
> +The CXL driver uses a combination of CDAT, HMAT, SRAT, and other data to
> +generate "whole path performance" data for a CXL device.
> --
> 2.49.0
>
next prev parent reply other threads:[~2025-05-15 3:41 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-15 0:09 [PATCH v2 0/3] cxl: Update CXL documentation for access coordinates calculation Dave Jiang
2025-05-15 0:09 ` [PATCH v2 1/3] cxl: docs/platform/cdat reference documentation Dave Jiang
2025-05-15 3:41 ` Gregory Price [this message]
2025-05-15 0:09 ` [PATCH v2 2/3] cxl: docs/platform/acpi/srat Add generic target documentation Dave Jiang
2025-05-15 0:09 ` [PATCH v2 3/3] cxl: doc/linux/access-coordinates Update access coordinates calculation methods Dave Jiang
2025-05-15 17:15 ` [PATCH v2 0/3] cxl: Update CXL documentation for access coordinates calculation Alison Schofield
2025-05-15 17:25 ` Dave Jiang
2025-05-15 23:51 ` Dave Jiang
2025-05-19 15:42 ` Jonathan Cameron
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=aCVifxz7t62uOfmn@gourry-fedora-PF4VCD3F \
--to=gourry@gourry.net \
--cc=alison.schofield@intel.com \
--cc=dan.j.williams@intel.com \
--cc=dave.jiang@intel.com \
--cc=dave@stgolabs.net \
--cc=ira.weiny@intel.com \
--cc=jonathan.cameron@huawei.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