AcoSail/kernel_optimize_test
8
0
Fork 0
forked from luck/tmp_suning_uos_patched
Code Pull Requests Activity

lightnvm: do not remove instance under global lock

Browse Source 
Currently all the target instances are removed under global nvm_lock.
This was needed to ensure that nvm_dev struct will not be freed by
hot unplug event during target removal. However, current implementation
has some drawbacks, since the same lock is used when new nvme subsystem
is registered, so we can have a situation, that due to long process of
target removal on drive A, registration (and listing in OS) of the
drive B will take a lot of time, since it will wait for that lock.

Now when we have kref which ensures that nvm_dev will not be freed in
the meantime, we can easily get rid of this lock for a time when we are
removing nvm targets.

Signed-off-by: Igor Konopko <igor.j.konopko@intel.com>
Reviewed-by: Javier González <javier@javigon.com>
Signed-off-by: Matias Bjørling <mb@lightnvm.io>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
This commit is contained in:
Igor Konopko 2019-05-04 20:38:09 +02:00 committed by Jens Axboe
parent e69397ea05
commit 843f2edbdd
1 changed files with 16 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
drivers/lightnvm/core.c
View File

@ -483,7 +483,6 @@ static void __nvm_remove_target(struct nvm_target *t, bool graceful)
/** /**
* nvm_remove_tgt - Removes a target from the media manager * nvm_remove_tgt - Removes a target from the media manager
* @dev: device
* @remove: ioctl structure with target name to remove. * @remove: ioctl structure with target name to remove.
* *
* Returns: * Returns:
@ -491,18 +490,27 @@ static void __nvm_remove_target(struct nvm_target *t, bool graceful)
* 1: on not found * 1: on not found
* <0: on error * <0: on error
*/ */
static int nvm_remove_tgt(struct nvm_dev *dev, struct nvm_ioctl_remove *remove) static int nvm_remove_tgt(struct nvm_ioctl_remove *remove)
{ {
struct nvm_target *t; struct nvm_target *t;
struct nvm_dev *dev;
mutex_lock(&dev->mlock); down_read(&nvm_lock);
t = nvm_find_target(dev, remove->tgtname); list_for_each_entry(dev, &nvm_devices, devices) {
if (!t) { mutex_lock(&dev->mlock);
t = nvm_find_target(dev, remove->tgtname);
if (t) {
mutex_unlock(&dev->mlock);
break;
}
mutex_unlock(&dev->mlock); mutex_unlock(&dev->mlock);
return 1;
} }
up_read(&nvm_lock);
if (!t)
return 1;
__nvm_remove_target(t, true); __nvm_remove_target(t, true);
mutex_unlock(&dev->mlock);
kref_put(&dev->ref, nvm_free); kref_put(&dev->ref, nvm_free);
return 0; return 0;
@ -1348,8 +1356,6 @@ static long nvm_ioctl_dev_create(struct file *file, void __user *arg)
static long nvm_ioctl_dev_remove(struct file *file, void __user *arg) static long nvm_ioctl_dev_remove(struct file *file, void __user *arg)
{ {
struct nvm_ioctl_remove remove; struct nvm_ioctl_remove remove;
struct nvm_dev *dev;
int ret = 0;
if (copy_from_user(&remove, arg, sizeof(struct nvm_ioctl_remove))) if (copy_from_user(&remove, arg, sizeof(struct nvm_ioctl_remove)))
return -EFAULT; return -EFAULT;
@ -1361,15 +1367,7 @@ static long nvm_ioctl_dev_remove(struct file *file, void __user *arg)
return -EINVAL; return -EINVAL;
} }
down_read(&nvm_lock); return nvm_remove_tgt(&remove);
list_for_each_entry(dev, &nvm_devices, devices) {
ret = nvm_remove_tgt(dev, &remove);
if (!ret)
break;
}
up_read(&nvm_lock);
return ret;
} }
/* kept for compatibility reasons */ /* kept for compatibility reasons */