Commit 194046a1 authored by Olof Johansson's avatar Olof Johansson Committed by Paul Mackerras

[POWERPC] MPIC: Minor optimization of ipi handler

Optimize MPIC IPIs, by passing in the IPI number as the argument to the
handler, since all we did was translate it back based on which mpic
the interrupt came though on (and that was always the primary mpic).
Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
Acked-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent 2c0b713f
...@@ -612,12 +612,11 @@ static inline void mpic_eoi(struct mpic *mpic) ...@@ -612,12 +612,11 @@ static inline void mpic_eoi(struct mpic *mpic)
} }
#ifdef CONFIG_SMP #ifdef CONFIG_SMP
static irqreturn_t mpic_ipi_action(int irq, void *dev_id) static irqreturn_t mpic_ipi_action(int irq, void *data)
{ {
struct mpic *mpic; long ipi = (long)data;
mpic = mpic_find(irq, NULL); smp_message_recv(ipi);
smp_message_recv(mpic_irq_to_hw(irq) - mpic->ipi_vecs[0]);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -1457,7 +1456,7 @@ unsigned int mpic_get_irq(void) ...@@ -1457,7 +1456,7 @@ unsigned int mpic_get_irq(void)
void mpic_request_ipis(void) void mpic_request_ipis(void)
{ {
struct mpic *mpic = mpic_primary; struct mpic *mpic = mpic_primary;
int i, err; long i, err;
static char *ipi_names[] = { static char *ipi_names[] = {
"IPI0 (call function)", "IPI0 (call function)",
"IPI1 (reschedule)", "IPI1 (reschedule)",
...@@ -1472,14 +1471,14 @@ void mpic_request_ipis(void) ...@@ -1472,14 +1471,14 @@ void mpic_request_ipis(void)
unsigned int vipi = irq_create_mapping(mpic->irqhost, unsigned int vipi = irq_create_mapping(mpic->irqhost,
mpic->ipi_vecs[0] + i); mpic->ipi_vecs[0] + i);
if (vipi == NO_IRQ) { if (vipi == NO_IRQ) {
printk(KERN_ERR "Failed to map IPI %d\n", i); printk(KERN_ERR "Failed to map IPI %ld\n", i);
break; break;
} }
err = request_irq(vipi, mpic_ipi_action, err = request_irq(vipi, mpic_ipi_action,
IRQF_DISABLED|IRQF_PERCPU, IRQF_DISABLED|IRQF_PERCPU,
ipi_names[i], mpic); ipi_names[i], (void *)i);
if (err) { if (err) {
printk(KERN_ERR "Request of irq %d for IPI %d failed\n", printk(KERN_ERR "Request of irq %d for IPI %ld failed\n",
vipi, i); vipi, i);
break; break;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment