Re: [RFE] xfstests: Add support for tmpfs filesystem

[Dave Chinner <david@fromorbit.com>]

On Thu, Aug 23, 2012 at 10:42:56PM +0200, Boris Ranto wrote:
> This is resent with the improved subject line.
> This is just a simple patch to get the tmpfs working. I've made the full
> xfstests run with the tmpfs as the target filesystem and it works quite
> well. Some tests passed properly, some tests failed (and some of them due
> to the nature of tmpfs).
> 
> I didn't change the xfstests logic to recognize a proper SCRATCH_DEV so the
> SCRATCH_DEV for tmpfs should be in the nfs form (with the : in it) for this
> to work properly.

The patch format isn't quite right. it is normally:

 Subject: One-line description

 <description>

 Signed-off-by: ....

 ---
 <diff>

As it is, it would be nice if you could rebase this on top of my
series that reworks xfstests - all the option parsing is changed,
the common.rc file is now in common/rc, and so on.

>           fi
>           ;;
> +    tmpfs)
> +        if [ -z "$SCRATCH_DEV" -o ! -d "$SCRATCH_MNT" ];
> +        then
> +            _notrun "this test requires a valid \$SCRATCH_MNT and unique
> $SCRATCH_DEV"
> +        fi
> +        ;;

The patch also has line wrap issues.

> @@ -1552,6 +1604,10 @@ _cleanup_testdir()
>      # umount testdir as it is $SCRATCH_MNT which could be used by xfs next
>      [ -n "$testdir" ] && $UMOUNT_PROG $testdir
>      ;;
> +    tmpfs)
> +    # umount testdir since it is actually a scrath mount

scratch mount.

Cheers,

Dave.

-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs