Commit d42045b2 authored by Denis Nagorny's avatar Denis Nagorny Committed by Tomasz Zawadzki
Browse files

bdevperf: test time wrong time fix 



spdk_bdevperf_shutdown_cb computed total time after the
bdevperf_test_done call. It's wrong. Let's get
g_shutdown_tsc delta when it is really needed.

Signed-off-by: default avatarDenis Nagorny <denisn@nvidia.com>
Change-Id: Ic4421305b16277af874008e293da8cb121c0244a
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9598


Community-CI: Mellanox Build Bot
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
parent 00d58714

test/bdev/bdevperf/bdevperf.c

+1 −2
Original line number Diff line number Diff line
@@ -389,6 +389,7 @@ bdevperf_test_done(void *ctx) 
	}



	if (g_shutdown) {

		g_shutdown_tsc = spdk_get_ticks() - g_shutdown_tsc;

		g_time_in_usec = g_shutdown_tsc * 1000000 / spdk_get_ticks_hz();

		printf("Received shutdown signal, test time was about %.6f seconds\n",

		       (double)g_time_in_usec / 1000000);
@@ -1925,8 +1926,6 @@ spdk_bdevperf_shutdown_cb(void) 
		return;

	}



	g_shutdown_tsc = spdk_get_ticks() - g_shutdown_tsc;



	/* Iterate jobs to stop all I/O */

	TAILQ_FOREACH_SAFE(job, &g_bdevperf.jobs, link, tmp) {

		spdk_thread_send_msg(job->thread, _bdevperf_job_drain, job);