Commit ab16bfc7 authored by Neil Brown's avatar Neil Brown Committed by NeilBrown

md: clear Blocked flag on failed devices when array is read-only.

The Blocked flag indicates that a device has failed but that this
fact hasn't been recorded in the metadata yet.  Writes to such
devices cannot be allowed until the metadata has been updated.

On a read-only array, the Blocked flag will never be cleared.
This prevents the device being removed from the array.

If the metadata is being handled by the kernel
(i.e. !mddev->external), then we can be sure that if the array is
switch to writable, then a metadata update will happen and will
record the failure.  So we don't need the flag set.

If metadata is externally managed, it is upto the external manager
to clear the 'blocked' flag.
Reported-by: default avatarXiaoNi <xni@redhat.com>
Signed-off-by: default avatarNeilBrown <neilb@suse.de>
parent 9a8c0fa8
...@@ -8130,6 +8130,15 @@ void md_check_recovery(struct mddev *mddev) ...@@ -8130,6 +8130,15 @@ void md_check_recovery(struct mddev *mddev)
int spares = 0; int spares = 0;
if (mddev->ro) { if (mddev->ro) {
struct md_rdev *rdev;
if (!mddev->external && mddev->in_sync)
/* 'Blocked' flag not needed as failed devices
* will be recorded if array switched to read/write.
* Leaving it set will prevent the device
* from being removed.
*/
rdev_for_each(rdev, mddev)
clear_bit(Blocked, &rdev->flags);
/* On a read-only array we can: /* On a read-only array we can:
* - remove failed devices * - remove failed devices
* - add already-in_sync devices if the array itself * - add already-in_sync devices if the array itself
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment