Commit b7692076 authored by Matias Bjørling's avatar Matias Bjørling Committed by Jens Axboe

lightnvm: use system block for mm initialization

Use system block information to register the appropriate media manager.
This enables the LightNVM subsystem to instantiate a media manager
selected by the user, instead of relying on automatic detection by each
media manager loaded in the kernel.

A device must now be initialized before it can proceed to initialize its
media manager. Upon initialization, the configured media manager is
automatically initialized as well.
Signed-off-by: default avatarMatias Bjørling <m@bjorling.me>
Signed-off-by: default avatarJens Axboe <axboe@fb.com>
parent 55696154
...@@ -106,6 +106,9 @@ struct nvmm_type *nvm_init_mgr(struct nvm_dev *dev) ...@@ -106,6 +106,9 @@ struct nvmm_type *nvm_init_mgr(struct nvm_dev *dev)
lockdep_assert_held(&nvm_lock); lockdep_assert_held(&nvm_lock);
list_for_each_entry(mt, &nvm_mgrs, list) { list_for_each_entry(mt, &nvm_mgrs, list) {
if (strncmp(dev->sb.mmtype, mt->name, NVM_MMTYPE_LEN))
continue;
ret = mt->register_mgr(dev); ret = mt->register_mgr(dev);
if (ret < 0) { if (ret < 0) {
pr_err("nvm: media mgr failed to init (%d) on dev %s\n", pr_err("nvm: media mgr failed to init (%d) on dev %s\n",
...@@ -569,9 +572,16 @@ int nvm_register(struct request_queue *q, char *disk_name, ...@@ -569,9 +572,16 @@ int nvm_register(struct request_queue *q, char *disk_name,
} }
} }
ret = nvm_get_sysblock(dev, &dev->sb);
if (!ret)
pr_err("nvm: device not initialized.\n");
else if (ret < 0)
pr_err("nvm: err (%d) on device initialization\n", ret);
/* register device with a supported media manager */ /* register device with a supported media manager */
down_write(&nvm_lock); down_write(&nvm_lock);
dev->mt = nvm_init_mgr(dev); if (ret > 0)
dev->mt = nvm_init_mgr(dev);
list_add(&dev->devices, &nvm_devices); list_add(&dev->devices, &nvm_devices);
up_write(&nvm_lock); up_write(&nvm_lock);
...@@ -1030,6 +1040,7 @@ static long __nvm_ioctl_dev_init(struct nvm_ioctl_dev_init *init) ...@@ -1030,6 +1040,7 @@ static long __nvm_ioctl_dev_init(struct nvm_ioctl_dev_init *init)
{ {
struct nvm_dev *dev; struct nvm_dev *dev;
struct nvm_sb_info info; struct nvm_sb_info info;
int ret;
down_write(&nvm_lock); down_write(&nvm_lock);
dev = nvm_find_nvm_dev(init->dev); dev = nvm_find_nvm_dev(init->dev);
...@@ -1044,7 +1055,17 @@ static long __nvm_ioctl_dev_init(struct nvm_ioctl_dev_init *init) ...@@ -1044,7 +1055,17 @@ static long __nvm_ioctl_dev_init(struct nvm_ioctl_dev_init *init)
strncpy(info.mmtype, init->mmtype, NVM_MMTYPE_LEN); strncpy(info.mmtype, init->mmtype, NVM_MMTYPE_LEN);
info.fs_ppa.ppa = -1; info.fs_ppa.ppa = -1;
return nvm_init_sysblock(dev, &info); ret = nvm_init_sysblock(dev, &info);
if (ret)
return ret;
memcpy(&dev->sb, &info, sizeof(struct nvm_sb_info));
down_write(&nvm_lock);
dev->mt = nvm_init_mgr(dev);
up_write(&nvm_lock);
return 0;
} }
static long nvm_ioctl_dev_init(struct file *file, void __user *arg) static long nvm_ioctl_dev_init(struct file *file, void __user *arg)
......
...@@ -301,6 +301,9 @@ struct nvm_dev { ...@@ -301,6 +301,9 @@ struct nvm_dev {
struct nvmm_type *mt; struct nvmm_type *mt;
void *mp; void *mp;
/* System blocks */
struct nvm_sb_info sb;
/* Device information */ /* Device information */
int nr_chnls; int nr_chnls;
int nr_planes; int nr_planes;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment