Commit 6ccc8c1a authored by Ben Walker's avatar Ben Walker Committed by Jim Harris
Browse files

test/nvme: Reset tool now only registers active namespaces 



Only active namespaces can be used for I/O.

Signed-off-by: default avatarBen Walker <benjamin.walker@intel.com>
Change-Id: Idf63e9f5e03244c1671288038fed54831463c3eb
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10098


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 2dbdb994

test/nvme/reset/reset.c

+3 −3
Original line number Diff line number Diff line
@@ -128,7 +128,7 @@ register_ns(struct spdk_nvme_ctrlr *ctrlr, struct spdk_nvme_ns *ns) 
static void

register_ctrlr(struct spdk_nvme_ctrlr *ctrlr)

{

	int nsid, num_ns;

	int nsid;

	struct spdk_nvme_ns *ns;

	struct ctrlr_entry *entry = malloc(sizeof(struct ctrlr_entry));
@@ -140,8 +140,8 @@ register_ctrlr(struct spdk_nvme_ctrlr *ctrlr) 
	entry->ctrlr = ctrlr;

	TAILQ_INSERT_TAIL(&g_controllers, entry, link);



	num_ns = spdk_nvme_ctrlr_get_num_ns(ctrlr);

	for (nsid = 1; nsid <= num_ns; nsid++) {

	for (nsid = spdk_nvme_ctrlr_get_first_active_ns(ctrlr); nsid != 0;

	     nsid = spdk_nvme_ctrlr_get_next_active_ns(ctrlr, nsid)) {

		ns = spdk_nvme_ctrlr_get_ns(ctrlr, nsid);

		if (ns == NULL) {

			continue;