Re: [PATCH] xfs/186: fix test for crc=0, ftype=1

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Fri, Jun 17, 2016 at 12:54:43PM -0500, Eric Sandeen wrote:
> Test 186 won't run when crcs are enabled, because
> attrv1 is not allowed with crc=1.
> 
> However, ftype is still allowed with crc=0, so
> this creates v3 directories, and xfs_db prints
> them as such (along with the filetype), which
> breaks the test output.
> 
> We can filter & replace to fix up the test in
> this case.
> 
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>

This fixes the problems I hit, and it looks fine to me.

But xfs/186 still fails with MKFS_OPTIONS="-m crc=1 -b size=512", but
that's a different issue, I'll send another patch to fix it.

Thanks,
Eryu

> ---
> 
> diff --git a/tests/xfs/186 b/tests/xfs/186
> index 8e18975..cf66623 100755
> --- a/tests/xfs/186
> +++ b/tests/xfs/186
> @@ -76,12 +76,17 @@ _rmv_eas()
>  	done
>  }
>  
> +# If filetype is available (-n ftype=1) will get v3 dirs;
> +# just filter/replace to make this look teh same as if we
> +# had v2 dirs, as we are not interested in this info.
>  _filter_inode()
>  {
>  	tee -a $seqres.full | \
>  		sed -e "s/core.forkoff/forkoff/g" | \
> -		egrep '^u.sfdir2|^a.sfattr|forkoff' | \
> -		egrep -v 'inumber|parent'
> +		egrep '^u.sfdir2|^u.sfdir3|^a.sfattr|forkoff' | \
> +		egrep -v 'inumber|parent' | \
> +		sed -e s/sfdir3/sfdir2/g | \
> +		grep -v filetype
>  }
>  
>  _filter_version()
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux