Re: [PATCH v3 05/25] cxl/mbox: Flag support for Dynamic Capacity Devices (DCD)

2024-09-02 Thread Li, Ming4
On 8/16/2024 10:44 PM, [email protected] wrote:
> From: Navneet Singh 
>
> 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 
> Reviewed-by: Jonathan Cameron 
> Reviewed-by: Fan Ni 
> Reviewed-by: Dave Jiang 
> Reviewed-by: Davidlohr Bueso 
> Co-developed-by: Ira Weiny 
> Signed-off-by: Ira Weiny 
>
Reviewed-by: Li Ming 




[PATCH v3 05/25] cxl/mbox: Flag support for Dynamic Capacity Devices (DCD)

2024-08-16 Thread ira . weiny
From: Navneet Singh 

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 
Reviewed-by: Jonathan Cameron 
Reviewed-by: Fan Ni 
Reviewed-by: Dave Jiang 
Reviewed-by: Davidlohr Bueso 
Co-developed-by: Ira Weiny 
Signed-off-by: Ira Weiny 

---
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 = 0x1
 };
 

-- 
2.45.2