From ed4b89aaabde38efc0e4aa2cd08d17ea59d930f1 Mon Sep 17 00:00:00 2001 From: GangCao Date: Thu, 15 Dec 2022 03:34:36 -0500 Subject: [PATCH] examples/bdevperf: reset the performance period for next run In the case of running by below commands: ./examples/bdev/bdevperf/bdevperf.py perform_tests This value needs to be reset in each run, otherwise, it shows wrong IOPS in the periodical way with -S parameter. Change-Id: Ib8de7611aa06675d4a16fc3aa130d99b508b7605 Signed-off-by: GangCao Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15960 Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris Reviewed-by: Changpeng Liu --- examples/bdev/bdevperf/bdevperf.c | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/examples/bdev/bdevperf/bdevperf.c b/examples/bdev/bdevperf/bdevperf.c index 925e02a17..f368d4ea2 100644 --- a/examples/bdev/bdevperf/bdevperf.c +++ b/examples/bdev/bdevperf/bdevperf.c @@ -2209,6 +2209,19 @@ bdevperf_run(void *arg1) bdevperf_construct_job_configs(); } +static void +rpc_perform_tests_reset(void) +{ + /* Reset g_run_rc to 0 for the next test run. */ + g_run_rc = 0; + + /* Reset g_stats to 0 for the next test run. */ + memset(&g_stats, 0, sizeof(g_stats)); + + /* Reset g_show_performance_period_num to 0 for the next test run. */ + g_show_performance_period_num = 0; +} + static void rpc_perform_tests_cb(void) { @@ -2226,11 +2239,7 @@ rpc_perform_tests_cb(void) "bdevperf failed with error %s", spdk_strerror(-g_run_rc)); } - /* Reset g_run_rc to 0 for the next test run. */ - g_run_rc = 0; - - /* Reset g_stats to 0 for the next test run. */ - memset(&g_stats, 0, sizeof(g_stats)); + rpc_perform_tests_reset(); } static void