From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zheng Liu Subject: Re: [PATCH 1/5] ext4: convert write_begin methods to stable_page_writes semantics Date: Mon, 22 Apr 2013 20:36:48 +0800 Message-ID: <20130422123648.GC2039@gmail.com> References: <1365966097-8968-1-git-send-email-dmonakhov@openvz.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-ext4@vger.kernel.org, jack@suse.cz To: Dmitry Monakhov Return-path: Received: from mail-pa0-f51.google.com ([209.85.220.51]:46239 "EHLO mail-pa0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751304Ab3DVMTf (ORCPT ); Mon, 22 Apr 2013 08:19:35 -0400 Received: by mail-pa0-f51.google.com with SMTP id jh10so3588074pab.10 for ; Mon, 22 Apr 2013 05:19:35 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1365966097-8968-1-git-send-email-dmonakhov@openvz.org> Sender: linux-ext4-owner@vger.kernel.org List-ID: On Sun, Apr 14, 2013 at 11:01:33PM +0400, Dmitry Monakhov wrote: > use wait_for_stable_page() instead of wait_on_page_writeback() > > Signed-off-by: Dmitry Monakhov Reviewed-by: Zheng Liu Regards, - Zheng > --- > fs/ext4/inode.c | 5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c > index 62189c8..1be5827 100644 > --- a/fs/ext4/inode.c > +++ b/fs/ext4/inode.c > @@ -1028,7 +1028,8 @@ retry_journal: > ext4_journal_stop(handle); > goto retry_grab; > } > - wait_on_page_writeback(page); > + /* In case writeback began while the page was unlocked */ > + wait_for_stable_page(page); > > if (ext4_should_dioread_nolock(inode)) > ret = __block_write_begin(page, pos, len, ext4_get_block_write); > @@ -2715,7 +2716,7 @@ retry_journal: > goto retry_grab; > } > /* In case writeback began while the page was unlocked */ > - wait_on_page_writeback(page); > + wait_for_stable_page(page); > > ret = __block_write_begin(page, pos, len, ext4_da_get_block_prep); > if (ret < 0) { > -- > 1.7.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html