{"id":2216055,"url":"http://patchwork.ozlabs.org/api/patches/2216055/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/patch/20260325184259.366-10-alireza.sanaee@huawei.com/","project":{"id":14,"url":"http://patchwork.ozlabs.org/api/projects/14/?format=json","name":"QEMU Development","link_name":"qemu-devel","list_id":"qemu-devel.nongnu.org","list_email":"qemu-devel@nongnu.org","web_url":"","scm_url":"","webscm_url":"","list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260325184259.366-10-alireza.sanaee@huawei.com>","list_archive_url":null,"date":"2026-03-25T18:42:57","name":"[9/9] hw/cxl: Add QMP status query for dynamic-capacity extent release","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"2de4c55b1d4e3c0e2b787cfd3378bc792fd96b6d","submitter":{"id":90159,"url":"http://patchwork.ozlabs.org/api/people/90159/?format=json","name":"Alireza Sanaee","email":"alireza.sanaee@huawei.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/qemu-devel/patch/20260325184259.366-10-alireza.sanaee@huawei.com/mbox/","series":[{"id":497484,"url":"http://patchwork.ozlabs.org/api/series/497484/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/list/?series=497484","date":"2026-03-25T18:42:48","name":"Application Specific Tagged Memory Support in CXL Type 3 Devices","version":1,"mbox":"http://patchwork.ozlabs.org/series/497484/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2216055/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2216055/checks/","tags":{},"related":[],"headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":"legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org\n (client-ip=209.51.188.17; helo=lists.gnu.org;\n envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n receiver=patchwork.ozlabs.org)","Received":["from lists.gnu.org (lists.gnu.org [209.51.188.17])\n\t(using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fgwsJ4BHHz1xy3\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 26 Mar 2026 05:48:20 +1100 (AEDT)","from localhost ([::1] helo=lists1p.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.90_1)\n\t(envelope-from <qemu-devel-bounces@nongnu.org>)\n\tid 1w5TH4-00068g-Qp; Wed, 25 Mar 2026 14:48:14 -0400","from eggs.gnu.org ([2001:470:142:3::10])\n by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <alireza.sanaee@huawei.com>)\n id 1w5TH0-00063s-Ts\n for qemu-devel@nongnu.org; Wed, 25 Mar 2026 14:48:11 -0400","from frasgout.his.huawei.com ([185.176.79.56])\n by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <alireza.sanaee@huawei.com>)\n id 1w5TGw-0000Jm-IY\n for qemu-devel@nongnu.org; Wed, 25 Mar 2026 14:48:10 -0400","from mail.maildlp.com (unknown [172.18.224.150])\n by frasgout.his.huawei.com (SkyGuard) with ESMTPS id 4fgwrq2fRtzJ469d;\n Thu, 26 Mar 2026 02:47:55 +0800 (CST)","from dubpeml500005.china.huawei.com (unknown [7.214.145.207])\n by mail.maildlp.com (Postfix) with ESMTPS id 5254F4056B;\n Thu, 26 Mar 2026 02:48:04 +0800 (CST)","from a2303103017.china.huawei.com (10.47.66.203) by\n dubpeml500005.china.huawei.com (7.214.145.207) with Microsoft SMTP Server\n (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n 15.2.1544.11; Wed, 25 Mar 2026 18:48:03 +0000"],"To":"<qemu-devel@nongnu.org>","CC":"<anisa.su@samsung.com>, <armbru@redhat.com>, <berrange@redhat.com>,\n <eblake@redhat.com>, <jonathan.cameron@huawei.com>,\n <linux-cxl@vger.kernel.org>, <linuxarm@huawei.com>, <lizhijian@fujitsu.com>,\n <mst@redhat.com>, <pbonzini@redhat.com>, <gourry@gourry.net>,\n <nifan.cxl@gmail.com>, <me@linux.beauty>","Subject":"[PATCH 9/9] hw/cxl: Add QMP status query for dynamic-capacity extent\n release","Date":"Wed, 25 Mar 2026 18:42:57 +0000","Message-ID":"<20260325184259.366-10-alireza.sanaee@huawei.com>","X-Mailer":"git-send-email 2.51.0.windows.2","In-Reply-To":"<20260325184259.366-1-alireza.sanaee@huawei.com>","References":"<20260325184259.366-1-alireza.sanaee@huawei.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Content-Type":"text/plain","X-Originating-IP":"[10.47.66.203]","X-ClientProxiedBy":"lhrpeml500011.china.huawei.com (7.191.174.215) To\n dubpeml500005.china.huawei.com (7.214.145.207)","Received-SPF":"pass client-ip=185.176.79.56;\n envelope-from=alireza.sanaee@huawei.com; helo=frasgout.his.huawei.com","X-Spam_score_int":"-41","X-Spam_score":"-4.2","X-Spam_bar":"----","X-Spam_report":"(-4.2 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3,\n RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001,\n RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001,\n SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no","X-Spam_action":"no action","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"qemu development <qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<https://lists.nongnu.org/archive/html/qemu-devel>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Reply-to":"Alireza Sanaee <alireza.sanaee@huawei.com>","From":"Alireza Sanaee via qemu development <qemu-devel@nongnu.org>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org"},"content":"Add a small status query so orchestration layers can check whether a tagged\nextent is still pending, committed, or no longer present after a release\nrequest.\n\nIntroduce the ExtentStatus QAPI struct and the\ncxl-release-dynamic-capacity-status command.\n\nSigned-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>\n---\n hw/mem/cxl_type3.c       | 85 ++++++++++++++++++++++++++++++++++++++++\n hw/mem/cxl_type3_stubs.c | 10 +++++\n qapi/cxl.json            | 46 ++++++++++++++++++++++\n 3 files changed, 141 insertions(+)","diff":"diff --git a/hw/mem/cxl_type3.c b/hw/mem/cxl_type3.c\nindex d5fa8a530a..8999b36e61 100644\n--- a/hw/mem/cxl_type3.c\n+++ b/hw/mem/cxl_type3.c\n@@ -2635,6 +2635,91 @@ void cxl_remove_memory_alias(CXLType3Dev *dcd, struct CXLFixedWindow *fw,\n     dcd->dc.direct_mr_bitmap &= ~(1u << hdm_id);\n }\n \n+/*\n+ * This function allows for a simple check to make sure that\n+ * our extent is removed. It can be used by an orchestration layer.\n+ */\n+ExtentStatus *\n+qmp_cxl_release_dynamic_capacity_status(const char *path,\n+                                        uint16_t hid, uint8_t rid,\n+                                        const char *tag,\n+                                        Error **errp)\n+{\n+    Object *obj;\n+    CXLType3Dev *dcd;\n+    CXLDCExtentList *list = NULL;\n+    CXLDCExtent *ent;\n+    QemuUUID uuid_req;\n+    ExtentStatus *res;\n+\n+    obj = object_resolve_path_type(path, TYPE_CXL_TYPE3, NULL);\n+    if (!obj) {\n+        error_setg(errp, \"Unable to resolve CXL type 3 device\");\n+        return NULL;\n+    }\n+\n+    dcd = CXL_TYPE3(obj);\n+    if (!dcd->dc.num_regions) {\n+        error_setg(errp, \"No dynamic capacity support from the device\");\n+        return NULL;\n+    }\n+\n+    if (rid >= dcd->dc.num_regions) {\n+        error_setg(errp, \"Region id is too large\");\n+        return NULL;\n+    }\n+\n+    if (!tag) {\n+        error_setg(errp, \"Tag must be valid\");\n+        return NULL;\n+    }\n+\n+    list = &dcd->dc.extents;\n+    qemu_uuid_parse(tag, &uuid_req);\n+    res = g_new0(ExtentStatus, 1);\n+\n+    /* Check committed extents */\n+    QTAILQ_FOREACH(ent, list, node) {\n+        QemuUUID uuid_ext;\n+        g_autofree char *uuid_str = NULL;\n+        memcpy(&uuid_ext.data, ent->tag, sizeof(ent->tag));\n+        if (qemu_uuid_is_equal(&uuid_req, &uuid_ext)) {\n+            uuid_str = qemu_uuid_unparse_strdup(&uuid_ext);\n+            res->status = g_strdup(\"Committed\");\n+            res->message =\n+                g_strdup_printf(\"Found committed extent with tag %s dpa 0x%\"\n+                                PRIx64 \" len 0x%\" PRIx64,\n+                                uuid_str, ent->start_dpa, ent->len);\n+            return res;\n+        }\n+    }\n+\n+    /* Check pending extents (not yet committed by kernel) */\n+    {\n+        CXLDCExtentGroup *group;\n+        QTAILQ_FOREACH(group, &dcd->dc.extents_pending, node) {\n+            QTAILQ_FOREACH(ent, &group->list, node) {\n+                QemuUUID uuid_ext;\n+                g_autofree char *uuid_str = NULL;\n+                memcpy(&uuid_ext.data, ent->tag, sizeof(ent->tag));\n+                if (qemu_uuid_is_equal(&uuid_req, &uuid_ext)) {\n+                    uuid_str = qemu_uuid_unparse_strdup(&uuid_ext);\n+                    res->status = g_strdup(\"Pending\");\n+                    res->message =\n+                        g_strdup_printf(\"Found pending extent with tag %s\"\n+                                        \" dpa 0x%\" PRIx64 \" len 0x%\" PRIx64,\n+                                        uuid_str, ent->start_dpa, ent->len);\n+                    return res;\n+                }\n+            }\n+        }\n+    }\n+\n+    res->status = g_strdup(\"Released\");\n+    res->message = g_strdup_printf(\"Tag %s released or not found\", tag);\n+    return res;\n+}\n+\n static void ct3_class_init(ObjectClass *oc, const void *data)\n {\n     DeviceClass *dc = DEVICE_CLASS(oc);\ndiff --git a/hw/mem/cxl_type3_stubs.c b/hw/mem/cxl_type3_stubs.c\nindex 98292a931c..17f9414142 100644\n--- a/hw/mem/cxl_type3_stubs.c\n+++ b/hw/mem/cxl_type3_stubs.c\n@@ -127,3 +127,13 @@ void qmp_cxl_release_dynamic_capacity(const char *path, uint16_t host_id,\n {\n     error_setg(errp, \"CXL Type 3 support is not compiled in\");\n }\n+\n+ExtentStatus *qmp_cxl_release_dynamic_capacity_status(const char *path,\n+                                                      uint16_t host_id,\n+                                                      uint8_t region,\n+                                                      const char *tag,\n+                                                      Error **errp)\n+{\n+    error_setg(errp, \"CXL Type 3 support is not compiled in\");\n+    return NULL;\n+}\ndiff --git a/qapi/cxl.json b/qapi/cxl.json\nindex 81d6198ba0..53325ba530 100644\n--- a/qapi/cxl.json\n+++ b/qapi/cxl.json\n@@ -636,3 +636,49 @@\n            },\n   'features': [ 'unstable' ]\n }\n+\n+##\n+# @ExtentStatus:\n+# This is an object that describes the status of an extent.\n+#\n+# @status:   String indicating the overall result, e.g. \"success\".\n+# @message:  Human-readable description of the outcome.\n+#\n+# Since: 9.1\n+##\n+{ 'struct': 'ExtentStatus',\n+      'data': { 'status': 'str', 'message': 'str' }\n+}\n+\n+##\n+# @cxl-release-dynamic-capacity-status:\n+#\n+# This commands checks if an extent tag has been released or not.\n+#\n+# @path: path to the CXL Dynamic Capacity Device in the QOM tree.\n+#\n+# @host-id: The \"Host ID\" field as defined in Compute Express Link\n+#     (CXL) Specification, Revision 3.1, Table 7-71.\n+#\n+# @region: The \"Region Number\" field as defined in Compute Express\n+#     Link Specification, Revision 3.1, Table 7-71.  Valid range\n+#     is from 0-7.\n+#\n+# @tag: The \"Tag\" field as defined in Compute Express Link (CXL)\n+#     Specification, Revision 3.1, Table 7-71.\n+#\n+# Features:\n+#\n+# @unstable: For now this command is subject to change.\n+#\n+# Since: 9.1\n+##\n+{ 'command': 'cxl-release-dynamic-capacity-status',\n+  'data': { 'path': 'str',\n+            'host-id': 'uint16',\n+            'region': 'uint8',\n+            'tag': 'str'\n+          },\n+  'features': [ 'unstable' ],\n+  'returns': 'ExtentStatus'\n+}\n","prefixes":["9/9"]}