diff options
Diffstat (limited to 'drivers/md/raid5.c')
| -rw-r--r-- | drivers/md/raid5.c | 25 |
1 files changed, 17 insertions, 8 deletions
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c index 08a1620b9f8c..77610b98d4e0 100644 --- a/drivers/md/raid5.c +++ b/drivers/md/raid5.c | |||
| @@ -420,21 +420,29 @@ static int raid5_end_read_request(struct bio * bi, unsigned int bytes_done, | |||
| 420 | clear_bit(R5_ReadError, &sh->dev[i].flags); | 420 | clear_bit(R5_ReadError, &sh->dev[i].flags); |
| 421 | clear_bit(R5_ReWrite, &sh->dev[i].flags); | 421 | clear_bit(R5_ReWrite, &sh->dev[i].flags); |
| 422 | } | 422 | } |
| 423 | if (atomic_read(&conf->disks[i].rdev->read_errors)) | ||
| 424 | atomic_set(&conf->disks[i].rdev->read_errors, 0); | ||
| 423 | } else { | 425 | } else { |
| 426 | int retry = 0; | ||
| 424 | clear_bit(R5_UPTODATE, &sh->dev[i].flags); | 427 | clear_bit(R5_UPTODATE, &sh->dev[i].flags); |
| 425 | if (conf->mddev->degraded) { | 428 | atomic_inc(&conf->disks[i].rdev->read_errors); |
| 429 | if (conf->mddev->degraded) | ||
| 426 | printk("R5: read error not correctable.\n"); | 430 | printk("R5: read error not correctable.\n"); |
| 427 | clear_bit(R5_ReadError, &sh->dev[i].flags); | 431 | else if (test_bit(R5_ReWrite, &sh->dev[i].flags)) |
| 428 | clear_bit(R5_ReWrite, &sh->dev[i].flags); | ||
| 429 | md_error(conf->mddev, conf->disks[i].rdev); | ||
| 430 | } else if (test_bit(R5_ReWrite, &sh->dev[i].flags)) { | ||
| 431 | /* Oh, no!!! */ | 432 | /* Oh, no!!! */ |
| 432 | printk("R5: read error NOT corrected!!\n"); | 433 | printk("R5: read error NOT corrected!!\n"); |
| 434 | else if (atomic_read(&conf->disks[i].rdev->read_errors) | ||
| 435 | > conf->max_nr_stripes) | ||
| 436 | printk("raid5: Too many read errors, failing device.\n"); | ||
| 437 | else | ||
| 438 | retry = 1; | ||
| 439 | if (retry) | ||
| 440 | set_bit(R5_ReadError, &sh->dev[i].flags); | ||
| 441 | else { | ||
| 433 | clear_bit(R5_ReadError, &sh->dev[i].flags); | 442 | clear_bit(R5_ReadError, &sh->dev[i].flags); |
| 434 | clear_bit(R5_ReWrite, &sh->dev[i].flags); | 443 | clear_bit(R5_ReWrite, &sh->dev[i].flags); |
| 435 | md_error(conf->mddev, conf->disks[i].rdev); | 444 | md_error(conf->mddev, conf->disks[i].rdev); |
| 436 | } else | 445 | } |
| 437 | set_bit(R5_ReadError, &sh->dev[i].flags); | ||
| 438 | } | 446 | } |
| 439 | rdev_dec_pending(conf->disks[i].rdev, conf->mddev); | 447 | rdev_dec_pending(conf->disks[i].rdev, conf->mddev); |
| 440 | #if 0 | 448 | #if 0 |
| @@ -1328,7 +1336,8 @@ static void handle_stripe(struct stripe_head *sh) | |||
| 1328 | /* If the failed drive is just a ReadError, then we might need to progress | 1336 | /* If the failed drive is just a ReadError, then we might need to progress |
| 1329 | * the repair/check process | 1337 | * the repair/check process |
| 1330 | */ | 1338 | */ |
| 1331 | if (failed == 1 && test_bit(R5_ReadError, &sh->dev[failed_num].flags) | 1339 | if (failed == 1 && ! conf->mddev->ro && |
| 1340 | test_bit(R5_ReadError, &sh->dev[failed_num].flags) | ||
| 1332 | && !test_bit(R5_LOCKED, &sh->dev[failed_num].flags) | 1341 | && !test_bit(R5_LOCKED, &sh->dev[failed_num].flags) |
| 1333 | && test_bit(R5_UPTODATE, &sh->dev[failed_num].flags) | 1342 | && test_bit(R5_UPTODATE, &sh->dev[failed_num].flags) |
| 1334 | ) { | 1343 | ) { |
