Commit f66041f6 authored by ChangSyun Peng's avatar ChangSyun Peng Committed by Kelsey Skunberg

md/raid5: Fix Force reconstruct-write io stuck in degraded raid5

BugLink: https://bugs.launchpad.net/bugs/1892822

commit a1c6ae3d upstream.

In degraded raid5, we need to read parity to do reconstruct-write when
data disks fail. However, we can not read parity from
handle_stripe_dirtying() in force reconstruct-write mode.

Reproducible Steps:

1. Create degraded raid5
mdadm -C /dev/md2 --assume-clean -l5 -n3 /dev/sda2 /dev/sdb2 missing
2. Set rmw_level to 0
echo 0 > /sys/block/md2/md/rmw_level
3. IO to raid5

Now some io may be stuck in raid5. We can use handle_stripe_fill() to read
the parity in this situation.

Cc: <stable@vger.kernel.org> # v4.4+
Reviewed-by: default avatarAlex Wu <alexwu@synology.com>
Reviewed-by: default avatarBingJing Chang <bingjingc@synology.com>
Reviewed-by: default avatarDanny Shih <dannyshih@synology.com>
Signed-off-by: default avatarChangSyun Peng <allenpeng@synology.com>
Signed-off-by: default avatarSong Liu <songliubraving@fb.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarKhalid Elmously <khalid.elmously@canonical.com>
Signed-off-by: default avatarIan May <ian.may@canonical.com>
Signed-off-by: default avatarKelsey Skunberg <kelsey.skunberg@canonical.com>
parent 1b3d3f91
...@@ -3345,6 +3345,7 @@ static int need_this_block(struct stripe_head *sh, struct stripe_head_state *s, ...@@ -3345,6 +3345,7 @@ static int need_this_block(struct stripe_head *sh, struct stripe_head_state *s,
* is missing/faulty, then we need to read everything we can. * is missing/faulty, then we need to read everything we can.
*/ */
if (sh->raid_conf->level != 6 && if (sh->raid_conf->level != 6 &&
sh->raid_conf->rmw_level != PARITY_DISABLE_RMW &&
sh->sector < sh->raid_conf->mddev->recovery_cp) sh->sector < sh->raid_conf->mddev->recovery_cp)
/* reconstruct-write isn't being forced */ /* reconstruct-write isn't being forced */
return 0; return 0;
...@@ -4479,7 +4480,7 @@ static void handle_stripe(struct stripe_head *sh) ...@@ -4479,7 +4480,7 @@ static void handle_stripe(struct stripe_head *sh)
* or to load a block that is being partially written. * or to load a block that is being partially written.
*/ */
if (s.to_read || s.non_overwrite if (s.to_read || s.non_overwrite
|| (conf->level == 6 && s.to_write && s.failed) || (s.to_write && s.failed)
|| (s.syncing && (s.uptodate + s.compute < disks)) || (s.syncing && (s.uptodate + s.compute < disks))
|| s.replacing || s.replacing
|| s.expanding) || s.expanding)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment