Btrfs: SB read failure should return EIO for __bread failure
authorAnand Jain <anand.jain@oracle.com>
Fri, 14 Aug 2015 10:32:51 +0000 (18:32 +0800)
committerDavid Sterba <dsterba@suse.com>
Tue, 29 Sep 2015 14:29:59 +0000 (16:29 +0200)
This will return EIO when __bread() fails to read SB,
instead of EINVAL.

Signed-off-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/disk-io.c
fs/btrfs/volumes.c

index 76734732eca6701feb49173b272de8068e13234a..357d9f9c99688f186c101fb00715a5c0395405a9 100644 (file)
@@ -2653,8 +2653,8 @@ int open_ctree(struct super_block *sb,
         * Read super block and check the signature bytes only
         */
        bh = btrfs_read_dev_super(fs_devices->latest_bdev);
-       if (!bh) {
-               err = -EINVAL;
+       if (IS_ERR(bh)) {
+               err = PTR_ERR(bh);
                goto fail_alloc;
        }
 
@@ -3196,6 +3196,7 @@ struct buffer_head *btrfs_read_dev_super(struct block_device *bdev)
        int i;
        u64 transid = 0;
        u64 bytenr;
+       int ret = -EINVAL;
 
        /* we would like to check all the supers, but that would make
         * a btrfs mount succeed after a mkfs from a different FS.
@@ -3209,13 +3210,20 @@ struct buffer_head *btrfs_read_dev_super(struct block_device *bdev)
                        break;
                bh = __bread(bdev, bytenr / 4096,
                                        BTRFS_SUPER_INFO_SIZE);
-               if (!bh)
+               /*
+                * If we fail to read from the underlying devices, as of now
+                * the best option we have is to mark it EIO.
+                */
+               if (!bh) {
+                       ret = -EIO;
                        continue;
+               }
 
                super = (struct btrfs_super_block *)bh->b_data;
                if (btrfs_super_bytenr(super) != bytenr ||
                    btrfs_super_magic(super) != BTRFS_MAGIC) {
                        brelse(bh);
+                       ret = -EINVAL;
                        continue;
                }
 
@@ -3227,6 +3235,10 @@ struct buffer_head *btrfs_read_dev_super(struct block_device *bdev)
                        brelse(bh);
                }
        }
+
+       if (!latest)
+               return ERR_PTR(ret);
+
        return latest;
 }
 
index 643879208f05c391a6aec6e2bd13bafe2303f65a..e8a618261218f58cc17d9ff7a21d754da146fc86 100644 (file)
@@ -211,8 +211,8 @@ btrfs_get_bdev_and_sb(const char *device_path, fmode_t flags, void *holder,
        }
        invalidate_bdev(*bdev);
        *bh = btrfs_read_dev_super(*bdev);
-       if (!*bh) {
-               ret = -EINVAL;
+       if (IS_ERR(*bh)) {
+               ret = PTR_ERR(*bh);
                blkdev_put(*bdev, flags);
                goto error;
        }
@@ -6746,8 +6746,8 @@ int btrfs_scratch_superblock(struct btrfs_device *device)
        struct btrfs_super_block *disk_super;
 
        bh = btrfs_read_dev_super(device->bdev);
-       if (!bh)
-               return -EINVAL;
+       if (IS_ERR(bh))
+               return PTR_ERR(bh);
        disk_super = (struct btrfs_super_block *)bh->b_data;
 
        memset(&disk_super->magic, 0, sizeof(disk_super->magic));