From: "Darrick J. Wong" <djwong@kernel.org>
To: fdmanana@kernel.org
Cc: fstests@vger.kernel.org, linux-btrfs@vger.kernel.org,
Filipe Manana <fdmanana@suse.com>
Subject: Re: [PATCH 1/2] fstests: redirect fsstress' stdout to $seqres.full instead of /dev/null
Date: Tue, 3 Oct 2023 08:24:37 -0700 [thread overview]
Message-ID: <20231003152437.GC21283@frogsfrogsfrogs> (raw)
In-Reply-To: <bbd99333b3c2dcd44c10958aeab43c22c91a738f.1696333874.git.fdmanana@suse.com>
On Tue, Oct 03, 2023 at 12:57:44PM +0100, fdmanana@kernel.org wrote:
> From: Filipe Manana <fdmanana@suse.com>
>
> Several tests are redirecting the output of fsstress to /dev/null and this
> makes it harder to debug a test failure because we have no way of knowing
> what was the seed used by fsstress, as fsstress outputs the seed it uses
> to stdout. Very often when such a test fails, I have to go modify to
> redirect stdout to the $seqres.full file and then run it in a loop until
> I find a seed that causes a failure.
>
> So modify all tests that redirect fsstress' output to /dev/null to instead
> redirect it to the $seqres.full file. Note that for some tests I've added
> the style ">> $seqres.full" (with a space after >>) while for others I did
> ">>$seqres.full" (no space) - the reason for this was to keep style
> consistency within each test case.
>
> Signed-off-by: Filipe Manana <fdmanana@suse.com>
> ---
> tests/btrfs/028 | 2 +-
> tests/btrfs/049 | 2 +-
> tests/btrfs/060 | 2 +-
> tests/btrfs/061 | 2 +-
> tests/btrfs/062 | 2 +-
> tests/btrfs/063 | 2 +-
> tests/btrfs/064 | 2 +-
> tests/btrfs/065 | 2 +-
> tests/btrfs/066 | 2 +-
> tests/btrfs/067 | 2 +-
> tests/btrfs/068 | 2 +-
> tests/btrfs/069 | 2 +-
> tests/btrfs/070 | 2 +-
> tests/btrfs/071 | 2 +-
> tests/btrfs/072 | 2 +-
> tests/btrfs/073 | 2 +-
> tests/btrfs/074 | 2 +-
> tests/btrfs/136 | 2 +-
> tests/btrfs/192 | 2 +-
> tests/btrfs/232 | 2 +-
> tests/btrfs/261 | 2 +-
> tests/btrfs/286 | 2 +-
> tests/ext4/057 | 2 +-
> tests/ext4/307 | 2 +-
> tests/generic/068 | 2 +-
> tests/generic/269 | 2 +-
> tests/generic/409 | 2 +-
> tests/generic/410 | 2 +-
> tests/generic/411 | 2 +-
> tests/generic/589 | 2 +-
> tests/xfs/051 | 2 +-
> tests/xfs/057 | 2 +-
> tests/xfs/297 | 2 +-
> tests/xfs/305 | 2 +-
> tests/xfs/538 | 2 +-
For the generic & xfs parts,
Acked-by: Darrick J. Wong <djwong@kernel.org>
(The rest are also probably ok but I'm much less familiar with the
btrfs-specific tests.)
--D
> 35 files changed, 35 insertions(+), 35 deletions(-)
>
> diff --git a/tests/btrfs/028 b/tests/btrfs/028
> index fe0678f8..4637469c 100755
> --- a/tests/btrfs/028
> +++ b/tests/btrfs/028
> @@ -35,7 +35,7 @@ args=`_scale_fsstress_args -z \
> -f fsync=10 -n 100000 -p 2 \
> -d $SCRATCH_MNT/stress_dir`
> echo "Run fsstress $args" >>$seqres.full
> -$FSSTRESS_PROG $args >/dev/null 2>&1 &
> +$FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo "Start balance" >>$seqres.full
> diff --git a/tests/btrfs/049 b/tests/btrfs/049
> index 9569c141..c48e4087 100755
> --- a/tests/btrfs/049
> +++ b/tests/btrfs/049
> @@ -42,7 +42,7 @@ args=`_scale_fsstress_args -z \
> -f write=10 -f creat=10 \
> -n 1000 -p 2 -d $SCRATCH_MNT/stress_dir`
> echo "Run fsstress $args" >>$seqres.full
> -$FSSTRESS_PROG $args >/dev/null 2>&1
> +$FSSTRESS_PROG $args >>$seqres.full
>
> # Start and pause balance to ensure it will be restored on remount
> echo "Start balance" >>$seqres.full
> diff --git a/tests/btrfs/060 b/tests/btrfs/060
> index 7dd4d2af..a0184891 100755
> --- a/tests/btrfs/060
> +++ b/tests/btrfs/060
> @@ -38,7 +38,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start balance worker: " >>$seqres.full
> diff --git a/tests/btrfs/061 b/tests/btrfs/061
> index 55f5625b..c1010413 100755
> --- a/tests/btrfs/061
> +++ b/tests/btrfs/061
> @@ -36,7 +36,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start balance worker: " >>$seqres.full
> diff --git a/tests/btrfs/062 b/tests/btrfs/062
> index 10f95111..818a0156 100755
> --- a/tests/btrfs/062
> +++ b/tests/btrfs/062
> @@ -37,7 +37,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start balance worker: " >>$seqres.full
> diff --git a/tests/btrfs/063 b/tests/btrfs/063
> index cef80771..2f771baf 100755
> --- a/tests/btrfs/063
> +++ b/tests/btrfs/063
> @@ -36,7 +36,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start balance worker: " >>$seqres.full
> diff --git a/tests/btrfs/064 b/tests/btrfs/064
> index f29e68ba..e9b46ce6 100755
> --- a/tests/btrfs/064
> +++ b/tests/btrfs/064
> @@ -46,7 +46,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> # Start both balance and replace in the background.
> diff --git a/tests/btrfs/065 b/tests/btrfs/065
> index b6c9dbad..c4b6aafe 100755
> --- a/tests/btrfs/065
> +++ b/tests/btrfs/065
> @@ -46,7 +46,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> # make sure the stop sign is not there
> diff --git a/tests/btrfs/066 b/tests/btrfs/066
> index 8d12af61..a29034bb 100755
> --- a/tests/btrfs/066
> +++ b/tests/btrfs/066
> @@ -38,7 +38,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> # make sure the stop sign is not there
> diff --git a/tests/btrfs/067 b/tests/btrfs/067
> index 970a23c4..709db155 100755
> --- a/tests/btrfs/067
> +++ b/tests/btrfs/067
> @@ -39,7 +39,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> # make sure the stop sign is not there
> diff --git a/tests/btrfs/068 b/tests/btrfs/068
> index e0bcc2ac..15fd41db 100755
> --- a/tests/btrfs/068
> +++ b/tests/btrfs/068
> @@ -39,7 +39,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> # make sure the stop sign is not there
> diff --git a/tests/btrfs/069 b/tests/btrfs/069
> index 6e798a2e..139dde48 100755
> --- a/tests/btrfs/069
> +++ b/tests/btrfs/069
> @@ -44,7 +44,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start replace worker: " >>$seqres.full
> diff --git a/tests/btrfs/070 b/tests/btrfs/070
> index f2e61ad3..54aa275c 100755
> --- a/tests/btrfs/070
> +++ b/tests/btrfs/070
> @@ -45,7 +45,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start replace worker: " >>$seqres.full
> diff --git a/tests/btrfs/071 b/tests/btrfs/071
> index 40230b11..6ebbd8cc 100755
> --- a/tests/btrfs/071
> +++ b/tests/btrfs/071
> @@ -44,7 +44,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start replace worker: " >>$seqres.full
> diff --git a/tests/btrfs/072 b/tests/btrfs/072
> index bcb0ea25..4b6b6fb5 100755
> --- a/tests/btrfs/072
> +++ b/tests/btrfs/072
> @@ -37,7 +37,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start scrub worker: " >>$seqres.full
> diff --git a/tests/btrfs/073 b/tests/btrfs/073
> index 26c5deb6..b1604f94 100755
> --- a/tests/btrfs/073
> +++ b/tests/btrfs/073
> @@ -36,7 +36,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start scrub worker: " >>$seqres.full
> diff --git a/tests/btrfs/074 b/tests/btrfs/074
> index 92e25c7c..9b22c620 100755
> --- a/tests/btrfs/074
> +++ b/tests/btrfs/074
> @@ -37,7 +37,7 @@ run_test()
>
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
>
> echo -n "Start defrag worker: " >>$seqres.full
> diff --git a/tests/btrfs/136 b/tests/btrfs/136
> index b9ab8270..70e836a5 100755
> --- a/tests/btrfs/136
> +++ b/tests/btrfs/136
> @@ -39,7 +39,7 @@ populate_data(){
> mkdir -p $data_path
> args=`_scale_fsstress_args -p 20 -n 100 $FSSTRESS_AVOID -d $data_path`
> echo "Run fsstress $args" >>$seqres.full
> - $FSSTRESS_PROG $args >/dev/null 2>&1 &
> + $FSSTRESS_PROG $args >>$seqres.full &
> fsstress_pid=$!
> wait $fsstress_pid
> }
> diff --git a/tests/btrfs/192 b/tests/btrfs/192
> index ea261b34..80588a3c 100755
> --- a/tests/btrfs/192
> +++ b/tests/btrfs/192
> @@ -140,7 +140,7 @@ pid1=$!
> delete_workload &
> pid2=$!
>
> -"$FSSTRESS_PROG" $fsstress_args > /dev/null &
> +"$FSSTRESS_PROG" $fsstress_args >> $seqres.full &
> sleep $runtime
>
> "$KILLALL_PROG" -q "$FSSTRESS_PROG" &> /dev/null
> diff --git a/tests/btrfs/232 b/tests/btrfs/232
> index 02c7e49d..e8a22f5e 100755
> --- a/tests/btrfs/232
> +++ b/tests/btrfs/232
> @@ -25,7 +25,7 @@ writer()
>
> while true; do
> args=`_scale_fsstress_args -p 20 -n 1000 $FSSTRESS_AVOID -d $SCRATCH_MNT/stressdir`
> - $FSSTRESS_PROG $args >/dev/null 2>&1
> + $FSSTRESS_PROG $args >> $seqres.full
> done
> }
>
> diff --git a/tests/btrfs/261 b/tests/btrfs/261
> index 50173de3..58fa8e75 100755
> --- a/tests/btrfs/261
> +++ b/tests/btrfs/261
> @@ -36,7 +36,7 @@ prepare_fs()
> # Then use fsstress to generate some extra contents.
> # Disable setattr related operations, as it may set NODATACOW which will
> # not allow us to use btrfs checksum to verify the content.
> - $FSSTRESS_PROG -f setattr=0 -d $SCRATCH_MNT -w -n 3000 > /dev/null 2>&1
> + $FSSTRESS_PROG -f setattr=0 -d $SCRATCH_MNT -w -n 3000 >> $seqres.full
> sync
>
> # Save the fssum of this fs
> diff --git a/tests/btrfs/286 b/tests/btrfs/286
> index f1ee129c..71f6d4bd 100755
> --- a/tests/btrfs/286
> +++ b/tests/btrfs/286
> @@ -36,7 +36,7 @@ workload()
> # Use nodatasum mount option, so all data won't have checksum.
> _scratch_mount -o nodatasum
>
> - $FSSTRESS_PROG -p 10 -n 200 -d $SCRATCH_MNT > /dev/null 2>&1
> + $FSSTRESS_PROG -p 10 -n 200 -d $SCRATCH_MNT >> $seqres.full
> sync
>
> # Generate fssum for later verification, here we only care
> diff --git a/tests/ext4/057 b/tests/ext4/057
> index 4006a07c..6babedb2 100755
> --- a/tests/ext4/057
> +++ b/tests/ext4/057
> @@ -42,7 +42,7 @@ _scratch_mount
>
> # Begin fsstress while modifying UUID
> fsstress_args=$(_scale_fsstress_args -d $SCRATCH_MNT -p 15 -n 999999)
> -$FSSTRESS_PROG $fsstress_args > /dev/null 2>&1 &
> +$FSSTRESS_PROG $fsstress_args >> $seqres.full &
> fsstress_pid=$!
>
> for n in $(seq 1 20); do
> diff --git a/tests/ext4/307 b/tests/ext4/307
> index db83a083..8b1cfc9e 100755
> --- a/tests/ext4/307
> +++ b/tests/ext4/307
> @@ -21,7 +21,7 @@ _workout()
> out=$SCRATCH_MNT/fsstress.$$
> args=`_scale_fsstress_args -p4 -n999 -f setattr=1 $FSSTRESS_AVOID -d $out`
> echo "fsstress $args" >> $seqres.full
> - $FSSTRESS_PROG $args > /dev/null 2>&1
> + $FSSTRESS_PROG $args >> $seqres.full
> find $out -type f > $out.list
> cat $out.list | xargs md5sum > $out.md5sum
> usage=`du -sch $out | tail -n1 | gawk '{ print $1 }'`
> diff --git a/tests/generic/068 b/tests/generic/068
> index eeddf6d1..af527fee 100755
> --- a/tests/generic/068
> +++ b/tests/generic/068
> @@ -57,7 +57,7 @@ touch $tmp.running
> # We do both read & write IO - not only is this more realistic,
> # but it also potentially tests atime updates
> FSSTRESS_ARGS=`_scale_fsstress_args -d $STRESS_DIR -p $procs -n $nops $FSSTRESS_AVOID`
> - $FSSTRESS_PROG $FSSTRESS_ARGS > /dev/null 2>&1
> + $FSSTRESS_PROG $FSSTRESS_ARGS >>$seqres.full
> done
>
> rm -r $STRESS_DIR/*
> diff --git a/tests/generic/269 b/tests/generic/269
> index 838e696d..b852f6bf 100755
> --- a/tests/generic/269
> +++ b/tests/generic/269
> @@ -23,7 +23,7 @@ _workout()
> out=$SCRATCH_MNT/fsstress.$$
> args=`_scale_fsstress_args -p128 -n999999999 -f setattr=1 $FSSTRESS_AVOID -d $out`
> echo "fsstress $args" >> $seqres.full
> - $FSSTRESS_PROG $args > /dev/null 2>&1 &
> + $FSSTRESS_PROG $args >> $seqres.full &
> pid=$!
> echo "Run dd writers in parallel"
> for ((i=0; i < num_iterations; i++))
> diff --git a/tests/generic/409 b/tests/generic/409
> index 7a5004ed..432befac 100755
> --- a/tests/generic/409
> +++ b/tests/generic/409
> @@ -58,7 +58,7 @@ fs_stress()
> -f chown=1 \
> -f getdents=1 \
> -f fiemap=1 \
> - -d $target >/dev/null
> + -d $target >>$seqres.full
> sync
> }
>
> diff --git a/tests/generic/410 b/tests/generic/410
> index f35f2f4a..8cc36d9f 100755
> --- a/tests/generic/410
> +++ b/tests/generic/410
> @@ -66,7 +66,7 @@ fs_stress()
> -f chown=1 \
> -f getdents=1 \
> -f fiemap=1 \
> - -d $target >/dev/null
> + -d $target >>$seqres.full
> sync
> }
>
> diff --git a/tests/generic/411 b/tests/generic/411
> index 9852a49d..b2b8d550 100755
> --- a/tests/generic/411
> +++ b/tests/generic/411
> @@ -49,7 +49,7 @@ fs_stress()
> -f chown=1 \
> -f getdents=1 \
> -f fiemap=1 \
> - -d $target >/dev/null
> + -d $target >>$seqres.full
> sync
> }
>
> diff --git a/tests/generic/589 b/tests/generic/589
> index c03cf1fc..bfc7407a 100755
> --- a/tests/generic/589
> +++ b/tests/generic/589
> @@ -48,7 +48,7 @@ fs_stress()
> {
> local target=$1
>
> - $FSSTRESS_PROG -n 50 -p 3 -d $target >/dev/null
> + $FSSTRESS_PROG -n 50 -p 3 -d $target >>$seqres.full
> sync
> }
>
> diff --git a/tests/xfs/051 b/tests/xfs/051
> index ea70cb50..1c670964 100755
> --- a/tests/xfs/051
> +++ b/tests/xfs/051
> @@ -38,7 +38,7 @@ _scratch_mount
>
> # Start a workload and shutdown the fs. The subsequent mount will require log
> # recovery.
> -$FSSTRESS_PROG -n 9999 -p 2 -w -d $SCRATCH_MNT > /dev/null 2>&1 &
> +$FSSTRESS_PROG -n 9999 -p 2 -w -d $SCRATCH_MNT >> $seqres.full &
> sleep 5
> _scratch_shutdown -f
> $KILLALL_PROG -q $FSSTRESS_PROG
> diff --git a/tests/xfs/057 b/tests/xfs/057
> index 9fb3f406..6af14c80 100755
> --- a/tests/xfs/057
> +++ b/tests/xfs/057
> @@ -56,7 +56,7 @@ _scratch_mkfs_sized $((1024 * 1024 * 500)) >> $seqres.full 2>&1 ||
> _scratch_mount
>
> # populate the fs with some data and cycle the mount to reset the log head/tail
> -$FSSTRESS_PROG -d $SCRATCH_MNT -z -fcreat=1 -p 4 -n 100000 > /dev/null 2>&1
> +$FSSTRESS_PROG -d $SCRATCH_MNT -z -fcreat=1 -p 4 -n 100000 >> $seqres.full
> _scratch_cycle_mount || _fail "cycle mount failed"
>
> # Pin the tail and start a file removal workload. File removal tends to
> diff --git a/tests/xfs/297 b/tests/xfs/297
> index 07f84c25..1d101876 100755
> --- a/tests/xfs/297
> +++ b/tests/xfs/297
> @@ -39,7 +39,7 @@ _scratch_mount
> STRESS_DIR="$SCRATCH_MNT/testdir"
> mkdir -p $STRESS_DIR
>
> -$FSSTRESS_PROG -d $STRESS_DIR -n 100 -p 1000 $FSSTRESS_AVOID >/dev/null 2>&1 &
> +$FSSTRESS_PROG -d $STRESS_DIR -n 100 -p 1000 $FSSTRESS_AVOID >>$seqres.full &
>
> # Freeze/unfreeze file system randomly
> echo "Start freeze/unfreeze randomly" | tee -a $seqres.full
> diff --git a/tests/xfs/305 b/tests/xfs/305
> index 41c7b7f8..d8a6712e 100755
> --- a/tests/xfs/305
> +++ b/tests/xfs/305
> @@ -36,7 +36,7 @@ _exercise()
> _qmount
> mkdir -p $QUOTA_DIR
>
> - $FSSTRESS_PROG -d $QUOTA_DIR -n 1000000 -p 100 $FSSTRESS_AVOID >/dev/null 2>&1 &
> + $FSSTRESS_PROG -d $QUOTA_DIR -n 1000000 -p 100 $FSSTRESS_AVOID >>$seqres.full &
> sleep 10
> $XFS_QUOTA_PROG -x -c "disable -$type" $SCRATCH_DEV
> sleep 5
> diff --git a/tests/xfs/538 b/tests/xfs/538
> index 2b5e97e5..0b5772a1 100755
> --- a/tests/xfs/538
> +++ b/tests/xfs/538
> @@ -63,7 +63,7 @@ $FSSTRESS_PROG -d $SCRATCH_MNT \
> -f readv=0 \
> -f stat=0 \
> -f aread=0 \
> - -f dread=0 > /dev/null 2>&1
> + -f dread=0 >> $seqres.full
>
> # success, all done
> status=0
> --
> 2.40.1
>
next prev parent reply other threads:[~2023-10-03 15:24 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-03 11:57 [PATCH 0/2] fstests: make some tests that use fsstress easier to debug fdmanana
2023-10-03 11:57 ` [PATCH 1/2] fstests: redirect fsstress' stdout to $seqres.full instead of /dev/null fdmanana
2023-10-03 15:24 ` Darrick J. Wong [this message]
2023-10-03 11:57 ` [PATCH 2/2] btrfs/192: use append operator to output log replay results to $seqres.full fdmanana
2023-10-03 14:06 ` Anand Jain
2023-10-04 8:06 ` [PATCH 0/2] fstests: make some tests that use fsstress easier to debug Qu Wenruo
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=20231003152437.GC21283@frogsfrogsfrogs \
--to=djwong@kernel.org \
--cc=fdmanana@kernel.org \
--cc=fdmanana@suse.com \
--cc=fstests@vger.kernel.org \
--cc=linux-btrfs@vger.kernel.org \
/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.