From owner-cvs-src@FreeBSD.ORG  Sat Oct 14 23:21:48 2006
Return-Path: <owner-cvs-src@FreeBSD.ORG>
X-Original-To: cvs-src@FreeBSD.org
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id D1E6016A415;
	Sat, 14 Oct 2006 23:21:48 +0000 (UTC) (envelope-from alc@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 8026843D4C;
	Sat, 14 Oct 2006 23:21:48 +0000 (GMT) (envelope-from alc@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.13.6/8.13.6) with ESMTP id k9ENLmrp032738;
	Sat, 14 Oct 2006 23:21:48 GMT (envelope-from alc@repoman.freebsd.org)
Received: (from alc@localhost)
	by repoman.freebsd.org (8.13.6/8.13.4/Submit) id k9ENLmK3032737;
	Sat, 14 Oct 2006 23:21:48 GMT (envelope-from alc)
Message-Id: <200610142321.k9ENLmK3032737@repoman.freebsd.org>
From: Alan Cox <alc@FreeBSD.org>
Date: Sat, 14 Oct 2006 23:21:48 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/sys/vm vnode_pager.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 14 Oct 2006 23:21:48 -0000

alc         2006-10-14 23:21:48 UTC

  FreeBSD src repository

  Modified files:
    sys/vm               vnode_pager.c 
  Log:
  Long ago, revision 1.22 of vm/vm_pager.h introduced a bug.  Specifically,
  it introduced a check after the call to file system's get pages method
  that assumes that the get pages method does not change the array of pages
  that is passed to it.  In the case of vnode_pager_generic_getpages(),
  this assumption has been incorrect.  The contents of the array of pages
  may be shifted by vnode_pager_generic_getpages().  Likely, the problem
  has been hidden by vnode_pager_haspage() limiting the set of pages that
  are passed to vnode_pager_generic_getpages() such that a shift never
  occurs.
  
  The fix implemented herein is to adjust the pointer to the array of pages
  rather than shifting the pages within the array.
  
  MFC after: 3 weeks
  Fix suggested by: tegge
  
  Revision  Changes    Path
  1.232     +1 -3      src/sys/vm/vnode_pager.c