Re: [PATCH v2 1/3] md/raid10: fix still_degraded being inverted in raid10_sync_request()

From: Mykola Marzhan

Date: Sun Jul 26 2026 - 08:26:26 EST


> - still_degraded = false;
> + still_degraded = true;

I posted the same one-liner on 19 July -- flagging it only so it is not
applied twice:

https://lore.kernel.org/linux-raid/20260719144227.940444-2-mykola@xxxxxxxxxxx/

No objection to yours going in, and I will drop mine if it does:

Reviewed-by: Mykola Marzhan <mykola@xxxxxxxxxxx>

Identical hunk and the same Fixes: tag. Mine picked up Reviewed-by from
Yu Kuai and Paul Menzel on that posting.
Your changelog is the better of the two: it carries a measured
reproducer where mine only reasons through the failure.

Your patch 3 and patch 2 of my 19 July series hit the same unit mismatch
-- raid10's recovery cursor is in per-device space while its bitmap is
indexed by array sectors -- but at different sites, and they do not
overlap:

https://lore.kernel.org/linux-raid/20260719144227.940444-3-mykola@xxxxxxxxxxx/

Yours converts the bitmap-reported span into per-device sectors inside
raid10_sync_request(); mine stops md_do_sync() handing
bitmap_ops->skip_sync_blocks() an offset raid10 cannot interpret. Both
are still needed. Yu Kuai has asked me to re-check mine against his
llbitmap reshape series, so mine is still under discussion.

Mykola