[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080826171532.2414c7a2@bike.lwn.net>
Date: Tue, 26 Aug 2008 17:15:32 -0600
From: Jonathan Corbet <corbet@....net>
To: LKML <linux-kernel@...r.kernel.org>
Cc: Greg KH <greg@...ah.com>, hjk@...utronix.de
Subject: [PATCH, RFC] uio BKL removal
I had a moment to dedicate to the BKL removal cause, so I went for the
UIO driver, which seemed simple. The main thing I found was that there
was locking around some idr accesses, but not all, so I filled that
in. With that in place, removing the BKL from uio_open() seems safe,
especially since none of the in-tree UIO drivers have open() or
release() methods.
(Incidentally, I don't see how uio_pdrv.c could ever work - who initializes
uioinfo?)
If there's no complaints, I'll feed this into linux-next via the
bkl-removal tree.
Thanks,
jon
---
UIO: BKL removal
Remove the BKL from the UIO driver, and add complete locking where needed
to serialize idr accesses.
Signed-off-by: Jonathan Corbet <corbet@....net>
diff --git a/drivers/uio/uio.c b/drivers/uio/uio.c
index 3a6934b..4f28f4b 100644
--- a/drivers/uio/uio.c
+++ b/drivers/uio/uio.c
@@ -47,6 +47,9 @@ static struct uio_class {
struct class *class;
} *uio_class;
+/* Protect idr accesses */
+static DEFINE_MUTEX(minor_lock);
+
/*
* attributes
*/
@@ -231,7 +234,6 @@ static void uio_dev_del_attributes(struct uio_device *idev)
static int uio_get_minor(struct uio_device *idev)
{
- static DEFINE_MUTEX(minor_lock);
int retval = -ENOMEM;
int id;
@@ -253,7 +255,9 @@ exit:
static void uio_free_minor(struct uio_device *idev)
{
+ mutex_lock(&minor_lock);
idr_remove(&uio_idr, idev->minor);
+ mutex_unlock(&minor_lock);
}
/**
@@ -297,8 +301,9 @@ static int uio_open(struct inode *inode, struct file *filep)
struct uio_listener *listener;
int ret = 0;
- lock_kernel();
+ mutex_lock(&minor_lock);
idev = idr_find(&uio_idr, iminor(inode));
+ mutex_unlock(&minor_lock);
if (!idev) {
ret = -ENODEV;
goto out;
@@ -324,18 +329,15 @@ static int uio_open(struct inode *inode, struct file *filep)
if (ret)
goto err_infoopen;
}
- unlock_kernel();
return 0;
err_infoopen:
-
kfree(listener);
-err_alloc_listener:
+err_alloc_listener:
module_put(idev->owner);
out:
- unlock_kernel();
return ret;
}
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists