Cleancache support in XFS

Ben Myers bpm at sgi.com
Wed May 1 11:20:44 CDT 2013


Hi James, 

On Wed, May 01, 2013 at 01:39:09PM +0100, James Dingwall wrote:
> In reference to: http://oss.sgi.com/archives/xfs/2012-05/msg00046.html
> 
> $ grep -r cleancache fs/xfs
> on the 3.9 kernel source suggests that no patch was submitted to
> enable cleancache for the XFS filesystem.  Since it was suggested
> that this could be a one liner I've had a go and my first effort is
> inline below.  While this seems to compile OK I have no experience
> in filesystems so I would appreciate it if anyone can point out that
> it is obviously wrong and likely to eat my data before I try booting
> the kernel.
> 
> If it seems a reasonable attempt what would be the best way to check
> that it isn't doing nasty things?

Hrm.. Looks like there is a doc in Documentation/vm/cleancache.txt which
includes a list of attributes the filesystem needs to have to work properly
with cleancache.  I'd suggest going over that list and the code by inspection
first, followed by some xfstests runs, and then wait for it to make a release
before trusting any data to it.  For a test machine that I'm willing to crater,
no worries.  But I wouldn't run this on my main workstation without doing the
above first.  We can probably explore this for 3.11.  Thanks for the patch.

Regards,
	Ben



More information about the xfs mailing list