Commit c744a65c authored Mar 19, 2012 by NeilBrown

md: don't set md arrays to readonly on shutdown.



It seems that with recent kernel, writeback can still be happening
while shutdown is happening, and consequently data can be written
after the md reboot notifier switches all arrays to read-only.
This causes a BUG.

So don't switch them to read-only - just mark them clean and
set 'safemode' to '2' which mean that immediately after any
write the array will be switch back to 'clean'.

This could result in the shutdown happening when array is marked
dirty, thus forcing a resync on reboot.  However if you reboot
without performing a "sync" first, you get to keep both halves.

This is suitable for any stable kernel (though there might be some
conflicts with obvious fixes in earlier kernels).

Cc: stable@vger.kernel.org
Signed-off-by: NeilBrown <neilb@suse.de>

parent dc10c643

drivers/md/md.c

+15 −22

Original line number	Diff line number	Diff line
		@@ -8157,17 +8157,10 @@ static int md_notify_reboot(struct notifier_block *this,
		struct mddev *mddev;
		int need_delay = 0;

		if ((code == SYS_DOWN) \|\| (code == SYS_HALT) \|\| (code == SYS_POWER_OFF)) {

		printk(KERN_INFO "md: stopping all md devices.\n");

		for_each_mddev(mddev, tmp) {
		if (mddev_trylock(mddev)) {
		/* Force a switch to readonly even array
		* appears to still be in use. Hence
		* the '100'.
		*/
		md_set_readonly(mddev, 100);
		__md_stop_writes(mddev);
		mddev->safemode = 2;
		mddev_unlock(mddev);
		}
		need_delay = 1;
		@@ -8180,7 +8173,7 @@ static int md_notify_reboot(struct notifier_block *this,
		*/
		if (need_delay)
		mdelay(1000*1);
		}

		return NOTIFY_DONE;
		}

Admin message