From owner-p4-projects@FreeBSD.ORG Wed Jul 11 15:39:14 2012 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id D217E1065676; Wed, 11 Jul 2012 15:39:13 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 949111065675 for ; Wed, 11 Jul 2012 15:39:13 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from skunkworks.freebsd.org (skunkworks.freebsd.org [IPv6:2001:4f8:fff6::2d]) by mx1.freebsd.org (Postfix) with ESMTP id 7E1CB8FC17 for ; Wed, 11 Jul 2012 15:39:13 +0000 (UTC) Received: from skunkworks.freebsd.org (localhost [127.0.0.1]) by skunkworks.freebsd.org (8.14.4/8.14.4) with ESMTP id q6BFdDVr078305 for ; Wed, 11 Jul 2012 15:39:13 GMT (envelope-from jhb@freebsd.org) Received: (from perforce@localhost) by skunkworks.freebsd.org (8.14.4/8.14.4/Submit) id q6BFdDHS078302 for perforce@freebsd.org; Wed, 11 Jul 2012 15:39:13 GMT (envelope-from jhb@freebsd.org) Date: Wed, 11 Jul 2012 15:39:13 GMT Message-Id: <201207111539.q6BFdDHS078302@skunkworks.freebsd.org> X-Authentication-Warning: skunkworks.freebsd.org: perforce set sender to jhb@freebsd.org using -f From: John Baldwin To: Perforce Change Reviews Precedence: bulk Cc: Subject: PERFORCE change 214248 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 11 Jul 2012 15:39:14 -0000 http://p4web.freebsd.org/@@214248?ac=10 Change 214248 by jhb@jhb_jhbbsd on 2012/07/11 15:38:26 Fix mismerge. Affected files ... .. //depot/projects/fadvise/sys/kern/vfs_vnops.c#11 edit Differences ... ==== //depot/projects/fadvise/sys/kern/vfs_vnops.c#11 (text+ko) ==== @@ -678,12 +678,6 @@ if (error == 0) #endif error = VOP_READ(vp, uio, ioflag, fp->f_cred); - mtx_lock(mtxp); - if (fp->f_vnread_flags & FOFFSET_LOCK_WAITING) - wakeup(&fp->f_vnread_flags); - fp->f_vnread_flags = 0; - mtx_unlock(mtxp); - } fp->f_nextoff = uio->uio_offset; VOP_UNLOCK(vp, 0); if (error == 0 && advice == POSIX_FADV_NOREUSE && @@ -706,24 +700,6 @@ start = offset; end = uio->uio_offset - 1; mtxp = mtx_pool_find(mtxpool_sleep, fp); - offset != uio->uio_offset) { - /* - * Use POSIX_FADV_DONTNEED to flush clean pages and - * buffers for the backing file after a - * POSIX_FADV_NOREUSE read(2). To optimize the common - * case of using POSIX_FADV_NOREUSE with sequential - * access, track the previous implicit DONTNEED - * request and grow this request to include the - * current read(2) in addition to the previous - * DONTNEED. With purely sequential access this will - * cause the DONTNEED requests to continously grow to - * cover all of the previously read regions of the - * file. This allows filesystem blocks that are - * accessed by multiple calls to read(2) to be flushed - * once the last read(2) finishes. - */ - start = offset; - end = uio->uio_offset - 1; mtx_lock(mtxp); if (fp->f_advice != NULL && fp->f_advice->fa_advice == POSIX_FADV_NOREUSE) {