On Thu, Oct 24, 2019 at 09:33:18AM +0800, Qu Wenruo wrote: > > I've briefly checked 'void *private', but looks like it can be removed > > as well, it's holds bi_private of the first_bio (btrfs_map_bio) and it's > > read in btrfs_end_bbio, but bbio is accessible and so bbio->private is > > bbio->orig_bio->bi_private. > > Would double check that, anyway it will be another patch. No rush, that was only a drive-by idea. The btrfs_bio could be improved, there are eg. some raid56-only members or other types can be narrowed or reordered so this can be done at the same time. More work compared to removing a completely unused member.
