diff options
author | Chris Mason <chris.mason@oracle.com> | 2010-10-28 15:30:42 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2010-10-29 11:25:46 -0400 |
commit | 18e503d695ff8ff9a43768555aa74575bf6b77f3 (patch) | |
tree | c17c34762dcf9988e9739ae0cdf5628885bf8828 /fs/btrfs/volumes.c | |
parent | 19fe0a8b787d7c7f9318975b5a8c6e7e5e54e925 (diff) |
Btrfs: fix raid code for removing missing drives
When btrfs is mounted in degraded mode, it has some internal structures
to track the missing devices. This missing device is setup as readonly,
but the mapping code can get upset when we try to write to it.
This changes the mapping code to return -EIO instead of oops when we try
to write to the readonly device.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/volumes.c')
-rw-r--r-- | fs/btrfs/volumes.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index dd318ff280b2..28681e729b1d 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c | |||
@@ -3034,8 +3034,7 @@ int btrfs_map_bio(struct btrfs_root *root, int rw, struct bio *bio, | |||
3034 | } | 3034 | } |
3035 | bio->bi_sector = multi->stripes[dev_nr].physical >> 9; | 3035 | bio->bi_sector = multi->stripes[dev_nr].physical >> 9; |
3036 | dev = multi->stripes[dev_nr].dev; | 3036 | dev = multi->stripes[dev_nr].dev; |
3037 | BUG_ON(rw == WRITE && !dev->writeable); | 3037 | if (dev && dev->bdev && (rw != WRITE || dev->writeable)) { |
3038 | if (dev && dev->bdev) { | ||
3039 | bio->bi_bdev = dev->bdev; | 3038 | bio->bi_bdev = dev->bdev; |
3040 | if (async_submit) | 3039 | if (async_submit) |
3041 | schedule_bio(root, dev, rw, bio); | 3040 | schedule_bio(root, dev, rw, bio); |