Skip to content

Commit

Permalink
stat: fix unified_rw_reporting=both BW and IOPS
Browse files Browse the repository at this point in the history
When unified_rw_reporting=both, the value for the significant_figures
option is not populated in the struct thread_stat containing the mixed
r/w/t data. Thus, when bandwidth and IOPS numerical values are converted
to strings, they are converted to "0".

This patch populates the significant figures member of struct
thread_stat for the mixed r/w/t data.

Fixes: #1844
Signed-off-by: Vincent Fu <[email protected]>
  • Loading branch information
vincentkfu committed Dec 4, 2024
1 parent dc03e37 commit 6f3de5c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions stat.c
Original file line number Diff line number Diff line change
Expand Up @@ -507,6 +507,7 @@ static struct thread_stat *gen_mixed_ddir_stats_from_ts(struct thread_stat *ts)
ts_lcl->slat_percentiles = ts->slat_percentiles;
ts_lcl->percentile_precision = ts->percentile_precision;
memcpy(ts_lcl->percentile_list, ts->percentile_list, sizeof(ts->percentile_list));
ts_lcl->sig_figs = ts->sig_figs;

sum_thread_stats(ts_lcl, ts);

Expand Down

0 comments on commit 6f3de5c

Please sign in to comment.