From patchwork Tue Aug 30 07:35:32 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Zorro Lang X-Patchwork-Id: 9304955 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id E6B53601C0 for ; Tue, 30 Aug 2016 07:35:44 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D661428A7D for ; Tue, 30 Aug 2016 07:35:44 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id CAD7228AFE; Tue, 30 Aug 2016 07:35:44 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from oss.sgi.com (oss.sgi.com [192.48.182.195]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 38FA928A7D for ; Tue, 30 Aug 2016 07:35:43 +0000 (UTC) Received: from oss.sgi.com (localhost [IPv6:::1]) by oss.sgi.com (Postfix) with ESMTP id 318957CA1; Tue, 30 Aug 2016 02:35:42 -0500 (CDT) X-Original-To: xfs@oss.sgi.com Delivered-To: xfs@oss.sgi.com Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id 6D6807CA0 for ; Tue, 30 Aug 2016 02:35:40 -0500 (CDT) Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by relay1.corp.sgi.com (Postfix) with ESMTP id 31C0D8F8039 for ; Tue, 30 Aug 2016 00:35:37 -0700 (PDT) X-ASG-Debug-ID: 1472542535-0bf57b531238ef30001-NocioJ Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by cuda.sgi.com with ESMTP id YjW0hjBFvkY897T5 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for ; Tue, 30 Aug 2016 00:35:36 -0700 (PDT) X-Barracuda-Envelope-From: zlang@redhat.com X-Barracuda-Effective-Source-IP: mx1.redhat.com[209.132.183.28] X-Barracuda-Apparent-Source-IP: 209.132.183.28 X-ASG-Whitelist: Client Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 19DFB1555D; Tue, 30 Aug 2016 07:35:35 +0000 (UTC) Received: from localhost (dhcp-13-34.nay.redhat.com [10.66.13.34]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u7U7ZXCc011490; Tue, 30 Aug 2016 03:35:34 -0400 From: Zorro Lang To: fstests@vger.kernel.org Subject: [PATCH] xfs/187: fix new sb_features2 ftype stop case running Date: Tue, 30 Aug 2016 15:35:32 +0800 X-ASG-Orig-Subj: [PATCH] xfs/187: fix new sb_features2 ftype stop case running Message-Id: <1472542532-16497-1-git-send-email-zlang@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Tue, 30 Aug 2016 07:35:35 +0000 (UTC) X-Barracuda-Connect: mx1.redhat.com[209.132.183.28] X-Barracuda-Start-Time: 1472542535 X-Barracuda-Encrypted: ECDHE-RSA-AES256-GCM-SHA384 X-Barracuda-URL: https://192.48.176.25:443/cgi-mod/mark.cgi X-Barracuda-Scan-Msg-Size: 4302 X-Virus-Scanned: by bsmtpd at sgi.com X-Barracuda-BRTS-Status: 1 Cc: Zorro Lang , xfs@oss.sgi.com X-BeenThere: xfs@oss.sgi.com X-Mailman-Version: 2.1.14 Precedence: list List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com X-Virus-Scanned: ClamAV using ClamSMTP This case is too old, at that time there's no "ftype" feature for XFS. Due to this case need to clear features2 bits when mkfs.xfs, so ftype bit stop case running for long time. We care 6 features2 bits in this case: "ATTR2, LAZYSBCOUNT, PROJID32BIT, CRC, FTYPE, FINOBT" ATTR2 and LAZYSBCOUNT bits will be tested in case. FINBOT will be disabled if CRC=0. So this patch only check and disable PROJID32BIT, CRC and FTYPE features when mkfs.xfs. Signed-off-by: Zorro Lang --- Hi, I think we shouldn't skip this case if user doesn't specify suitable MKFS_OPTIONS and MOUNT_OPTIONS. Because this case need complex options, nearly no one will specify these all options for xfstests at same time. Thanks, Zorro tests/xfs/187 | 51 ++++++++++++++++++++++++++++----------------------- 1 file changed, 28 insertions(+), 23 deletions(-) diff --git a/tests/xfs/187 b/tests/xfs/187 index 836b924..5e7c677 100755 --- a/tests/xfs/187 +++ b/tests/xfs/187 @@ -31,7 +31,6 @@ seq=`basename $0` seqres=$RESULT_DIR/$seq echo "QA output created by $seq" -here=`pwd` tmp=/tmp/$$ status=1 # failure is the default! trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -58,24 +57,32 @@ _supported_os Linux _require_scratch _require_attrs -_require_attr_v1 -_require_projid16bit rm -f $seqres.full - +_scratch_mkfs >/dev/null 2>&1 # 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. -export MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0" -export MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0" +MKFS_OPTIONS="" +MOUNT_OPTIONS="" +ver2=`$XFS_DB_PROG -c version $SCRATCH_DEV | sed -n -e "s/,/ /g" \ + -e "s/.*MOREBITS\(.*\)/\1/p"` +# clear features2 bits which we won't test +for b in $ver2; do + case $b in + CRC) + MKFS_OPTIONS="$MKFS_OPTIONS -m crc=0" + ;; + PROJID32BIT) + MKFS_OPTIONS="$MKFS_OPTIONS -i projid32bit=0" + ;; + FTYPE) + MKFS_OPTIONS="$MKFS_OPTIONS -n ftype=0" + ;; + esac +done # 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_mkfs -i attr=1 -l lazy-count=0 >/dev/null 2>&1 $XFS_DB_PROG -c version $SCRATCH_DEV 2>&1 >$tmp.db if grep -i morebits $tmp.db then @@ -90,13 +97,13 @@ 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_mkfs -i attr=2 -l lazy-count=0 >/dev/null 2>&1 $XFS_DB_PROG -r -c version $SCRATCH_DEV 2>&1 | _filter_version echo "" echo "noattr2 fs" echo "" _scratch_mount -o noattr2 -$UMOUNT_PROG $SCRATCH_MNT +_scratch_unmount $XFS_DB_PROG -r -c version $SCRATCH_DEV 2>&1 | _filter_version # adding an EA will ensure the ATTR1 flag is turned on @@ -105,7 +112,7 @@ 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_mkfs -i attr=2 -l lazy-count=0 >/dev/null 2>&1 $XFS_DB_PROG -r -c version $SCRATCH_DEV 2>&1 | _filter_version echo "" echo "noattr2 fs" @@ -115,8 +122,8 @@ cd $SCRATCH_MNT touch testfile $SETFATTR_PROG -n user.test -v 0xbabe testfile $GETFATTR_PROG testfile -cd $here -$UMOUNT_PROG $SCRATCH_MNT +cd - >/dev/null +_scratch_unmount $XFS_DB_PROG -r -c version $SCRATCH_DEV 2>&1 | _filter_version echo "" @@ -125,16 +132,14 @@ echo "" echo "" echo "attr2 fs" echo "" -_scratch_mkfs -i attr=2 $MKFS_LAZY >/dev/null 2>&1 +_scratch_mkfs -i attr=2 -l lazy-count=1 >/dev/null 2>&1 $XFS_DB_PROG -r -c version $SCRATCH_DEV 2>&1 | _filter_version echo "" echo "noattr2 fs" echo "" _scratch_mount -o noattr2 -cd $SCRATCH_MNT -touch testfile -cd $here -$UMOUNT_PROG $SCRATCH_MNT +touch $SCRATCH_MNT/testfile +_scratch_unmount $XFS_DB_PROG -r -c version $SCRATCH_DEV 2>&1 | _filter_version # success, all done