[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