summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWang Shilong <wangsl.fnst@cn.fujitsu.com>2014-04-24 11:19:17 +0800
committerDavid Sterba <dsterba@suse.cz>2014-10-10 10:38:16 +0200
commit897c98de73627c10e9bf68798e98b3477dfc3bb0 (patch)
tree0b2403673281cf0bb0e52bcc4ae234e245904ace
parentbf17a5e43ebf7db63a64fe3e584af88e48b27355 (diff)
btrfs-progs: fsck: avoid pinning same block several times
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> Signed-off-by: David Sterba <dsterba@suse.cz>
-rw-r--r--cmds-check.c9
1 files changed, 9 insertions, 0 deletions
diff --git a/cmds-check.c b/cmds-check.c
index 23dbd0a0..afdd044d 100644
--- a/cmds-check.c
+++ b/cmds-check.c
@@ -6121,6 +6121,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);