device-dax: kill NR_DEV_DAX
authorDan Williams <dan.j.williams@intel.com>
Mon, 8 May 2017 19:33:53 +0000 (12:33 -0700)
committerDan Williams <dan.j.williams@intel.com>
Tue, 9 May 2017 16:08:22 +0000 (09:08 -0700)
There is no point to ask how many device-dax instances the kernel should
support. Since we are already using a dynamic major number, just allow
the max number of minors by default and be done. This also fixes the
fact that the proposed max for the NR_DEV_DAX range was larger than what
could be supported by alloc_chrdev_region().

Fixes: ba09c01d2fa8 ("dax: convert to the cdev api")
Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
Tested-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/dax/Kconfig
drivers/dax/super.c

index 6b7e20eae16c63865acd49682ae78c43e7f528b6..b79aa8f7a49714441fe5cb204f45a48fe4c6034f 100644 (file)
@@ -28,9 +28,4 @@ config DEV_DAX_PMEM
 
          Say Y if unsure
 
-config NR_DEV_DAX
-       int "Maximum number of Device-DAX instances"
-       default 32768
-       range 256 2147483647
-
 endif
index e8998d15f3cd16330ca29de648f476d35cff580a..ebf43f531adaff85da90968529ec522b2660776a 100644 (file)
 #include <linux/dax.h>
 #include <linux/fs.h>
 
-static int nr_dax = CONFIG_NR_DEV_DAX;
-module_param(nr_dax, int, S_IRUGO);
-MODULE_PARM_DESC(nr_dax, "max number of dax device instances");
-
 static dev_t dax_devt;
 DEFINE_STATIC_SRCU(dax_srcu);
 static struct vfsmount *dax_mnt;
@@ -331,7 +327,7 @@ struct dax_device *alloc_dax(void *private, const char *__host,
        if (__host && !host)
                return NULL;
 
-       minor = ida_simple_get(&dax_minor_ida, 0, nr_dax, GFP_KERNEL);
+       minor = ida_simple_get(&dax_minor_ida, 0, MINORMASK+1, GFP_KERNEL);
        if (minor < 0)
                goto err_minor;
 
@@ -475,8 +471,7 @@ static int __init dax_fs_init(void)
        if (rc)
                return rc;
 
-       nr_dax = max(nr_dax, 256);
-       rc = alloc_chrdev_region(&dax_devt, 0, nr_dax, "dax");
+       rc = alloc_chrdev_region(&dax_devt, 0, MINORMASK+1, "dax");
        if (rc)
                __dax_fs_exit();
        return rc;
@@ -484,7 +479,7 @@ static int __init dax_fs_init(void)
 
 static void __exit dax_fs_exit(void)
 {
-       unregister_chrdev_region(dax_devt, nr_dax);
+       unregister_chrdev_region(dax_devt, MINORMASK+1);
        ida_destroy(&dax_minor_ida);
        __dax_fs_exit();
 }