On Mon, Jan 07, 2019 at 01:41:47PM +0800, Qu Wenruo wrote:
> +/*
> + * Helper structure to keep record of a file tree whose reloc
> + * root needs to be cleaned up.
> + *
> + * Since reloc_control is used less frequently than btrfs_root, this should
> + * prevent us to add another structure in btrfs_root.
> + */
> +struct dirty_source_root {
> + struct rb_node node;
> +
> + /* Root must be file tree */
The naming starts here and is used in the following patches too: the
naming is IMO confusing, here it's still ok but variables like
'file_bytenr' or 'file_parent' sound weird in context of relocation.
The file tree is the subvolume right, or would it be confusing in
another way?