cf8405fc24
We had not held mutex while removing bdev name or alias from bdev name tree for most cases. Fix these in this patch. spdk_bdev_unregister() already holds g_bdev_mgr.mutex when removing name, and so we do not need to change it. spdk_bdev_close() had not held g_bdev_mgr.mutex. What we want to lock is only when removing name from name tree, that is, calling bdev_name_del() in bdev_unregister_unsafe(). However, we need to keep hierarchical lock ordering. Hence get and free g_bdev_mgr.mutex outside of bdev->internal.mutex. Signed-off-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Change-Id: I4e2c8604e27c8603725efa9bc0bee2013eccb2ac Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8527 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> |
||
|---|---|---|
| .. | ||
| bdev_internal.h | ||
| bdev_rpc.c | ||
| bdev_zone.c | ||
| bdev.c | ||
| Makefile | ||
| part.c | ||
| scsi_nvme.c | ||
| spdk_bdev.map | ||
| vtune.c | ||