diff options
author | NeilBrown <neilb@suse.de> | 2010-05-07 19:44:26 +1000 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2010-05-18 15:28:00 +1000 |
commit | 75a73a29e520a6ce982b0da6dd8b7560ae3faa90 (patch) | |
tree | 1bcb14b69e2ae7673bf29be6b3f27aee6e896070 /drivers/md/md.c | |
parent | af3a2cd6b8a479345786e7fe5e199ad2f6240e56 (diff) | |
download | op-kernel-dev-75a73a29e520a6ce982b0da6dd8b7560ae3faa90.zip op-kernel-dev-75a73a29e520a6ce982b0da6dd8b7560ae3faa90.tar.gz |
md: restore ability of spare drives to spin down.
Some time ago we stopped the clean/active metadata updates
from being written to a 'spare' device in most cases so that
it could spin down and say spun down. Device failure/removal
etc are still recorded on spares.
However commit 51d5668cb2e3fd1827a55 broke this 50% of the time,
depending on whether the event count is even or odd.
The change log entry said:
This means that the alignment between 'odd/even' and
'clean/dirty' might take a little longer to attain,
how ever the code makes no attempt to create that alignment, so it
could take arbitrarily long.
So when we find that clean/dirty is not aligned with odd/even,
force a second metadata-update immediately. There are already cases
where a second metadata-update is needed immediately (e.g. when a
device fails during the metadata update). We just piggy-back on that.
Reported-by: Joe Bryant <tenminjoe@yahoo.com>
Signed-off-by: NeilBrown <neilb@suse.de>
Cc: stable@kernel.org
Diffstat (limited to 'drivers/md/md.c')
-rw-r--r-- | drivers/md/md.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index b8a0fcf..fec4abc 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -2150,12 +2150,18 @@ repeat: if (!mddev->in_sync || mddev->recovery_cp != MaxSector) { /* not clean */ /* .. if the array isn't clean, an 'even' event must also go * to spares. */ - if ((mddev->events&1)==0) + if ((mddev->events&1)==0) { nospares = 0; + sync_req = 2; /* force a second update to get the + * even/odd in sync */ + } } else { /* otherwise an 'odd' event must go to spares */ - if ((mddev->events&1)) + if ((mddev->events&1)) { nospares = 0; + sync_req = 2; /* force a second update to get the + * even/odd in sync */ + } } } |