From: Navneet Singh <navneet.si...@intel.com>

Per the CXL 3.1 specification software must check the Command Effects
Log (CEL) for dynamic capacity command support.

Detect support for the DCD commands while reading the CEL, including:

        Get DC Config
        Get DC Extent List
        Add DC Response
        Release DC

Signed-off-by: Navneet Singh <navneet.si...@intel.com>
Reviewed-by: Jonathan Cameron <jonathan.came...@huawei.com>
Reviewed-by: Fan Ni <fan...@samsung.com>
Reviewed-by: Dave Jiang <dave.ji...@intel.com>
Reviewed-by: Davidlohr Bueso <d...@stgolabs.net>
Co-developed-by: Ira Weiny <ira.we...@intel.com>
Signed-off-by: Ira Weiny <ira.we...@intel.com>

---
Changes:
[iweiny: Keep tags for this early simple patch]
[Davidlohr: update commit message]
[djiang: Fix misalignment]
---
 drivers/cxl/core/mbox.c | 33 +++++++++++++++++++++++++++++++++
 drivers/cxl/cxlmem.h    | 15 +++++++++++++++
 2 files changed, 48 insertions(+)

diff --git a/drivers/cxl/core/mbox.c b/drivers/cxl/core/mbox.c
index e5cdeafdf76e..8eb196858abe 100644
--- a/drivers/cxl/core/mbox.c
+++ b/drivers/cxl/core/mbox.c
@@ -164,6 +164,34 @@ static void cxl_set_security_cmd_enabled(struct 
cxl_security_state *security,
        }
 }
 
+static bool cxl_is_dcd_command(u16 opcode)
+{
+#define CXL_MBOX_OP_DCD_CMDS 0x48
+
+       return (opcode >> 8) == CXL_MBOX_OP_DCD_CMDS;
+}
+
+static void cxl_set_dcd_cmd_enabled(struct cxl_memdev_state *mds,
+                                   u16 opcode)
+{
+       switch (opcode) {
+       case CXL_MBOX_OP_GET_DC_CONFIG:
+               set_bit(CXL_DCD_ENABLED_GET_CONFIG, mds->dcd_cmds);
+               break;
+       case CXL_MBOX_OP_GET_DC_EXTENT_LIST:
+               set_bit(CXL_DCD_ENABLED_GET_EXTENT_LIST, mds->dcd_cmds);
+               break;
+       case CXL_MBOX_OP_ADD_DC_RESPONSE:
+               set_bit(CXL_DCD_ENABLED_ADD_RESPONSE, mds->dcd_cmds);
+               break;
+       case CXL_MBOX_OP_RELEASE_DC:
+               set_bit(CXL_DCD_ENABLED_RELEASE, mds->dcd_cmds);
+               break;
+       default:
+               break;
+       }
+}
+
 static bool cxl_is_poison_command(u16 opcode)
 {
 #define CXL_MBOX_OP_POISON_CMDS 0x43
@@ -745,6 +773,11 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, 
size_t size, u8 *cel)
                        enabled++;
                }
 
+               if (cxl_is_dcd_command(opcode)) {
+                       cxl_set_dcd_cmd_enabled(mds, opcode);
+                       enabled++;
+               }
+
                dev_dbg(dev, "Opcode 0x%04x %s\n", opcode,
                        enabled ? "enabled" : "unsupported by driver");
        }
diff --git a/drivers/cxl/cxlmem.h b/drivers/cxl/cxlmem.h
index afb53d058d62..f2f8b567e0e7 100644
--- a/drivers/cxl/cxlmem.h
+++ b/drivers/cxl/cxlmem.h
@@ -238,6 +238,15 @@ struct cxl_event_state {
        struct mutex log_lock;
 };
 
+/* Device enabled DCD commands */
+enum dcd_cmd_enabled_bits {
+       CXL_DCD_ENABLED_GET_CONFIG,
+       CXL_DCD_ENABLED_GET_EXTENT_LIST,
+       CXL_DCD_ENABLED_ADD_RESPONSE,
+       CXL_DCD_ENABLED_RELEASE,
+       CXL_DCD_ENABLED_MAX
+};
+
 /* Device enabled poison commands */
 enum poison_cmd_enabled_bits {
        CXL_POISON_ENABLED_LIST,
@@ -454,6 +463,7 @@ struct cxl_dev_state {
  *                (CXL 2.0 8.2.9.5.1.1 Identify Memory Device)
  * @mbox_mutex: Mutex to synchronize mailbox access.
  * @firmware_version: Firmware version for the memory device.
+ * @dcd_cmds: List of DCD commands implemented by memory device
  * @enabled_cmds: Hardware commands found enabled in CEL.
  * @exclusive_cmds: Commands that are kernel-internal only
  * @total_bytes: sum of all possible capacities
@@ -482,6 +492,7 @@ struct cxl_memdev_state {
        size_t lsa_size;
        struct mutex mbox_mutex; /* Protects device mailbox and firmware */
        char firmware_version[0x10];
+       DECLARE_BITMAP(dcd_cmds, CXL_DCD_ENABLED_MAX);
        DECLARE_BITMAP(enabled_cmds, CXL_MEM_COMMAND_ID_MAX);
        DECLARE_BITMAP(exclusive_cmds, CXL_MEM_COMMAND_ID_MAX);
        u64 total_bytes;
@@ -555,6 +566,10 @@ enum cxl_opcode {
        CXL_MBOX_OP_UNLOCK              = 0x4503,
        CXL_MBOX_OP_FREEZE_SECURITY     = 0x4504,
        CXL_MBOX_OP_PASSPHRASE_SECURE_ERASE     = 0x4505,
+       CXL_MBOX_OP_GET_DC_CONFIG       = 0x4800,
+       CXL_MBOX_OP_GET_DC_EXTENT_LIST  = 0x4801,
+       CXL_MBOX_OP_ADD_DC_RESPONSE     = 0x4802,
+       CXL_MBOX_OP_RELEASE_DC          = 0x4803,
        CXL_MBOX_OP_MAX                 = 0x10000
 };
 

-- 
2.45.2


Reply via email to