From fc909ccfc7cd6e89c7176b22b4bd3b49c5b188e1 Mon Sep 17 00:00:00 2001 From: ragnarkj Date: Wed, 14 Jul 2021 11:19:11 +0100 Subject: [PATCH] Fix off-by-one error in verbose ops/sec messages. (#382) Summary: Upstream pull request #311 (https://github.com/hpc/ior/pull/311) added support for bencmarking directory renames, inserting an extra entry into the list of MDTEST operations, but missed updating some existing uses. This change fixes the remaining uses, to correct misleading ops/sec messages. Co-authored-by: Ragnar Kjorstad --- src/mdtest.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mdtest.c b/src/mdtest.c index cf2e373..14d02d3 100644 --- a/src/mdtest.c +++ b/src/mdtest.c @@ -1338,13 +1338,13 @@ void file_test(const int iteration, const int ntasks, const char *path, rank_pro o.items *= o.num_dirs_in_tree_calc; } - VERBOSE(1,-1," File creation : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_CREATE_NUM], o.summary_table[iteration].rate[4]); + VERBOSE(1,-1," File creation : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_CREATE_NUM], o.summary_table[iteration].rate[MDTEST_FILE_CREATE_NUM]); if(o.summary_table[iteration].stonewall_time[MDTEST_FILE_CREATE_NUM]){ VERBOSE(1,-1," File creation (stonewall): %14.3f sec, %14.3f ops/sec", o.summary_table[iteration].stonewall_time[MDTEST_FILE_CREATE_NUM], o.summary_table[iteration].stonewall_item_sum[MDTEST_FILE_CREATE_NUM]); } - VERBOSE(1,-1," File stat : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_STAT_NUM], o.summary_table[iteration].rate[5]); - VERBOSE(1,-1," File read : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_READ_NUM], o.summary_table[iteration].rate[6]); - VERBOSE(1,-1," File removal : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_REMOVE_NUM], o.summary_table[iteration].rate[7]); + VERBOSE(1,-1," File stat : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_STAT_NUM], o.summary_table[iteration].rate[MDTEST_FILE_STAT_NUM]); + VERBOSE(1,-1," File read : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_READ_NUM], o.summary_table[iteration].rate[MDTEST_FILE_READ_NUM]); + VERBOSE(1,-1," File removal : %14.3f sec, %14.3f ops/sec", res->time[MDTEST_FILE_REMOVE_NUM], o.summary_table[iteration].rate[MDTEST_FILE_REMOVE_NUM]); } char const * mdtest_test_name(int i){