From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f65.google.com ([209.85.220.65]:33294 "EHLO mail-pa0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750909AbcEULPj (ORCPT ); Sat, 21 May 2016 07:15:39 -0400 Received: by mail-pa0-f65.google.com with SMTP id f8so1408368pag.0 for ; Sat, 21 May 2016 04:15:39 -0700 (PDT) Date: Sat, 21 May 2016 19:15:33 +0800 From: Eryu Guan To: gregkh@linuxfoundation.org Cc: tytso@mit.edu, stable@vger.kernel.org, tmhikaru@gmail.com Subject: Re: FAILED: patch "[PATCH] ext4: iterate over buffer heads correctly in" failed to apply to 4.5-stable tree Message-ID: <20160521111533.GE10350@eguan.usersys.redhat.com> References: <1460249484173221@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1460249484173221@kroah.com> Sender: stable-owner@vger.kernel.org List-ID: Hi greg, On Sat, Apr 09, 2016 at 05:51:24PM -0700, gregkh@linuxfoundation.org wrote: > > The patch below does not apply to the 4.5-stable tree. > If someone wants it applied there, or to any other stable or longterm > tree, then please email the backport, including the original git commit > id to . 4.5 kernel has this patch already, so this failure is expected. But 4.4 stable needs it as well, and the patch should be applied cleanly. Can you please consider adding commit 6ffe77bad545 ("ext4: iterate over buffer heads correctly in move_extent_per_page()") to 4.4 stable tree? Thanks! Eryu > > thanks, > > greg k-h > > ------------------ original commit in Linus's tree ------------------ > > From 87f9a031af48defee9f34c6aaf06d6f1988c244d Mon Sep 17 00:00:00 2001 > From: Eryu Guan > Date: Sun, 21 Feb 2016 18:38:44 -0500 > Subject: [PATCH] ext4: iterate over buffer heads correctly in > move_extent_per_page() > > In commit bcff24887d00 ("ext4: don't read blocks from disk after extents > being swapped") bh is not updated correctly in the for loop and wrong > data has been written to disk. generic/324 catches this on sub-page > block size ext4. > > Fixes: bcff24887d00 ("ext4: don't read blocks from disk after extentsbeing swapped") > Signed-off-by: Eryu Guan > Signed-off-by: Theodore Ts'o > Cc: stable@vger.kernel.org > > diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c > index e032a0423e35..4098acc701c3 100644 > --- a/fs/ext4/move_extent.c > +++ b/fs/ext4/move_extent.c > @@ -390,6 +390,7 @@ data_copy: > *err = ext4_get_block(orig_inode, orig_blk_offset + i, bh, 0); > if (*err < 0) > break; > + bh = bh->b_this_page; > } > if (!*err) > *err = block_commit_write(pagep[0], from, from + replaced_size); >