Hi, On Tue, Dec 11, 2012 at 05:44:10PM +0100, Koen De Wit wrote: > Tests included: > - Creating a reflink and overwriting the original contents > - Reflinking a directory tree > - Moving/deleting reflinks > - Diskspace consumption checks > - Cross-filesystem copies (should fail) > - Cross-subvolume copies David did a review and I agree with all of his comments. I have a few suggestions to extend test scope: * check if reflink expectedly fails when it's done on the same filesystem but mounted to 2 different directories (for example bind mount the TEST partition to SCRATCH dir and do the reflinks) * most of the files are very small and will get inlined, this covers only a small range of problems that could arise from reflinks. I'd like to see this included in the initial test version so the test covers majority of the cases and we can add specific corner cases later * larger files allow to test reflinks on files that comprise of interesting mix of holes and data (eg. what happens if I reflink half of hole and data extent at once?), reflink beginning or end of a file to another file david _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs