[PATCH] XFS: Kill filestreams cache flush
Christoph Hellwig
hch at infradead.org
Sat Jan 2 05:53:38 CST 2010
On Sat, Jan 02, 2010 at 01:38:20PM +1100, Dave Chinner wrote:
> The filestreams cache flush is not needed in the sync code as it
> does not affect data writeback, and it is now not used by the
> growfs code, either, so kill it.
I have to admit I never understood the purpose of this call.
What xfs_fstrm_free_func does is:
- clear the filesystems flag on the inode
- decrement pagf_fstrms
- drop references on the inode (and parent if known)
- free the filestream item
Nothing of that should matter for sync, although dropping inode
references during sync might generally be a good idea.
More information about the xfs
mailing list