Re: [PATCH 11/11 v3] btrfs: Use btrfs_transaction::pinned_extents

Nikolay Borisov
2020-02-06T19:40:39Z

On 6.02.20 г. 20:10 ч., David Sterba wrote:
> On Fri, Jan 24, 2020 at 05:18:30PM +0200, Nikolay Borisov wrote:
>> --- a/fs/btrfs/transaction.c
>> +++ b/fs/btrfs/transaction.c
>> @@ -334,6 +334,7 @@ static noinline int join_transaction(struct btrfs_fs_info *fs_info,
>> list_add_tail(&cur_trans->list, &fs_info->trans_list);
>> extent_io_tree_init(fs_info, &cur_trans->dirty_pages,
>> IO_TREE_TRANS_DIRTY_PAGES, fs_info->btree_inode);
>> + extent_io_tree_init(fs_info, &cur_trans->pinned_extents, 0, NULL);
>
> What's the reason there's no symbolic name for pinned_extents? Also 0
> matches IO_TREE_FS_INFO_EXCLUDED_EXTENTS because it's first in the enum
> list.
>

No reason, I'll change it in next version :)