Btrfs: fix unfinished readahead thread for raid5/6 degraded mounting
authorWang Shilong <wangsl.fnst@cn.fujitsu.com>
Wed, 11 Jun 2014 02:55:22 +0000 (10:55 +0800)
committerChris Mason <clm@fb.com>
Fri, 13 Jun 2014 16:52:21 +0000 (09:52 -0700)
Steps to reproduce:

 # mkfs.btrfs -f /dev/sd[b-f] -m raid5 -d raid5
 # mkfs.ext4 /dev/sdc --->corrupt one of btrfs device
 # mount /dev/sdb /mnt -o degraded
 # btrfs scrub start -BRd /mnt

This is because readahead would skip missing device, this is not true
for RAID5/6, because REQ_GET_READ_MIRRORS return 1 for RAID5/6 block
mapping. If expected data locates in missing device, readahead thread
would not call __readahead_hook() which makes event @rc->elems=0
wait forever.

Fix this problem by checking return value of btrfs_map_block(),we
can only skip missing device safely if there are several mirrors.

Signed-off-by: Wang Shilong <wangsl.fnst@cn.fujitsu.com>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/reada.c

index 30947f923620678f9bc1a27fda49359c6b36e9a4..09230cf3a2447b3541826f21b2f3fe1dcce202ce 100644 (file)
@@ -428,8 +428,13 @@ static struct reada_extent *reada_find_extent(struct btrfs_root *root,
                        continue;
                }
                if (!dev->bdev) {
-                       /* cannot read ahead on missing device */
-                       continue;
+                       /*
+                        * cannot read ahead on missing device, but for RAID5/6,
+                        * REQ_GET_READ_MIRRORS return 1. So don't skip missing
+                        * device for such case.
+                        */
+                       if (nzones > 1)
+                               continue;
                }
                if (dev_replace_is_ongoing &&
                    dev == fs_info->dev_replace.tgtdev) {