On Fri, Jan 09, 2015 at 03:52:45AM -0800, Jaegeuk Kim wrote:
> This patch add the f2fs support for xfs/087 with goingdown.
> 

xfs/087 is not really a test that should be made generic/shared;
it's testing a bunch of different XFS specific mount options and log
recovery behaviour which has no relevance to any other filesystem.

If you want to take xfs specific tests that are only XFS specific
because of the shutdowns they use, look at these ones:

        xfs/053         (data exposure)
        xfs/137         (data vs filesize)
        xfs/138         (data vs filesize vs truncate)
        xfs/139         (data vs filesize vs partial truncate)
        xfs/140         (data vs filesize vs extending truncate)
        xfs/179         (data vs filesize w/ fsync)
        xfs/180         (data vs filesize w/ sync)
        xfs/182         (data vs filesize w/ recovery)
        xfs/200         (recovery vs ro-block device)
        xfs/306         (fsstress vs recovery)

These can all be made generic with very little effort -
"_require_shutdown" is probably all that is necessary for most of
them....

Also, we really are trying not to add more "shared" tests - with the
correct "_requires_foo" rules we can just place those tests in
generic/ and filesystems that don't support "foo" will avoid it.

Cheers,

Dave.
-- 
Dave Chinner
da...@fromorbit.com

------------------------------------------------------------------------------
New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
GigeNET is offering a free month of service with a new server in Ashburn.
Choose from 2 high performing configs, both with 100TB of bandwidth.
Higher redundancy.Lower latency.Increased capacity.Completely compliant.
http://p.sf.net/sfu/gigenet
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to