Commit d7f4b4ef authored Oct 09, 2024 by David Sterba

btrfs: drop unused transaction parameter from btrfs_qgroup_add_swapped_blocks()



The caller replace_path() runs under transaction but we don't need it in
btrfs_qgroup_add_swapped_blocks().

Reviewed-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>

parent 2651f432

fs/btrfs/qgroup.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -4692,8 +4692,7 @@ void btrfs_qgroup_clean_swapped_blocks(struct btrfs_root *root)
		* BOTH POINTERS ARE BEFORE TREE SWAP
		* @last_snapshot: last snapshot generation of the subvolume tree
		*/
		int btrfs_qgroup_add_swapped_blocks(struct btrfs_trans_handle *trans,
		struct btrfs_root *subvol_root,
		int btrfs_qgroup_add_swapped_blocks(struct btrfs_root *subvol_root,
		struct btrfs_block_group *bg,
		struct extent_buffer *subvol_parent, int subvol_slot,
		struct extent_buffer *reloc_parent, int reloc_slot,

fs/btrfs/qgroup.h

+1 −2

Original line number	Diff line number	Diff line
		@@ -439,8 +439,7 @@ void btrfs_qgroup_init_swapped_blocks(
		struct btrfs_qgroup_swapped_blocks *swapped_blocks);

		void btrfs_qgroup_clean_swapped_blocks(struct btrfs_root *root);
		int btrfs_qgroup_add_swapped_blocks(struct btrfs_trans_handle *trans,
		struct btrfs_root *subvol_root,
		int btrfs_qgroup_add_swapped_blocks(struct btrfs_root *subvol_root,
		struct btrfs_block_group *bg,
		struct extent_buffer *subvol_parent, int subvol_slot,
		struct extent_buffer *reloc_parent, int reloc_slot,

fs/btrfs/relocation.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1244,7 +1244,7 @@ int replace_path(struct btrfs_trans_handle trans, struct reloc_control rc,
		* The real subtree rescan is delayed until we have new
		* CoW on the subtree root node before transaction commit.
		*/
		ret = btrfs_qgroup_add_swapped_blocks(trans, dest,
		ret = btrfs_qgroup_add_swapped_blocks(dest,
		rc->block_group, parent, slot,
		path->nodes[level], path->slots[level],
		last_snapshot);