Commit 29d94b7f authored by Alexey Marchuk's avatar Alexey Marchuk Committed by Tomasz Zawadzki
Browse files

nvmf: Stop all listeners when destroying transport 



When subsystem is destroyed, it removes its listeners,
however transport level listeners remain active.
This patch removes all transport listerners when
the transport is being destroyed.

Fixes issue 2353/

Signed-off-by: default avatarAlexey Marchuk <alexeymar@mellanox.com>
Change-Id: Ica7bcb0052b626aa62d0da9049bb8f216027dc49
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11307


Community-CI: Mellanox Build Bot
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
parent c708e6df

lib/nvmf/transport.c

+8 −0
Original line number Diff line number Diff line
@@ -284,6 +284,8 @@ int 
spdk_nvmf_transport_destroy(struct spdk_nvmf_transport *transport,

			    spdk_nvmf_transport_destroy_done_cb cb_fn, void *cb_arg)

{

	struct spdk_nvmf_listener *listener, *listener_tmp;



	if (transport->data_buf_pool != NULL) {

		if (spdk_mempool_count(transport->data_buf_pool) !=

		    transport->opts.num_shared_buffers) {
@@ -294,6 +296,12 @@ spdk_nvmf_transport_destroy(struct spdk_nvmf_transport *transport, 
		spdk_mempool_free(transport->data_buf_pool);

	}



	TAILQ_FOREACH_SAFE(listener, &transport->listeners, link, listener_tmp) {

		TAILQ_REMOVE(&transport->listeners, listener, link);

		transport->ops->stop_listen(transport, &listener->trid);

		free(listener);

	}



	return transport->ops->destroy(transport, cb_fn, cb_arg);

}