Commit 830778a1 authored Jan 14, 2014 by NeilBrown

md: ensure metadata is writen after raid level change.



level_store() currently does not make sure the metadata is
updates to reflect the new raid level.  It simply sets MD_CHANGE_DEVS.

Any level with a ->thread will quickly notice this and update the
metadata.  However RAID0 and Linear do not have a thread so no
metadata update happens until the array is stopped.  At that point the
metadata is written.

This is later that we would like.  While the delay doesn't risk any
data it can cause confusion.  So if there is no md thread, immediately
update the metadata after a level change.

Reported-by: Richard Michael <rmichael@edgeofthenet.org>
Signed-off-by: NeilBrown <neilb@suse.de>

parent 0b59bb64

drivers/md/md.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -3611,6 +3611,8 @@ level_store(struct mddev mddev, const char buf, size_t len)
		pers->run(mddev);
		set_bit(MD_CHANGE_DEVS, &mddev->flags);
		mddev_resume(mddev);
		if (!mddev->thread)
		md_update_sb(mddev, 1);
		sysfs_notify(&mddev->kobj, NULL, "level");
		md_new_event(mddev);
		return rv;

Admin message