bcachefs: Fix trans->locked assert

Kent Overstreet · Kent Overstreet · commit 319fef29e965 · 2024-06-05T10:44:08.000-04:00
in bch2_move_data_btree, we might start with the trans unlocked from a
previous loop iteration - we need a trans_begin() before iter_init().

Signed-off-by: Kent Overstreet &lt;kent.overstreet@linux.dev&gt;
diff --git a/fs/bcachefs/move.c b/fs/bcachefs/move.c
@@ -547,6 +547,7 @@ static int bch2_move_data_btree(struct moving_context *ctxt,
 		ctxt->stats->pos	= BBPOS(btree_id, start);
 	}
 
+	bch2_trans_begin(trans);
 	bch2_trans_iter_init(trans, &iter, btree_id, start,
 			     BTREE_ITER_prefetch|
 			     BTREE_ITER_all_snapshots);

Original file line number	Diff line number	Diff line change
`@@ -547,6 +547,7 @@ static int bch2_move_data_btree(struct moving_context *ctxt,`
`547`	`547`	`ctxt->stats->pos = BBPOS(btree_id, start);`
`548`	`548`	`}`
`549`	`549`
	`550`	`+ bch2_trans_begin(trans);`
`550`	`551`	`bch2_trans_iter_init(trans, &iter, btree_id, start,`
`551`	`552`	`BTREE_ITER_prefetch\|`
`552`	`553`	`BTREE_ITER_all_snapshots);`