intel: Add enable-lat-stats function.

This adds the cli function for lat-stats-tracking. Enabling it allows
the drive to start tracking latency statistics during i/o.

diff --git a/plugins/intel/intel-nvme.c b/plugins/intel/intel-nvme.c
index 807f2d2a7b837fe5349e480d05d68b487ad61d4f..f083419d53cc12138acf68ce87057fe88bdfbde2 100644 (file)
--- a/plugins/intel/intel-nvme.c
+++ b/plugins/intel/intel-nvme.c
@@ -1215,3 +1215,90 @@ static int get_internal_log(int argc, char **argv, struct command *command,
        free(intel);
        return err;
 }
+
+static int enable_lat_stats_tracking(int argc, char **argv,
+               struct command *command, struct plugin *plugin)
+{
+       int err, fd;
+       const char *desc = (
+                       "Enable/Disable Intel Latency Statistics Tracking.\n"
+                       "No argument prints current status.");
+       const char *enable_desc = "Enable LST";
+       const char *disable_desc = "Disable LST";
+       const __u32 nsid = 0;
+       const __u8 fid = 0xe2;
+       const __u8 sel = 0;
+       const __u32 cdw11 = 0x0;
+       const __u32 cdw12 = 0x0;
+       const __u32 data_len = 32;
+       const __u32 save = 0;
+       __u32 result;
+       void *buf = NULL;
+
+       struct config {
+               bool enable, disable;
+       };
+
+       struct config cfg = {
+               .enable = false,
+               .disable = false,
+       };
+
+       const struct argconfig_commandline_options command_line_options[] = {
+               {"enable", 'e', "", CFG_NONE, &cfg.enable, no_argument, enable_desc},
+               {"disable", 'd', "", CFG_NONE, &cfg.disable, no_argument, disable_desc},
+               {NULL}
+       };
+
+       fd = parse_and_open(argc, argv, desc, command_line_options, &cfg,
+                       sizeof(cfg));
+
+       enum Option {
+               None = -1,
+               True = 1,
+               False = 0,
+       };
+
+       enum Option option = None;
+
+       if (cfg.enable && cfg.disable)
+               printf("Cannot enable and disable simultaneously.");
+       else if (cfg.enable || cfg.disable)
+               option = cfg.enable;
+
+       if (fd < 0)
+               return fd;
+       switch (option) {
+       case None:
+               err = nvme_get_feature(fd, nsid, fid, sel, cdw11, data_len, buf,
+                                       &result);
+               if (!err) {
+                       printf(
+                               "Latency Statistics Tracking (FID 0x%X) is currently (%i).\n",
+                               fid, result);
+               } else {
+                       printf("Could not read feature id 0xE2.");
+                       return err;
+               }
+               break;
+       case True:
+       case False:
+               err = nvme_set_feature(fd, nsid, fid, option, cdw12, save,
+                               data_len, buf, &result);
+               if (err > 0) {
+                       fprintf(stderr, "NVMe Status:%s(%x)\n",
+                                       nvme_status_to_string(err), err);
+               } else if (err < 0) {
+                       perror("Enable latency tracking");
+                       fprintf(stderr, "Command failed while parsing.");
+               } else {
+                       printf("Successfully set enable bit for FID (0x%X) to %i.\n",
+                               fid, option);
+               }
+               break;
+       default:
+               printf("%d not supported.", option);
+               return EINVAL;
+       }
+       return fd;
+}

diff --git a/plugins/intel/intel-nvme.h b/plugins/intel/intel-nvme.h
index 6d70efa401d091de34f00f2539de2517e657803b..335e901e5d164f96d14e68cca59a416c010a471f 100644 (file)
--- a/plugins/intel/intel-nvme.h
+++ b/plugins/intel/intel-nvme.h
@@ -9,11 +9,12 @@
 PLUGIN(NAME("intel", "Intel vendor specific extensions"),
        COMMAND_LIST(
                ENTRY("id-ctrl", "Send NVMe Identify Controller", id_ctrl)
-               ENTRY("smart-log-add", "Retrieve Intel SMART Log, show it", get_additional_smart_log)
+               ENTRY("internal-log", "Retrieve Intel internal firmware log, save it", get_internal_log)
+               ENTRY("lat-stats", "Retrieve Intel IO Latency Statistics log, show it", get_lat_stats_log)
+               ENTRY("lat-stats-tracking", "Enable and disable Latency Statistics logging.", enable_lat_stats_tracking)
                ENTRY("market-name", "Retrieve Intel Marketing Name log, show it", get_market_log)
+               ENTRY("smart-log-add", "Retrieve Intel SMART Log, show it", get_additional_smart_log)
                ENTRY("temp-stats", "Retrieve Intel Temperature Statistics log, show it", get_temp_stats_log)
-               ENTRY("lat-stats", "Retrieve Intel IO Latency Statistics log, show it", get_lat_stats_log)
-               ENTRY("internal-log", "Retrieve Intel internal firmware log, save it", get_internal_log)
        )
 );