Commit fcccf167 authored by Cunyin Chang's avatar Cunyin Chang Committed by Jim Harris
Browse files

nvmf: close all the bdevs when delete subsystems. 



Change-Id: Ie3c7c8d960b0074b86e983e0e8bf04de2d99f61f
Signed-off-by: default avatarCunyin Chang <cunyin.chang@intel.com>
Reviewed-on: https://review.gerrithub.io/390365


Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
parent 30dbca86

lib/nvmf/subsystem.c

+9 −0
Original line number Diff line number Diff line
@@ -171,6 +171,15 @@ spdk_nvmf_subsystem_delete_done(void *io_device, void *ctx, int status) 
{

	struct spdk_nvmf_tgt *tgt = io_device;

	struct spdk_nvmf_subsystem *subsystem = ctx;

	struct spdk_nvmf_ns *ns;



	for (ns = spdk_nvmf_subsystem_get_first_ns(subsystem); ns != NULL;

	     ns = spdk_nvmf_subsystem_get_next_ns(subsystem, ns)) {

		if (ns->bdev == NULL) {

			continue;

		}

		spdk_bdev_close(ns->desc);

	}



	free(subsystem->ns);

test/unit/lib/nvmf/ctrlr_discovery.c/ctrlr_discovery_ut.c

+5 −0
Original line number Diff line number Diff line
@@ -46,6 +46,11 @@ spdk_bdev_open(struct spdk_bdev *bdev, bool write, spdk_bdev_remove_cb_t remove_ 
	return 0;

}



void

spdk_bdev_close(struct spdk_bdev_desc *desc)

{

}



const char *

spdk_bdev_get_name(const struct spdk_bdev *bdev)

{

test/unit/lib/nvmf/subsystem.c/subsystem_ut.c

+5 −0
Original line number Diff line number Diff line
@@ -160,6 +160,11 @@ spdk_bdev_open(struct spdk_bdev *bdev, bool write, spdk_bdev_remove_cb_t remove_ 
	return 0;

}



void

spdk_bdev_close(struct spdk_bdev_desc *desc)

{

}



const char *

spdk_bdev_get_name(const struct spdk_bdev *bdev)

{