md: fix race when unfreezing sync_action
authorNeilBrown <neilb@suse.de>
Thu, 28 May 2015 07:53:29 +0000 (17:53 +1000)
committerNeilBrown <neilb@suse.de>
Thu, 28 May 2015 08:04:45 +0000 (18:04 +1000)
commit56ccc1125bc141cf63927eda7febff4216dea2d3
tree3af5cac150be4c0cf4d2e0d46dc9fba1328b8670
parent626f2092c85ac847bb80b3257eb6a565dec32278
md: fix race when unfreezing sync_action

A recent change removed the need for locking around writing
to "sync_action" (and various other places), but introduced a
subtle race.
When e.g. setting 'reshape' on a 'frozen' array, the 'frozen'
flag is cleared before 'reshape' is set, so the md thread can
get in and start trying recovery - which isn't wanted.

So instead of clearing MD_RECOVERY_FROZEN for any command
except 'frozen', only clear it when each specific command
is parsed.  This allows the handling of 'reshape' to clear
the bit while a lock is held.

Also remove some places where we set MD_RECOVERY_NEEDED,
as it is always set on non-error exit of the function.

Signed-off-by: NeilBrown <neilb@suse.de>
Fixes: 6791875e2e53 ("md: make reconfig_mutex optional for writes to md sysfs files.")
drivers/md/md.c