1 files changed, 23 insertions, 28 deletions

diff --git a/libbcachefs/backpointers.c b/libbcachefs/backpointers.c
index 7660a254..405823d1 100644
--- a/libbcachefs/backpointers.c
+++ b/libbcachefs/backpointers.c
@@ -242,6 +242,9 @@ btree:
 	    memcmp(bkey_s_c_to_backpointer(k).v, &bp, sizeof(bp))) {
 		struct printbuf buf = PRINTBUF;
 
+		if (!test_bit(BCH_FS_CHECK_BACKPOINTERS_DONE, &c->flags))
+			goto err;
+
 		prt_printf(&buf, "backpointer not found when deleting");
 		prt_newline(&buf);
 		printbuf_indent_add(&buf, 2);
@@ -261,12 +264,9 @@ btree:
 		prt_printf(&buf, "for ");
 		bch2_bkey_val_to_text(&buf, c, orig_k);
 
-		if (!test_bit(BCH_FS_CHECK_BACKPOINTERS_DONE, &c->flags)) {
-			bch_err(c, "%s", buf.buf);
-		} else {
-			ret = -EIO;
-			bch2_trans_inconsistent(trans, "%s", buf.buf);
-		}
+		bch_err(c, "%s", buf.buf);
+		bch2_inconsistent_error(c);
+		ret = -EIO;
 		printbuf_exit(&buf);
 		goto err;
 	}
@@ -283,7 +283,6 @@ int bch2_bucket_backpointer_add(struct btree_trans *trans,
 				struct bkey_s_c orig_k)
 {
 	struct bch_fs *c = trans->c;
-	struct bch_dev *ca;
 	struct bch_backpointer *bps = alloc_v4_backpointers(&a->v);
 	unsigned i, nr = BCH_ALLOC_V4_NR_BACKPOINTERS(&a->v);
 	struct bkey_i_backpointer *bp_k;
@@ -317,11 +316,10 @@ int bch2_bucket_backpointer_add(struct btree_trans *trans,
 		prt_printf(&buf, "for ");
 		bch2_bkey_val_to_text(&buf, c, orig_k);
 
-		if (!test_bit(BCH_FS_CHECK_BACKPOINTERS_DONE, &c->flags))
-			bch_err(c, "%s", buf.buf);
-		else {
-			bch2_trans_inconsistent(trans, "%s", buf.buf);
-			printbuf_exit(&buf);
+		bch_err(c, "%s", buf.buf);
+		printbuf_exit(&buf);
+		if (test_bit(BCH_FS_CHECK_BACKPOINTERS_DONE, &c->flags)) {
+			bch2_inconsistent_error(c);
 			return -EIO;
 		}
 	}
@@ -334,18 +332,9 @@ int bch2_bucket_backpointer_add(struct btree_trans *trans,
 	}
 
 	/* Overflow: use backpointer btree */
-	bp_k = bch2_trans_kmalloc(trans, sizeof(*bp_k));
-	ret = PTR_ERR_OR_ZERO(bp_k);
-	if (ret)
-		return ret;
-
-	ca = bch_dev_bkey_exists(c, a->k.p.inode);
 
-	bkey_backpointer_init(&bp_k->k_i);
-	bp_k->k.p = bucket_pos_to_bp(c, a->k.p, bp.bucket_offset);
-	bp_k->v = bp;
-
-	bch2_trans_iter_init(trans, &bp_iter, BTREE_ID_backpointers, bp_k->k.p,
+	bch2_trans_iter_init(trans, &bp_iter, BTREE_ID_backpointers,
+			     bucket_pos_to_bp(c, a->k.p, bp.bucket_offset),
 			     BTREE_ITER_INTENT|
 			     BTREE_ITER_SLOTS|
 			     BTREE_ITER_WITH_UPDATES);
@@ -369,16 +358,22 @@ int bch2_bucket_backpointer_add(struct btree_trans *trans,
 		prt_printf(&buf, "for ");
 		bch2_bkey_val_to_text(&buf, c, orig_k);
 
-		if (!test_bit(BCH_FS_CHECK_BACKPOINTERS_DONE, &c->flags))
-			bch_err(c, "%s", buf.buf);
-		else {
-			bch2_trans_inconsistent(trans, "%s", buf.buf);
-			printbuf_exit(&buf);
+		bch_err(c, "%s", buf.buf);
+		printbuf_exit(&buf);
+		if (test_bit(BCH_FS_CHECK_BACKPOINTERS_DONE, &c->flags)) {
+			bch2_inconsistent_error(c);
 			ret = -EIO;
 			goto err;
 		}
 	}
 
+	bp_k = bch2_bkey_alloc(trans, &bp_iter, backpointer);
+	ret = PTR_ERR_OR_ZERO(bp_k);
+	if (ret)
+		goto err;
+
+	bp_k->v = bp;
+
 	ret = bch2_trans_update(trans, &bp_iter, &bp_k->k_i, 0);
 err:
 	bch2_trans_iter_exit(trans, &bp_iter);