[2/3] Btrfs-progs: fsck: avoid pinning same block several times
diff mbox

Message ID 1398309558-29509-2-git-send-email-wangsl.fnst@cn.fujitsu.com
State Accepted
Headers show

Commit Message

Wang Shilong April 24, 2014, 3:19 a.m. UTC
This can not only give some speedups but also avoid forever loop
with a really broken filesystem.

Signed-off-by: Wang Shilong <wangsl.fnst@cn.fujitsu.com>
---
 cmds-check.c | 9 +++++++++
 1 file changed, 9 insertions(+)

Patch
diff mbox

diff --git a/cmds-check.c b/cmds-check.c
index 15806a4..e6fb380 100644
--- a/cmds-check.c
+++ b/cmds-check.c
@@ -6189,6 +6189,15 @@  static int pin_down_tree_blocks(struct btrfs_fs_info *fs_info,
 	int ret;
 	int i;
 
+	/*
+	 * If we have pinned this block before, don't pin it again.
+	 * This can not only avoid forever loop with broken filesystem
+	 * but also give us some speedups.
+	 */
+	if (test_range_bit(&fs_info->pinned_extents, eb->start,
+			   eb->start + eb->len - 1, EXTENT_DIRTY, 0))
+		return 0;
+
 	btrfs_pin_extent(fs_info, eb->start, eb->len);
 
 	leafsize = btrfs_super_leafsize(fs_info->super_copy);