From: Dave Chinner <dchinner@xxxxxxxxxx> Upstream changes mean that "-o noattr2" no longer removes the on-disk ATTR2 feature bit from the superblock and so the failure case this test exercises is no longer guaranteed to be a failure case. Hence the test is now useless as a regression test, so remove it. Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx> --- tests/xfs/187 | 117 ---------------------------------------------- tests/xfs/187.out | 39 ---------------- 2 files changed, 156 deletions(-) delete mode 100755 tests/xfs/187 delete mode 100644 tests/xfs/187.out diff --git a/tests/xfs/187 b/tests/xfs/187 deleted file mode 100755 index 3f8c7ef8..00000000 --- a/tests/xfs/187 +++ /dev/null @@ -1,117 +0,0 @@ -#! /bin/bash -# SPDX-License-Identifier: GPL-2.0 -# Copyright (c) 2008 Silicon Graphics, Inc. All Rights Reserved. -# -# FS QA Test No. 187 -# -# To test out the noattr2 flag which is broken in pv#980021 -# Given an existing attr2 filesystem, we should be able to mount -# as noattr2 and go back to an attr1 filesystem. -# -# Test the case where there are no more features2 bits on and -# so the morebitsbit should be off. -# -. ./common/preamble -_begin_fstest attr auto quick - -_filter_version() -{ - tee -a $seqres.full | tr ',' '\n' | egrep 'ATTR|MORE|LAZY' -} - -# Import common functions. -. ./common/filter -. ./common/attr - -# real QA test starts here -_supported_fs xfs - -_require_scratch -_require_attrs -_require_attr_v1 -_require_projid16bit - -# Reset the options so that we can control what is going on here -export MKFS_OPTIONS="" -export MOUNT_OPTIONS="" - -# lazysb, attr2 and other feature bits are held in features2 and will require -# morebitsbit on So test with lazysb and without it to see if the morebitsbit is -# okay etc. If the mkfs defaults change, these need to change as well. -MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0" -MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0" - -# ftype is also stored in features2, so we have to detect its presence in -# mkfs and disable it here too. -if _scratch_mkfs --help 2>&1 | grep -q "ftype="; then - MKFS_NO_LAZY="$MKFS_NO_LAZY -n ftype=0" - MKFS_LAZY="$MKFS_LAZY -n ftype=0" -fi - -# Make sure that when we think we are testing with morebits off -# that we really are. -_scratch_mkfs -i attr=1 $MKFS_NO_LAZY >/dev/null 2>&1 -_scratch_xfs_db -c version 2>&1 >$tmp.db -if grep -i morebits $tmp.db -then - echo "" - echo "Need to update test $seq so that initial subtests do not use features2" - echo "" - exit -fi - -echo "" -echo "*** 1. test attr2 mkfs and then noattr2 mount ***" -echo "" -echo "attr2 fs" -echo "" -_scratch_mkfs -i attr=2 $MKFS_NO_LAZY >/dev/null 2>&1 -_scratch_xfs_db -r -c version 2>&1 | _filter_version -echo "" -echo "noattr2 fs" -echo "" -_scratch_mount -o noattr2 -$UMOUNT_PROG $SCRATCH_MNT -_scratch_xfs_db -r -c version 2>&1 | _filter_version - -# adding an EA will ensure the ATTR1 flag is turned on -echo "" -echo "*** 2. test attr2 mkfs and then noattr2 mount with 1 EA ***" -echo "" -echo "attr2 fs" -echo "" -_scratch_mkfs -i attr=2 $MKFS_NO_LAZY >/dev/null 2>&1 -_scratch_xfs_db -r -c version 2>&1 | _filter_version -echo "" -echo "noattr2 fs" -echo "" -_scratch_mount -o noattr2 -cd $SCRATCH_MNT -touch testfile -$SETFATTR_PROG -n user.test -v 0xbabe testfile -_getfattr testfile -cd $here -$UMOUNT_PROG $SCRATCH_MNT -_scratch_xfs_db -r -c version 2>&1 | _filter_version - -echo "" -echo "*** 3. test noattr2 mount and lazy sb ***" -echo "" -echo "" -echo "attr2 fs" -echo "" -_scratch_mkfs -i attr=2 $MKFS_LAZY >/dev/null 2>&1 -_scratch_xfs_db -r -c version 2>&1 | _filter_version -echo "" -echo "noattr2 fs" -echo "" -_scratch_mount -o noattr2 -cd $SCRATCH_MNT -touch testfile -cd $here -$UMOUNT_PROG $SCRATCH_MNT -_scratch_xfs_db -r -c version 2>&1 | _filter_version - -# success, all done -status=0 -exit diff --git a/tests/xfs/187.out b/tests/xfs/187.out deleted file mode 100644 index e7e20212..00000000 --- a/tests/xfs/187.out +++ /dev/null @@ -1,39 +0,0 @@ -QA output created by 187 - -*** 1. test attr2 mkfs and then noattr2 mount *** - -attr2 fs - -MOREBITS -ATTR2 - -noattr2 fs - - -*** 2. test attr2 mkfs and then noattr2 mount with 1 EA *** - -attr2 fs - -MOREBITS -ATTR2 - -noattr2 fs - -# file: testfile -user.test - -ATTR - -*** 3. test noattr2 mount and lazy sb *** - - -attr2 fs - -MOREBITS -ATTR2 -LAZYSBCOUNT - -noattr2 fs - -MOREBITS -LAZYSBCOUNT -- 2.31.1