md: use current request time as base for ktime comparisons

Request coalescing logic uses 'prev_flush_start' as base to
compare the current request start time. 'prev_flush_start' is
updated in other context.

This patch changes this by using ktime comparison base to
'req_start' for better readability of code.

Signed-off-by: Pankaj Gupta <pankaj.gupta@cloud.ionos.com>
Signed-off-by: Song Liu <songliubraving@fb.com>

diff --git a/drivers/md/md.c b/drivers/md/md.c
index a0998ad6388cf0d736f1cc8e6bf1adbcc5efb7c9..1a3a6150123e4d2114878907e5bd012bcc4f6e42 100644 (file)
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -667,10 +667,10 @@ bool md_flush_request(struct mddev *mddev, struct bio *bio)
         */
        wait_event_lock_irq(mddev->sb_wait,
                            !mddev->flush_bio ||
-                           ktime_after(mddev->prev_flush_start, req_start),
+                           ktime_before(req_start, mddev->prev_flush_start),
                            mddev->lock);
        /* new request after previous flush is completed */
-       if (!ktime_after(mddev->prev_flush_start, req_start)) {
+       if (ktime_after(req_start, mddev->prev_flush_start)) {
                WARN_ON(mddev->flush_bio);
                mddev->flush_bio = bio;
                bio = NULL;