From: Yu Kuai <yuku...@huawei.com>

daemon_work() will be called by daemon thread, on the one hand, daemon
thread doesn't have strict wake-up time; on the other hand, too much
work are put to daemon thread, like handle sync IO, handle failed
or specail normal IO, handle recovery, and so on. Hence daemon thread
may be too busy to clear dirty bits in time.

Make bitmap_ops->daemon_work() optional and following patches will use
separate async work to clear dirty bits for the new bitmap.

Signed-off-by: Yu Kuai <yuku...@huawei.com>
Reviewed-by: Christoph Hellwig <h...@lst.de>
Reviewed-by: Hannes Reinecke <h...@suse.de>
---
 drivers/md/md.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/md.c b/drivers/md/md.c
index 13c17c45dd45..1ab6bba63890 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -9885,7 +9885,7 @@ static void unregister_sync_thread(struct mddev *mddev)
  */
 void md_check_recovery(struct mddev *mddev)
 {
-       if (md_bitmap_enabled(mddev, false))
+       if (md_bitmap_enabled(mddev, false) && mddev->bitmap_ops->daemon_work)
                mddev->bitmap_ops->daemon_work(mddev);
 
        if (signal_pending(current)) {
-- 
2.39.2


Reply via email to