diff options
author | nwhitehorn <nwhitehorn@FreeBSD.org> | 2009-01-25 17:50:53 +0000 |
---|---|---|
committer | nwhitehorn <nwhitehorn@FreeBSD.org> | 2009-01-25 17:50:53 +0000 |
commit | d8a0e03d761654d8775e052e4132c83f77f2fa1b (patch) | |
tree | 09312eac278620d3595f6b8a4c48fb67fe09214a /sys/powerpc | |
parent | ba4371ba78cc526812a2d1a2ce1cda9b2d2898a7 (diff) | |
download | FreeBSD-src-d8a0e03d761654d8775e052e4132c83f77f2fa1b.zip FreeBSD-src-d8a0e03d761654d8775e052e4132c83f77f2fa1b.tar.gz |
Fix a race condition where interrupts set up after boot could be enabled in
the PIC before the interrupt handler was set. If the interrupt triggered in
that window, then the interrupt vector would be disabled.
Reported by: Marco Trillo
Diffstat (limited to 'sys/powerpc')
-rw-r--r-- | sys/powerpc/powerpc/intr_machdep.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/sys/powerpc/powerpc/intr_machdep.c b/sys/powerpc/powerpc/intr_machdep.c index 3e07348..99b3c2d 100644 --- a/sys/powerpc/powerpc/intr_machdep.c +++ b/sys/powerpc/powerpc/intr_machdep.c @@ -243,7 +243,7 @@ powerpc_setup_intr(const char *name, u_int irq, driver_filter_t filter, driver_intr_t handler, void *arg, enum intr_type flags, void **cookiep) { struct powerpc_intr *i; - int error; + int error, enable = 0; i = intr_lookup(irq); if (i == NULL) @@ -258,13 +258,16 @@ powerpc_setup_intr(const char *name, u_int irq, driver_filter_t filter, i->cntp = &intrcnt[i->vector]; - if (!cold) - PIC_ENABLE(pic, i->irq, i->vector); + enable = 1; } error = intr_event_add_handler(i->event, name, filter, handler, arg, intr_priority(flags), flags, cookiep); intrcnt_setname(i->event->ie_fullname, i->vector); + + if (!cold && enable) + PIC_ENABLE(pic, i->irq, i->vector); + return (error); } |