bcachefs: Don't skip triggers in fcollapse()

With backpointers this doesn't work anymore - backpointers always need
to be updated to point to the new extent position.

Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
This commit is contained in:
Kent Overstreet
2022-04-08 23:54:14 -04:00
committed by Kent Overstreet
parent 84c72755b9
commit 7c4ca54ae6

View File

@@ -2894,13 +2894,7 @@ static long bchfs_fcollapse_finsert(struct bch_inode_info *inode,
next_pos = insert ? bkey_start_pos(&delete.k) : delete.k.p;
if (copy.k->k.size == k.k->size) {
/*
* If we're moving the entire extent, we can skip
* running triggers:
*/
trigger_flags |= BTREE_TRIGGER_NORUN;
} else {
if (copy.k->k.size != k.k->size) {
/* We might end up splitting compressed extents: */
unsigned nr_ptrs =
bch2_bkey_nr_ptrs_allocated(bkey_i_to_s_c(copy.k));