Merge branch 'master' of https://github.com/bvanassche/fio into master
[fio.git] / stat.c
diff --git a/stat.c b/stat.c
index 601652a6e08b3f1c2da03db0764bbe5ecaf38128..23657cee6a7702c3aacd81783acb1516b3960a1b 100644 (file)
--- a/stat.c
+++ b/stat.c
@@ -414,6 +414,18 @@ static void display_lat(const char *name, unsigned long long min,
        free(maxp);
 }
 
+static double convert_agg_kbytes_percent(struct group_run_stats *rs, int ddir, int mean)
+{
+       double p_of_agg = 100.0;
+       if (rs && rs->agg[ddir] > 1024) {
+               p_of_agg = mean * 100 / (double) (rs->agg[ddir] / 1024.0);
+
+               if (p_of_agg > 100.0)
+                       p_of_agg = 100.0;
+       }
+       return p_of_agg;
+}
+
 static void show_ddir_status(struct group_run_stats *rs, struct thread_stat *ts,
                             int ddir, struct buf_output *out)
 {
@@ -482,27 +494,43 @@ static void show_ddir_status(struct group_run_stats *rs, struct thread_stat *ts,
                display_lat("clat", min, max, mean, dev, out);
        if (calc_lat(&ts->lat_stat[ddir], &min, &max, &mean, &dev))
                display_lat(" lat", min, max, mean, dev, out);
-       if (calc_lat(&ts->clat_high_prio_stat[ddir], &min, &max, &mean, &dev))
-               display_lat("prio_clat", min, max, mean, dev, out);
+       if (calc_lat(&ts->clat_high_prio_stat[ddir], &min, &max, &mean, &dev)) {
+               display_lat(ts->lat_percentiles ? "high prio_lat" : "high prio_clat",
+                               min, max, mean, dev, out);
+               if (calc_lat(&ts->clat_low_prio_stat[ddir], &min, &max, &mean, &dev))
+                       display_lat(ts->lat_percentiles ? "low prio_lat" : "low prio_clat",
+                                       min, max, mean, dev, out);
+       }
+
+       if (ts->slat_percentiles && ts->slat_stat[ddir].samples > 0)
+               show_clat_percentiles(ts->io_u_plat[FIO_SLAT][ddir],
+                                       ts->slat_stat[ddir].samples,
+                                       ts->percentile_list,
+                                       ts->percentile_precision, "slat", out);
+       if (ts->clat_percentiles && ts->clat_stat[ddir].samples > 0)
+               show_clat_percentiles(ts->io_u_plat[FIO_CLAT][ddir],
+                                       ts->clat_stat[ddir].samples,
+                                       ts->percentile_list,
+                                       ts->percentile_precision, "clat", out);
+       if (ts->lat_percentiles && ts->lat_stat[ddir].samples > 0)
+               show_clat_percentiles(ts->io_u_plat[FIO_LAT][ddir],
+                                       ts->lat_stat[ddir].samples,
+                                       ts->percentile_list,
+                                       ts->percentile_precision, "lat", out);
 
        if (ts->clat_percentiles || ts->lat_percentiles) {
-               const char *name = ts->clat_percentiles ? "clat" : " lat";
+               const char *name = ts->lat_percentiles ? "lat" : "clat";
                char prio_name[32];
                uint64_t samples;
 
-               if (ts->clat_percentiles)
-                       samples = ts->clat_stat[ddir].samples;
-               else
+               if (ts->lat_percentiles)
                        samples = ts->lat_stat[ddir].samples;
-
-               show_clat_percentiles(ts->io_u_plat[FIO_CLAT][ddir],
-                                       samples,
-                                       ts->percentile_list,
-                                       ts->percentile_precision, name, out);
+               else
+                       samples = ts->clat_stat[ddir].samples;
 
                /* Only print this if some high and low priority stats were collected */
                if (ts->clat_high_prio_stat[ddir].samples > 0 &&
-                       ts->clat_prio_stat[ddir].samples > 0)
+                       ts->clat_low_prio_stat[ddir].samples > 0)
                {
                        sprintf(prio_name, "high prio (%.2f%%) %s",
                                        100. * (double) ts->clat_high_prio_stat[ddir].samples / (double) samples,
@@ -513,14 +541,15 @@ static void show_ddir_status(struct group_run_stats *rs, struct thread_stat *ts,
                                                ts->percentile_precision, prio_name, out);
 
                        sprintf(prio_name, "low prio (%.2f%%) %s",
-                                       100. * (double) ts->clat_prio_stat[ddir].samples / (double) samples,
+                                       100. * (double) ts->clat_low_prio_stat[ddir].samples / (double) samples,
                                        name);
-                       show_clat_percentiles(ts->io_u_plat_prio[ddir],
-                                               ts->clat_prio_stat[ddir].samples,
+                       show_clat_percentiles(ts->io_u_plat_low_prio[ddir],
+                                               ts->clat_low_prio_stat[ddir].samples,
                                                ts->percentile_list,
                                                ts->percentile_precision, prio_name, out);
                }
        }
+
        if (calc_lat(&ts->bw_stat[ddir], &min, &max, &mean, &dev)) {
                double p_of_agg = 100.0, fkb_base = (double)rs->kb_base;
                const char *bw_str;
@@ -534,11 +563,7 @@ static void show_ddir_status(struct group_run_stats *rs, struct thread_stat *ts,
                else
                        bw_str = "kB";
 
-               if (rs->agg[ddir]) {
-                       p_of_agg = mean * 100 / (double) (rs->agg[ddir] / 1024);
-                       if (p_of_agg > 100.0)
-                               p_of_agg = 100.0;
-               }
+               p_of_agg = convert_agg_kbytes_percent(rs, ddir, mean);
 
                if (rs->unit_base == 1) {
                        min *= 8.0;
@@ -937,7 +962,7 @@ void json_array_add_disk_util(struct disk_util_stat *dus,
        obj = json_create_object();
        json_array_add_value_object(array, obj);
 
-       json_object_add_value_string(obj, "name", dus->name);
+       json_object_add_value_string(obj, "name", (const char *)dus->name);
        json_object_add_value_int(obj, "read_ios", dus->s.ios[0]);
        json_object_add_value_int(obj, "write_ios", dus->s.ios[1]);
        json_object_add_value_int(obj, "read_merges", dus->s.merges[0]);
@@ -1170,12 +1195,17 @@ static void show_ddir_status_terse(struct thread_stat *ts,
        else
                log_buf(out, ";%llu;%llu;%f;%f", 0ULL, 0ULL, 0.0, 0.0);
 
-       if (ts->clat_percentiles || ts->lat_percentiles) {
+       if (ts->lat_percentiles)
+               len = calc_clat_percentiles(ts->io_u_plat[FIO_LAT][ddir],
+                                       ts->lat_stat[ddir].samples,
+                                       ts->percentile_list, &ovals, &maxv,
+                                       &minv);
+       else if (ts->clat_percentiles)
                len = calc_clat_percentiles(ts->io_u_plat[FIO_CLAT][ddir],
                                        ts->clat_stat[ddir].samples,
                                        ts->percentile_list, &ovals, &maxv,
                                        &minv);
-       else
+       else
                len = 0;
 
        for (i = 0; i < FIO_IO_U_LIST_MAX_LEN; i++) {
@@ -1221,18 +1251,63 @@ static void show_ddir_status_terse(struct thread_stat *ts,
        }
 }
 
+static struct json_object *add_ddir_lat_json(struct thread_stat *ts, uint32_t percentiles,
+               struct io_stat *lat_stat, uint64_t *io_u_plat)
+{
+       char buf[120];
+       double mean, dev;
+       unsigned int i, len;
+       struct json_object *lat_object, *percentile_object, *clat_bins_object;
+       unsigned long long min, max, maxv, minv, *ovals = NULL;
+
+       if (!calc_lat(lat_stat, &min, &max, &mean, &dev)) {
+               min = max = 0;
+               mean = dev = 0.0;
+       }
+       lat_object = json_create_object();
+       json_object_add_value_int(lat_object, "min", min);
+       json_object_add_value_int(lat_object, "max", max);
+       json_object_add_value_float(lat_object, "mean", mean);
+       json_object_add_value_float(lat_object, "stddev", dev);
+       json_object_add_value_int(lat_object, "N", lat_stat->samples);
+
+       if (percentiles && lat_stat->samples) {
+               len = calc_clat_percentiles(io_u_plat, lat_stat->samples,
+                               ts->percentile_list, &ovals, &maxv, &minv);
+
+               if (len > FIO_IO_U_LIST_MAX_LEN)
+                       len = FIO_IO_U_LIST_MAX_LEN;
+
+               percentile_object = json_create_object();
+               json_object_add_value_object(lat_object, "percentile", percentile_object);
+               for (i = 0; i < len; i++) {
+                       snprintf(buf, sizeof(buf), "%f", ts->percentile_list[i].u.f);
+                       json_object_add_value_int(percentile_object, buf, ovals[i]);
+               }
+               free(ovals);
+
+               if (output_format & FIO_OUTPUT_JSON_PLUS) {
+                       clat_bins_object = json_create_object();
+                       json_object_add_value_object(lat_object, "bins", clat_bins_object);
+
+                       for(i = 0; i < FIO_IO_U_PLAT_NR; i++)
+                               if (io_u_plat[i]) {
+                                       snprintf(buf, sizeof(buf), "%llu", plat_idx_to_val(i));
+                                       json_object_add_value_int(clat_bins_object, buf, io_u_plat[i]);
+                               }
+               }
+       }
+
+       return lat_object;
+}
+
 static void add_ddir_status_json(struct thread_stat *ts,
                struct group_run_stats *rs, int ddir, struct json_object *parent)
 {
-       unsigned long long min, max, minv, maxv;
+       unsigned long long min, max;
        unsigned long long bw_bytes, bw;
-       unsigned long long *ovals = NULL;
        double mean, dev, iops;
-       unsigned int len;
-       int i;
-       struct json_object *dir_object, *tmp_object, *percentile_object = NULL,
-               *clat_bins_object = NULL;
-       char buf[120];
+       struct json_object *dir_object, *tmp_object;
        double p_of_agg = 100.0;
 
        assert(ddir_rw(ddir) || ddir_sync(ddir));
@@ -1266,233 +1341,55 @@ static void add_ddir_status_json(struct thread_stat *ts,
                json_object_add_value_int(dir_object, "short_ios", ts->short_io_u[ddir]);
                json_object_add_value_int(dir_object, "drop_ios", ts->drop_io_u[ddir]);
 
-               if (!calc_lat(&ts->slat_stat[ddir], &min, &max, &mean, &dev)) {
-                       min = max = 0;
-                       mean = dev = 0.0;
-               }
-               tmp_object = json_create_object();
+               tmp_object = add_ddir_lat_json(ts, ts->slat_percentiles,
+                               &ts->slat_stat[ddir], ts->io_u_plat[FIO_SLAT][ddir]);
                json_object_add_value_object(dir_object, "slat_ns", tmp_object);
-               json_object_add_value_int(tmp_object, "min", min);
-               json_object_add_value_int(tmp_object, "max", max);
-               json_object_add_value_float(tmp_object, "mean", mean);
-               json_object_add_value_float(tmp_object, "stddev", dev);
-
-               if (!calc_lat(&ts->clat_stat[ddir], &min, &max, &mean, &dev)) {
-                       min = max = 0;
-                       mean = dev = 0.0;
-               }
-               tmp_object = json_create_object();
+
+               tmp_object = add_ddir_lat_json(ts, ts->clat_percentiles,
+                               &ts->clat_stat[ddir], ts->io_u_plat[FIO_CLAT][ddir]);
                json_object_add_value_object(dir_object, "clat_ns", tmp_object);
-               json_object_add_value_int(tmp_object, "min", min);
-               json_object_add_value_int(tmp_object, "max", max);
-               json_object_add_value_float(tmp_object, "mean", mean);
-               json_object_add_value_float(tmp_object, "stddev", dev);
-       } else {
-               if (!calc_lat(&ts->sync_stat, &min, &max, &mean, &dev)) {
-                       min = max = 0;
-                       mean = dev = 0.0;
-               }
 
-               tmp_object = json_create_object();
+               tmp_object = add_ddir_lat_json(ts, ts->lat_percentiles,
+                               &ts->lat_stat[ddir], ts->io_u_plat[FIO_LAT][ddir]);
                json_object_add_value_object(dir_object, "lat_ns", tmp_object);
+       } else {
                json_object_add_value_int(dir_object, "total_ios", ts->total_io_u[DDIR_SYNC]);
-               json_object_add_value_int(tmp_object, "min", min);
-               json_object_add_value_int(tmp_object, "max", max);
-               json_object_add_value_float(tmp_object, "mean", mean);
-               json_object_add_value_float(tmp_object, "stddev", dev);
-       }
-
-       if (ts->clat_percentiles || ts->lat_percentiles) {
-               if (ddir_rw(ddir)) {
-                       uint64_t samples;
-
-                       if (ts->clat_percentiles)
-                               samples = ts->clat_stat[ddir].samples;
-                       else
-                               samples = ts->lat_stat[ddir].samples;
-
-                       len = calc_clat_percentiles(ts->io_u_plat[FIO_CLAT][ddir],
-                                       samples, ts->percentile_list, &ovals,
-                                       &maxv, &minv);
-               } else {
-                       len = calc_clat_percentiles(ts->io_u_sync_plat,
-                                       ts->sync_stat.samples,
-                                       ts->percentile_list, &ovals, &maxv,
-                                       &minv);
-               }
-
-               if (len > FIO_IO_U_LIST_MAX_LEN)
-                       len = FIO_IO_U_LIST_MAX_LEN;
-       } else
-               len = 0;
-
-       if (ts->clat_percentiles) {
-               percentile_object = json_create_object();
-               json_object_add_value_object(tmp_object, "percentile", percentile_object);
-               for (i = 0; i < len; i++) {
-                       snprintf(buf, sizeof(buf), "%f",
-                                ts->percentile_list[i].u.f);
-                       json_object_add_value_int(percentile_object, buf,
-                                                 ovals[i]);
-               }
-       }
-
-       free(ovals);
-
-       if (output_format & FIO_OUTPUT_JSON_PLUS && ts->clat_percentiles) {
-               clat_bins_object = json_create_object();
-               json_object_add_value_object(tmp_object, "bins",
-                                            clat_bins_object);
-
-               for(i = 0; i < FIO_IO_U_PLAT_NR; i++) {
-                       if (ddir_rw(ddir)) {
-                               if (ts->io_u_plat[FIO_CLAT][ddir][i]) {
-                                       snprintf(buf, sizeof(buf), "%llu", plat_idx_to_val(i));
-                                       json_object_add_value_int(clat_bins_object, buf, ts->io_u_plat[FIO_CLAT][ddir][i]);
-                               }
-                       } else {
-                               if (ts->io_u_sync_plat[i]) {
-                                       snprintf(buf, sizeof(buf), "%llu", plat_idx_to_val(i));
-                                       json_object_add_value_int(clat_bins_object, buf, ts->io_u_sync_plat[i]);
-                               }
-                       }
-               }
+               tmp_object = add_ddir_lat_json(ts, ts->lat_percentiles | ts->clat_percentiles,
+                               &ts->sync_stat, ts->io_u_sync_plat);
+               json_object_add_value_object(dir_object, "lat_ns", tmp_object);
        }
 
+       if (!ddir_rw(ddir))
+               return;
 
        /* Only print PRIO latencies if some high priority samples were gathered */
        if (ts->clat_high_prio_stat[ddir].samples > 0) {
-               /* START OF HIGH PRIO CLAT */
-           if (!calc_lat(&ts->clat_high_prio_stat[ddir], &min, &max, &mean, &dev)) {
-                       min = max = 0;
-                       mean = dev = 0.0;
-               }
-               tmp_object = json_create_object();
-               json_object_add_value_object(dir_object, "clat_prio",
-                               tmp_object);
-               json_object_add_value_int(tmp_object, "samples",
-                               ts->clat_high_prio_stat[ddir].samples);
-               json_object_add_value_int(tmp_object, "min", min);
-               json_object_add_value_int(tmp_object, "max", max);
-               json_object_add_value_float(tmp_object, "mean", mean);
-               json_object_add_value_float(tmp_object, "stddev", dev);
-
-               if (ts->clat_percentiles) {
-                       len = calc_clat_percentiles(ts->io_u_plat_high_prio[ddir],
-                                               ts->clat_high_prio_stat[ddir].samples,
-                                               ts->percentile_list, &ovals, &maxv,
-                                               &minv);
-               } else
-                       len = 0;
-
-               percentile_object = json_create_object();
-               json_object_add_value_object(tmp_object, "percentile", percentile_object);
-               for (i = 0; i < FIO_IO_U_LIST_MAX_LEN; i++) {
-                       if (i >= len) {
-                               json_object_add_value_int(percentile_object, "0.00", 0);
-                               continue;
-                       }
-                       snprintf(buf, sizeof(buf), "%f", ts->percentile_list[i].u.f);
-                       json_object_add_value_int(percentile_object, (const char *)buf, ovals[i]);
-               }
+               const char *high, *low;
 
-               if (output_format & FIO_OUTPUT_JSON_PLUS) {
-                       clat_bins_object = json_create_object();
-                       json_object_add_value_object(tmp_object, "bins", clat_bins_object);
-                       for(i = 0; i < FIO_IO_U_PLAT_NR; i++) {
-                               snprintf(buf, sizeof(buf), "%d", i);
-                               json_object_add_value_int(clat_bins_object, (const char *)buf,
-                                               ts->io_u_plat_high_prio[ddir][i]);
-                       }
-                       json_object_add_value_int(clat_bins_object, "FIO_IO_U_PLAT_BITS",
-                                       FIO_IO_U_PLAT_BITS);
-                       json_object_add_value_int(clat_bins_object, "FIO_IO_U_PLAT_VAL",
-                                       FIO_IO_U_PLAT_VAL);
-                       json_object_add_value_int(clat_bins_object, "FIO_IO_U_PLAT_NR",
-                                       FIO_IO_U_PLAT_NR);
-               }
-               /* END OF HIGH PRIO CLAT */
-
-               /* START OF PRIO CLAT */
-           if (!calc_lat(&ts->clat_prio_stat[ddir], &min, &max, &mean, &dev)) {
-                       min = max = 0;
-                       mean = dev = 0.0;
-               }
-               tmp_object = json_create_object();
-               json_object_add_value_object(dir_object, "clat_low_prio",
-                               tmp_object);
-               json_object_add_value_int(tmp_object, "samples",
-                               ts->clat_prio_stat[ddir].samples);
-               json_object_add_value_int(tmp_object, "min", min);
-               json_object_add_value_int(tmp_object, "max", max);
-               json_object_add_value_float(tmp_object, "mean", mean);
-               json_object_add_value_float(tmp_object, "stddev", dev);
-
-               if (ts->clat_percentiles) {
-                       len = calc_clat_percentiles(ts->io_u_plat_prio[ddir],
-                                               ts->clat_prio_stat[ddir].samples,
-                                               ts->percentile_list, &ovals, &maxv,
-                                               &minv);
-               } else
-                       len = 0;
-
-               percentile_object = json_create_object();
-               json_object_add_value_object(tmp_object, "percentile", percentile_object);
-               for (i = 0; i < FIO_IO_U_LIST_MAX_LEN; i++) {
-                       if (i >= len) {
-                               json_object_add_value_int(percentile_object, "0.00", 0);
-                               continue;
-                       }
-                       snprintf(buf, sizeof(buf), "%f", ts->percentile_list[i].u.f);
-                       json_object_add_value_int(percentile_object, (const char *)buf, ovals[i]);
-               }
-
-               if (output_format & FIO_OUTPUT_JSON_PLUS) {
-                       clat_bins_object = json_create_object();
-                       json_object_add_value_object(tmp_object, "bins", clat_bins_object);
-                       for(i = 0; i < FIO_IO_U_PLAT_NR; i++) {
-                               snprintf(buf, sizeof(buf), "%d", i);
-                               json_object_add_value_int(clat_bins_object, (const char *)buf,
-                                               ts->io_u_plat_prio[ddir][i]);
-                       }
-                       json_object_add_value_int(clat_bins_object, "FIO_IO_U_PLAT_BITS",
-                                       FIO_IO_U_PLAT_BITS);
-                       json_object_add_value_int(clat_bins_object, "FIO_IO_U_PLAT_VAL",
-                                       FIO_IO_U_PLAT_VAL);
-                       json_object_add_value_int(clat_bins_object, "FIO_IO_U_PLAT_NR",
-                                       FIO_IO_U_PLAT_NR);
+               if (ts->lat_percentiles) {
+                       high = "lat_high_prio";
+                       low = "lat_low_prio";
+               } else {
+                       high = "clat_high_prio";
+                       low = "clat_low_prio";
                }
-               /* END OF PRIO CLAT */
-       }
 
-       if (!ddir_rw(ddir))
-               return;
+               tmp_object = add_ddir_lat_json(ts, ts->clat_percentiles | ts->lat_percentiles,
+                               &ts->clat_high_prio_stat[ddir], ts->io_u_plat_high_prio[ddir]);
+               json_object_add_value_object(dir_object, high, tmp_object);
 
-       if (!calc_lat(&ts->lat_stat[ddir], &min, &max, &mean, &dev)) {
-               min = max = 0;
-               mean = dev = 0.0;
+               tmp_object = add_ddir_lat_json(ts, ts->clat_percentiles | ts->lat_percentiles,
+                               &ts->clat_low_prio_stat[ddir], ts->io_u_plat_low_prio[ddir]);
+               json_object_add_value_object(dir_object, low, tmp_object);
        }
-       tmp_object = json_create_object();
-       json_object_add_value_object(dir_object, "lat_ns", tmp_object);
-       json_object_add_value_int(tmp_object, "min", min);
-       json_object_add_value_int(tmp_object, "max", max);
-       json_object_add_value_float(tmp_object, "mean", mean);
-       json_object_add_value_float(tmp_object, "stddev", dev);
-       if (ts->lat_percentiles)
-               json_object_add_value_object(tmp_object, "percentile", percentile_object);
-       if (output_format & FIO_OUTPUT_JSON_PLUS && ts->lat_percentiles)
-               json_object_add_value_object(tmp_object, "bins", clat_bins_object);
 
        if (calc_lat(&ts->bw_stat[ddir], &min, &max, &mean, &dev)) {
-               if (rs->agg[ddir]) {
-                       p_of_agg = mean * 100 / (double) (rs->agg[ddir] / 1024);
-                       if (p_of_agg > 100.0)
-                               p_of_agg = 100.0;
-               }
+               p_of_agg = convert_agg_kbytes_percent(rs, ddir, mean);
        } else {
                min = max = 0;
                p_of_agg = mean = dev = 0.0;
        }
+
        json_object_add_value_int(dir_object, "bw_min", min);
        json_object_add_value_int(dir_object, "bw_max", max);
        json_object_add_value_float(dir_object, "bw_agg", p_of_agg);
@@ -1613,12 +1510,8 @@ static void json_add_job_opts(struct json_object *root, const char *name,
        json_object_add_value_object(root, name, dir_object);
 
        flist_for_each(entry, opt_list) {
-               const char *pos = "";
-
                p = flist_entry(entry, struct print_option, list);
-               if (p->value)
-                       pos = p->value;
-               json_object_add_value_string(dir_object, p->name, pos);
+               json_object_add_value_string(dir_object, p->name, p->value);
        }
 }
 
@@ -1987,7 +1880,7 @@ void sum_thread_stats(struct thread_stat *dst, struct thread_stat *src,
                if (!dst->unified_rw_rep) {
                        sum_stat(&dst->clat_stat[l], &src->clat_stat[l], first, false);
                        sum_stat(&dst->clat_high_prio_stat[l], &src->clat_high_prio_stat[l], first, false);
-                       sum_stat(&dst->clat_prio_stat[l], &src->clat_prio_stat[l], first, false);
+                       sum_stat(&dst->clat_low_prio_stat[l], &src->clat_low_prio_stat[l], first, false);
                        sum_stat(&dst->slat_stat[l], &src->slat_stat[l], first, false);
                        sum_stat(&dst->lat_stat[l], &src->lat_stat[l], first, false);
                        sum_stat(&dst->bw_stat[l], &src->bw_stat[l], first, true);
@@ -1999,8 +1892,8 @@ void sum_thread_stats(struct thread_stat *dst, struct thread_stat *src,
                                dst->runtime[l] = src->runtime[l];
                } else {
                        sum_stat(&dst->clat_stat[0], &src->clat_stat[l], first, false);
-                       sum_stat(&dst->clat_high_prio_stat[l], &src->clat_high_prio_stat[l], first, false);
-                       sum_stat(&dst->clat_prio_stat[l], &src->clat_prio_stat[l], first, false);
+                       sum_stat(&dst->clat_high_prio_stat[0], &src->clat_high_prio_stat[l], first, false);
+                       sum_stat(&dst->clat_low_prio_stat[0], &src->clat_low_prio_stat[l], first, false);
                        sum_stat(&dst->slat_stat[0], &src->slat_stat[l], first, false);
                        sum_stat(&dst->lat_stat[0], &src->lat_stat[l], first, false);
                        sum_stat(&dst->bw_stat[0], &src->bw_stat[l], first, true);
@@ -2068,10 +1961,10 @@ void sum_thread_stats(struct thread_stat *dst, struct thread_stat *src,
                for (m = 0; m < FIO_IO_U_PLAT_NR; m++) {
                        if (!dst->unified_rw_rep) {
                                dst->io_u_plat_high_prio[k][m] += src->io_u_plat_high_prio[k][m];
-                               dst->io_u_plat_prio[k][m] += src->io_u_plat_prio[k][m];
+                               dst->io_u_plat_low_prio[k][m] += src->io_u_plat_low_prio[k][m];
                        } else {
                                dst->io_u_plat_high_prio[0][m] += src->io_u_plat_high_prio[k][m];
-                               dst->io_u_plat_prio[0][m] += src->io_u_plat_prio[k][m];
+                               dst->io_u_plat_low_prio[0][m] += src->io_u_plat_low_prio[k][m];
                        }
 
                }
@@ -2107,7 +2000,7 @@ void init_thread_stat(struct thread_stat *ts)
                ts->bw_stat[j].min_val = -1UL;
                ts->iops_stat[j].min_val = -1UL;
                ts->clat_high_prio_stat[j].min_val = -1UL;
-               ts->clat_prio_stat[j].min_val = -1UL;
+               ts->clat_low_prio_stat[j].min_val = -1UL;
        }
        ts->sync_stat.min_val = -1UL;
        ts->groupid = -1;
@@ -2177,6 +2070,7 @@ void __show_run_stats(void)
 
                ts->clat_percentiles = td->o.clat_percentiles;
                ts->lat_percentiles = td->o.lat_percentiles;
+               ts->slat_percentiles = td->o.slat_percentiles;
                ts->percentile_precision = td->o.percentile_precision;
                memcpy(ts->percentile_list, td->o.percentile_list, sizeof(td->o.percentile_list));
                opt_lists[j] = &td->opt_list;
@@ -2460,8 +2354,6 @@ void __show_running_run_stats(void)
        fio_sem_up(stat_sem);
 }
 
-static bool status_interval_init;
-static struct timespec status_time;
 static bool status_file_disabled;
 
 #define FIO_STATUS_FILE                "fio-dump-status"
@@ -2504,16 +2396,6 @@ static int check_status_file(void)
 
 void check_for_running_stats(void)
 {
-       if (status_interval) {
-               if (!status_interval_init) {
-                       fio_gettime(&status_time, NULL);
-                       status_interval_init = true;
-               } else if (mtime_since_now(&status_time) >= status_interval) {
-                       show_running_run_stats();
-                       fio_gettime(&status_time, NULL);
-                       return;
-               }
-       }
        if (check_status_file()) {
                show_running_run_stats();
                return;
@@ -2736,7 +2618,7 @@ void reset_io_stats(struct thread_data *td)
 
        for (i = 0; i < DDIR_RWDIR_CNT; i++) {
                reset_io_stat(&ts->clat_high_prio_stat[i]);
-               reset_io_stat(&ts->clat_prio_stat[i]);
+               reset_io_stat(&ts->clat_low_prio_stat[i]);
                reset_io_stat(&ts->clat_stat[i]);
                reset_io_stat(&ts->slat_stat[i]);
                reset_io_stat(&ts->lat_stat[i]);
@@ -2751,7 +2633,7 @@ void reset_io_stats(struct thread_data *td)
 
                for (j = 0; j < FIO_IO_U_PLAT_NR; j++) {
                        ts->io_u_plat_high_prio[i][j] = 0;
-                       ts->io_u_plat_prio[i][j] = 0;
+                       ts->io_u_plat_low_prio[i][j] = 0;
                        if (!i)
                                ts->io_u_sync_plat[j] = 0;
                }
@@ -2855,7 +2737,7 @@ static unsigned long add_log_sample(struct thread_data *td,
                        return diff;
        }
 
-       _add_stat_to_log(iolog, elapsed, td->o.log_max != 0, priority_bit);
+       __add_stat_to_log(iolog, ddir, elapsed, td->o.log_max != 0, priority_bit);
 
        iolog->avg_last[ddir] = elapsed - (this_window - iolog->avg_msec);
        return iolog->avg_msec;
@@ -2900,19 +2782,27 @@ void add_sync_clat_sample(struct thread_stat *ts, unsigned long long nsec)
        add_stat_sample(&ts->sync_stat, nsec);
 }
 
-static void add_clat_percentile_sample(struct thread_stat *ts,
-                               unsigned long long nsec, enum fio_ddir ddir, uint8_t priority_bit)
+static void add_lat_percentile_sample_noprio(struct thread_stat *ts,
+                               unsigned long long nsec, enum fio_ddir ddir, enum fio_lat lat)
 {
        unsigned int idx = plat_val_to_idx(nsec);
        assert(idx < FIO_IO_U_PLAT_NR);
 
-       ts->io_u_plat[FIO_CLAT][ddir][idx]++;
+       ts->io_u_plat[lat][ddir][idx]++;
+}
+
+static void add_lat_percentile_sample(struct thread_stat *ts,
+                               unsigned long long nsec, enum fio_ddir ddir, uint8_t priority_bit,
+                               enum fio_lat lat)
+{
+       unsigned int idx = plat_val_to_idx(nsec);
 
-       if (!priority_bit) {
-               ts->io_u_plat_prio[ddir][idx]++;
-       } else {
+       add_lat_percentile_sample_noprio(ts, nsec, ddir, lat);
+
+       if (!priority_bit)
+               ts->io_u_plat_low_prio[ddir][idx]++;
+       else
                ts->io_u_plat_high_prio[ddir][idx]++;
-       }
 }
 
 void add_clat_sample(struct thread_data *td, enum fio_ddir ddir,
@@ -2929,10 +2819,11 @@ void add_clat_sample(struct thread_data *td, enum fio_ddir ddir,
 
        add_stat_sample(&ts->clat_stat[ddir], nsec);
 
-       if (priority_bit) {
-               add_stat_sample(&ts->clat_high_prio_stat[ddir], nsec);
-       } else {
-               add_stat_sample(&ts->clat_prio_stat[ddir], nsec);
+       if (!ts->lat_percentiles) {
+               if (priority_bit)
+                       add_stat_sample(&ts->clat_high_prio_stat[ddir], nsec);
+               else
+                       add_stat_sample(&ts->clat_low_prio_stat[ddir], nsec);
        }
 
        if (td->clat_log)
@@ -2940,7 +2831,10 @@ void add_clat_sample(struct thread_data *td, enum fio_ddir ddir,
                               offset, priority_bit);
 
        if (ts->clat_percentiles) {
-               add_clat_percentile_sample(ts, nsec, ddir, priority_bit);
+               if (ts->lat_percentiles)
+                       add_lat_percentile_sample_noprio(ts, nsec, ddir, FIO_CLAT);
+               else
+                       add_lat_percentile_sample(ts, nsec, ddir, priority_bit, FIO_CLAT);
        }
 
        if (iolog && iolog->hist_msec) {
@@ -2986,7 +2880,7 @@ void add_clat_sample(struct thread_data *td, enum fio_ddir ddir,
 }
 
 void add_slat_sample(struct thread_data *td, enum fio_ddir ddir,
-                       unsigned long usec, unsigned long long bs, uint64_t offset,
+                       unsigned long long nsec, unsigned long long bs, uint64_t offset,
                        uint8_t priority_bit)
 {
        const bool needs_lock = td_async_processing(td);
@@ -2998,12 +2892,15 @@ void add_slat_sample(struct thread_data *td, enum fio_ddir ddir,
        if (needs_lock)
                __td_io_u_lock(td);
 
-       add_stat_sample(&ts->slat_stat[ddir], usec);
+       add_stat_sample(&ts->slat_stat[ddir], nsec);
 
        if (td->slat_log)
-               add_log_sample(td, td->slat_log, sample_val(usec), ddir, bs, offset,
+               add_log_sample(td, td->slat_log, sample_val(nsec), ddir, bs, offset,
                        priority_bit);
 
+       if (ts->slat_percentiles)
+               add_lat_percentile_sample_noprio(ts, nsec, ddir, FIO_SLAT);
+
        if (needs_lock)
                __td_io_u_unlock(td);
 }
@@ -3027,9 +2924,14 @@ void add_lat_sample(struct thread_data *td, enum fio_ddir ddir,
                add_log_sample(td, td->lat_log, sample_val(nsec), ddir, bs,
                               offset, priority_bit);
 
-       if (ts->lat_percentiles)
-               add_clat_percentile_sample(ts, nsec, ddir, priority_bit);
+       if (ts->lat_percentiles) {
+               add_lat_percentile_sample(ts, nsec, ddir, priority_bit, FIO_LAT);
+               if (priority_bit)
+                       add_stat_sample(&ts->clat_high_prio_stat[ddir], nsec);
+               else
+                       add_stat_sample(&ts->clat_low_prio_stat[ddir], nsec);
 
+       }
        if (needs_lock)
                __td_io_u_unlock(td);
 }
@@ -3232,3 +3134,4 @@ uint32_t *io_u_block_info(struct thread_data *td, struct io_u *io_u)
        assert(idx < td->ts.nr_block_infos);
        return info;
 }
+