Commit 218fd85d authored by Dariusz Stojaczyk's avatar Dariusz Stojaczyk Committed by Jim Harris
Browse files

bdevio: check spdk_app_parse_args return code 



This makes us exit immediately on '-h' or any parsing error.

Change-Id: I6bd2fc9238bcecba6df1e62a8db42ce79ad73524
Signed-off-by: default avatarDariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/422686


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
parent df531e7f

test/bdev/bdevio/bdevio.c

+8 −5
Original line number Diff line number Diff line
@@ -952,7 +952,7 @@ bdevio_parse_arg(int ch, char *arg) 
int

main(int argc, char **argv)

{

	int			num_failures;

	int			rc;

	struct spdk_app_opts	opts = {};



	spdk_app_opts_init(&opts);
@@ -962,11 +962,14 @@ main(int argc, char **argv) 
	opts.reactor_mask = "0x7";

	opts.mem_size = 1024;



	spdk_app_parse_args(argc, argv, &opts, "", NULL,

			    bdevio_parse_arg, bdevio_usage);

	if ((rc = spdk_app_parse_args(argc, argv, &opts, "", NULL,

				      bdevio_parse_arg, bdevio_usage)) !=

	    SPDK_APP_PARSE_ARGS_SUCCESS) {

		return rc;

	}



	num_failures = spdk_app_start(&opts, test_main, NULL, NULL);

	rc = spdk_app_start(&opts, test_main, NULL, NULL);

	spdk_app_fini();



	return num_failures;

	return rc;

}