nvme: add get and set features specific status code string

Since the status code is duplicated with the fw-download status code.

Signed-off-by: Tokunori Ikegami <[email protected]>

Author: ikegami-t

libnvme/src/nvme/util.h

@@ -103,6 +103,25 @@ nvme_sanitize_ns_status_to_string(__u16 sc)
 	return NULL;
 };
 
+/**
+ * nvme_set_features_status_to_string() - Returns set features status string.
+ * @sc: Return status code from an set features command
+ *
+ * Return: The set features status string if it is a specific status code.
+ */
+static inline const char *
+nvme_set_features_status_to_string(__u16 sc)
+{
+	switch (sc) {
+	case NVME_SC_OVERLAPPING_RANGE:
+		return "Overlapping Range: LBA range type data structure";
+	default:
+		break;
+	}
+
+	return NULL;
+};
+
 /**
  * nvme_opcode_status_to_string() - Returns nvme opcode status string.
  * @status: Return status from an nvme passthrough command
@@ -120,8 +139,18 @@ nvme_opcode_status_to_string(int status, bool admin, __u8 opcode)
 	const char *s = NULL;
 
 	if (status >= 0 && sct == NVME_SCT_CMD_SPECIFIC) {
-		if (admin && opcode == nvme_admin_sanitize_ns)
-			s = nvme_sanitize_ns_status_to_string(sc);
+		if (admin) {
+			switch (opcode) {
+			case nvme_admin_sanitize_ns:
+				s = nvme_sanitize_ns_status_to_string(sc);
+				break;
+			case nvme_admin_set_features:
+				s = nvme_set_features_status_to_string(sc);
+				break;
+			default:
+				break;
+			}
+		}
 	}
 
 	if (s)

nvme.c

@@ -6995,7 +6995,8 @@ static int set_feature(int argc, char **argv, struct command *acmd, struct plugi
 	err = nvme_set_features(hdl, cfg.nsid, cfg.fid, cfg.sv, cfg.value, cfg.cdw12,
 			0, cfg.uidx, 0, buf, cfg.data_len, &result);
 	if (err) {
-		nvme_show_err("set-feature", err);
+		nvme_show_admin_cmd_err("set-feature", nvme_admin_set_features,
+					err);
 		return err;
 	}