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 _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
