diff options
author | Paul Mackerras <paulus@samba.org> | 2005-09-28 06:29:44 -0400 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2005-09-28 06:29:44 -0400 |
commit | beeca08738c4c4024c81a591812bfe38f8c436c0 (patch) | |
tree | 2d668f8529c54562c13ee06f2777d0f145f92c1e /kernel/irq | |
parent | 20c8c2106305729e7d5e06f6c3d390e965a3dd34 (diff) |
Don't call a NULL ack function in the generic IRQ code.
Some IRQ controllers don't need an ack function (e.g. OpenPIC on
PPC platforms) and for them we'd rather not have the overhead
of doing an indirect call to a function that does nothing.
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'kernel/irq')
-rw-r--r-- | kernel/irq/handle.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/kernel/irq/handle.c b/kernel/irq/handle.c index 3ff7b925c387..51df337b37db 100644 --- a/kernel/irq/handle.c +++ b/kernel/irq/handle.c | |||
@@ -117,14 +117,16 @@ fastcall unsigned int __do_IRQ(unsigned int irq, struct pt_regs *regs) | |||
117 | /* | 117 | /* |
118 | * No locking required for CPU-local interrupts: | 118 | * No locking required for CPU-local interrupts: |
119 | */ | 119 | */ |
120 | desc->handler->ack(irq); | 120 | if (desc->handler->ack) |
121 | desc->handler->ack(irq); | ||
121 | action_ret = handle_IRQ_event(irq, regs, desc->action); | 122 | action_ret = handle_IRQ_event(irq, regs, desc->action); |
122 | desc->handler->end(irq); | 123 | desc->handler->end(irq); |
123 | return 1; | 124 | return 1; |
124 | } | 125 | } |
125 | 126 | ||
126 | spin_lock(&desc->lock); | 127 | spin_lock(&desc->lock); |
127 | desc->handler->ack(irq); | 128 | if (desc->handler->ack) |
129 | desc->handler->ack(irq); | ||
128 | /* | 130 | /* |
129 | * REPLAY is when Linux resends an IRQ that was dropped earlier | 131 | * REPLAY is when Linux resends an IRQ that was dropped earlier |
130 | * WAITING is used by probe to mark irqs that are being tested | 132 | * WAITING is used by probe to mark irqs that are being tested |