md: test mddev->flags more safely in md_check_recovery.
authorNeilBrown <neilb@suse.de>
Wed, 27 Nov 2013 23:34:18 +0000 (10:34 +1100)
committerNeilBrown <neilb@suse.de>
Thu, 28 Nov 2013 00:00:08 +0000 (11:00 +1100)
commit142d44c310819e1965ca70b4d55d7679f5797e25
treef6c369bd20d2c3213a5dcbb738d9e13a97f6f951
parent0c775d5208284700de423e6746259da54a42e1f5
md: test mddev->flags more safely in md_check_recovery.

commit 7a0a5355cbc71efa md: Don't test all of mddev->flags at once.
made most tests on mddev->flags safer, but missed one.

When
commit 260fa034ef7a4ff8b7306 md: avoid deadlock when dirty buffers during md_stop.
added MD_STILL_CLOSED, this caused md_check_recovery to misbehave.
It can think there is something to do but find nothing.  This can
lead to the md thread spinning during array shutdown.

https://bugzilla.kernel.org/show_bug.cgi?id=65721

Reported-and-tested-by: Richard W.M. Jones <rjones@redhat.com>
Fixes: 260fa034ef7a4ff8b7306
Cc: stable@vger.kernel.org (3.12)
Signed-off-by: NeilBrown <neilb@suse.de>
drivers/md/md.c