From: Chandan Rajendra <chandan@linux.vnet.ibm.com>
To: amir73il@gmail.com, eguan@redhat.com
Cc: Chandan Rajendra <chandan@linux.vnet.ibm.com>,
fstests@vger.kernel.org, miklos@szeredi.hu,
linux-unionfs@vger.kernel.org
Subject: [PATCH V2] overlay: Test constant d_ino feature
Date: Thu, 31 Aug 2017 12:05:32 +0530 [thread overview]
Message-ID: <20170831063532.7505-1-chandan@linux.vnet.ibm.com> (raw)
This commit adds a test to verify constant d_ino feature. The following
scenarios are checked,
- Parent's (i.e. "..") d_ino must always be calculated because a pure
dir can be residing inside a merged dir.
- d_ino for "." must always be calculated because the present directory
can have a copy-up origin.
- Verify d_ino values corresponding to "." and ".." entries of a pure
lower dir.
- Verify d_ino of ".." entry of a merged dir.
- Verify pure lower residing in dir which has another lower layer pure
lower residing in dir which has another lower layer.
Signed-off-by: Chandan Rajendra <chandan@linux.vnet.ibm.com>
---
Changelog:
v1->v2:
1. Removed tests copied over from overlay/017.
2. Removed src/t_dir_ino.c.
3. Add code for testing,
- d_ino of . and .. entries of pure_lower_dir.
- d_ino of .. entry of merged_dir.
- Verify pure lower residing in dir which has another lower layer.
Thanks to Amir for suggesting these tests.
tests/overlay/037 | 144 ++++++++++++++++++++++++++++++++++++++++++++++++++
tests/overlay/037.out | 2 +
tests/overlay/group | 1 +
3 files changed, 147 insertions(+)
create mode 100755 tests/overlay/037
create mode 100644 tests/overlay/037.out
diff --git a/tests/overlay/037 b/tests/overlay/037
new file mode 100755
index 0000000..225d2b9
--- /dev/null
+++ b/tests/overlay/037
@@ -0,0 +1,144 @@
+#! /bin/bash
+# FSQA Test No. 037
+#
+# Test constant d_ino numbers
+#
+#-----------------------------------------------------------------------
+#
+# Copyright (C) 2017 IBM Corporation. All Rights Reserved.
+# Author: Chandan Rajendra <chandan@linux.vnet.ibm.com>
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it would be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write the Free Software Foundation,
+# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+#-----------------------------------------------------------------------
+#
+
+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
+
+# real QA test starts here
+_supported_fs overlay
+_supported_os Linux
+_require_scratch
+_require_test_program "af_unix"
+_require_test_program "t_dir_type"
+
+rm -f $seqres.full
+
+_scratch_mkfs >>$seqres.full 2>&1
+
+# Create our test files.
+lowerdir=$OVL_BASE_SCRATCH_MNT/$OVL_LOWER
+mkdir -p $lowerdir
+
+pure_lower_dir=$lowerdir/pure_lower_dir
+impure_dir=$lowerdir/impure_dir
+merged_dir=$lowerdir/merged_dir
+
+mkdir $pure_lower_dir
+mkdir $impure_dir
+mkdir $merged_dir
+
+impure_dir_st_ino=$(stat -c '%i' $impure_dir)
+pure_lower_dir_st_ino=$(stat -c '%i' $pure_lower_dir)
+
+_scratch_mount
+
+impure_dir=$SCRATCH_MNT/impure_dir
+pure_upper_dir=$impure_dir/pure_upper_dir
+
+mkdir -p $pure_upper_dir
+
+# Parent's (i.e. "..") d_ino must always be calculated because a pure
+# dir can be residing inside a merged dir.
+parent_d=$($here/src/t_dir_type $pure_upper_dir $impure_dir_st_ino)
+[[ $parent_d != ".. d" ]] && \
+ echo "Pure dir inside a merged dir: Invalid d_ino reported for .."
+
+# d_ino for "." must always be calculated because the present
+# directory can have a copy-up origin.
+current_d=$($here/src/t_dir_type $impure_dir $impure_dir_st_ino)
+[[ $current_d != ". d" ]] && echo "Impure dir: Invalid d_ino reported for ."
+
+# Verify d_ino values corresponding to "." and ".." entries of a
+# pure lower dir.
+ovl_root_st_ino=$(stat -c '%i' $SCRATCH_MNT/)
+pure_lower_dir=$SCRATCH_MNT/pure_lower_dir
+
+parent_d=$($here/src/t_dir_type $pure_lower_dir $ovl_root_st_ino)
+[[ $parent_d != ".. d" ]] && echo "Pure lower dir: Invalid d_ino reported for .."
+
+current_d=$($here/src/t_dir_type $pure_lower_dir $pure_lower_dir_st_ino)
+[[ $current_d != ". d" ]] && echo "Pure lower dir: Invalid d_ino reported for ."
+
+# Verify d_ino of ".." entry of a merged dir.
+merged_dir=$SCRATCH_MNT/merged_dir
+touch $merged_dir/file.bin
+
+parent_d=$($here/src/t_dir_type $merged_dir $ovl_root_st_ino)
+[[ $parent_d != ".. d" ]] && echo "Merged dir: Invalid d_ino reported for .."
+
+_scratch_unmount
+
+# Verify pure lower residing in dir which has another lower layer
+_scratch_mkfs
+
+OVL_LOWER0=${OVL_LOWER}0
+OVL_LOWER1=${OVL_LOWER}1
+
+mkdir -p $OVL_BASE_SCRATCH_MNT/$OVL_LOWER0
+mkdir -p $OVL_BASE_SCRATCH_MNT/$OVL_LOWER1
+mkdir -p $OVL_BASE_SCRATCH_MNT/$OVL_UPPER
+mkdir -p $OVL_BASE_SCRATCH_MNT/$OVL_WORK
+
+lowerdir0=$OVL_BASE_SCRATCH_MNT/$OVL_LOWER0
+lowerdir1=$OVL_BASE_SCRATCH_MNT/$OVL_LOWER1
+upperdir=$OVL_BASE_SCRATCH_MNT/$OVL_UPPER
+workdir=$OVL_BASE_SCRATCH_MNT/$OVL_WORK
+
+mkdir -p $lowerdir0/dir0
+mkdir -p $lowerdir1/dir0/dir1
+
+dir0_st_ino=$(stat -c '%i' $lowerdir0/dir0)
+
+$MOUNT_PROG -t overlay overlay -o lowerdir=$lowerdir0:$lowerdir1 \
+ -o upperdir=$upperdir -o workdir=$workdir $SCRATCH_MNT
+
+# Copy-up dir0
+touch $SCRATCH_MNT/dir0/file.bin
+
+parent_d=$($here/src/t_dir_type $SCRATCH_MNT/dir0/dir1 $dir0_st_ino)
+[[ $parent_d != ".. d" ]] && \
+ echo "Pure lower in dir which has another lower layer: Invalid d_ino reported for .."
+
+$UMOUNT_PROG $SCRATCH_MNT
+
+echo "Silence is golden"
+status=0
+exit
diff --git a/tests/overlay/037.out b/tests/overlay/037.out
new file mode 100644
index 0000000..5c3a30a
--- /dev/null
+++ b/tests/overlay/037.out
@@ -0,0 +1,2 @@
+QA output created by 037
+Silence is golden
diff --git a/tests/overlay/group b/tests/overlay/group
index 4cc1d74..b2460e1 100644
--- a/tests/overlay/group
+++ b/tests/overlay/group
@@ -39,3 +39,4 @@
034 auto quick copyup hardlink
035 auto quick mount
036 auto quick mount
+037 auto quick copyup
--
2.9.5
next reply other threads:[~2017-08-31 6:35 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-31 6:35 Chandan Rajendra [this message]
2017-08-31 7:09 ` [PATCH V2] overlay: Test constant d_ino feature Amir Goldstein
2017-08-31 15:12 ` Chandan Rajendra
2017-08-31 15:37 ` Amir Goldstein
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170831063532.7505-1-chandan@linux.vnet.ibm.com \
--to=chandan@linux.vnet.ibm.com \
--cc=amir73il@gmail.com \
--cc=eguan@redhat.com \
--cc=fstests@vger.kernel.org \
--cc=linux-unionfs@vger.kernel.org \
--cc=miklos@szeredi.hu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.