diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2014-03-04 15:19:16 +0100 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2014-03-04 17:37:55 +0100 |
| commit | 5c331c8626f5d39722d07101c699c8e794f5629d (patch) | |
| tree | 31472b882b476598f9d79bebf44da818856c8687 /drivers/irqchip/irq-xtensa-mx.c | |
| parent | sh: Use irq_set_affinity instead of homebrewn code (diff) | |
| download | linux-5c331c8626f5d39722d07101c699c8e794f5629d.tar.xz linux-5c331c8626f5d39722d07101c699c8e794f5629d.zip | |
irqchip: xtensa: Select only an online cpu
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>
Diffstat (limited to 'drivers/irqchip/irq-xtensa-mx.c')
| -rw-r--r-- | drivers/irqchip/irq-xtensa-mx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/irqchip/irq-xtensa-mx.c b/drivers/irqchip/irq-xtensa-mx.c index f693f1bc1348..e1c2f9632893 100644 --- a/drivers/irqchip/irq-xtensa-mx.c +++ b/drivers/irqchip/irq-xtensa-mx.c @@ -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; |