Date: Sun, 28 Jan 2001 22:39:35 -0800 From: Jason Evans <jasone@canonware.com> To: John Baldwin <jhb@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, Bosko Milekic <bmilekic@FreeBSD.org> Subject: Re: cvs commit: src/sys/i386/i386 mp_machdep.c Message-ID: <20010128223935.W87569@canonware.com> In-Reply-To: <XFMail.010128221910.jhb@FreeBSD.org>; from jhb@FreeBSD.org on Sun, Jan 28, 2001 at 10:19:10PM -0800 References: <20010127222552.S87569@canonware.com> <XFMail.010128221910.jhb@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Jan 28, 2001 at 10:19:10PM -0800, John Baldwin wrote: > On 28-Jan-01 Jason Evans wrote: > > On Sat, Jan 27, 2001 at 10:14:25PM -0800, John Baldwin wrote: > >> Actually, what I suggested to jasone (and kind of preferred myself) was to > >> set > >> each curproc in the AP's when we initialized the idleprocs. This has the > >> benefit of being MI: > > > > Actually, I didn't forget it; the machine wouldn't boot with that change in > > place, so I took it back out. > > Did you do the check for NULL sot that you didn't change curproc on the boot > processor? If not you probably hung trying to release/recurse on Giant. > Without this or Bosko's version of the fix, the AP's have a NULL curproc when > they try to grab the ap_boot_lock and bad things can happen. I can't say for sure; I don't think the change was present at the times I generated the diffs, so I have no record of exactly what the code looked like. Chances are good though that I made exactly the mistake you describe. Jason To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010128223935.W87569>