Date: Thu, 23 Dec 2010 15:17:28 +0000 (UTC) From: John Baldwin <jhb@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/x86/x86 io_apic.c Message-ID: <201012231517.oBNFHcQe036941@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
jhb 2010-12-23 15:17:28 UTC
FreeBSD src repository
Modified files:
sys/x86/x86 io_apic.c
Log:
SVN rev 216679 on 2010-12-23 15:17:28Z by jhb
Drop the icu_lock spinlock while pausing briefly after masking the
interrupt in the I/O APIC before moving it to a different CPU. If the
interrupt had been triggered by the I/O APIC after locking icu_lock but
before we masked the pin in the I/O APIC, then this could cause the
interrupt to be pending on the "old" CPU and it would finally trigger
after we had moved the interrupt to the new CPU. This could cause us to
panic as there was no interrupt source associated with the old IDT vector
on the old CPU. Dropping the lock after the interrupt is masked but
before it is moved allows the interrupt to fire and be handled in this
case before it is moved.
Tested by: Daniel Braniss danny of cs huji ac il
MFC after: 1 week
Revision Changes Path
1.4 +2 -0 src/sys/x86/x86/io_apic.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201012231517.oBNFHcQe036941>