osd: Convert to new IDA API
authorMatthew Wilcox <willy@infradead.org>
Mon, 11 Jun 2018 19:41:25 +0000 (15:41 -0400)
committerMatthew Wilcox <willy@infradead.org>
Wed, 22 Aug 2018 03:54:17 +0000 (23:54 -0400)
Slightly simpler code.

Signed-off-by: Matthew Wilcox <willy@infradead.org>
drivers/scsi/osd/osd_uld.c

index 0e56f1eb05dc5caa711360dab68fcb7584d71b71..eaf36ccf58db55779033f73c936a5dec586b58c1 100644 (file)
@@ -423,19 +423,11 @@ static int osd_probe(struct device *dev)
        if (scsi_device->type != TYPE_OSD)
                return -ENODEV;
 
-       do {
-               if (!ida_pre_get(&osd_minor_ida, GFP_KERNEL))
-                       return -ENODEV;
-
-               error = ida_get_new(&osd_minor_ida, &minor);
-       } while (error == -EAGAIN);
-
-       if (error)
-               return error;
-       if (minor >= SCSI_OSD_MAX_MINOR) {
-               error = -EBUSY;
-               goto err_retract_minor;
-       }
+       minor = ida_alloc_max(&osd_minor_ida, SCSI_OSD_MAX_MINOR, GFP_KERNEL);
+       if (minor == -ENOSPC)
+               return -EBUSY;
+       if (minor < 0)
+               return -ENODEV;
 
        error = -ENOMEM;
        oud = kzalloc(sizeof(*oud), GFP_KERNEL);
@@ -499,7 +491,7 @@ static int osd_probe(struct device *dev)
 err_free_osd:
        put_device(&oud->class_dev);
 err_retract_minor:
-       ida_remove(&osd_minor_ida, minor);
+       ida_free(&osd_minor_ida, minor);
        return error;
 }
 
@@ -514,7 +506,7 @@ static int osd_remove(struct device *dev)
        }
 
        cdev_device_del(&oud->cdev, &oud->class_dev);
-       ida_remove(&osd_minor_ida, oud->minor);
+       ida_free(&osd_minor_ida, oud->minor);
        put_device(&oud->class_dev);
 
        return 0;