plugin/ocp_get_feature_fid_c5h:Added the OCP Get Feature FID=C5h command api

Enabled the Get Feature Command FID=C5h command api with nsid field Values

Reviewed-by: Karthik Balan <[email protected]>
Reviewed-by: Arunpandian J <[email protected]>
Signed-off-by: Vigneshwaran Saravanan/Vigneshwaran Saravanan <[email protected]>

Author: VigneshwaranSaravana

Documentation/nvme-ocp-get-latency-monitor.txt

@@ -0,0 +1,58 @@
+nvme-ocp-get-latency-monitor(1)
+=========================================
+
+NAME
+----
+nvme-ocp-get-latency-monitor - Define and print get-latency-monitor value
+
+SYNOPSIS
+--------
+[verse]
+'nvme ocp get-latency-monitor' <device> [--sel=<select> | -S <select>] [--nsid | -n]
+
+DESCRIPTION
+-----------
+Define get-latency-monitor.
+No argument prints current mode.
+
+The <device> parameter is mandatory and may be either the NVMe character
+device (ex: /dev/nvme0) or block device (ex: /dev/nvme0n1).
+
+This will only work on OCP compliant devices supporting this feature.
+Results for any other device are undefined.
+
+On success it returns 0, error code otherwise.
+
+OPTIONS
+-------
+
+-n <nsid>::
+--nsid=<nsid>::
+	NSID: This field specifies Valid, Invalid and
+	Inactive NSID value:
+-S <select>::
+--sel=<select>::
+	Select (SEL): This field specifies which value of the attributes
+	to return in the provided data:
++
+[]
+|==================
+|Select|Description
+|0|Current
+|1|Default
+|2|Saved
+|3|Supported capabilities
+|4-7|Reserved
+|==================
+
+EXAMPLES
+--------
+* Has the program issue a get-latency-monitor to retrieve the 0xC5 get features.
++
+------------
+# nvme ocp get-latency-monitor /dev/nvme0
+------------
+
+NVME
+----
+Part of the nvme-user suite.

completions/_nvme

@@ -396,6 +396,20 @@ _nvme () {
 				_arguments '*:: :->subcmds'
 				_describe -t commands "nvme ocp set-telemetry-profile options" _ocp_set_telemetry_profile_feature
 				;;
+			(get-latency-monitor)
+				local _ocp_get_latency_monitor_feature
+				_ocp_get_latency_monitor_feature=(
+				/dev/nvme':supply a device to use (required)'
+				--sel=':select from 0 - current, 1 - default, 2 - saved, 3 - supported'
+				-S':alias to --sel'
+				--nsid=':valid, invalid and inactive nsid'
+				-n':alias to --nsid'
+				--no-uuid':Skip UUID index search'
+				-u':alias for --no-uuid'
+				)
+				_arguments '*:: :->subcmds'
+				_describe -t commands "nvme ocp get-latency-monitor options" _ocp_get_latency_monitor_feature
+				;;
 			(*)
 				_files
 				;;
@@ -2816,6 +2830,7 @@ _nvme () {
 			get-error-injection':get error injection'
 			set-error-injection':set error injection'
 			hardware-component-log':retrieve hardware component log'
+			get-latency-monitor':Get Latency Monitor Feature'
 			)
 			_arguments '*:: :->subcmds'
 			_describe -t commands "nvme ocp options" _ocp

completions/bash-nvme-completion.sh

@@ -1616,6 +1616,10 @@ plugin_ocp_opts () {
 		opts+=" --comp-id= -i --list -l --verbose -v \
 			--output-format -o --timeout= -t"
 			;;
+		"get-latency-monitor")
+		opts+=" --sel= -S \
+			--nsid= -n"
+			;;
 		"help")
 		opts+=$NO_OPTS
 			;;
@@ -1697,7 +1701,7 @@ _nvme_subcmds () {
 			telemetry-string-log set-telemetry-profile \
 			set-dssd-async-event-config get-dssd-async-event-config \
 			get-error-injection set-error-injection \
-			hardware-component-log"
+			hardware-component-log get-latency-monitor"
 	)
 
 	# Associative array mapping plugins to corresponding option completions

plugins/ocp/ocp-nvme.c

@@ -423,6 +423,65 @@ int ocp_set_latency_monitor_feature(int argc, char **argv, struct command *cmd,
 	return err;
 }
 
+static int ocp_get_latency_monitor_feature(int argc, char **argv, struct command *cmd,
+					 struct plugin *plugin)
+{
+	const char *desc = "Define Issue Get Feature command (FID : 0xC5) Latency Monitor";
+	const char *sel = "[0-3]: current/default/saved/supported/";
+	const char *nsid = "Byte[04-07]:Namespace Identifier Valid/Invalid/Inactive";
+	struct nvme_dev *dev;
+	__u32 result;
+	int err;
+
+	struct config {
+		__u8 sel;
+		__u32 nsid;
+	};
+
+	struct config cfg = {
+		.sel = 0,
+		.nsid = 0,
+	};
+
+	OPT_ARGS(opts) = {
+		OPT_BYTE("sel", 'S', &cfg.sel, sel),
+		OPT_BYTE("nsid", 'n', &cfg.nsid, nsid),
+		OPT_FLAG("no-uuid", 'u', NULL, no_uuid),
+		OPT_END()
+	};
+
+	err = parse_and_open(&dev, argc, argv, desc, opts);
+	if (err)
+		return err;
+
+	struct nvme_get_features_args args = {
+		.args_size  = sizeof(args),
+		.fd         = dev_fd(dev),
+		.fid        = OCP_FID_LM,
+		.nsid       = cfg.nsid,
+		.sel        = cfg.sel,
+		.cdw11      = 0,
+		.uuidx      = 0,
+		.data_len   = 0,
+		.data       = NULL,
+		.timeout    = NVME_DEFAULT_IOCTL_TIMEOUT,
+		.result     = &result,
+	};
+
+	err = nvme_get_features(&args);
+	if (!err) {
+		printf("get-feature:0xC5 %s value: %#08x\n",
+		nvme_select_to_string(cfg.sel), result);
+
+		if (cfg.sel == NVME_GET_FEATURES_SEL_SUPPORTED)
+			nvme_show_select_result(0xC5, result);
+	} else {
+		nvme_show_error("Could not get feature: 0xC5");
+	}
+
+	return err;
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////////////////////////

plugins/ocp/ocp-nvme.h

@@ -42,6 +42,8 @@ PLUGIN(NAME("ocp", "OCP cloud SSD extensions", OCP_PLUGIN_VERSION),
 		      get_enable_ieee1667_silo)
 		ENTRY("set-enable-ieee1667-silo", "enable IEEE1667 silo", set_enable_ieee1667_silo)
 		ENTRY("hardware-component-log", "retrieve hardware component log", hwcomp_log)
+		ENTRY("get-latency-monitor", "Get Latency Monitor Feature"
+		, ocp_get_latency_monitor_feature)
 	)
 );