Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933441Ab1ERSRn (ORCPT ); Wed, 18 May 2011 14:17:43 -0400 Received: from li9-11.members.linode.com ([67.18.176.11]:33063 "EHLO test.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932709Ab1ERSRj (ORCPT ); Wed, 18 May 2011 14:17:39 -0400 Date: Wed, 18 May 2011 14:17:11 -0400 From: "Ted Ts'o" To: "Darrick J. Wong" Cc: Jan Kara , Alexander Viro , OGAWA Hirofumi , Jens Axboe , "Martin K. Petersen" , Jeff Layton , Dave Chinner , linux-kernel , Dave Hansen , Christoph Hellwig , linux-mm@kvack.org, Chris Mason , Joel Becker , linux-scsi , linux-fsdevel , linux-ext4@vger.kernel.org, Mingming Cao Subject: Re: [5/7] ext4: Wait for writeback to complete while making pages writable Message-ID: <20110518181711.GB13693@thunk.org> Mail-Followup-To: Ted Ts'o , "Darrick J. Wong" , Jan Kara , Alexander Viro , OGAWA Hirofumi , Jens Axboe , "Martin K. Petersen" , Jeff Layton , Dave Chinner , linux-kernel , Dave Hansen , Christoph Hellwig , linux-mm@kvack.org, Chris Mason , Joel Becker , linux-scsi , linux-fsdevel , linux-ext4@vger.kernel.org, Mingming Cao References: <20110509230356.19566.48351.stgit@elm3c44.beaverton.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110509230356.19566.48351.stgit@elm3c44.beaverton.ibm.com> User-Agent: Mutt/1.5.20 (2009-06-14) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: tytso@thunk.org X-SA-Exim-Scanned: No (on test.thunk.org); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 877 Lines: 21 On Mon, May 09, 2011 at 01:03:56PM -0000, Darrick J. Wong wrote: > In order to stabilize pages during disk writes, ext4_page_mkwrite must wait for > writeback operations to complete before making a page writable. Furthermore, > the function must return locked pages, and recheck the writeback status if the > page lock is ever dropped. The "someone could wander in" part of this patch > was suggested by Chris Mason. > > Signed-off-by: Darrick J. Wong > > --- > fs/ext4/inode.c | 24 +++++++++++++++++++----- > 1 files changed, 19 insertions(+), 5 deletions(-) Added to the ext4 tree, thanks. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/