bcachefs: Fix racy use of jiffies

Calculate the timeout, then check if it's positive before calling
schedule_timeout().

Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>

diff --git a/fs/bcachefs/journal_reclaim.c b/fs/bcachefs/journal_reclaim.c
index ace291f175dd6b0f45d6fcabd3a30391b7a47911..3d8fc2642425ec8461e703646ac417a4dd4377c6 100644 (file)
--- a/fs/bcachefs/journal_reclaim.c
+++ b/fs/bcachefs/journal_reclaim.c
@@ -758,10 +758,12 @@ static int bch2_journal_reclaim_thread(void *arg)
                        journal_empty = fifo_empty(&j->pin);
                        spin_unlock(&j->lock);
 
+                       long timeout = j->next_reclaim - jiffies;
+
                        if (journal_empty)
                                schedule();
-                       else if (time_after(j->next_reclaim, jiffies))
-                               schedule_timeout(j->next_reclaim - jiffies);
+                       else if (timeout > 0)
+                               schedule_timeout(timeout);
                        else
                                break;
                }