diff options
author | Artur Paszkiewicz <artur.paszkiewicz@intel.com> | 2017-09-29 16:54:19 -0400 |
---|---|---|
committer | Shaohua Li <shli@fb.com> | 2017-10-16 22:06:34 -0400 |
commit | 07719ff767dcd8cc42050f185d332052f3816546 (patch) | |
tree | a52816ee10590641fa161d379f28cfaa605bdbf6 /drivers/md/raid5-ppl.c | |
parent | 611426e2737235cf05e1b8f27d2502b96a5e05d9 (diff) |
raid5-ppl: check recovery_offset when performing ppl recovery
If starting an array that is undergoing rebuild, make ppl recovery honor
the recovery_offset of a member disk and don't read data that is not yet
in-sync.
Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Signed-off-by: Shaohua Li <shli@fb.com>
Diffstat (limited to 'drivers/md/raid5-ppl.c')
-rw-r--r-- | drivers/md/raid5-ppl.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/md/raid5-ppl.c b/drivers/md/raid5-ppl.c index 76d6245427b8..628c0bf7b9fd 100644 --- a/drivers/md/raid5-ppl.c +++ b/drivers/md/raid5-ppl.c | |||
@@ -758,7 +758,8 @@ static int ppl_recover_entry(struct ppl_log *log, struct ppl_header_entry *e, | |||
758 | (unsigned long long)sector); | 758 | (unsigned long long)sector); |
759 | 759 | ||
760 | rdev = conf->disks[dd_idx].rdev; | 760 | rdev = conf->disks[dd_idx].rdev; |
761 | if (!rdev) { | 761 | if (!rdev || (!test_bit(In_sync, &rdev->flags) && |
762 | sector >= rdev->recovery_offset)) { | ||
762 | pr_debug("%s:%*s data member disk %d missing\n", | 763 | pr_debug("%s:%*s data member disk %d missing\n", |
763 | __func__, indent, "", dd_idx); | 764 | __func__, indent, "", dd_idx); |
764 | update_parity = false; | 765 | update_parity = false; |