bcachefs: Delete an incorrect bch2_trans_unlock()
authorKent Overstreet <kent.overstreet@linux.dev>
Mon, 15 May 2023 03:01:14 +0000 (23:01 -0400)
committerKent Overstreet <kent.overstreet@linux.dev>
Sun, 22 Oct 2023 21:10:01 +0000 (17:10 -0400)
commita49bd8c007e4f4840f8c4d7fe7d62c7bdc7fffca
tree9f7ed9e4c9d7afff7ee44555484788cede6f638e
parentd598a9b7e27158d3b6972077e2f7296f279c2e8b
bcachefs: Delete an incorrect bch2_trans_unlock()

These deletes a bch2_trans_unlock() call from __bch2_move_data(). It was
redundant; bch2_move_extent() has the correct unlock call, and it was
buggy because when move_extent calls bch2_extent_drop_ptrs() we don't
want the transaction to be unlocked yet - this fixes a btree_iter.c
assertion.

Fixes https://github.com/koverstreet/bcachefs/issues/511.

Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
fs/bcachefs/move.c