vserver 1.9.5.x5
[linux-2.6.git] / drivers / base / power / suspend.c
index 86f9e6e..8e1ace2 100644 (file)
@@ -11,7 +11,7 @@
 #include <linux/device.h>
 #include "power.h"
 
-extern int sysdev_suspend(u32 state);
+extern int sysdev_suspend(pm_message_t state);
 
 /*
  * The entries in the dpm_active list are in a depth first order, simply
@@ -35,7 +35,7 @@ extern int sysdev_suspend(u32 state);
  *     @state: Power state device is entering.
  */
 
-int suspend_device(struct device * dev, u32 state)
+int suspend_device(struct device * dev, pm_message_t state)
 {
        int error = 0;
 
@@ -63,44 +63,50 @@ int suspend_device(struct device * dev, u32 state)
  *     If we hit a failure with any of the devices, call device_resume()
  *     above to bring the suspended devices back to life.
  *
- *     Note this function leaves dpm_sem held to
- *     a) block other devices from registering.
- *     b) prevent other PM operations from happening after we've begun.
- *     c) make sure we're exclusive when we disable interrupts.
- *
  */
 
-int device_suspend(u32 state)
+int device_suspend(pm_message_t state)
 {
        int error = 0;
 
        down(&dpm_sem);
-       while(!list_empty(&dpm_active)) {
+       down(&dpm_list_sem);
+       while (!list_empty(&dpm_active) && error == 0) {
                struct list_head * entry = dpm_active.prev;
                struct device * dev = to_device(entry);
+
+               get_device(dev);
+               up(&dpm_list_sem);
+
                error = suspend_device(dev, state);
 
-               if (!error) {
-                       list_del(&dev->power.entry);
-                       list_add(&dev->power.entry, &dpm_off);
-               } else if (error == -EAGAIN) {
-                       list_del(&dev->power.entry);
-                       list_add(&dev->power.entry, &dpm_off_irq);
-               } else {
+               down(&dpm_list_sem);
+
+               /* Check if the device got removed */
+               if (!list_empty(&dev->power.entry)) {
+                       /* Move it to the dpm_off or dpm_off_irq list */
+                       if (!error) {
+                               list_del(&dev->power.entry);
+                               list_add(&dev->power.entry, &dpm_off);
+                       } else if (error == -EAGAIN) {
+                               list_del(&dev->power.entry);
+                               list_add(&dev->power.entry, &dpm_off_irq);
+                               error = 0;
+                       }
+               }
+               if (error)
                        printk(KERN_ERR "Could not suspend device %s: "
                                "error %d\n", kobject_name(&dev->kobj), error);
-                       goto Error;
-               }
+               put_device(dev);
        }
- Done:
+       up(&dpm_list_sem);
+       if (error)
+               dpm_resume();
        up(&dpm_sem);
        return error;
- Error:
-       dpm_resume();
-       goto Done;
 }
 
-EXPORT_SYMBOL(device_suspend);
+EXPORT_SYMBOL_GPL(device_suspend);
 
 
 /**
@@ -112,7 +118,7 @@ EXPORT_SYMBOL(device_suspend);
  *     done, power down system devices.
  */
 
-int device_power_down(u32 state)
+int device_power_down(pm_message_t state)
 {
        int error = 0;
        struct device * dev;
@@ -132,5 +138,5 @@ int device_power_down(u32 state)
        goto Done;
 }
 
-EXPORT_SYMBOL(device_power_down);
+EXPORT_SYMBOL_GPL(device_power_down);