Re: [PATCH v4 1/8] xfs: add EOFBLOCKS inode tagging/untagging

Dave Chinner
2012-09-28T07:02:44Z

On Thu, Sep 27, 2012 at 01:45:45PM -0400, Brian Foster wrote:
> Add the XFS_ICI_EOFBLOCKS_TAG inode tag to identify inodes with
> speculatively preallocated blocks beyond EOF. An inode is tagged
> when speculative preallocation occurs and untagged either via
> truncate down or when post-EOF blocks are freed via release or
> reclaim.
>
> The tag management is intentionally not aggressive to prefer
> simplicity over the complexity of handling all the corner cases
> under which post-EOF blocks could be freed (i.e., forward
> truncation, fallocate, write error conditions, etc.). This means
> that a tagged inode may or may not have post-EOF blocks after a
> period of time. The tag is eventually cleared when the inode is
> released or reclaimed.
>
> Signed-off-by: Brian Foster <bfoster@redhat.com>

Apart from the fact this conflicts with my xfssyncd killing patchset
and xfs_sync.c no longer exists, it looks fine.

Can you rebase this on top of my patch series? Mostly it is simply
making your changes to xfs_icache.c rather than xfs_sync.c as that's
where all this inode cache radix tree walking code is now.....

Cheers,

Dave.
--
Dave Chinner
david@fromorbit.com