From mboxrd@z Thu Jan 1 00:00:00 1970 Message-Id: <20060912144903.770953000@chello.nl> References: <20060912143049.278065000@chello.nl> Subject: [PATCH 07/20] nfs: add a comment explaining the use of PG_private in the NFS client Content-Disposition: inline; filename=nfs_PG_private_comment.patch Date: Tue, 12 Sep 2006 17:25:49 +0200 From: Peter Zijlstra Sender: owner-linux-mm@kvack.org Return-Path: To: linux-mm@kvack.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org Cc: Linus Torvalds , Andrew Morton , David Miller , Rik van Riel , Daniel Phillips , Peter Zijlstra , Trond Myklebust List-ID: Add a little comment explaining the use of PG_private in the NFS client. Signed-off-by: Peter Zijlstra CC: Trond Myklebust --- fs/nfs/write.c | 5 +++++ 1 file changed, 5 insertions(+) Index: linux-2.6/fs/nfs/write.c =================================================================== --- linux-2.6.orig/fs/nfs/write.c +++ linux-2.6/fs/nfs/write.c @@ -417,6 +417,11 @@ static int nfs_inode_add_request(struct if (nfs_have_delegation(inode, FMODE_WRITE)) nfsi->change_attr++; } + /* + * The PG_private bit is unfortunately needed if we want to fix the + * hole in the mmap semantics. If we do not set it, then the VM will + * fail to call the "releasepage" address ops. + */ SetPagePrivate(req->wb_page); nfsi->npages++; atomic_inc(&req->wb_count); -- -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org