Re: [PATCH] XFS: Kill filestreams cache flush

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

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.