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.
