On Mon, Jun 08, 2026 at 04:19:47PM +0800, Dave Jiang wrote:
>
>
> On 5/23/26 2:50 AM, Anisa Su wrote:
> > From: Ira Weiny <[email protected]>
> >
> > Dynamic capacity partitions are exposed as a singular dynamic ram
> > partition.
> >
> > Add CXL library support to read this partition information.
> >
> > Signed-off-by: Ira Weiny <[email protected]>
>
> Missing Anisa sign off.
>
> Can probably squash this and the next commit so the usage is shown for the
> reviewer.
>
> DJ
>
> > ---
> > Documentation/cxl/lib/libcxl.txt | 6 +++--
> > cxl/lib/libcxl.c | 43 ++++++++++++++++++++++++++++++++
> > cxl/lib/libcxl.sym | 4 +++
> > cxl/lib/private.h | 3 +++
> > cxl/libcxl.h | 10 +++++++-
> > 5 files changed, 63 insertions(+), 3 deletions(-)
> >
> > diff --git a/Documentation/cxl/lib/libcxl.txt
> > b/Documentation/cxl/lib/libcxl.txt
> > index 5c3ebd4..9921ac1 100644
> > --- a/Documentation/cxl/lib/libcxl.txt
> > +++ b/Documentation/cxl/lib/libcxl.txt
> > @@ -74,6 +74,7 @@ int cxl_memdev_get_major(struct cxl_memdev *memdev);
> > int cxl_memdev_get_minor(struct cxl_memdev *memdev);
> > unsigned long long cxl_memdev_get_pmem_size(struct cxl_memdev *memdev);
> > unsigned long long cxl_memdev_get_ram_size(struct cxl_memdev *memdev);
> > +unsigned long long cxl_memdev_get_dynamic_ram_a_size(struct cxl_memdev
> > *memdev);
> > const char *cxl_memdev_get_firmware_version(struct cxl_memdev *memdev);
> > size_t cxl_memdev_get_label_size(struct cxl_memdev *memdev);
> > int cxl_memdev_nvdimm_bridge_active(struct cxl_memdev *memdev);
> > @@ -93,8 +94,8 @@ The character device node for command submission can be
> > found by default
> > at /dev/cxl/mem%d, or created with a major / minor returned from
> > cxl_memdev_get_{major,minor}().
> >
> > -The 'pmem_size' and 'ram_size' attributes return the current
> > -provisioning of DPA (Device Physical Address / local capacity) in the
> > +The 'pmem_size', 'ram_size', and 'dynamic_ram_a_size' attributes return the
> > +current provisioning of DPA (Device Physical Address / local capacity) in
> > the
> > device.
> >
> > cxl_memdev_get_numa_node() returns the affinitized CPU node number if
> > @@ -453,6 +454,7 @@ enum cxl_decoder_mode {
> > CXL_DECODER_MODE_MIXED,
> > CXL_DECODER_MODE_PMEM,
> > CXL_DECODER_MODE_RAM,
> > + CXL_DECODER_MODE_DYNAMIC_RAM_A,
> > };
> > enum cxl_decoder_mode cxl_decoder_get_mode(struct cxl_decoder *decoder);
> > int cxl_decoder_set_mode(struct cxl_decoder *decoder, enum
> > cxl_decoder_mode mode);
> > diff --git a/cxl/lib/libcxl.c b/cxl/lib/libcxl.c
> > index e55a7b4..be0bc03 100644
> > --- a/cxl/lib/libcxl.c
> > +++ b/cxl/lib/libcxl.c
> > @@ -501,6 +501,9 @@ CXL_EXPORT bool cxl_region_qos_class_mismatch(struct
> > cxl_region *region)
> > } else if (region->mode == CXL_DECODER_MODE_PMEM) {
> > if (root_decoder->qos_class != memdev->pmem_qos_class)
> > return true;
> > + } else if (region->mode == CXL_DECODER_MODE_DYNAMIC_RAM_A) {
> > + if (root_decoder->qos_class !=
> > memdev->dynamic_ram_a_qos_class)
> > + return true;
> > }
> > }
> >
> > @@ -1426,6 +1429,10 @@ static void *add_cxl_memdev(void *parent, int id,
> > const char *cxlmem_base)
> > if (sysfs_read_attr(ctx, path, buf) == 0)
> > memdev->ram_size = strtoull(buf, NULL, 0);
> >
> > + sprintf(path, "%s/dynamic_ram_a/size", cxlmem_base);
> > + if (sysfs_read_attr(ctx, path, buf) == 0)
> > + memdev->dynamic_ram_a_size = strtoull(buf, NULL, 0);
> > +
> > sprintf(path, "%s/pmem/qos_class", cxlmem_base);
> > if (sysfs_read_attr(ctx, path, buf) < 0)
> > memdev->pmem_qos_class = CXL_QOS_CLASS_NONE;
> > @@ -1438,6 +1445,12 @@ static void *add_cxl_memdev(void *parent, int id,
> > const char *cxlmem_base)
> > else
> > memdev->ram_qos_class = atoi(buf);
> >
> > + sprintf(path, "%s/dynamic_ram_a/qos_class", cxlmem_base);
> > + if (sysfs_read_attr(ctx, path, buf) < 0)
> > + memdev->dynamic_ram_a_qos_class = CXL_QOS_CLASS_NONE;
> > + else
> > + memdev->dynamic_ram_a_qos_class = atoi(buf);
> > +
> > sprintf(path, "%s/payload_max", cxlmem_base);
> > if (sysfs_read_attr(ctx, path, buf) == 0) {
> > memdev->payload_max = strtoull(buf, NULL, 0);
> > @@ -1685,6 +1698,11 @@ CXL_EXPORT unsigned long long
> > cxl_memdev_get_ram_size(struct cxl_memdev *memdev)
> > return memdev->ram_size;
> > }
> >
> > +CXL_EXPORT unsigned long long cxl_memdev_get_dynamic_ram_a_size(struct
> > cxl_memdev *memdev)
> > +{
> > + return memdev->dynamic_ram_a_size;
> > +}
> > +
> > CXL_EXPORT int cxl_memdev_get_pmem_qos_class(struct cxl_memdev *memdev)
> > {
> > return memdev->pmem_qos_class;
> > @@ -1695,6 +1713,11 @@ CXL_EXPORT int cxl_memdev_get_ram_qos_class(struct
> > cxl_memdev *memdev)
> > return memdev->ram_qos_class;
> > }
> >
> > +CXL_EXPORT int cxl_memdev_get_dynamic_ram_a_qos_class(struct cxl_memdev
> > *memdev)
> > +{
> > + return memdev->dynamic_ram_a_qos_class;
> > +}
> > +
> > CXL_EXPORT const char *cxl_memdev_get_firmware_verison(struct cxl_memdev
> > *memdev)
> > {
> > return memdev->firmware_version;
> > @@ -2465,6 +2488,8 @@ static void *add_cxl_decoder(void *parent, int id,
> > const char *cxldecoder_base)
> > decoder->mode = CXL_DECODER_MODE_MIXED;
> > else if (strcmp(buf, "none") == 0)
> > decoder->mode = CXL_DECODER_MODE_NONE;
> > + else if (strcmp(buf, "dynamic_ram_a") == 0)
> > + decoder->mode = CXL_DECODER_MODE_DYNAMIC_RAM_A;
> > else
> > decoder->mode = CXL_DECODER_MODE_MIXED;
> > } else
> > @@ -2504,6 +2529,7 @@ static void *add_cxl_decoder(void *parent, int id,
> > const char *cxldecoder_base)
> > case CXL_PORT_SWITCH:
> > decoder->pmem_capable = true;
> > decoder->volatile_capable = true;
> > + decoder->dynamic_ram_a_capable = true;
> > decoder->mem_capable = true;
> > decoder->accelmem_capable = true;
> > sprintf(path, "%s/locked", cxldecoder_base);
> > @@ -2528,6 +2554,7 @@ static void *add_cxl_decoder(void *parent, int id,
> > const char *cxldecoder_base)
> > { "cap_type3", &decoder->mem_capable },
> > { "cap_ram", &decoder->volatile_capable },
> > { "cap_pmem", &decoder->pmem_capable },
> > + { "cap_dynamic_ram_a", &decoder->dynamic_ram_a_capable
> > },
> > { "locked", &decoder->locked },
> > };
> >
> > @@ -2778,6 +2805,9 @@ CXL_EXPORT int cxl_decoder_set_mode(struct
> > cxl_decoder *decoder,
> > case CXL_DECODER_MODE_RAM:
> > sprintf(buf, "ram");
> > break;
> > + case CXL_DECODER_MODE_DYNAMIC_RAM_A:
> > + sprintf(buf, "dynamic_ram_a");
> > + break;
> > default:
> > err(ctx, "%s: unsupported mode: %d\n",
> > cxl_decoder_get_devname(decoder), mode);
> > @@ -2829,6 +2859,11 @@ CXL_EXPORT bool
> > cxl_decoder_is_volatile_capable(struct cxl_decoder *decoder)
> > return decoder->volatile_capable;
> > }
> >
> > +CXL_EXPORT bool cxl_decoder_is_dynamic_ram_a_capable(struct cxl_decoder
> > *decoder)
> > +{
> > + return decoder->dynamic_ram_a_capable;
> > +}
> > +
> > CXL_EXPORT bool cxl_decoder_is_mem_capable(struct cxl_decoder *decoder)
> > {
> > return decoder->mem_capable;
> > @@ -2903,6 +2938,8 @@ static struct cxl_region
> > *cxl_decoder_create_region(struct cxl_decoder *decoder,
> > sprintf(path, "%s/create_pmem_region", decoder->dev_path);
> > else if (mode == CXL_DECODER_MODE_RAM)
> > sprintf(path, "%s/create_ram_region", decoder->dev_path);
> > + else if (mode == CXL_DECODER_MODE_DYNAMIC_RAM_A)
> > + sprintf(path, "%s/create_dynamic_ram_a_region",
> > decoder->dev_path);
> >
> > rc = sysfs_read_attr(ctx, path, buf);
> > if (rc < 0) {
> > @@ -2954,6 +2991,12 @@ cxl_decoder_create_ram_region(struct cxl_decoder
> > *decoder)
> > return cxl_decoder_create_region(decoder, CXL_DECODER_MODE_RAM);
> > }
> >
> > +CXL_EXPORT struct cxl_region *
> > +cxl_decoder_create_dynamic_ram_a_region(struct cxl_decoder *decoder)
> > +{
> > + return cxl_decoder_create_region(decoder,
> > CXL_DECODER_MODE_DYNAMIC_RAM_A);
> > +}
> > +
> > CXL_EXPORT int cxl_decoder_get_nr_targets(struct cxl_decoder *decoder)
> > {
> > return decoder->nr_targets;
> > diff --git a/cxl/lib/libcxl.sym b/cxl/lib/libcxl.sym
> > index ed4429f..258bdd3 100644
> > --- a/cxl/lib/libcxl.sym
> > +++ b/cxl/lib/libcxl.sym
> > @@ -294,6 +294,10 @@ global:
> > cxl_memdev_get_fwctl;
> > cxl_fwctl_get_major;
> > cxl_fwctl_get_minor;
> > + cxl_memdev_get_dynamic_ram_a_size;
> > + cxl_memdev_get_dynamic_ram_a_qos_class;
> > + cxl_decoder_is_dynamic_ram_a_capable;
> > + cxl_decoder_create_dynamic_ram_a_region;
> > } LIBECXL_8;
> >
> > LIBCXL_10 {
Shouldn't new exported symbols go in a fresh top-level node ?
Something like LIBCXL_12 ? please note that Patch 4 has the same
issue.
Please let me know if I'm wrong or misunderstand anything.
Best regards,
Richard Cheng.
> > diff --git a/cxl/lib/private.h b/cxl/lib/private.h
> > index d2d71fc..37b7b06 100644
> > --- a/cxl/lib/private.h
> > +++ b/cxl/lib/private.h
> > @@ -52,8 +52,10 @@ struct cxl_memdev {
> > struct list_node list;
> > unsigned long long pmem_size;
> > unsigned long long ram_size;
> > + unsigned long long dynamic_ram_a_size;
> > int ram_qos_class;
> > int pmem_qos_class;
> > + int dynamic_ram_a_qos_class;
> > int payload_max;
> > size_t lsa_size;
> > struct kmod_module *module;
> > @@ -159,6 +161,7 @@ struct cxl_decoder {
> > unsigned int interleave_granularity;
> > bool pmem_capable;
> > bool volatile_capable;
> > + bool dynamic_ram_a_capable;
> > bool mem_capable;
> > bool accelmem_capable;
> > bool locked;
> > diff --git a/cxl/libcxl.h b/cxl/libcxl.h
> > index e91af90..fd41122 100644
> > --- a/cxl/libcxl.h
> > +++ b/cxl/libcxl.h
> > @@ -75,8 +75,10 @@ struct cxl_fwctl *cxl_memdev_get_fwctl(struct cxl_memdev
> > *memdev);
> > struct cxl_ctx *cxl_memdev_get_ctx(struct cxl_memdev *memdev);
> > unsigned long long cxl_memdev_get_pmem_size(struct cxl_memdev *memdev);
> > unsigned long long cxl_memdev_get_ram_size(struct cxl_memdev *memdev);
> > +unsigned long long cxl_memdev_get_dynamic_ram_a_size(struct cxl_memdev
> > *memdev);
> > int cxl_memdev_get_pmem_qos_class(struct cxl_memdev *memdev);
> > int cxl_memdev_get_ram_qos_class(struct cxl_memdev *memdev);
> > +int cxl_memdev_get_dynamic_ram_a_qos_class(struct cxl_memdev *memdev);
> > const char *cxl_memdev_get_firmware_verison(struct cxl_memdev *memdev);
> > bool cxl_memdev_fw_update_in_progress(struct cxl_memdev *memdev);
> > size_t cxl_memdev_fw_update_get_remaining(struct cxl_memdev *memdev);
> > @@ -210,6 +212,7 @@ enum cxl_decoder_mode {
> > CXL_DECODER_MODE_MIXED,
> > CXL_DECODER_MODE_PMEM,
> > CXL_DECODER_MODE_RAM,
> > + CXL_DECODER_MODE_DYNAMIC_RAM_A,
> > };
> >
> > static inline const char *cxl_decoder_mode_name(enum cxl_decoder_mode mode)
> > @@ -219,9 +222,10 @@ static inline const char *cxl_decoder_mode_name(enum
> > cxl_decoder_mode mode)
> > [CXL_DECODER_MODE_MIXED] = "mixed",
> > [CXL_DECODER_MODE_PMEM] = "pmem",
> > [CXL_DECODER_MODE_RAM] = "ram",
> > + [CXL_DECODER_MODE_DYNAMIC_RAM_A] = "dynamic_ram_a",
> > };
> >
> > - if (mode < CXL_DECODER_MODE_NONE || mode > CXL_DECODER_MODE_RAM)
> > + if (mode < CXL_DECODER_MODE_NONE || mode >
> > CXL_DECODER_MODE_DYNAMIC_RAM_A)
> > mode = CXL_DECODER_MODE_NONE;
> > return names[mode];
> > }
> > @@ -235,6 +239,8 @@ cxl_decoder_mode_from_ident(const char *ident)
> > return CXL_DECODER_MODE_RAM;
> > else if (strcmp(ident, "pmem") == 0)
> > return CXL_DECODER_MODE_PMEM;
> > + else if (strcmp(ident, "dynamic_ram_a") == 0)
> > + return CXL_DECODER_MODE_DYNAMIC_RAM_A;
> > return CXL_DECODER_MODE_NONE;
> > }
> >
> > @@ -264,6 +270,7 @@ cxl_decoder_get_target_type(struct cxl_decoder
> > *decoder);
> > bool cxl_decoder_is_pmem_capable(struct cxl_decoder *decoder);
> > bool cxl_decoder_is_volatile_capable(struct cxl_decoder *decoder);
> > bool cxl_decoder_is_mem_capable(struct cxl_decoder *decoder);
> > +bool cxl_decoder_is_dynamic_ram_a_capable(struct cxl_decoder *decoder);
> > bool cxl_decoder_is_accelmem_capable(struct cxl_decoder *decoder);
> > bool cxl_decoder_is_locked(struct cxl_decoder *decoder);
> > unsigned int
> > @@ -272,6 +279,7 @@ unsigned int cxl_decoder_get_interleave_ways(struct
> > cxl_decoder *decoder);
> > struct cxl_region *cxl_decoder_get_region(struct cxl_decoder *decoder);
> > struct cxl_region *cxl_decoder_create_pmem_region(struct cxl_decoder
> > *decoder);
> > struct cxl_region *cxl_decoder_create_ram_region(struct cxl_decoder
> > *decoder);
> > +struct cxl_region *cxl_decoder_create_dynamic_ram_a_region(struct
> > cxl_decoder *decoder);
> > struct cxl_decoder *cxl_decoder_get_by_name(struct cxl_ctx *ctx,
> > const char *ident);
> > struct cxl_memdev *cxl_decoder_get_memdev(struct cxl_decoder *decoder);
>
>
