bcachefs: Don't rewind to run a recovery pass we already ran

Fix a small regression from the "run recovery passes" rewrite, which
enabled async recovery passes.

This fixes getting stuck in a loop in recovery.

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

diff --git a/fs/bcachefs/recovery_passes.c b/fs/bcachefs/recovery_passes.c
index dabb29b08ad0a83f0c2ddb5c359bc3c896bb012c..212658cb97dd2a9e9bf83a6379b4219e49a25540 100644 (file)
--- a/fs/bcachefs/recovery_passes.c
+++ b/fs/bcachefs/recovery_passes.c
@@ -315,7 +315,9 @@ int __bch2_run_explicit_recovery_pass(struct bch_fs *c,
                goto out;
 
        bool in_recovery = test_bit(BCH_FS_in_recovery, &c->flags);
-       bool rewind = in_recovery && r->curr_pass > pass;
+       bool rewind = in_recovery &&
+               r->curr_pass > pass &&
+               !(r->passes_complete & BIT_ULL(pass));
        bool ratelimit = flags & RUN_RECOVERY_PASS_ratelimit;
 
        if (!(in_recovery && (flags & RUN_RECOVERY_PASS_nopersistent))) {