* [PATCH v3 1/2] generic/608: Remove drop cache and umount & mount cycle
@ 2021-03-30 2:06 Xiao Yang
2021-03-30 2:06 ` [PATCH v3 2/2] generic/608: Add new regression test Xiao Yang
0 siblings, 1 reply; 3+ messages in thread
From: Xiao Yang @ 2021-03-30 2:06 UTC (permalink / raw)
To: fstests; +Cc: darrick.wong, lihao2018.fnst, david, xifeng, ira.weiny, Xiao Yang
Since commit 77573fa310d9, toggling FS_XFLAG_DAX on an existing file can make
S_DAX on the file change immediately when all applications close the file.
As a result, we don't need the redundant drop cache and umount & mount cycle.
Signed-off-by: Xiaoli Feng <xifeng@redhat.com>
Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
---
tests/generic/608 | 41 ++++++++++++++++++-----------------------
1 file changed, 18 insertions(+), 23 deletions(-)
diff --git a/tests/generic/608 b/tests/generic/608
index 861bbff5..5ca2442f 100755
--- a/tests/generic/608
+++ b/tests/generic/608
@@ -3,10 +3,10 @@
# Copyright (c) 2020 Fujitsu. All Rights Reserved.
#
# FS QA Test 608
-# Change FS_XFLAG_DAX on an existing file and check if S_DAX on
-# the file can take effect immediately by the following steps:
-# 1) Stop all applications which are using the file.
-# 2) Do drop_caches or umount & mount cycle.
+# Toggling FS_XFLAG_DAX on an existing file can make S_DAX on the
+# file change immediately when all applications close the file.
+# It's a regression test for:
+# 'commit 77573fa310d9 ("fs: Kill DCACHE_DONTCACHE dentry even if DCACHE_REFERENCED is set")'
seq=`basename $0`
seqres=$RESULT_DIR/$seq
@@ -36,7 +36,7 @@ _require_dax_iflag
_require_xfs_io_command "lsattr" "-v"
_require_xfs_io_command "statx" "-r"
-test_drop_caches()
+test_enable_dax()
{
local t_file=$SCRATCH_MNT/testfile
@@ -49,24 +49,19 @@ test_drop_caches()
$XFS_IO_PROG -c 'chattr +x' $t_file
_check_xflag $t_file 1
- _check_s_dax $t_file 0
-
- # One application is using test file and check if S_DAX on
- # the file doesn't take effect immediately by drop_caches
- echo 2 > /proc/sys/vm/drop_caches
+ # One application is using test file and S_DAX
+ # on the file is not changed immediately
_check_s_dax $t_file 0
exec 3<&-
- # No application is using test file and check if S_DAX on
- # the file takes effect immediately by drop_caches
- echo 2 > /proc/sys/vm/drop_caches
+ # No application is using test file and S_DAX
+ # on the file is changed immediately
_check_s_dax $t_file 1
}
-test_cycle_mount()
+test_disable_dax()
{
- local option=$1
local t_dir=$SCRATCH_MNT/testdir
local t_file=$t_dir/testfile
@@ -81,29 +76,29 @@ test_cycle_mount()
$XFS_IO_PROG -c 'chattr -x' $t_file
_check_xflag $t_file 0
+ # One application is using test file and S_DAX
+ # on the file is not changed immediately
_check_s_dax $t_file 1
exec 3<&-
- # No application is using test file and check if S_DAX on
- # the file takes effect immediately by umount & mount
- _scratch_cycle_mount "$option"
+ # No application is using test file and S_DAX
+ # on the file is changed immediately
_check_s_dax $t_file 0
}
do_tests()
{
local mount_option=$1
- local cycle_mount_option=$2
_scratch_mount "$mount_option"
# Make sure the root dir doesn't have FS_XFLAG_DAX set before we start.
$XFS_IO_PROG -c "chattr -x" $SCRATCH_MNT &>> $seqres.full
- test_drop_caches
-
- test_cycle_mount "$cycle_mount_option"
+ # Do test for commit 77573fa310d9
+ test_enable_dax
+ test_disable_dax
_scratch_unmount
}
@@ -111,7 +106,7 @@ do_tests()
_scratch_mkfs >> $seqres.full 2>&1
# Mount with dax option
-do_tests "-o dax=inode" "dax=inode"
+do_tests "-o dax=inode"
# Mount without dax option
export MOUNT_OPTIONS=""
--
2.21.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH v3 2/2] generic/608: Add new regression test
2021-03-30 2:06 [PATCH v3 1/2] generic/608: Remove drop cache and umount & mount cycle Xiao Yang
@ 2021-03-30 2:06 ` Xiao Yang
2021-03-30 4:57 ` Ira Weiny
0 siblings, 1 reply; 3+ messages in thread
From: Xiao Yang @ 2021-03-30 2:06 UTC (permalink / raw)
To: fstests; +Cc: darrick.wong, lihao2018.fnst, david, xifeng, ira.weiny, Xiao Yang
Write data into a file and then enable DAX on the file immediately,
the written data which is still in the buffer should be synchronized
to disk instead of discarded when the corresponding inode is evicted.
It's a regression test for commit 88149082bb8e.
Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com>
---
tests/generic/608 | 29 +++++++++++++++++++++++++++++
1 file changed, 29 insertions(+)
diff --git a/tests/generic/608 b/tests/generic/608
index 5ca2442f..c5bdac32 100755
--- a/tests/generic/608
+++ b/tests/generic/608
@@ -7,6 +7,12 @@
# file change immediately when all applications close the file.
# It's a regression test for:
# 'commit 77573fa310d9 ("fs: Kill DCACHE_DONTCACHE dentry even if DCACHE_REFERENCED is set")'
+#
+# Write data into a file and then enable DAX on the file immediately,
+# the written data which is still in the buffer should be synchronized
+# to disk instead of discarded when the corresponding inode is evicted.
+# It's a regression test for:
+# 'commit 88149082bb8e ("fs: Handle I_DONTCACHE in iput_final() instead of generic_drop_inode()"'
seq=`basename $0`
seqres=$RESULT_DIR/$seq
@@ -87,6 +93,26 @@ test_disable_dax()
_check_s_dax $t_file 0
}
+test_buffered_data_lost()
+{
+ local t_file=$SCRATCH_MNT/datafile
+
+ # Write data into a file
+ echo "Buffered data" > $t_file
+
+ # Then enable DAX on the file immediately
+ $XFS_IO_PROG -c 'chattr +x' $t_file
+
+ # Without commit 77573fa310d9, ensure inode can
+ # be evicted by drop_caches
+ echo 2 > /proc/sys/vm/drop_caches
+
+ # The written data which is still in the buffer should not be lost
+ grep -q "Buffered data" $t_file || echo "Buffered data is lost"
+
+ rm -f $t_file
+}
+
do_tests()
{
local mount_option=$1
@@ -100,6 +126,9 @@ do_tests()
test_enable_dax
test_disable_dax
+ # Do test for commit 88149082bb8e
+ test_buffered_data_lost
+
_scratch_unmount
}
--
2.21.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3 2/2] generic/608: Add new regression test
2021-03-30 2:06 ` [PATCH v3 2/2] generic/608: Add new regression test Xiao Yang
@ 2021-03-30 4:57 ` Ira Weiny
0 siblings, 0 replies; 3+ messages in thread
From: Ira Weiny @ 2021-03-30 4:57 UTC (permalink / raw)
To: Xiao Yang; +Cc: fstests, darrick.wong, lihao2018.fnst, david, xifeng
On Tue, Mar 30, 2021 at 10:06:55AM +0800, Xiao Yang wrote:
> Write data into a file and then enable DAX on the file immediately,
> the written data which is still in the buffer should be synchronized
> to disk instead of discarded when the corresponding inode is evicted.
>
> It's a regression test for commit 88149082bb8e.
>
> Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com>
LGTM
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
> ---
> tests/generic/608 | 29 +++++++++++++++++++++++++++++
> 1 file changed, 29 insertions(+)
>
> diff --git a/tests/generic/608 b/tests/generic/608
> index 5ca2442f..c5bdac32 100755
> --- a/tests/generic/608
> +++ b/tests/generic/608
> @@ -7,6 +7,12 @@
> # file change immediately when all applications close the file.
> # It's a regression test for:
> # 'commit 77573fa310d9 ("fs: Kill DCACHE_DONTCACHE dentry even if DCACHE_REFERENCED is set")'
> +#
> +# Write data into a file and then enable DAX on the file immediately,
> +# the written data which is still in the buffer should be synchronized
> +# to disk instead of discarded when the corresponding inode is evicted.
> +# It's a regression test for:
> +# 'commit 88149082bb8e ("fs: Handle I_DONTCACHE in iput_final() instead of generic_drop_inode()"'
>
> seq=`basename $0`
> seqres=$RESULT_DIR/$seq
> @@ -87,6 +93,26 @@ test_disable_dax()
> _check_s_dax $t_file 0
> }
>
> +test_buffered_data_lost()
> +{
> + local t_file=$SCRATCH_MNT/datafile
> +
> + # Write data into a file
> + echo "Buffered data" > $t_file
> +
> + # Then enable DAX on the file immediately
> + $XFS_IO_PROG -c 'chattr +x' $t_file
> +
> + # Without commit 77573fa310d9, ensure inode can
> + # be evicted by drop_caches
> + echo 2 > /proc/sys/vm/drop_caches
> +
> + # The written data which is still in the buffer should not be lost
> + grep -q "Buffered data" $t_file || echo "Buffered data is lost"
> +
> + rm -f $t_file
> +}
> +
> do_tests()
> {
> local mount_option=$1
> @@ -100,6 +126,9 @@ do_tests()
> test_enable_dax
> test_disable_dax
>
> + # Do test for commit 88149082bb8e
> + test_buffered_data_lost
> +
> _scratch_unmount
> }
>
> --
> 2.21.0
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-03-30 4:58 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-30 2:06 [PATCH v3 1/2] generic/608: Remove drop cache and umount & mount cycle Xiao Yang
2021-03-30 2:06 ` [PATCH v3 2/2] generic/608: Add new regression test Xiao Yang
2021-03-30 4:57 ` Ira Weiny
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).