irqchip: xtensa: Select only an online cpu
authorThomas Gleixner <tglx@linutronix.de>
Tue, 4 Mar 2014 14:19:16 +0000 (15:19 +0100)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 4 Mar 2014 16:37:55 +0000 (17:37 +0100)
The user space interface does not filter out offline cpus. It merily
verifies that the mask contains at least one online cpu. So the
selector in the irq chip implementation needs to make sure to pick
only an online cpu because otherwise:

     Offline Core 1
     Set affinity to 0xe
     Selector will pick first set bit, i.e. core 1

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Chris Zankel <chris@zankel.net>
Cc: xtensa <linux-xtensa@linux-xtensa.org>
drivers/irqchip/irq-xtensa-mx.c

index f693f1b..e1c2f96 100644 (file)
@@ -122,7 +122,7 @@ static int xtensa_mx_irq_retrigger(struct irq_data *d)
 static int xtensa_mx_irq_set_affinity(struct irq_data *d,
                const struct cpumask *dest, bool force)
 {
-       unsigned mask = 1u << cpumask_any(dest);
+       unsigned mask = 1u << cpumask_any_and(dest, cpu_online_mask);
 
        set_er(mask, MIROUT(d->hwirq - HW_IRQ_MX_BASE));
        return 0;