[PATCH 07/11] nvmet: implement the FDP feature
Guixin Liu
kanie at linux.alibaba.com
Mon Oct 12 19:21:05 PDT 2026
Implement Set Features and Get Features for FID 1Dh. The Endurance
Group Identifier in Command Dword 11 names the namespace directly,
since nvmet identifies each namespace as its own endurance group
(endgid = nsid). Get Features reports FDPE set with FDPCIDX 0.
Set Features is served locally: the bridge runs the one configuration
that the backing device is already in, so setting the feature to that
same value is a no-op that succeeds, and any other value is a change
of a feature whose endurance group has namespaces, which the spec
aborts with Command Sequence Error. That also covers disabling,
which would strand the write streams already handed out to hosts.
The namespace lookup takes a namespace reference, so that a namespace
being disabled concurrently with the command cannot go away under
it. Report the feature in the Features log page under the same
per-subsystem condition as the FDPS bit.
Assisted-by: LLM
Signed-off-by: Guixin Liu <kanie at linux.alibaba.com>
---
drivers/nvme/target/admin-cmd.c | 9 ++++
drivers/nvme/target/fdp.c | 77 +++++++++++++++++++++++++++++++++
drivers/nvme/target/nvmet.h | 10 +++++
3 files changed, 96 insertions(+)
diff --git a/drivers/nvme/target/admin-cmd.c b/drivers/nvme/target/admin-cmd.c
index b293ce418e2c3..93fb9c3d500a3 100644
--- a/drivers/nvme/target/admin-cmd.c
+++ b/drivers/nvme/target/admin-cmd.c
@@ -600,6 +600,9 @@ static void nvmet_execute_get_log_page_features(struct nvmet_req *req)
cpu_to_le32(NVME_FIS_FSUPP | NVME_FIS_NSCPE);
features->fis[NVME_FEAT_RESV_MASK] =
cpu_to_le32(NVME_FIS_FSUPP | NVME_FIS_NSCPE);
+ if (IS_ENABLED(CONFIG_NVME_TARGET_FDP))
+ features->fis[NVME_FEAT_FDP] =
+ cpu_to_le32(NVME_FIS_FSUPP | NVME_FIS_EGSCPE);
status = nvmet_copy_to_sgl(req, 0, features, sizeof(*features));
kfree(features);
@@ -1385,6 +1388,9 @@ void nvmet_execute_set_features(struct nvmet_req *req)
case NVME_FEAT_RESV_MASK:
status = nvmet_set_feat_resv_notif_mask(req, cdw11);
break;
+ case NVME_FEAT_FDP:
+ status = nvmet_set_feat_fdp(req);
+ break;
default:
req->error_loc = offsetof(struct nvme_common_command, cdw10);
status = NVME_SC_INVALID_FIELD | NVME_STATUS_DNR;
@@ -1562,6 +1568,9 @@ void nvmet_execute_get_features(struct nvmet_req *req)
case NVME_FEAT_RESV_MASK:
status = nvmet_get_feat_resv_notif_mask(req);
break;
+ case NVME_FEAT_FDP:
+ status = nvmet_get_feat_fdp(req);
+ break;
default:
req->error_loc =
offsetof(struct nvme_common_command, cdw10);
diff --git a/drivers/nvme/target/fdp.c b/drivers/nvme/target/fdp.c
index 17949e3665293..0327970a9cb9d 100644
--- a/drivers/nvme/target/fdp.c
+++ b/drivers/nvme/target/fdp.c
@@ -12,6 +12,10 @@
#include "../host/nvme.h"
#include "nvmet.h"
+/* Flexible Data Placement feature (FID 1Dh), Command Dword 12 */
+#define NVME_FDP_CDW12_FDPE (1U << 0)
+#define NVME_FDP_CDW12_FDPCIDX_MASK (0xffU << 8)
+
bool nvmet_fdp_subsys_configured(struct nvmet_subsys *subsys)
{
struct nvmet_ns *ns;
@@ -83,3 +87,76 @@ void nvmet_fdp_exit_ns(struct nvmet_ns *ns)
ns->fdp.backend_q = NULL;
ns->fdp.backend_ctrl = NULL;
}
+
+
+/*
+ * Each namespace is its own endurance group, so the Endurance Group
+ * Identifier in Command Dword 11, and in the Log Specific Identifier
+ * of the FDP log pages, directly names the namespace. Returns the
+ * namespace with a reference held.
+ */
+static struct nvmet_ns *nvmet_fdp_lookup_ns(struct nvmet_req *req, u32 endgid)
+{
+ struct nvmet_subsys *subsys = nvmet_req_subsys(req);
+ struct nvmet_ns *ns;
+
+ ns = xa_load(&subsys->namespaces, endgid);
+ if (!ns || !ns->enabled || !nvmet_fdp_enabled(ns))
+ return NULL;
+
+ if (!percpu_ref_tryget_live(&ns->ref))
+ return NULL;
+
+ return ns;
+}
+
+/*
+ * The bridge runs the one configuration that the backing device is
+ * already in, so setting the feature to that same value is the only
+ * transition that can be served. The spec aborts any other value
+ * change of a feature whose endurance group has namespaces with
+ * Command Sequence Error, which also covers disabling: pulling FDP
+ * from under a live namespace would strand the write streams already
+ * handed out to hosts.
+ */
+u16 nvmet_set_feat_fdp(struct nvmet_req *req)
+{
+ u32 cdw12 = le32_to_cpu(req->cmd->features.dword12);
+ struct nvmet_ns *ns;
+ u16 status;
+
+ ns = nvmet_fdp_lookup_ns(req,
+ le32_to_cpu(req->cmd->features.dword11));
+ if (!ns)
+ return NVME_SC_INVALID_FIELD | NVME_STATUS_DNR;
+
+ if (cdw12 & ~(NVME_FDP_CDW12_FDPCIDX_MASK | NVME_FDP_CDW12_FDPE)) {
+ status = NVME_SC_INVALID_FIELD | NVME_STATUS_DNR;
+ } else if ((cdw12 & NVME_FDP_CDW12_FDPE) &&
+ !(cdw12 & NVME_FDP_CDW12_FDPCIDX_MASK)) {
+ nvmet_set_result(req, FDPCFG_FDPE);
+ status = 0;
+ } else {
+ status = NVME_SC_CMD_SEQ_ERROR | NVME_STATUS_DNR;
+ }
+
+ percpu_ref_put(&ns->ref);
+ return status;
+}
+
+u16 nvmet_get_feat_fdp(struct nvmet_req *req)
+{
+ struct nvmet_ns *ns;
+
+ ns = nvmet_fdp_lookup_ns(req,
+ le32_to_cpu(req->cmd->features.dword11));
+ if (!ns)
+ return NVME_SC_INVALID_FIELD | NVME_STATUS_DNR;
+
+ /* the namespace is up, so FDP is enabled on the backing device */
+ nvmet_set_result(req, FDPCFG_FDPE);
+
+ percpu_ref_put(&ns->ref);
+ return 0;
+}
+
diff --git a/drivers/nvme/target/nvmet.h b/drivers/nvme/target/nvmet.h
index 07880db4468ad..e95bafe1fb019 100644
--- a/drivers/nvme/target/nvmet.h
+++ b/drivers/nvme/target/nvmet.h
@@ -997,6 +997,8 @@ static inline bool nvmet_fdp_enabled(struct nvmet_ns *ns)
int nvmet_fdp_init_ns(struct nvmet_ns *ns);
void nvmet_fdp_exit_ns(struct nvmet_ns *ns);
bool nvmet_fdp_subsys_configured(struct nvmet_subsys *subsys);
+u16 nvmet_set_feat_fdp(struct nvmet_req *req);
+u16 nvmet_get_feat_fdp(struct nvmet_req *req);
#else
static inline bool nvmet_fdp_enabled(struct nvmet_ns *ns)
{
@@ -1013,6 +1015,14 @@ static inline bool nvmet_fdp_subsys_configured(struct nvmet_subsys *subsys)
{
return false;
}
+static inline u16 nvmet_set_feat_fdp(struct nvmet_req *req)
+{
+ return NVME_SC_INVALID_FIELD | NVME_STATUS_DNR;
+}
+static inline u16 nvmet_get_feat_fdp(struct nvmet_req *req)
+{
+ return NVME_SC_INVALID_FIELD | NVME_STATUS_DNR;
+}
#endif
/*
--
2.43.7
More information about the Linux-nvme
mailing list