Commit 56c67f01 authored by Konrad Sztyber's avatar Konrad Sztyber Committed by Tomasz Zawadzki
Browse files

nvme_compliance: move parse_args() near main 



It doens't make sense for the function parsing command-line arguments to
be defined in the middle of test functions.

Signed-off-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Change-Id: Ib1eced1908fc5c80e64f166eb87cbff1947f8f0b
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19289


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
parent 7cd209b3

test/nvme/compliance/nvme_compliance.c

+22 −22
Original line number Diff line number Diff line
@@ -734,28 +734,6 @@ fabric_property_get(void) 
	spdk_nvme_detach(ctrlr);

}



static int

parse_args(int argc, char **argv, struct spdk_env_opts *opts)

{

	int op;



	while ((op = getopt(argc, argv, "gr:")) != -1) {

		switch (op) {

		case 'g':

			opts->hugepage_single_segments = true;

			break;

		case 'r':

			g_trid_str = optarg;

			break;

		default:

			SPDK_ERRLOG("Unknown op '%c'\n", op);

			return -1;

		}

	}



	return 0;

}



static void

admin_set_features_number_of_queues(void)

{
@@ -1446,6 +1424,28 @@ admin_create_io_sq_shared_cq(void) 
	spdk_nvme_detach(ctrlr);

}



static int

parse_args(int argc, char **argv, struct spdk_env_opts *opts)

{

	int op;



	while ((op = getopt(argc, argv, "gr:")) != -1) {

		switch (op) {

		case 'g':

			opts->hugepage_single_segments = true;

			break;

		case 'r':

			g_trid_str = optarg;

			break;

		default:

			SPDK_ERRLOG("Unknown op '%c'\n", op);

			return -1;

		}

	}



	return 0;

}



int

main(int argc, char **argv)

{