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