diff options
author | gibbs <gibbs@FreeBSD.org> | 2012-04-06 21:19:28 +0000 |
---|---|---|
committer | gibbs <gibbs@FreeBSD.org> | 2012-04-06 21:19:28 +0000 |
commit | ea2a8fbda52185e0617e71fb627826dfe9a850d8 (patch) | |
tree | 97a809e4ac0fe0bacc4243d77ac5b19950ea9edc /sys/x86 | |
parent | 4bd8f1d188e53a16720092a323327ecc0325dadc (diff) | |
download | FreeBSD-src-ea2a8fbda52185e0617e71fb627826dfe9a850d8.zip FreeBSD-src-ea2a8fbda52185e0617e71fb627826dfe9a850d8.tar.gz |
Fix interrupt load balancing regression, introduced in revision
222813, that left all un-pinned interrupts assigned to CPU 0.
sys/x86/x86/intr_machdep.c:
In intr_shuffle_irqs(), remove CPU_SETOF() call that initialized
the "intr_cpus" cpuset to only contain CPU0.
This initialization is too late and nullifies the results of calls
the intr_add_cpu() that occur much earlier in the boot process.
Since "intr_cpus" is statically initialized to the empty set, and
all processors, including the BSP, already add themselves to
"intr_cpus" no special initialization for the BSP is necessary.
MFC after: 3 days
Diffstat (limited to 'sys/x86')
-rw-r--r-- | sys/x86/x86/intr_machdep.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sys/x86/x86/intr_machdep.c b/sys/x86/x86/intr_machdep.c index cdab05c..fef227e 100644 --- a/sys/x86/x86/intr_machdep.c +++ b/sys/x86/x86/intr_machdep.c @@ -525,9 +525,6 @@ intr_shuffle_irqs(void *arg __unused) return; #endif - /* The BSP is always a valid target. */ - CPU_SETOF(0, &intr_cpus); - /* Don't bother on UP. */ if (mp_ncpus == 1) return; |