[PATCH v7] iomap: make inline data support more flexible
Matthew Wilcox
willy at infradead.org
Sat Jul 24 05:40:51 AEST 2021
On Sat, Jul 24, 2021 at 01:41:31AM +0800, Gao Xiang wrote:
> Add support for reading inline data content into the page cache from
> nonzero page-aligned file offsets. This enables the EROFS tailpacking
> mode where the last few bytes of the file are stored right after the
> inode.
>
> The buffered write path remains untouched since EROFS cannot be used
> for testing. It'd be better to be implemented if upcoming real users
> care and provide a real pattern rather than leave untested dead code
> around.
My one complaint with this version is the subject line. It's a bit vague.
I went with:
iomap: Support file tail packing
I also wrote a changelog entry that reads:
The existing inline data support only works for cases where the entire
file is stored as inline data. For larger files, EROFS stores the
initial blocks separately and then can pack a small tail adjacent to
the inode. Generalise inline data to allow for tail packing. Tails
may not cross a page boundary in memory.
... but I'm not sure that's necessarily better than what you've written
here.
> Cc: Christoph Hellwig <hch at lst.de>
> Cc: Darrick J. Wong <djwong at kernel.org>
> Cc: Matthew Wilcox <willy at infradead.org>
> Cc: Andreas Gruenbacher <andreas.gruenbacher at gmail.com>
> Tested-by: Huang Jianan <huangjianan at oppo.com> # erofs
> Signed-off-by: Gao Xiang <hsiangkao at linux.alibaba.com>
Reviewed-by: Matthew Wilcox (Oracle) <willy at infradead.org>
More information about the Linux-erofs
mailing list