From patchwork Thu Oct 4 04:35:00 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chao Yu X-Patchwork-Id: 10625585 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id A12CE1731 for ; Thu, 4 Oct 2018 04:35:20 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 813152903A for ; Thu, 4 Oct 2018 04:35:20 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6F5662904B; Thu, 4 Oct 2018 04:35:20 +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=-8.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 060912903A for ; Thu, 4 Oct 2018 04:35:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726813AbeJDL0k (ORCPT ); Thu, 4 Oct 2018 07:26:40 -0400 Received: from mail.kernel.org ([198.145.29.99]:50502 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726735AbeJDL0k (ORCPT ); Thu, 4 Oct 2018 07:26:40 -0400 Received: from localhost.localdomain (unknown [58.212.179.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id A794020877; Thu, 4 Oct 2018 04:35:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1538627717; bh=CGgvEPqkYwwrZvKsfrA2V5o14hN4C1V5N9P0qnR1uv8=; h=From:To:Cc:Subject:Date:From; b=if0W1v2WREFANTIMIj1dRTmwu96El6gLCXWUmOIMnrnuGfKGkRm63ydNbUSe8Ig4h a1HmXlo2tB4oSKCk/JbGn2XT3Sra73yHEgNsMFRxrf5awD7n2BuOS5ipOkdoRnqZqs o2xdOz8RiXGl+x0QTMlPNd7U3GUM5ed9fnEU2qWY= From: Chao Yu To: fstests@vger.kernel.org Cc: guaneryu@gmail.com, linux-f2fs-devel@lists.sourceforge.net, Chao Yu Subject: [PATCH 1/3] generic: test project id recovery after power failure Date: Thu, 4 Oct 2018 12:35:00 +0800 Message-Id: <20181004043502.15005-1-chao@kernel.org> X-Mailer: git-send-email 2.18.0 Sender: fstests-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Chao Yu After fsync, filesystem should guarantee inode metadata including project id being persisted, so even after sudden power-cut, during mount, we should recover project_id fields correctly, in order to not loss those meta info. So adding this testcase to check whether generic filesystem can guarantee that. Signed-off-by: Chao Yu --- tests/generic/506 | 97 +++++++++++++++++++++++++++++++++++++++++++ tests/generic/506.out | 2 + tests/generic/group | 1 + 3 files changed, 100 insertions(+) create mode 100644 tests/generic/506 create mode 100644 tests/generic/506.out diff --git a/tests/generic/506 b/tests/generic/506 new file mode 100644 index 00000000..ca035a91 --- /dev/null +++ b/tests/generic/506 @@ -0,0 +1,97 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# Copyright (c) 2018 Huawei. All Rights Reserved. +# +# FS QA Test 506 +# +# This testcase is trying to test recovery flow of generic filesystem, w/ below +# steps, once project id changes, after we fsync that file, we can expect that +# project id can be recovered after sudden power-cuts. +# 1. touch testfile; +# 1.1 sync (optional) +# 2. chattr -p 100 testfile; +# 3. xfs_io -f testfile -c "fsync"; +# 4. godown; +# 5. umount; +# 6. mount; +# 7. check project id +# +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 + +_cleanup() +{ + cd / + rm -f $tmp.* +} + +# get standard environment, filters and checks +. ./common/rc +. ./common/filter +. ./common/quota + +# remove previous $seqres.full before test +rm -f $seqres.full + +# real QA test starts here +_supported_fs generic +_supported_os Linux + +_require_test_lsattr +_require_command "$CHATTR_PROG" chattr + +_require_prjquota $SCRATCH_DEV +_require_scratch +_require_scratch_shutdown + +_scratch_mkfs >/dev/null 2>&1 +_require_metadata_journaling $SCRATCH_DEV + +testfile=$SCRATCH_MNT/testfile + +do_check() +{ + _scratch_mount + + touch $testfile + + if [ "$1" == "sync" ]; then + sync + fi + + chattr -p 100 $testfile + + before=`lsattr -p $testfile` + + $XFS_IO_PROG -f $testfile -c "fsync" | _filter_xfs_io + + _scratch_shutdown | tee -a $seqres.full + _scratch_cycle_mount + + after=`lsattr -p $testfile` + + # check inode's uid/gid + if [ "$before" != "$after" ]; then + echo "Before: $before" + echo "After : $after" + fi + echo "Before: $before" >> $seqres.full + echo "After : $after" >> $seqres.full + + rm $testfile + _scratch_unmount +} + +echo "Silence is golden" + +do_check +do_check sync + +status=0 +exit diff --git a/tests/generic/506.out b/tests/generic/506.out new file mode 100644 index 00000000..cb46b201 --- /dev/null +++ b/tests/generic/506.out @@ -0,0 +1,2 @@ +QA output created by 506 +Silence is golden diff --git a/tests/generic/group b/tests/generic/group index 4da0e188..c7d42ff9 100644 --- a/tests/generic/group +++ b/tests/generic/group @@ -508,3 +508,4 @@ 503 auto quick dax punch collapse zero 504 auto quick locks 505 shutdown auto quick metadata +506 shutdown auto quick metadata quota From patchwork Thu Oct 4 04:35:01 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chao Yu X-Patchwork-Id: 10625587 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 3ECD316B1 for ; Thu, 4 Oct 2018 04:35:22 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 2EAF72903A for ; Thu, 4 Oct 2018 04:35:22 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 229812904B; Thu, 4 Oct 2018 04:35:22 +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=-8.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A859C2903A for ; Thu, 4 Oct 2018 04:35:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726840AbeJDL0m (ORCPT ); Thu, 4 Oct 2018 07:26:42 -0400 Received: from mail.kernel.org ([198.145.29.99]:50520 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726735AbeJDL0m (ORCPT ); Thu, 4 Oct 2018 07:26:42 -0400 Received: from localhost.localdomain (unknown [58.212.179.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 591CB21473; Thu, 4 Oct 2018 04:35:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1538627720; bh=TG3ipQK7nQ58TNQtZDrRakA2MpuRSquWGfH4J5NjyF4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=PB0t/sOa7sNIEBqkB4A05NIa6p67ToPvXqFDFmrwhcamn4OO1s7fBXVafkJXH/Pz9 HBjyIW2D37WSe39oxAlfiyArVMrzA37gLEC9eJQ6BdQy396hB6iktno56Vtt2B0eqX hUJWeBlBZxANpJOaHNK3+LAfdI4XIEibV12Wm3Ug= From: Chao Yu To: fstests@vger.kernel.org Cc: guaneryu@gmail.com, linux-f2fs-devel@lists.sourceforge.net, Chao Yu Subject: [PATCH 2/3] generic: test i_flags recovery after power failure Date: Thu, 4 Oct 2018 12:35:01 +0800 Message-Id: <20181004043502.15005-2-chao@kernel.org> X-Mailer: git-send-email 2.18.0 In-Reply-To: <20181004043502.15005-1-chao@kernel.org> References: <20181004043502.15005-1-chao@kernel.org> Sender: fstests-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Chao Yu After fsync, filesystem should guarantee inode metadata including i_flags being persisted, so even after sudden power-cut, during mount, we should recover i_flags fields correctly, in order to not loss those meta info. So adding this testcase to check whether generic filesystem can guarantee that. We only check below attribute modification which most filesystem supports: - no atime updates (A) - secure deletion (s) - synchronous updates (S) - undeletable (u) Signed-off-by: Chao Yu --- tests/generic/507 | 132 ++++++++++++++++++++++++++++++++++++++++++ tests/generic/507.out | 2 + tests/generic/group | 1 + 3 files changed, 135 insertions(+) create mode 100755 tests/generic/507 create mode 100644 tests/generic/507.out diff --git a/tests/generic/507 b/tests/generic/507 new file mode 100755 index 00000000..2c718f8d --- /dev/null +++ b/tests/generic/507 @@ -0,0 +1,132 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# Copyright (c) 2018 Huawei. All Rights Reserved. +# +# FS QA Test 507 +# +# This testcase is trying to test recovery flow of generic filesystem, w/ below +# steps, once i_flags changes, after we fsync that file, we can expect that +# i_flags can be recovered after sudden power-cuts. +# 1. touch testfile; +# 1.1 sync (optional) +# 2. chattr +[AsSu] testfile +# 3. xfs_io -f testfile -c "fsync"; +# 4. godown; +# 5. umount; +# 6. mount; +# 7. check i_flags +# 8. chattr -[AsSu] testfile +# 9. xfs_io -f testfile -c "fsync"; +# 10. godown; +# 11. umount; +# 12. mount; +# 13. check i_flags +# +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 + +_cleanup() +{ + cd / + rm -f $tmp.* +} + +# get standard environment, filters and checks +. ./common/rc +. ./common/filter + +# remove previous $seqres.full before test +rm -f $seqres.full + +# real QA test starts here +_supported_fs generic +_supported_os Linux + +_require_test_lsattr +_require_command "$CHATTR_PROG" chattr + +_require_scratch +_require_scratch_shutdown + +_scratch_mkfs >/dev/null 2>&1 +_require_metadata_journaling $SCRATCH_DEV + +testfile=$SCRATCH_MNT/testfile +opt="AsSu" + +do_check() +{ + attr=$1 + + _scratch_mount + + touch $testfile + + if [ "$2" == "sync" ]; then + echo "sync" >> $seqres.full + sync + fi + + echo "Test chattr +$1" >> $seqres.full + + # add attribute + chattr +$attr $testfile + + before=`lsattr $testfile` + + $XFS_IO_PROG -f $testfile -c "fsync" | _filter_xfs_io + + _scratch_shutdown | tee -a $seqres.full + _scratch_cycle_mount + + after=`lsattr $testfile` + + # check inode's i_flags + if [ "$before" != "$after" ]; then + echo "Before: $before" + echo "After : $after" + fi + echo "Before: $before" >> $seqres.full + echo "After : $after" >> $seqres.full + + echo "Test chattr -$1" >> $seqres.full + + # delete attribute + chattr -$attr $testfile + + before=`lsattr $testfile` + + $XFS_IO_PROG -f $testfile -c "fsync" | _filter_xfs_io + + _scratch_shutdown | tee -a $seqres.full + _scratch_cycle_mount + + after=`lsattr $testfile` + + # check inode's i_flags + if [ "$before" != "$after" ]; then + echo "Before: $before" + echo "After : $after" + fi + echo "Before: $before" >> $seqres.full + echo "After : $after" >> $seqres.full + + rm $testfile + _scratch_unmount +} + +echo "Silence is golden" + +for i in `seq ${#opt}`; do + do_check ${opt:$i-1:1} + do_check ${opt:$i-1:1} sync +done + +status=0 +exit diff --git a/tests/generic/507.out b/tests/generic/507.out new file mode 100644 index 00000000..3d8672f8 --- /dev/null +++ b/tests/generic/507.out @@ -0,0 +1,2 @@ +QA output created by 507 +Silence is golden diff --git a/tests/generic/group b/tests/generic/group index c7d42ff9..e94253dc 100644 --- a/tests/generic/group +++ b/tests/generic/group @@ -509,3 +509,4 @@ 504 auto quick locks 505 shutdown auto quick metadata 506 shutdown auto quick metadata quota +507 shutdown auto quick metadata From patchwork Thu Oct 4 04:35:02 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chao Yu X-Patchwork-Id: 10625589 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id BDB6C1731 for ; Thu, 4 Oct 2018 04:35:26 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id AE8B52903A for ; Thu, 4 Oct 2018 04:35:26 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id A2ABA2904B; Thu, 4 Oct 2018 04:35:26 +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=-8.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 3C92B2903A for ; Thu, 4 Oct 2018 04:35:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726852AbeJDL0r (ORCPT ); Thu, 4 Oct 2018 07:26:47 -0400 Received: from mail.kernel.org ([198.145.29.99]:50536 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726735AbeJDL0r (ORCPT ); Thu, 4 Oct 2018 07:26:47 -0400 Received: from localhost.localdomain (unknown [58.212.179.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id D6ECD21477; Thu, 4 Oct 2018 04:35:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1538627724; bh=7DG3DZePoIk0xzp9b15tk9gh/slPtJWMgVAofz8CU0U=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=XrRosAJj61F0qEAV/e8Qgdgz738iVgF7v4lAOZuI2V6iK0t8r5ZOPY3zQwlnNdaAa uNwJGEEC5q7csR6ATGlX+ynQGqneyrzS0OqqQaq+Yu8lSxHFvz422E5+YEJhF8wLYv WcyjgvYuwEK1Kw+xkyQ27N7W/JBrhirUvJSncmwA= From: Chao Yu To: fstests@vger.kernel.org Cc: guaneryu@gmail.com, linux-f2fs-devel@lists.sourceforge.net, Chao Yu Subject: [PATCH 3/3] generic: test creation time recovery after power failure Date: Thu, 4 Oct 2018 12:35:02 +0800 Message-Id: <20181004043502.15005-3-chao@kernel.org> X-Mailer: git-send-email 2.18.0 In-Reply-To: <20181004043502.15005-1-chao@kernel.org> References: <20181004043502.15005-1-chao@kernel.org> Sender: fstests-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Chao Yu After fsync, filesystem should guarantee inode metadata including creation time being persisted, so even after sudden power-cut, during mount, we should recover i_crtime_{,nsec} fields correctly, in order to not loss those meta info. So adding this testcase to check whether generic filesystem can guarantee that. Note that, it needs inode creation time support on specified filesystem. Signed-off-by: Chao Yu --- tests/generic/508 | 96 +++++++++++++++++++++++++++++++++++++++++++ tests/generic/508.out | 2 + tests/generic/group | 1 + 3 files changed, 99 insertions(+) create mode 100755 tests/generic/508 create mode 100644 tests/generic/508.out diff --git a/tests/generic/508 b/tests/generic/508 new file mode 100755 index 00000000..61cd3397 --- /dev/null +++ b/tests/generic/508 @@ -0,0 +1,96 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# Copyright (c) 2018 Huawei. All Rights Reserved. +# +# FS QA Test 508 +# +# This testcase is trying to test recovery flow of generic filesystem, it needs +# creation time support on specified filesystem. +# With below steps, once the file is created, creation time attribute should be +# valid on the file, after we fsync that file, it expects creation time can be +# recovered after sudden power-cuts. +# 1. touch testfile; +# 1.1 sync (optional) +# 2. xfs_io -f testfile -c "fsync"; +# 3. godown; +# 4. umount; +# 5. mount; +# 6. check creation time +# +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 + +_cleanup() +{ + cd / + rm -f $tmp.* +} + +# get standard environment, filters and checks +. ./common/rc +. ./common/filter +. ./common/quota + +# remove previous $seqres.full before test +rm -f $seqres.full + +# real QA test starts here +_supported_fs generic +_supported_os Linux + +_require_test_lsattr +_require_xfs_io_command "statx" "-v" + +_require_scratch +_require_scratch_shutdown + +_scratch_mkfs >/dev/null 2>&1 +_require_metadata_journaling $SCRATCH_DEV + +testfile=$SCRATCH_MNT/testfile + +do_check() +{ + _scratch_mount + + touch $testfile + + if [ "$1" == "sync" ]; then + sync + fi + + before=`$XFS_IO_PROG -f $testfile -c "statx -v" | grep btime` || \ + _notrun "inode creation time not supported by this filesystem" + + $XFS_IO_PROG -f $testfile -c "fsync" | _filter_xfs_io + + _scratch_shutdown | tee -a $seqres.full + _scratch_cycle_mount + + after=`$XFS_IO_PROG -f $testfile -c "statx -v" | grep btime` + + # check inode's uid/gid + if [ "$before" != "$after" ]; then + echo "Before: $before" + echo "After : $after" + fi + echo "Before: $before" >> $seqres.full + echo "After : $after" >> $seqres.full + + rm $testfile + _scratch_unmount +} + +echo "Silence is golden" + +do_check +do_check sync + +status=0 +exit diff --git a/tests/generic/508.out b/tests/generic/508.out new file mode 100644 index 00000000..a5ceec05 --- /dev/null +++ b/tests/generic/508.out @@ -0,0 +1,2 @@ +QA output created by 508 +Silence is golden diff --git a/tests/generic/group b/tests/generic/group index e94253dc..8f26ee6a 100644 --- a/tests/generic/group +++ b/tests/generic/group @@ -510,3 +510,4 @@ 505 shutdown auto quick metadata 506 shutdown auto quick metadata quota 507 shutdown auto quick metadata +508 shutdown auto quick metadata