linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ext4: move_extent explicitly invalidate page buffers
@ 2014-10-23 11:48 Dmitry Monakhov
  2014-10-23 13:03 ` Dmitry Monakhov
  0 siblings, 1 reply; 8+ messages in thread
From: Dmitry Monakhov @ 2014-10-23 11:48 UTC (permalink / raw)
  To: linux-ext4; +Cc: tytso, wangxg.fnst, Dmitry Monakhov

In hard core test-cases such as ext4/301, ext4/302 some bh may
becomes dirty so try_to_release_page() will fail and result in
false positive EBUSY failures. We can easily fix that by
explicit ->invalidatepage() after we holds page which is locked
and uptodate.

Tested-by: Xiaoguang Wang<wangxg.fnst@cn.fujitsu.com>
Signed-off-by: Dmitry Monakhov <dmonakhov@openvz.org>
---
 fs/ext4/move_extent.c |   10 +++++++---
 1 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c
index c2b2b02..76c45b6 100644
--- a/fs/ext4/move_extent.c
+++ b/fs/ext4/move_extent.c
@@ -348,8 +348,9 @@ again:
 		     !try_to_release_page(pagep[0], 0)) ||
 		    (page_has_private(pagep[1]) &&
 		     !try_to_release_page(pagep[1], 0))) {
-			*err = -EBUSY;
-			goto drop_data_sem;
+			/* One of buffers is busy, fall back data copy */
+			ext4_double_up_write_data_sem(orig_inode, donor_inode);
+			goto data_copy;
 		}
 		replaced_count = ext4_swap_extents(handle, orig_inode,
 						   donor_inode, orig_blk_offset,
@@ -360,12 +361,15 @@ again:
 		goto unlock_pages;
 	}
 data_copy:
-	*err = mext_page_mkuptodate(pagep[0], from, from + replaced_size);
+	/* In order to drop all buffers we have to make page fully uptodate */
+	*err = mext_page_mkuptodate(pagep[0], 0, PAGE_CACHE_SIZE);
 	if (*err)
 		goto unlock_pages;
 
 	/* At this point all buffers in range are uptodate, old mapping layout
 	 * is no longer required, try to drop it now. */
+	do_invalidatepage(pagep[0], 0, PAGE_CACHE_SIZE);
+	do_invalidatepage(pagep[1], 0, PAGE_CACHE_SIZE);
 	if ((page_has_private(pagep[0]) && !try_to_release_page(pagep[0], 0)) ||
 	    (page_has_private(pagep[1]) && !try_to_release_page(pagep[1], 0))) {
 		*err = -EBUSY;
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-01-19  7:05 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-23 11:48 [PATCH] ext4: move_extent explicitly invalidate page buffers Dmitry Monakhov
2014-10-23 13:03 ` Dmitry Monakhov
2014-12-10  9:21   ` Xiaoguang Wang
2014-12-10 10:25     ` Dmitry Monakhov
2014-12-10 11:04       ` Xiaoguang Wang
2015-01-11  9:50       ` Xiaoguang Wang
2015-01-13 13:32         ` Dmitry Monakhov
2015-01-19  7:00           ` Xiaoguang Wang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).