From: Niklas Cassel <niklas.cassel@xxxxxxx>
The name for add_lat_percentile_sample() is confusing, since the function
actually adds a percentile sample to io_u_plat_{low,high}_prio,
yet it doesn't have prio as part of the function name.
Rename the function so that it is more obvious that this function should
be used if you want to add a prio percentile sample.
Signed-off-by: Niklas Cassel <niklas.cassel@xxxxxxx>
---
stat.c | 13 ++++++++-----
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/stat.c b/stat.c
index e0dc99b6..93f3a8ff 100644
--- a/stat.c
+++ b/stat.c
@@ -3061,9 +3061,10 @@ static void add_lat_percentile_sample_noprio(struct thread_stat *ts,
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,
- bool high_prio, enum fio_lat lat)
+static void add_lat_percentile_prio_sample(struct thread_stat *ts,
+ unsigned long long nsec,
+ enum fio_ddir ddir,
+ bool high_prio, enum fio_lat lat)
{
unsigned int idx = plat_val_to_idx(nsec);
@@ -3104,7 +3105,8 @@ void add_clat_sample(struct thread_data *td, enum fio_ddir ddir,
if (ts->lat_percentiles)
add_lat_percentile_sample_noprio(ts, nsec, ddir, FIO_CLAT);
else
- add_lat_percentile_sample(ts, nsec, ddir, high_prio, FIO_CLAT);
+ add_lat_percentile_prio_sample(ts, nsec, ddir, high_prio,
+ FIO_CLAT);
}
if (iolog && iolog->hist_msec) {
@@ -3195,7 +3197,8 @@ void add_lat_sample(struct thread_data *td, enum fio_ddir ddir,
offset, ioprio);
if (ts->lat_percentiles) {
- add_lat_percentile_sample(ts, nsec, ddir, high_prio, FIO_LAT);
+ add_lat_percentile_prio_sample(ts, nsec, ddir, high_prio,
+ FIO_LAT);
if (high_prio)
add_stat_sample(&ts->clat_high_prio_stat[ddir], nsec);
else
--
2.33.1
