* [PATCH v2 1/2] clean-up; Remove left files after test finishes
@ 2022-09-22 18:55 Pavel Reichl
2022-09-22 18:55 ` [PATCH v2 2/2] g/299: skip test early if falloc is not supported Pavel Reichl
2022-09-29 1:58 ` [PATCH v2 1/2] clean-up; Remove left files after test finishes Zorro Lang
0 siblings, 2 replies; 4+ messages in thread
From: Pavel Reichl @ 2022-09-22 18:55 UTC (permalink / raw)
To: fstests
Fix file leak in _get_max_file_size - This is obviously mostly
problematic for FS lacking support for sparse files.
There seems to be some seek_sanity_testfile files that are not
cleaned up and take up space:
-rwxr-xr-x. 1 root root 8.0G Sep 22 13:39 seek_sanity_testfile10
g/394: Clean up test files taking space:
-rwxr-xr-x. 1 root root 1.0G Sep 22 14:58 394.1183899
-rwxr-xr-x. 1 root root 1.0G Sep 22 14:58 394.1183899-1
-rwxr-xr-x. 1 root root 0 sep 22 14:58 394.1183899+1
Signed-off-by: Pavel Reichl <preichl@redhat.com>
---
common/rc | 1 +
tests/generic/285 | 2 +-
tests/generic/394 | 2 +-
tests/generic/436 | 2 +-
tests/generic/445 | 2 +-
tests/generic/448 | 2 +-
6 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/common/rc b/common/rc
index 228fcb37..c9078649 100644
--- a/common/rc
+++ b/common/rc
@@ -4637,6 +4637,7 @@ _get_max_file_size()
l=$m
fi
done
+ rm -f $testfile
echo $l
}
diff --git a/tests/generic/285 b/tests/generic/285
index 6c755b5f..d1850021 100755
--- a/tests/generic/285
+++ b/tests/generic/285
@@ -28,7 +28,7 @@ _require_test_program "seek_sanity_test"
# Override the default cleanup function.
_cleanup()
{
- eval "rm -f $BASE_TEST_FILE.*"
+ rm -f $BASE_TEST_FILE*
}
_run_seek_sanity_test $BASE_TEST_FILE > $seqres.full 2>&1 ||
diff --git a/tests/generic/394 b/tests/generic/394
index fc03e2f3..cbc2ce13 100755
--- a/tests/generic/394
+++ b/tests/generic/394
@@ -14,7 +14,7 @@ _cleanup()
{
cd /
ulimit -f unlimited
- rm -f $tmp.*
+ rm -f $tmp.* $TEST_DIR/$seq.*
}
# Import common functions.
diff --git a/tests/generic/436 b/tests/generic/436
index 7fb24b41..d54af436 100755
--- a/tests/generic/436
+++ b/tests/generic/436
@@ -16,7 +16,7 @@ BASE_TEST_FILE=$TEST_DIR/seek_sanity_testfile
# Override the default cleanup function.
_cleanup()
{
- rm -f $tmp.* $BASE_TEST_FILE.*
+ rm -f $tmp.* $BASE_TEST_FILE*
}
# Import common functions.
diff --git a/tests/generic/445 b/tests/generic/445
index 05d1c5b3..d9e9f84b 100755
--- a/tests/generic/445
+++ b/tests/generic/445
@@ -16,7 +16,7 @@ BASE_TEST_FILE=$TEST_DIR/seek_sanity_testfile
# Override the default cleanup function.
_cleanup()
{
- rm -f $tmp.* $BASE_TEST_FILE.*
+ rm -f $tmp.* $BASE_TEST_FILE*
}
# Import common functions.
diff --git a/tests/generic/448 b/tests/generic/448
index d66967bc..11945549 100755
--- a/tests/generic/448
+++ b/tests/generic/448
@@ -15,7 +15,7 @@ BASE_TEST_FILE=$TEST_DIR/seek_sanity_testfile_$seq
_cleanup()
{
cd /
- rm -f $tmp.* $BASE_TEST_FILE
+ rm -f $tmp.* $BASE_TEST_FILE*
}
# Import common functions.
--
2.37.3
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH v2 2/2] g/299: skip test early if falloc is not supported
2022-09-22 18:55 [PATCH v2 1/2] clean-up; Remove left files after test finishes Pavel Reichl
@ 2022-09-22 18:55 ` Pavel Reichl
2022-09-29 2:04 ` Zorro Lang
2022-09-29 1:58 ` [PATCH v2 1/2] clean-up; Remove left files after test finishes Zorro Lang
1 sibling, 1 reply; 4+ messages in thread
From: Pavel Reichl @ 2022-09-22 18:55 UTC (permalink / raw)
To: fstests
Move the _require_xfs_io_command "falloc" above the
potencially 'expensive' function _get_max_file_size
Signed-off-by: Pavel Reichl <preichl@redhat.com>
---
tests/generic/299 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/generic/299 b/tests/generic/299
index 8e1a112c..1bb371a3 100755
--- a/tests/generic/299
+++ b/tests/generic/299
@@ -24,6 +24,7 @@ _require_scratch
_require_odirect
_require_aio
_require_block_device $SCRATCH_DEV
+_require_xfs_io_command "falloc"
NUM_JOBS=$((4*LOAD_FACTOR))
BLK_DEV_SIZE=`blockdev --getsz $SCRATCH_DEV`
@@ -96,7 +97,6 @@ filename=buffered-aio-verifier
EOF
_require_fio $fio_config
-_require_xfs_io_command "falloc"
_scratch_mkfs >> $seqres.full 2>&1
_scratch_mount
--
2.37.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 2/2] g/299: skip test early if falloc is not supported
2022-09-22 18:55 ` [PATCH v2 2/2] g/299: skip test early if falloc is not supported Pavel Reichl
@ 2022-09-29 2:04 ` Zorro Lang
0 siblings, 0 replies; 4+ messages in thread
From: Zorro Lang @ 2022-09-29 2:04 UTC (permalink / raw)
To: Pavel Reichl; +Cc: fstests
On Thu, Sep 22, 2022 at 08:55:02PM +0200, Pavel Reichl wrote:
> Move the _require_xfs_io_command "falloc" above the
> potencially 'expensive' function _get_max_file_size
>
> Signed-off-by: Pavel Reichl <preichl@redhat.com>
> ---
Hmm... looks like the `_require_xfs_io_command "falloc"` helps to save a helper
"_require_falloc" which detects if fs (kernel part) supports fallocate, although
it trys to make sure xfs_io supports falloc command.
Reviewed-by: Zorro Lang <zlang@redhat.com>
Thanks,
Zorro
> tests/generic/299 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/generic/299 b/tests/generic/299
> index 8e1a112c..1bb371a3 100755
> --- a/tests/generic/299
> +++ b/tests/generic/299
> @@ -24,6 +24,7 @@ _require_scratch
> _require_odirect
> _require_aio
> _require_block_device $SCRATCH_DEV
> +_require_xfs_io_command "falloc"
>
> NUM_JOBS=$((4*LOAD_FACTOR))
> BLK_DEV_SIZE=`blockdev --getsz $SCRATCH_DEV`
> @@ -96,7 +97,6 @@ filename=buffered-aio-verifier
> EOF
>
> _require_fio $fio_config
> -_require_xfs_io_command "falloc"
>
> _scratch_mkfs >> $seqres.full 2>&1
> _scratch_mount
> --
> 2.37.3
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 1/2] clean-up; Remove left files after test finishes
2022-09-22 18:55 [PATCH v2 1/2] clean-up; Remove left files after test finishes Pavel Reichl
2022-09-22 18:55 ` [PATCH v2 2/2] g/299: skip test early if falloc is not supported Pavel Reichl
@ 2022-09-29 1:58 ` Zorro Lang
1 sibling, 0 replies; 4+ messages in thread
From: Zorro Lang @ 2022-09-29 1:58 UTC (permalink / raw)
To: Pavel Reichl; +Cc: fstests
On Thu, Sep 22, 2022 at 08:55:01PM +0200, Pavel Reichl wrote:
> Fix file leak in _get_max_file_size - This is obviously mostly
> problematic for FS lacking support for sparse files.
>
> There seems to be some seek_sanity_testfile files that are not
> cleaned up and take up space:
>
> -rwxr-xr-x. 1 root root 8.0G Sep 22 13:39 seek_sanity_testfile10
>
> g/394: Clean up test files taking space:
>
> -rwxr-xr-x. 1 root root 1.0G Sep 22 14:58 394.1183899
> -rwxr-xr-x. 1 root root 1.0G Sep 22 14:58 394.1183899-1
> -rwxr-xr-x. 1 root root 0 sep 22 14:58 394.1183899+1
>
> Signed-off-by: Pavel Reichl <preichl@redhat.com>
> ---
This patch looks good to me, although I doubt _get_max_file_size() really can
give you a proper max file size for your exfat testing. I even doubt if those
cases test max file size are suit for exfat which doesn't support sparse file
and fallocate. Anyway, you can deal with that problem later, this patch can be
merged at first. Thanks.
Reviewed-by: Zorro Lang <zlang@redhat.com>
> common/rc | 1 +
> tests/generic/285 | 2 +-
> tests/generic/394 | 2 +-
> tests/generic/436 | 2 +-
> tests/generic/445 | 2 +-
> tests/generic/448 | 2 +-
> 6 files changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/common/rc b/common/rc
> index 228fcb37..c9078649 100644
> --- a/common/rc
> +++ b/common/rc
> @@ -4637,6 +4637,7 @@ _get_max_file_size()
> l=$m
> fi
> done
> + rm -f $testfile
> echo $l
> }
>
> diff --git a/tests/generic/285 b/tests/generic/285
> index 6c755b5f..d1850021 100755
> --- a/tests/generic/285
> +++ b/tests/generic/285
> @@ -28,7 +28,7 @@ _require_test_program "seek_sanity_test"
> # Override the default cleanup function.
> _cleanup()
> {
> - eval "rm -f $BASE_TEST_FILE.*"
> + rm -f $BASE_TEST_FILE*
> }
>
> _run_seek_sanity_test $BASE_TEST_FILE > $seqres.full 2>&1 ||
> diff --git a/tests/generic/394 b/tests/generic/394
> index fc03e2f3..cbc2ce13 100755
> --- a/tests/generic/394
> +++ b/tests/generic/394
> @@ -14,7 +14,7 @@ _cleanup()
> {
> cd /
> ulimit -f unlimited
> - rm -f $tmp.*
> + rm -f $tmp.* $TEST_DIR/$seq.*
> }
>
> # Import common functions.
> diff --git a/tests/generic/436 b/tests/generic/436
> index 7fb24b41..d54af436 100755
> --- a/tests/generic/436
> +++ b/tests/generic/436
> @@ -16,7 +16,7 @@ BASE_TEST_FILE=$TEST_DIR/seek_sanity_testfile
> # Override the default cleanup function.
> _cleanup()
> {
> - rm -f $tmp.* $BASE_TEST_FILE.*
> + rm -f $tmp.* $BASE_TEST_FILE*
> }
>
> # Import common functions.
> diff --git a/tests/generic/445 b/tests/generic/445
> index 05d1c5b3..d9e9f84b 100755
> --- a/tests/generic/445
> +++ b/tests/generic/445
> @@ -16,7 +16,7 @@ BASE_TEST_FILE=$TEST_DIR/seek_sanity_testfile
> # Override the default cleanup function.
> _cleanup()
> {
> - rm -f $tmp.* $BASE_TEST_FILE.*
> + rm -f $tmp.* $BASE_TEST_FILE*
> }
>
> # Import common functions.
> diff --git a/tests/generic/448 b/tests/generic/448
> index d66967bc..11945549 100755
> --- a/tests/generic/448
> +++ b/tests/generic/448
> @@ -15,7 +15,7 @@ BASE_TEST_FILE=$TEST_DIR/seek_sanity_testfile_$seq
> _cleanup()
> {
> cd /
> - rm -f $tmp.* $BASE_TEST_FILE
> + rm -f $tmp.* $BASE_TEST_FILE*
> }
>
> # Import common functions.
> --
> 2.37.3
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-09-29 2:04 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-22 18:55 [PATCH v2 1/2] clean-up; Remove left files after test finishes Pavel Reichl
2022-09-22 18:55 ` [PATCH v2 2/2] g/299: skip test early if falloc is not supported Pavel Reichl
2022-09-29 2:04 ` Zorro Lang
2022-09-29 1:58 ` [PATCH v2 1/2] clean-up; Remove left files after test finishes Zorro Lang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox