pps: convert to idr_alloc()
authorTejun Heo <tj@kernel.org>
Thu, 28 Feb 2013 01:04:38 +0000 (17:04 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 28 Feb 2013 03:10:18 +0000 (19:10 -0800)
Convert to the much saner new idr interface.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Rodolfo Giometti <giometti@enneenne.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/pps/kapi.c
drivers/pps/pps.c

index f197e8ea185c4099abc1289cd1b6cf74947f3bed..cdad4d95b20eed060e6bc961da9bb27ab23865fb 100644 (file)
@@ -102,7 +102,7 @@ struct pps_device *pps_register_source(struct pps_source_info *info,
                goto pps_register_source_exit;
        }
 
-       /* These initializations must be done before calling idr_get_new()
+       /* These initializations must be done before calling idr_alloc()
         * in order to avoid reces into pps_event().
         */
        pps->params.api_version = PPS_API_VERS;
index 6437703eb10fdfa484097a692af6ee2c20559b4b..7173e3ad475d9d9f8e6a169106691c18ad2fea56 100644 (file)
@@ -295,29 +295,21 @@ int pps_register_cdev(struct pps_device *pps)
        dev_t devt;
 
        mutex_lock(&pps_idr_lock);
-       /* Get new ID for the new PPS source */
-       if (idr_pre_get(&pps_idr, GFP_KERNEL) == 0) {
-               mutex_unlock(&pps_idr_lock);
-               return -ENOMEM;
-       }
-
-       /* Now really allocate the PPS source.
-        * After idr_get_new() calling the new source will be freely available
-        * into the kernel.
+       /*
+        * Get new ID for the new PPS source.  After idr_alloc() calling
+        * the new source will be freely available into the kernel.
         */
-       err = idr_get_new(&pps_idr, pps, &pps->id);
-       mutex_unlock(&pps_idr_lock);
-
-       if (err < 0)
-               return err;
-
-       pps->id &= MAX_IDR_MASK;
-       if (pps->id >= PPS_MAX_SOURCES) {
-               pr_err("%s: too many PPS sources in the system\n",
-                                       pps->info.name);
-               err = -EBUSY;
-               goto free_idr;
+       err = idr_alloc(&pps_idr, pps, 0, PPS_MAX_SOURCES, GFP_KERNEL);
+       if (err < 0) {
+               if (err == -ENOSPC) {
+                       pr_err("%s: too many PPS sources in the system\n",
+                              pps->info.name);
+                       err = -EBUSY;
+               }
+               goto out_unlock;
        }
+       pps->id = err;
+       mutex_unlock(&pps_idr_lock);
 
        devt = MKDEV(MAJOR(pps_devt), pps->id);
 
@@ -351,8 +343,8 @@ del_cdev:
 free_idr:
        mutex_lock(&pps_idr_lock);
        idr_remove(&pps_idr, pps->id);
+out_unlock:
        mutex_unlock(&pps_idr_lock);
-
        return err;
 }