From 2d8fc1c3e6925eed72a8efe60d89d42fc3a00b00 Mon Sep 17 00:00:00 2001 From: Ben Walker Date: Wed, 3 Nov 2021 12:20:13 -0700 Subject: [PATCH] test/nvme: Reset tool now only registers active namespaces Only active namespaces can be used for I/O. Signed-off-by: Ben Walker Change-Id: Idf63e9f5e03244c1671288038fed54831463c3eb Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10098 Tested-by: SPDK CI Jenkins Community-CI: Broadcom CI Reviewed-by: Changpeng Liu Reviewed-by: Shuhei Matsumoto Reviewed-by: Aleksey Marchuk Reviewed-by: Jim Harris --- test/nvme/reset/reset.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/nvme/reset/reset.c b/test/nvme/reset/reset.c index e5562f20d..754494529 100644 --- a/test/nvme/reset/reset.c +++ b/test/nvme/reset/reset.c @@ -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;