[PATCH -next] md: wake up mdmon after setting badblocks

From: Yu Kuai
Date: Thu Aug 01 2024 - 08:55:28 EST


From: Yu Kuai <yukuai3@xxxxxxxxxx>

For external super_block, mdmon rely on "mddev->sysfs_state" to update
super_block. However, rdev_set_badblocks() will set sb_flags without
waking up mdmon, which might cauing IO hang due to suepr_block can't be
updated.

This problem is found by code review.

Signed-off-by: Yu Kuai <yukuai3@xxxxxxxxxx>
---
drivers/md/md.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/md/md.c b/drivers/md/md.c
index 23cc77d51676..06d6ee8cd543 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -9831,10 +9831,12 @@ int rdev_set_badblocks(struct md_rdev *rdev, sector_t s, int sectors,
/* Make sure they get written out promptly */
if (test_bit(ExternalBbl, &rdev->flags))
sysfs_notify_dirent_safe(rdev->sysfs_unack_badblocks);
- sysfs_notify_dirent_safe(rdev->sysfs_state);
set_mask_bits(&mddev->sb_flags, 0,
BIT(MD_SB_CHANGE_CLEAN) | BIT(MD_SB_CHANGE_PENDING));
md_wakeup_thread(rdev->mddev->thread);
+
+ sysfs_notify_dirent_safe(rdev->sysfs_state);
+ sysfs_notify_dirent_safe(mddev->sysfs_state);
return 1;
} else
return 0;
--
2.39.2