Commit 42cefebe authored by Tomasz Zawadzki's avatar Tomasz Zawadzki Committed by Jim Harris
Browse files

test/bdevio: exit with error when no bdevs were configured 



Tests shouldn't report success when no actual tests were ran.
Using bdevio without any bdevs configured, does not serve
any purpose and should report failure.

Change-Id: I489a0d03f5b5c2482dede56cad92befd97d24057
Signed-off-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454620


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
parent 871c8053

test/bdev/bdevio/bdevio.c

+5 −0
Original line number Diff line number Diff line
@@ -141,6 +141,11 @@ bdevio_construct_targets(void) 
		bdev = spdk_bdev_next_leaf(bdev);

	}



	if (g_io_targets == NULL) {

		SPDK_ERRLOG("No bdevs to perform tests on\n");

		return -1;

	}



	return 0;

}