Commit fa6f7a16 authored by Daniel Verkamp's avatar Daniel Verkamp
Browse files

nvme: improve error messages in set_num_qpairs 



The functions they were referring to have been renamed; rather than
fixing up the function names, use the spec-defined NVMe command names so
it's more understandable. (The second message was also incorrectly
referring to "set" instead of "get", which is fixed as well.)

Change-Id: Id140a91c837d8c913760d2f55318472689c00f45
Signed-off-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-on: https://review.gerrithub.io/407593


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
parent a4a497d5

lib/nvme/nvme_ctrlr.c

+2 −2
Original line number Diff line number Diff line
@@ -901,7 +901,7 @@ nvme_ctrlr_set_num_qpairs(struct spdk_nvme_ctrlr *ctrlr) 
		spdk_nvme_qpair_process_completions(ctrlr->adminq, 0);

	}

	if (spdk_nvme_cpl_is_error(&status.cpl)) {

		SPDK_ERRLOG("nvme_set_num_queues failed!\n");

		SPDK_ERRLOG("Set Features - Number of Queues failed!\n");

	}



	/* Obtain the number of queues allocated using Get Features. */
@@ -915,7 +915,7 @@ nvme_ctrlr_set_num_qpairs(struct spdk_nvme_ctrlr *ctrlr) 
		spdk_nvme_qpair_process_completions(ctrlr->adminq, 0);

	}

	if (spdk_nvme_cpl_is_error(&status.cpl)) {

		SPDK_ERRLOG("nvme_set_num_queues failed!\n");

		SPDK_ERRLOG("Get Features - Number of Queues failed!\n");

		ctrlr->opts.num_io_queues = 0;

	} else {

		/*