On Tue, Jun 23, 2020 at 6:17 AM Michal Soltys [off-list ref] wrote:

Hmm.. this is weird, as I think I marked every instance of set_bit
MD_SB_CHANGE_PENDING.
Would you mind confirm those are to the other array with something like:

diff --git i/drivers/md/md.c w/drivers/md/md.c
index dbbc8a50e2ed2..e91acfdcec032 100644
--- i/drivers/md/md.c
+++ w/drivers/md/md.c

@@ -8480,7 +8480,7 @@ bool md_write_start(struct mddev *mddev, struct bio *bi)
                         mddev->in_sync = 0;
                         set_bit(MD_SB_CHANGE_CLEAN, &mddev->sb_flags);
                         set_bit(MD_SB_CHANGE_PENDING, &mddev->sb_flags);
-                       pr_info("%s set MD_SB_CHANGE_PENDING\n", __func__);
+                       pr_info("%s: md: %s set

MD_SB_CHANGE_PENDING\n", __func__, mdname(mddev));
                         md_wakeup_thread(mddev->thread);
                         did_change = 1;
                 }

Thanks,
Song