summaryrefslogtreecommitdiffstats
path: root/drivers/md
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2011-10-18 12:13:47 +1100
committerNeilBrown <neilb@suse.de>2011-10-18 12:13:47 +1100
commitd30519fc59c5cc2f7772fa67b16b1a2426d36c95 (patch)
tree3aae0a21c16816e1989ea90db23a5b123a5a6a12 /drivers/md
parent34db0cd60f8a1f4ab73d118a8be3797c20388223 (diff)
downloadop-kernel-dev-d30519fc59c5cc2f7772fa67b16b1a2426d36c95.zip
op-kernel-dev-d30519fc59c5cc2f7772fa67b16b1a2426d36c95.tar.gz
md: clear In_sync bit on devices added to an active array.
When we add a device to an active array it can be meaningful to set the 'insync' flag. This indicates that the device is in-sync with the array except for locations recorded in the bitmap. A bitmap-based recovery can then bring it completely in-sync. Internally we move that flag to 'saved_raid_disk' but forgot to clear In_sync like we do in add_new_disk. So clear In_sync after moving its value to saved_raid_disk. Reported-by: Andrei Warkentin <andreiw@vmware.com> Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md')
-rw-r--r--drivers/md/md.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c
index 7a49177..0ea3485 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -2722,6 +2722,7 @@ slot_store(struct md_rdev *rdev, const char *buf, size_t len)
rdev->saved_raid_disk = slot;
else
rdev->saved_raid_disk = -1;
+ clear_bit(In_sync, &rdev->flags);
err = rdev->mddev->pers->
hot_add_disk(rdev->mddev, rdev);
if (err) {
OpenPOWER on IntegriCloud