bcachefs: bch2_bucket_is_movable() -> BTREE_ITER_CACHED

BTREE_ITER_CACHED should really be the default for cached btrees - this
is an easy mistake to make.

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

diff --git a/fs/bcachefs/movinggc.c b/fs/bcachefs/movinggc.c
index 6755310f5ebc8d10234bdb0ef981c8d6256a522d..4d8d013fe4fc1f10bd55941969ef5bcc2e2ced37 100644 (file)
--- a/fs/bcachefs/movinggc.c
+++ b/fs/bcachefs/movinggc.c
@@ -46,7 +46,7 @@ static int bch2_bucket_is_movable(struct btree_trans *trans,
        if (bch2_bucket_is_open(trans->c, bucket.inode, bucket.offset))
                return 0;
 
-       bch2_trans_iter_init(trans, &iter, BTREE_ID_alloc, bucket, 0);
+       bch2_trans_iter_init(trans, &iter, BTREE_ID_alloc, bucket, BTREE_ITER_CACHED);
        k = bch2_btree_iter_peek_slot(&iter);
        ret = bkey_err(k);
        bch2_trans_iter_exit(trans, &iter);