diff options
| author | NeilBrown <neilb@suse.de> | 2009-08-18 10:35:26 +1000 |
|---|---|---|
| committer | NeilBrown <neilb@suse.de> | 2009-08-18 10:35:26 +1000 |
| commit | 80ffb3cceaefa405f2ecd46d66500ed8d53efe74 (patch) | |
| tree | 867d6ec951663255076ea42fab29a31eac4e51c0 | |
| parent | 4d484a4a7a5126410eed5f8dd329a33f6eeed068 (diff) | |
| download | linux-80ffb3cceaefa405f2ecd46d66500ed8d53efe74.tar.bz2 | |
Fix new incorrect error return from do_md_stop.
Recent commit c8c00a6915a2e3d10416e8bdd3138429beb96210
changed the exit paths in do_md_stop and was not quite
careful enough. There is one path were 'err' now needs
to be cleared but it isn't.
So setting an array to readonly (with mdadm --readonly) will
work, but will incorrectly report and error: ENXIO.
Signed-off-by: NeilBrown <neilb@suse.de>
| -rw-r--r-- | drivers/md/md.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 103f2d33fa89..9dd872000cec 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -4364,6 +4364,7 @@ static int do_md_stop(mddev_t * mddev, int mode, int is_open) if (mode == 1) set_disk_ro(disk, 1); clear_bit(MD_RECOVERY_FROZEN, &mddev->recovery); + err = 0; } out: mutex_unlock(&mddev->open_mutex); |