ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bdevperf: prevent multiple performance statistics poller registering

Browse Source 
Current implementation allows to run bdevperf with --wait-for-rpc and
without -z option. As the result several statistics pollers will be
registered. Let's add guardian to prevent this.
In the case of multiple test running guardian also will be helpful.

Signed-off-by: Denis Nagorny <denisn@nvidia.com>
Change-Id: I885f9dba6fa324fe4fbaba104d3058bf9f686c19
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9613
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
This commit is contained in:
Denis Nagorny 2021-09-24 11:10:18 +03:00 committed by Keith Lucas
parent d822ddc7dd
commit ed31d5156d
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/bdev/bdevperf/bdevperf.c
View File

@ -1077,7 +1077,7 @@ bdevperf_test(void)
/* Start a timer to dump performance numbers */ /* Start a timer to dump performance numbers */
g_shutdown_tsc = spdk_get_ticks(); g_shutdown_tsc = spdk_get_ticks();
if (g_show_performance_real_time) { if (g_show_performance_real_time && !g_perf_timer) {
g_perf_timer = SPDK_POLLER_REGISTER(performance_statistics_thread, NULL, g_perf_timer = SPDK_POLLER_REGISTER(performance_statistics_thread, NULL,
g_show_performance_period_in_usec); g_show_performance_period_in_usec);
} }