Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 9 Apr 2012 10:30:10 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        "Justin T. Gibbs" <gibbs@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r233961 - head/sys/x86/x86
Message-ID:  <201204091030.10717.jhb@freebsd.org>
In-Reply-To: <201204062119.q36LJTKR026564@svn.freebsd.org>
References:  <201204062119.q36LJTKR026564@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, April 06, 2012 5:19:29 pm Justin T. Gibbs wrote:
> Author: gibbs
> Date: Fri Apr  6 21:19:28 2012
> New Revision: 233961
> URL: http://svn.freebsd.org/changeset/base/233961
> 
> Log:
>   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

Ah, nice catch!

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201204091030.10717.jhb>