Commit 0238b5c4 authored by wuzhouhui's avatar wuzhouhui Committed by Jim Harris
Browse files

bdev/ftl: unlock g_ftl_bdev_lock before unregister ftl_bdev 



There is no need to lock g_ftl_bdev_lock when unregister a ftl_bdev.
Besides, the destructor of ftl_bdev will lock it again.

Change-Id: I99870483183879d9422584dbac6e154f605daea8
Signed-off-by: default avatarwuzhouhui <wuzhouhui@kingsoft.com>
Reviewed-on: https://review.gerrithub.io/c/444794


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent b9e462e4

lib/bdev/ftl/bdev_ftl.c

+1 −2
Original line number Diff line number Diff line
@@ -887,9 +887,8 @@ bdev_ftl_delete_bdev(const char *name, spdk_bdev_unregister_cb cb_fn, void *cb_a 


	LIST_FOREACH_SAFE(ftl_bdev, &g_ftl_bdevs, list_entry, tmp) {

		if (strcmp(ftl_bdev->bdev.name, name) == 0) {

			spdk_bdev_unregister(&ftl_bdev->bdev, cb_fn, cb_arg);



			pthread_mutex_unlock(&g_ftl_bdev_lock);

			spdk_bdev_unregister(&ftl_bdev->bdev, cb_fn, cb_arg);

			return;

		}

	}