From owner-cvs-src@FreeBSD.ORG  Sat Aug 25 19:14:31 2007
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 0ADD916A419;
	Sat, 25 Aug 2007 19:14:31 +0000 (UTC) (envelope-from alc@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id E76C113C459;
	Sat, 25 Aug 2007 19:14:30 +0000 (UTC) (envelope-from alc@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id l7PJEUk2097138;
	Sat, 25 Aug 2007 19:14:30 GMT (envelope-from alc@repoman.freebsd.org)
Received: (from alc@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id l7PJEUTZ097136;
	Sat, 25 Aug 2007 19:14:30 GMT (envelope-from alc)
Message-Id: <200708251914.l7PJEUTZ097136@repoman.freebsd.org>
From: Alan Cox <alc@FreeBSD.org>
Date: Sat, 25 Aug 2007 19:14:30 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: RELENG_6
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, 25 Aug 2007 19:14:31 -0000

alc         2007-08-25 19:14:30 UTC

  FreeBSD src repository

  Modified files:        (Branch: RELENG_6)
    sys/vm               vnode_pager.c 
  Log:
  MFC revision 1.231
    Change vnode_pager_addr() such that on returning it distinguishes between
    an error returned by VOP_BMAP() and a hole in the file.
  
    Change the callers to vnode_pager_addr() such that they return
    VM_PAGER_ERROR when VOP_BMAP fails instead of a zero-filled page.
  
  MFC revision 1.232
    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.
  
  Revision   Changes    Path
  1.221.2.8  +25 -22    src/sys/vm/vnode_pager.c