From owner-cvs-all@FreeBSD.ORG  Thu Mar  4 20:46:32 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 78B0F16A4CE; Thu,  4 Mar 2004 20:46:32 -0800 (PST)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 5DCEF43D1F; Thu,  4 Mar 2004 20:46:32 -0800 (PST)
	(envelope-from alc@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.10/8.12.10) with ESMTP id i254kWGe046657;
	Thu, 4 Mar 2004 20:46:32 -0800 (PST)
	(envelope-from alc@repoman.freebsd.org)
Received: (from alc@localhost)
	by repoman.freebsd.org (8.12.10/8.12.10/Submit) id i254kWoF046656;
	Thu, 4 Mar 2004 20:46:32 -0800 (PST)
	(envelope-from alc)
Message-Id: <200403050446.i254kWoF046656@repoman.freebsd.org>
From: Alan Cox <alc@FreeBSD.org>
Date: Thu, 4 Mar 2004 20:46:32 -0800 (PST)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/vm vm_contig.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 05 Mar 2004 04:46:32 -0000

alc         2004/03/04 20:46:32 PST

  FreeBSD src repository

  Modified files:
    sys/vm               vm_contig.c 
  Log:
  In the last revision, I introduced a physical contiguity check that is both
  unnecessary and wrong.  While it is necessary to verify that the page is
  still free after dropping and reacquiring the free page queue lock, the
  physical contiguity of the page can not change, making this check
  unnecessary.  This check was wrong in that it could cause an out-of-bounds
  array access.
  
  Tested by:      rwatson
  
  Revision  Changes    Path
  1.32      +1 -3      src/sys/vm/vm_contig.c