Date: Mon, 2 Mar 2009 09:14:45 +0100 From: Pawel Jakub Dawidek <pjd@FreeBSD.org> To: Alan Cox <alc@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/vm vm_page.c Message-ID: <20090302081445.GA2993@garage.freebsd.pl> In-Reply-To: <200712190547.lBJ5lpCU039889@repoman.freebsd.org> References: <200712190547.lBJ5lpCU039889@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--UugvWAfsgieZRqgk Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Dec 19, 2007 at 05:47:51AM +0000, Alan Cox wrote: > alc 2007-12-19 05:47:51 UTC >=20 > FreeBSD src repository >=20 > Modified files: > sys/vm vm_page.c=20 > Log: > Eliminate redundant code from vm_page_startup(). > =20 > Revision Changes Path > 1.363 +0 -17 src/sys/vm/vm_page.c >=20 > http://cvsweb.FreeBSD.org/src/sys/vm/vm_page.c.diff?r1=3D1.362&r2=3D1.363= &f=3Dh > --- src/sys/vm/vm_page.c 2007/12/11 21:20:34 1.362 > +++ src/sys/vm/vm_page.c 2007/12/19 05:47:50 1.363 > @@ -97,7 +97,7 @@ > */ > =20 > #include <sys/cdefs.h> > -__FBSDID("$FreeBSD: /usr/local/www/cvsroot/FreeBSD/src/sys/vm/vm_page.c,= v 1.362 2007/12/11 21:20:34 alc Exp $"); > +__FBSDID("$FreeBSD: /usr/local/www/cvsroot/FreeBSD/src/sys/vm/vm_page.c,= v 1.363 2007/12/19 05:47:50 alc Exp $"); > =20 > #include <sys/param.h> > #include <sys/systm.h> > @@ -201,7 +201,6 @@ vm_offset_t > vm_page_startup(vm_offset_t vaddr) > { > vm_offset_t mapped; > - vm_size_t npages; > vm_paddr_t page_range; > vm_paddr_t new_end; > int i; > @@ -216,9 +215,6 @@ vm_page_startup(vm_offset_t vaddr) > vm_paddr_t low_water, high_water; > int biggestone; > =20 > - vm_paddr_t total; > - > - total =3D 0; > biggestsize =3D 0; > biggestone =3D 0; > nblocks =3D 0; > @@ -244,7 +240,6 @@ vm_page_startup(vm_offset_t vaddr) > if (phys_avail[i + 1] > high_water) > high_water =3D phys_avail[i + 1]; > ++nblocks; > - total +=3D size; > } > =20 > end =3D phys_avail[biggestone+1]; > @@ -308,8 +303,6 @@ vm_page_startup(vm_offset_t vaddr) > #else > #error "Either VM_PHYSSEG_DENSE or VM_PHYSSEG_SPARSE must be defined." > #endif > - npages =3D (total - (page_range * sizeof(struct vm_page)) - > - (end - new_end)) / PAGE_SIZE; > end =3D new_end; > =20 > /* > @@ -345,16 +338,6 @@ vm_page_startup(vm_offset_t vaddr) > vm_page_array_size =3D page_range; > =20 > /* > - * This assertion tests the hypothesis that npages and total are > - * redundant. XXX > - */ > - page_range =3D 0; > - for (i =3D 0; phys_avail[i + 1] !=3D 0; i +=3D 2) > - page_range +=3D atop(phys_avail[i + 1] - phys_avail[i]); > - KASSERT(page_range =3D=3D npages, > - ("vm_page_startup: inconsistent page counts")); > - > - /* > * Initialize the physical memory allocator. > */ > vm_phys_init(); This assertion is triggered on my 7-STABLE machine. It was fine when it was running 6.1, but now after upgrade it panics. If this assert is bogus could you MFC it to 7-STABLE as well? --=20 Pawel Jakub Dawidek http://www.wheel.pl pjd@FreeBSD.org http://www.FreeBSD.org FreeBSD committer Am I Evil? Yes, I Am! --UugvWAfsgieZRqgk Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.4 (FreeBSD) iD8DBQFJq5V0ForvXbEpPzQRAjGoAKDXWk/uUDTlTspWyuhbc5ygyVFhQACgujca L7wsokqGfEi0TDDdMmFKP6w= =qmDR -----END PGP SIGNATURE----- --UugvWAfsgieZRqgk--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090302081445.GA2993>