lkml.org 
[lkml]   [2005]   [Mar]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[1/9] [RFC] Steps to fixing the driver model locking

ChangeSet@1.2230, 2005-03-21 10:41:04-08:00, mochel@digitalimplant.org
[driver core] Add a semaphore to struct device to synchronize calls to its driver.

This adds a per-device semaphore that is taken before every call from the core to a
driver method. This prevents e.g. simultaneous calls to the ->suspend() or ->resume()
and ->probe() or ->release(), potentially saving a whole lot of headaches.

It also moves us a step closer to removing the bus rwsem, since it protects the fields
in struct device that are modified by the core.



Signed-off-by: Patrick Mochel <mochel@digitalimplant.org>

diff -Nru a/drivers/base/bus.c b/drivers/base/bus.c
--- a/drivers/base/bus.c 2005-03-21 12:30:51 -08:00
+++ b/drivers/base/bus.c 2005-03-21 12:30:51 -08:00
@@ -283,18 +283,22 @@
*/
int driver_probe_device(struct device_driver * drv, struct device * dev)
{
+ int error = 0;
+
if (drv->bus->match && !drv->bus->match(dev, drv))
return -ENODEV;

+ down(&dev->sem);
dev->driver = drv;
if (drv->probe) {
- int error = drv->probe(dev);
+ error = drv->probe(dev);
if (error) {
dev->driver = NULL;
+ up(&dev->sem);
return error;
}
}
-
+ up(&dev->sem);
device_bind_driver(dev);
return 0;
}
@@ -385,7 +389,10 @@

void device_release_driver(struct device * dev)
{
- struct device_driver * drv = dev->driver;
+ struct device_driver * drv;
+
+ down(&dev->sem);
+ drv = dev->driver;
if (drv) {
sysfs_remove_link(&drv->kobj, kobject_name(&dev->kobj));
sysfs_remove_link(&dev->kobj, "driver");
@@ -395,6 +402,7 @@
drv->remove(dev);
dev->driver = NULL;
}
+ up(&dev->sem);
}


diff -Nru a/drivers/base/core.c b/drivers/base/core.c
--- a/drivers/base/core.c 2005-03-21 12:30:51 -08:00
+++ b/drivers/base/core.c 2005-03-21 12:30:51 -08:00
@@ -215,6 +215,7 @@
INIT_LIST_HEAD(&dev->driver_list);
INIT_LIST_HEAD(&dev->bus_list);
INIT_LIST_HEAD(&dev->dma_pools);
+ init_MUTEX(&dev->sem);
}

/**
diff -Nru a/drivers/base/power/resume.c b/drivers/base/power/resume.c
--- a/drivers/base/power/resume.c 2005-03-21 12:30:51 -08:00
+++ b/drivers/base/power/resume.c 2005-03-21 12:30:51 -08:00
@@ -22,9 +22,13 @@

int resume_device(struct device * dev)
{
+ int error = 0;
+
+ down(&dev->sem);
if (dev->bus && dev->bus->resume)
- return dev->bus->resume(dev);
- return 0;
+ error = dev->bus->resume(dev);
+ up(&dev->sem);
+ return error;
}


diff -Nru a/drivers/base/power/suspend.c b/drivers/base/power/suspend.c
--- a/drivers/base/power/suspend.c 2005-03-21 12:30:51 -08:00
+++ b/drivers/base/power/suspend.c 2005-03-21 12:30:51 -08:00
@@ -41,11 +41,11 @@

dev_dbg(dev, "suspending\n");

+ down(&dev->sem);
dev->power.prev_state = dev->power.power_state;
-
if (dev->bus && dev->bus->suspend && !dev->power.power_state)
error = dev->bus->suspend(dev, state);
-
+ up(&dev->sem);
return error;
}

diff -Nru a/include/linux/device.h b/include/linux/device.h
--- a/include/linux/device.h 2005-03-21 12:30:51 -08:00
+++ b/include/linux/device.h 2005-03-21 12:30:51 -08:00
@@ -265,6 +265,10 @@
struct kobject kobj;
char bus_id[BUS_ID_SIZE]; /* position on parent bus */

+ struct semaphore sem; /* semaphore to synchronize calls to
+ * its driver.
+ */
+
struct bus_type * bus; /* type of bus device is on */
struct device_driver *driver; /* which driver has allocated this
device */
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 14:11    [W:0.055 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site