md: don't update metadata when stopping a read-only array.
authorNeilBrown <neilb@suse.de>
Wed, 24 Apr 2013 01:42:42 +0000 (11:42 +1000)
committerNeilBrown <neilb@suse.de>
Wed, 24 Apr 2013 01:42:42 +0000 (11:42 +1000)
read-only arrays should stay that way as much as possible.
Updating the metadata - which could be triggered by a re-add
while assembling the array metadata - should be avoided.

Signed-off-by: NeilBrown <neilb@suse.de>
drivers/md/md.c

index 491afda..3013228 100644 (file)
@@ -5293,7 +5293,8 @@ static void __md_stop_writes(struct mddev *mddev)
        bitmap_flush(mddev);
        md_super_wait(mddev);
 
-       if (!mddev->in_sync || mddev->flags) {
+       if (mddev->ro == 0 &&
+           (!mddev->in_sync || mddev->flags)) {
                /* mark array as shutdown cleanly */
                mddev->in_sync = 1;
                md_update_sb(mddev, 1);