diff options
| author | NeilBrown <neilb@suse.com> | 2016-06-02 08:19:52 +0200 |
|---|---|---|
| committer | Shaohua Li <shli@fb.com> | 2016-06-13 20:54:12 +0200 |
| commit | 8430e7e0af9a15063b90343e3beebc164c8e90f3 (patch) | |
| tree | 4facf6ea5e596b014ceff1792de6006b3cb6bb27 | |
| parent | raid1/raid10: slow down resync if there is non-resync activity pending (diff) | |
| download | linux-8430e7e0af9a15063b90343e3beebc164c8e90f3.tar.xz linux-8430e7e0af9a15063b90343e3beebc164c8e90f3.zip | |
md: disconnect device from personality before trying to remove it.
When the HOT_REMOVE_DISK ioctl is used to remove a device, we
call remove_and_add_spares() which will remove it from the personality
if possible. This improves the chances that the removal will succeed.
When writing "remove" to dev-XX/state, we don't. So that can fail more easily.
So add the remove_and_add_spares() into "remove" handling.
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Shaohua Li <shli@fb.com>
| -rw-r--r-- | drivers/md/md.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index d0c1e79da49e..0793754eeffd 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -2595,6 +2595,8 @@ state_store(struct md_rdev *rdev, const char *buf, size_t len) else err = -EBUSY; } else if (cmd_match(buf, "remove")) { + clear_bit(Blocked, &rdev->flags); + remove_and_add_spares(rdev->mddev, rdev); if (rdev->raid_disk >= 0) err = -EBUSY; else { |