bcachefs: Don't unlock the trans if ret doesn't match BCH_ERR_operation_blocked

Reported-by: syzbot+d540192e763531d307ff@syzkaller.appspotmail.com
Signed-off-by: Alan Huang <mmpgouride@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>

diff --git a/fs/bcachefs/btree_update_interior.c b/fs/bcachefs/btree_update_interior.c
index 7bf1bd6a6e928677370c5a8d4930b3f2151715e7..553059b33bfd62c5c306bf4a0335d41c79e86e01 100644 (file)
--- a/fs/bcachefs/btree_update_interior.c
+++ b/fs/bcachefs/btree_update_interior.c
@@ -1287,10 +1287,11 @@ bch2_btree_update_start(struct btree_trans *trans, struct btree_path *path,
 
                do {
                        ret = bch2_btree_reserve_get(trans, as, nr_nodes, target, flags, &cl);
-
+                       if (!bch2_err_matches(ret, BCH_ERR_operation_blocked))
+                               break;
                        bch2_trans_unlock(trans);
                        bch2_wait_on_allocator(c, &cl);
-               } while (bch2_err_matches(ret, BCH_ERR_operation_blocked));
+               } while (1);
        }
 
        if (ret) {