3 files changed, 28 insertions, 16 deletions

diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
index e255341682c8..69b4ddb7de3b 100644
--- a/drivers/base/power/main.c
+++ b/drivers/base/power/main.c
@@ -23,6 +23,7 @@
 #include <linux/pm.h>
 #include <linux/resume-trace.h>
 #include <linux/rwsem.h>
+#include <linux/interrupt.h>
 
 #include "../base.h"
 #include "power.h"
@@ -349,7 +350,8 @@ static int resume_device_noirq(struct device *dev, pm_message_t state)
  *      Execute the appropriate "noirq resume" callback for all devices marked
  *      as DPM_OFF_IRQ.
  *
- *      Must be called with interrupts disabled and only one CPU running.
+ *      Must be called under dpm_list_mtx.  Device drivers should not receive
+ *      interrupts while it's being executed.
  */
 static void dpm_power_up(pm_message_t state)
 {
@@ -370,14 +372,13 @@ static void dpm_power_up(pm_message_t state)
  *      device_power_up - Turn on all devices that need special attention.
  *      @state: PM transition of the system being carried out.
  *
- *      Power on system devices, then devices that required we shut them down
- *      with interrupts disabled.
- *
- *      Must be called with interrupts disabled.
+ *      Call the "early" resume handlers and enable device drivers to receive
+ *      interrupts.
  */
 void device_power_up(pm_message_t state)
 {
         dpm_power_up(state);
+        resume_device_irqs();
 }
 EXPORT_SYMBOL_GPL(device_power_up);
 
@@ -602,16 +603,17 @@ static int suspend_device_noirq(struct device *dev, pm_message_t state)
  *      device_power_down - Shut down special devices.
  *      @state: PM transition of the system being carried out.
  *
- *      Power down devices that require interrupts to be disabled.
- *      Then power down system devices.
+ *      Prevent device drivers from receiving interrupts and call the "late"
+ *      suspend handlers.
  *
- *      Must be called with interrupts disabled and only one CPU running.
+ *      Must be called under dpm_list_mtx.
  */
 int device_power_down(pm_message_t state)
 {
         struct device *dev;
         int error = 0;
 
+        suspend_device_irqs();
         list_for_each_entry_reverse(dev, &dpm_list, power.entry) {
                 error = suspend_device_noirq(dev, state);
                 if (error) {
@@ -621,7 +623,7 @@ int device_power_down(pm_message_t state)
                 dev->power.status = DPM_OFF_IRQ;
         }
         if (error)
-                dpm_power_up(resume_event(state));
+                device_power_up(resume_event(state));
         return error;
 }
 EXPORT_SYMBOL_GPL(device_power_down);
diff --git a/drivers/base/sys.c b/drivers/base/sys.c
index cbd36cf59a0f..76ce75bad91e 100644
--- a/drivers/base/sys.c
+++ b/drivers/base/sys.c
@@ -22,6 +22,7 @@
 #include <linux/pm.h>
 #include <linux/device.h>
 #include <linux/mutex.h>
+#include <linux/interrupt.h>
 
 #include "base.h"
 
@@ -369,6 +370,13 @@ int sysdev_suspend(pm_message_t state)
         struct sysdev_driver *drv, *err_drv;
         int ret;
 
+        pr_debug("Checking wake-up interrupts\n");
+
+        /* Return error code if there are any wake-up interrupts pending */
+        ret = check_wakeup_irqs();
+        if (ret)
+                return ret;
+
         pr_debug("Suspending System Devices\n");
 
         list_for_each_entry_reverse(cls, &system_kset->list, kset.kobj.entry) {
diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c
index 3ccd348d112d..0d61db1e7b49 100644
--- a/drivers/xen/manage.c
+++ b/drivers/xen/manage.c
@@ -39,12 +39,6 @@ static int xen_suspend(void *data)
 
         BUG_ON(!irqs_disabled());
 
-        err = device_power_down(PMSG_SUSPEND);
-        if (err) {
-                printk(KERN_ERR "xen_suspend: device_power_down failed: %d\n",
-                       err);
-                return err;
-        }
         err = sysdev_suspend(PMSG_SUSPEND);
         if (err) {
                 printk(KERN_ERR "xen_suspend: sysdev_suspend failed: %d\n",
@@ -69,7 +63,6 @@ static int xen_suspend(void *data)
         xen_mm_unpin_all();
 
         sysdev_resume();
-        device_power_up(PMSG_RESUME);
 
         if (!*cancelled) {
                 xen_irq_resume();
@@ -108,6 +101,12 @@ static void do_suspend(void)
         /* XXX use normal device tree? */
         xenbus_suspend();
 
+        err = device_power_down(PMSG_SUSPEND);
+        if (err) {
+                printk(KERN_ERR "device_power_down failed: %d\n", err);
+                goto resume_devices;
+        }
+
         err = stop_machine(xen_suspend, &cancelled, cpumask_of(0));
         if (err) {
                 printk(KERN_ERR "failed to start xen_suspend: %d\n", err);
@@ -120,6 +119,9 @@ static void do_suspend(void)
         } else
                 xenbus_suspend_cancel();
 
+        device_power_up(PMSG_RESUME);
+
+resume_devices:
         device_resume(PMSG_RESUME);
 
         /* Make sure timer events get retriggered on all CPUs */