Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 2 Apr 2007 15:30:41 +0100 (BST)
From:      Robert Watson <rwatson@FreeBSD.org>
To:        John Baldwin <jhb@freebsd.org>
Cc:        Marcel Moolenaar <marcel@freebsd.org>, Perforce Change Reviews <perforce@freebsd.org>
Subject:   Re: PERFORCE change 117140 for review
Message-ID:  <20070402152910.A82995@fledge.watson.org>
In-Reply-To: <200704020915.03985.jhb@freebsd.org>
References:  <200704012152.l31LqHuB022635@repoman.freebsd.org> <200704020915.03985.jhb@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On Mon, 2 Apr 2007, John Baldwin wrote:

>> @@ -40,6 +40,7 @@
>>  	int		pc_inside_intr;					\
>>  	struct pmap	*pc_curpmap;		/* current pmap */	\
>>  	struct thread   *pc_fputhread;          /* current fpu user */  \
>> +	int		pc_bsp:1;					\
>
> FYI, other places in MI code assume that CPU 0 (pc_cpuid) is the BSP.

I realized the hard way, the other day, that the boot thread itself can 
migrate to another CPU during sysinit processing, once SMP is kicked off.  No 
more KASSERT(curcpu == 0, ()) for me. :-)

Robert N M Watson
Computer Laboratory
University of Cambridge



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