On Mon, May 21, 2018 at 12:27:19PM +0300, Nikolay Borisov wrote: > At the moment alloc_reserved_tree_block takes 8 friggin arguments. The irony > is that all of those are really derived from 3 structures. This series ends > up > reducing the arguments to 3. As a result some code, private to > alloc_reserved_tree_block > is moved from run_delayed_tree_ref. Patch 1 removes the fs_info argument, > Patch 2 replaces all arguments derived from btrfs_delayed_ref_node with the > nodes itself, Patch 3 replaces the last 2 arguments with > btrfs_delayed_extent_op > and finally, Patch 4 simplifies the code a bit by removing one extra check. > > All in all this doesn't provide any functional changes but makes the code > easier to follow. Also it has survived 2 xfstest runs. > > Nikolay Borisov (4): > btrfs: Remove fs_info argument from alloc_reserved_tree_block > btrfs: Simplify alloc_reserved_tree_block interface > btrfs: Pass btrfs_delayed_extent_op to alloc_reserved_tree_block > btrfs: Streamline shared ref check in alloc_reserved_tree_block
Passed fstests and added to misc-next, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html