Re: [PATCH] generic/251: check min and max length and minlen for FSTRIM

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Oct 21, 2023 at 09:14:48PM +0800, Zorro Lang wrote:
> On Thu, Oct 19, 2023 at 07:36:27AM -0700, Darrick J. Wong wrote:
> > From: Darrick J. Wong <djwong@xxxxxxxxxx>
> > 
> > Every now and then, this test fails with the following output when
> > running against my development tree when configured with an 8k fs block
> > size:
> > 
> > --- a/tests/generic/251.out	2023-07-11 12:18:21.624971186 -0700
> > +++ b/tests/generic/251.out.bad	2023-10-15 20:54:44.636000000 -0700
> > @@ -1,2 +1,4677 @@
> >  QA output created by 251
> >  Running the test: done.
> > +fstrim: /opt: FITRIM ioctl failed: Invalid argument
> > +fstrim: /opt: FITRIM ioctl failed: Invalid argument
> > ...
> > +fstrim: /opt: FITRIM ioctl failed: Invalid argument
> > 
> > Dumping the exact fstrim command lines to seqres.full produces this at
> > the end:
> > 
> > /usr/sbin/fstrim -m 32544k -o 30247k -l 4k /opt
> > /usr/sbin/fstrim -m 32544k -o 30251k -l 4k /opt
> > ...
> > /usr/sbin/fstrim -m 32544k -o 30255k -l 4k /opt
> > 
> > The count of failure messages is the same as the count as the "-l 4k"
> > fstrim invocations.  Since this is an 8k-block filesystem, the -l
> > parameter is clearly incorrect.  The test computes random -m and -l
> > options.
> > 
> > Therefore, create helper functions to guess at the minimum and maximum
> > length and minlen parameters that can be used with the fstrim program.
> > In the inner loop of the test, make sure that our choices for -m and -l
> > fall within those constraints.
> > 
> > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx>
> > ---
> 
> Hi Darrick, with this patch I 100% hit below failure (on default 4k xfs
> and ext4):
> 
> # ./check generic/251
> FSTYP         -- xfs (debug)
> PLATFORM      -- Linux/x86_64 hp-dl380pg8-01 6.6.0-rc6-mainline+ #7 SMP PREEMPT_DYNAMIC Thu Oct 19 22:34:28 CST 2023
> MKFS_OPTIONS  -- -f /dev/loop0
> MOUNT_OPTIONS -- -o context=system_u:object_r:root_t:s0 /dev/loop0 /mnt/scratch
> 
> generic/251 260s ... [failed, exit status 1]- output mismatch (see /root/git/xfstests/results//generic/251.out.bad)
>     --- tests/generic/251.out   2022-04-29 23:07:23.263498297 +0800
>     +++ /root/git/xfstests/results//generic/251.out.bad 2023-10-21 21:02:37.687088360 +0800
>     @@ -1,2 +1,5 @@
>      QA output created by 251
>      Running the test: done.
>     +5834a5835
>     +> aa60581221897d3d7dd60458e1cca2fa  ./results/generic/251.full
>     +!!!Checksums has changed - Filesystem possibly corrupted!!!\n
>     ...
>     (Run 'diff -u /root/git/xfstests/tests/generic/251.out /root/git/xfstests/results//generic/251.out.bad'  to see the entire diff)

Huh.  I don't see that on ext4 on my machine.  Can you send me all your
/root/git/xfstests/results//generic/251* files so that I can have a
look?

--D

> Ran: generic/251
> Failures: generic/251
> Failed 1 of 1 tests
> 
> And test passed without this patch.
> 
> # ./check generic/251
> FSTYP         -- xfs (debug)
> PLATFORM      -- Linux/x86_64 hp-dl380pg8-01 6.6.0-rc6-mainline+ #7 SMP PREEMPT_DYNAMIC Thu Oct 19 22:34:28 CST 2023
> MKFS_OPTIONS  -- -f /dev/loop0
> MOUNT_OPTIONS -- -o context=system_u:object_r:root_t:s0 /dev/loop0 /mnt/scratch
> 
> generic/251 260s ...  249s
> Ran: generic/251
> Passed all 1 tests
> 
> Thanks,
> Zorro
> 
> >  tests/generic/251 |   59 ++++++++++++++++++++++++++++++++++++++++++++++-------
> >  1 file changed, 51 insertions(+), 8 deletions(-)
> > 
> > diff --git a/tests/generic/251 b/tests/generic/251
> > index 8ee74980cc..40cfd7c381 100755
> > --- a/tests/generic/251
> > +++ b/tests/generic/251
> > @@ -53,14 +53,46 @@ _fail()
> >  	kill $mypid 2> /dev/null
> >  }
> >  
> > -_guess_max_minlen()
> > +# Set FSTRIM_{MIN,MAX}_MINLEN to the lower and upper bounds of the -m(inlen)
> > +# parameter to fstrim on the scratch filesystem.
> > +set_minlen_constraints()
> >  {
> > -	mmlen=100000
> > -	while [ $mmlen -gt 1 ]; do
> > +	local mmlen
> > +
> > +	for ((mmlen = 100000; mmlen > 0; mmlen /= 2)); do
> >  		$FSTRIM_PROG -l $(($mmlen*2))k -m ${mmlen}k $SCRATCH_MNT &> /dev/null && break
> > -		mmlen=$(($mmlen/2))
> >  	done
> > -	echo $mmlen
> > +	test $mmlen -gt 0 || \
> > +		_notrun "could not determine maximum FSTRIM minlen param"
> > +	FSTRIM_MAX_MINLEN=$mmlen
> > +
> > +	for ((mmlen = 1; mmlen < FSTRIM_MAX_MINLEN; mmlen *= 2)); do
> > +		$FSTRIM_PROG -l $(($mmlen*2))k -m ${mmlen}k $SCRATCH_MNT &> /dev/null && break
> > +	done
> > +	test $mmlen -le $FSTRIM_MAX_MINLEN || \
> > +		_notrun "could not determine minimum FSTRIM minlen param"
> > +	FSTRIM_MIN_MINLEN=$mmlen
> > +}
> > +
> > +# Set FSTRIM_{MIN,MAX}_LEN to the lower and upper bounds of the -l(ength)
> > +# parameter to fstrim on the scratch filesystem.
> > +set_length_constraints()
> > +{
> > +	local mmlen
> > +
> > +	for ((mmlen = 100000; mmlen > 0; mmlen /= 2)); do
> > +		$FSTRIM_PROG -l ${mmlen}k $SCRATCH_MNT &> /dev/null && break
> > +	done
> > +	test $mmlen -gt 0 || \
> > +		_notrun "could not determine maximum FSTRIM length param"
> > +	FSTRIM_MAX_LEN=$mmlen
> > +
> > +	for ((mmlen = 1; mmlen < FSTRIM_MAX_LEN; mmlen *= 2)); do
> > +		$FSTRIM_PROG -l ${mmlen}k $SCRATCH_MNT &> /dev/null && break
> > +	done
> > +	test $mmlen -le $FSTRIM_MAX_LEN || \
> > +		_notrun "could not determine minimum FSTRIM length param"
> > +	FSTRIM_MIN_LEN=$mmlen
> >  }
> >  
> >  ##
> > @@ -70,13 +102,24 @@ _guess_max_minlen()
> >  ##
> >  fstrim_loop()
> >  {
> > +	set_minlen_constraints
> > +	set_length_constraints
> > +	echo "MINLEN max=$FSTRIM_MAX_MINLEN min=$FSTRIM_MIN_MINLEN" >> $seqres.full
> > +	echo "LENGTH max=$FSTRIM_MAX_LEN min=$FSTRIM_MIN_LEN" >> $seqres.full
> > +
> >  	trap "_destroy_fstrim; exit \$status" 2 15
> >  	fsize=$(_discard_max_offset_kb "$SCRATCH_MNT" "$SCRATCH_DEV")
> > -	mmlen=$(_guess_max_minlen)
> >  
> >  	while true ; do
> > -		step=$((RANDOM*$RANDOM+4))
> > -		minlen=$(((RANDOM*($RANDOM%2+1))%$mmlen))
> > +		while true; do
> > +			step=$((RANDOM*$RANDOM+4))
> > +			test "$step" -ge "$FSTRIM_MIN_LEN" && break
> > +		done
> > +		while true; do
> > +			minlen=$(( (RANDOM * (RANDOM % 2 + 1)) % FSTRIM_MAX_MINLEN ))
> > +			test "$minlen" -ge "$FSTRIM_MIN_MINLEN" && break
> > +		done
> > +
> >  		start=$RANDOM
> >  		if [ $((RANDOM%10)) -gt 7 ]; then
> >  			$FSTRIM_PROG $SCRATCH_MNT &
> > 
> 



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux