Date: Thu, 4 Mar 2004 23:28:04 -0500 (EST) From: Robert Watson <rwatson@freebsd.org> To: "Alan L. Cox" <alc@imimic.com> Cc: Marcel Moolenaar <marcel@xcllnt.net> Subject: Re: bug in vm_contig.c? [was: Re: ACPI crash with recent changes] Message-ID: <Pine.NEB.3.96L.1040304232746.56322C-100000@fledge.watson.org> In-Reply-To: <4047F3F8.1080207@imimic.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 4 Mar 2004, Alan L. Cox wrote: > Please test the attached patch instead. In order to avoid a race > condition, it is necessary for the loop to begin with "start" rather > than "start + 1". What is both unnecessary and wrong is the physical > contiguity check. (My bad. :-() In other words, the test that the page > is still free must be repeated because of the dropping of the free page > queue lock, but contiguity once true is always true. Using this patch, my testbox boots just fine. Robert N M Watson FreeBSD Core Team, TrustedBSD Projects robert@fledge.watson.org Senior Research Scientist, McAfee Research > > Regards, > Alan > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.NEB.3.96L.1040304232746.56322C-100000>