genirq: Do not consider disabled wakeup irqs
authorThomas Gleixner <tglx@linutronix.de>
Fri, 4 May 2012 15:56:16 +0000 (17:56 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Fri, 4 May 2012 21:38:50 +0000 (23:38 +0200)
If an wakeup interrupt has been disabled before the suspend code
disables all interrupts then we have to ignore the pending flag.

Otherwise we would abort suspend over and over as nothing clears the
pending flag because the interrupt is disabled.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: NeilBrown <neilb@suse.de>
kernel/irq/pm.c

index 15e53b1766a653e4634eea3278e19354211378a1..cb228bf217603d6e841c1028dd610598ae133bff 100644 (file)
@@ -103,8 +103,13 @@ int check_wakeup_irqs(void)
        int irq;
 
        for_each_irq_desc(irq, desc) {
+               /*
+                * Only interrupts which are marked as wakeup source
+                * and have not been disabled before the suspend check
+                * can abort suspend.
+                */
                if (irqd_is_wakeup_set(&desc->irq_data)) {
-                       if (desc->istate & IRQS_PENDING)
+                       if (desc->depth == 1 && desc->istate & IRQS_PENDING)
                                return -EBUSY;
                        continue;
                }