Re: [PATCH 27/34] fuzzy: port fsx and fsstress loop to use --duration

["Darrick J. Wong" <djwong@kernel.org>]

On Wed, Feb 05, 2025 at 11:50:52AM +1100, Dave Chinner wrote:
> On Tue, Feb 04, 2025 at 01:29:20PM -0800, Darrick J. Wong wrote:
> > From: Darrick J. Wong <djwong@kernel.org>
> > 
> > Quite a while ago, I added --duration= arguments to fsx and fsstress,
> > and apparently I forgot to update the scrub stress loops to use them.
> > Replace the usage of timeout(1) for the remount_period versions of the
> > loop to clean up that code; and convert the non-remount loop so that
> > they don't run over time.
> 
> ....
> 
> > @@ -1115,7 +1124,8 @@ __stress_scrub_fsstress_loop() {
> >  			# anything.
> >  			test "$mode" = "rw" && __stress_scrub_clean_scratch && continue
> >  
> > -			_run_fsstress_bg $args $rw_arg >> $seqres.full
> > +			duration=$(___stress_scrub_duration "$end" "$remount_period")
> > +			_run_fsstress_bg $duration $args $rw_arg >> $seqres.full
> >  			sleep $remount_period
> >  			_kill_fsstress
> 
> Why does this need to run fsstress in the background any more? If it
> is only going to run for $remount_period, then run it in the
> foreground and get rid of the sleep/kill that stopped it after
> $remount_period. i.e. doesn't this:
> 
> -			_run_fsstress_bg $args $rw_arg >> $seqres.full
> +			duration=$(___stress_scrub_duration "$end" "$remount_period")
> +			_run_fsstress $duration $args $rw_arg >> $seqres.full
> -			sleep $remount_period
> -			_kill_fsstress
> 
> do the same thing, only cleaner?

It does, I was probably just moving too fast with the mechanical changes
when I wrote this.  Will fix for the next draft.

--D

> 
> -Dave.
> -- 
> Dave Chinner
> david@fromorbit.com
>