diff options
author | NeilBrown <neilb@suse.com> | 2016-06-02 02:19:52 -0400 |
---|---|---|
committer | Shaohua Li <shli@fb.com> | 2016-06-13 14:54:12 -0400 |
commit | 8430e7e0af9a15063b90343e3beebc164c8e90f3 (patch) | |
tree | 4facf6ea5e596b014ceff1792de6006b3cb6bb27 /drivers/md | |
parent | 7ac504472263180745ac94242f1d253eb7284e48 (diff) |
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>
Diffstat (limited to 'drivers/md')
-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) | |||
2595 | else | 2595 | else |
2596 | err = -EBUSY; | 2596 | err = -EBUSY; |
2597 | } else if (cmd_match(buf, "remove")) { | 2597 | } else if (cmd_match(buf, "remove")) { |
2598 | clear_bit(Blocked, &rdev->flags); | ||
2599 | remove_and_add_spares(rdev->mddev, rdev); | ||
2598 | if (rdev->raid_disk >= 0) | 2600 | if (rdev->raid_disk >= 0) |
2599 | err = -EBUSY; | 2601 | err = -EBUSY; |
2600 | else { | 2602 | else { |