OSDN Git Service

samples/vfio-mdev/mbochs: expand minor range when registering chrdev region
authorChengguang Xu <cgxu519@gmx.com>
Tue, 12 Feb 2019 05:59:30 +0000 (13:59 +0800)
committerAlex Williamson <alex.williamson@redhat.com>
Tue, 12 Feb 2019 20:21:05 +0000 (13:21 -0700)
Actually, total amount of available minor number
for a single major is MINORMARK + 1. So expand
minor range when registering chrdev region.

Signed-off-by: Chengguang Xu <cgxu519@gmx.com>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
samples/vfio-mdev/mbochs.c

index ca7960a..b038aa9 100644 (file)
@@ -1448,13 +1448,13 @@ static int __init mbochs_dev_init(void)
 {
        int ret = 0;
 
-       ret = alloc_chrdev_region(&mbochs_devt, 0, MINORMASK, MBOCHS_NAME);
+       ret = alloc_chrdev_region(&mbochs_devt, 0, MINORMASK + 1, MBOCHS_NAME);
        if (ret < 0) {
                pr_err("Error: failed to register mbochs_dev, err: %d\n", ret);
                return ret;
        }
        cdev_init(&mbochs_cdev, &vd_fops);
-       cdev_add(&mbochs_cdev, mbochs_devt, MINORMASK);
+       cdev_add(&mbochs_cdev, mbochs_devt, MINORMASK + 1);
        pr_info("%s: major %d\n", __func__, MAJOR(mbochs_devt));
 
        mbochs_class = class_create(THIS_MODULE, MBOCHS_CLASS_NAME);
@@ -1483,7 +1483,7 @@ failed2:
        class_destroy(mbochs_class);
 failed1:
        cdev_del(&mbochs_cdev);
-       unregister_chrdev_region(mbochs_devt, MINORMASK);
+       unregister_chrdev_region(mbochs_devt, MINORMASK + 1);
        return ret;
 }
 
@@ -1494,7 +1494,7 @@ static void __exit mbochs_dev_exit(void)
 
        device_unregister(&mbochs_dev);
        cdev_del(&mbochs_cdev);
-       unregister_chrdev_region(mbochs_devt, MINORMASK);
+       unregister_chrdev_region(mbochs_devt, MINORMASK + 1);
        class_destroy(mbochs_class);
        mbochs_class = NULL;
 }