diff options
author | NeilBrown | 2012-02-14 11:10:10 +1100 |
---|---|---|
committer | NeilBrown | 2012-02-14 11:10:10 +1100 |
commit | fae8cc5ed0714953b1ad7cf86f030d2177278424 (patch) | |
tree | f61f54aaff4122671153e63cad86914566bce389 /lib/ts_fsm.c | |
parent | f53e29fc87b6fb76db0043202ec2c3450caa5ff3 (diff) |
md/raid10: fix handling of error on last working device in array.
If we get a read error on the last working device in a RAID10 which
contains the target block, then we don't fail the device (which is
good) but we don't abort retries, which is wrong.
We end up in an infinite loop retrying the read on the one device.
This patch fixes the problem in two places:
1/ in raid10_end_read_request we don't even ask for a retry if this
was the last usable device. This is efficient but a little racy
and will sometimes retry when it should not.
2/ in handle_read_error we are careful to exclude any device from
retry which we tried to mark as faulty (that might have failed if
it was the last device). This is race-free but less efficient.
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'lib/ts_fsm.c')
0 files changed, 0 insertions, 0 deletions