diff options
| author | Rakesh Pandit | 2017-10-13 14:45:50 +0200 |
|---|---|---|
| committer | Jens Axboe | 2017-10-13 16:34:57 +0200 |
| commit | 900148296b78c61aa8c443dc594c0da968c3be53 (patch) | |
| tree | 916dfc2000208ddb8cc5e829ddef190cf9049cff /drivers/lightnvm/core.c | |
| parent | mtip32xx: Clean up unused variables (diff) | |
| download | kernel-qcow2-linux-900148296b78c61aa8c443dc594c0da968c3be53.tar.gz kernel-qcow2-linux-900148296b78c61aa8c443dc594c0da968c3be53.tar.xz kernel-qcow2-linux-900148296b78c61aa8c443dc594c0da968c3be53.zip | |
lightnvm: prevent target type module removal when in use
If target type module e.g. pblk here is unloaded (rmmod) while module
is in use (after creating target) system crashes. We fix this by
using module API refcnt.
Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
Signed-off-by: Matias Bjørling <m@bjorling.me>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/lightnvm/core.c')
| -rw-r--r-- | drivers/lightnvm/core.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/lightnvm/core.c b/drivers/lightnvm/core.c index ddae430b6eae..60e163be5a89 100644 --- a/drivers/lightnvm/core.c +++ b/drivers/lightnvm/core.c @@ -22,6 +22,7 @@ #include <linux/types.h> #include <linux/sem.h> #include <linux/bitmap.h> +#include <linux/module.h> #include <linux/moduleparam.h> #include <linux/miscdevice.h> #include <linux/lightnvm.h> @@ -316,6 +317,8 @@ static int nvm_create_tgt(struct nvm_dev *dev, struct nvm_ioctl_create *create) list_add_tail(&t->list, &dev->targets); mutex_unlock(&dev->mlock); + __module_get(tt->owner); + return 0; err_sysfs: if (tt->exit) @@ -351,6 +354,7 @@ static void __nvm_remove_target(struct nvm_target *t) nvm_remove_tgt_dev(t->dev, 1); put_disk(tdisk); + module_put(t->type->owner); list_del(&t->list); kfree(t); |