summaryrefslogtreecommitdiffstats
path: root/drivers/md
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2013-07-04 16:38:16 +1000
committerNeilBrown <neilb@suse.de>2013-07-04 16:42:52 +1000
commitfdcfbbb653b27964c4daa4d2bcb364259c257e7d (patch)
tree1ea7bffe6fbd61874922fb9da20c0a9157a9d71c /drivers/md
parent78eaa0d4cbcdb345992fa3dd22b3bcbb473cc064 (diff)
downloadop-kernel-dev-fdcfbbb653b27964c4daa4d2bcb364259c257e7d.zip
op-kernel-dev-fdcfbbb653b27964c4daa4d2bcb364259c257e7d.tar.gz
md/raid5: allow 5-device RAID6 to be reshaped to 4-device.
There is a bug in 'check_reshape' for raid5.c To checks that the new minimum number of devices is large enough (which is good), but it does so also after the reshape has started (bad). This is bad because - the calculation is now wrong as mddev->raid_disks has changed already, and - it is pointless because it is now too late to stop. So only perform that test when reshape has not been committed to. Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md')
-rw-r--r--drivers/md/raid5.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c
index cd9aab9..2bf094a 100644
--- a/drivers/md/raid5.c
+++ b/drivers/md/raid5.c
@@ -5914,7 +5914,7 @@ static int check_reshape(struct mddev *mddev)
return 0; /* nothing to do */
if (has_failed(conf))
return -EINVAL;
- if (mddev->delta_disks < 0) {
+ if (mddev->delta_disks < 0 && mddev->reshape_position == MaxSector) {
/* We might be able to shrink, but the devices must
* be made bigger first.
* For raid6, 4 is the minimum size.
OpenPOWER on IntegriCloud