Re: [PATCH 23/40] btrfs: store an inode pointer in struct btrfs_bio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Mar 23, 2022 at 08:54:21AM +0800, Qu Wenruo wrote:
> Something I want to avoid is to futher increasing the size of btrfs_bio.
>
> For buffered uncompressed IO, we can grab the inode from the first page.
> For direct IO we have bio->bi_private (btrfs_dio_private).
> For compressed IO, it's bio->bi_private again (compressed_bio).
>
> Do the saved code lines really validate the memory usage for all bios?

This isn't about the saved lines.  It allows to remove the async submit
and completion container structures that both point to an inode, and
later on the dio_private structure.  So overall it actually is a major
memory saving.



[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux