From owner-cvs-all@FreeBSD.ORG Mon Mar 2 08:40:49 2009 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7468C1065672; Mon, 2 Mar 2009 08:40:49 +0000 (UTC) (envelope-from pjd@garage.freebsd.pl) Received: from mail.garage.freebsd.pl (chello087206045082.chello.pl [87.206.45.82]) by mx1.freebsd.org (Postfix) with ESMTP id B4FF28FC22; Mon, 2 Mar 2009 08:40:47 +0000 (UTC) (envelope-from pjd@garage.freebsd.pl) Received: by mail.garage.freebsd.pl (Postfix, from userid 65534) id 235504569A; Mon, 2 Mar 2009 09:14:18 +0100 (CET) Received: from localhost (pjd.wheel.pl [10.0.1.1]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.garage.freebsd.pl (Postfix) with ESMTP id 195FD45684; Mon, 2 Mar 2009 09:14:11 +0100 (CET) Date: Mon, 2 Mar 2009 09:14:45 +0100 From: Pawel Jakub Dawidek To: Alan Cox Message-ID: <20090302081445.GA2993@garage.freebsd.pl> References: <200712190547.lBJ5lpCU039889@repoman.freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="UugvWAfsgieZRqgk" Content-Disposition: inline In-Reply-To: <200712190547.lBJ5lpCU039889@repoman.freebsd.org> User-Agent: Mutt/1.4.2.3i X-PGP-Key-URL: http://people.freebsd.org/~pjd/pjd.asc X-OS: FreeBSD 8.0-CURRENT i386 X-Spam-Checker-Version: SpamAssassin 3.0.4 (2005-06-05) on mail.garage.freebsd.pl X-Spam-Level: X-Spam-Status: No, score=-5.9 required=3.0 tests=ALL_TRUSTED,BAYES_00 autolearn=ham version=3.0.4 Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/vm vm_page.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 02 Mar 2009 08:40:50 -0000 --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 > -__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 > #include > @@ -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--