On Fri, Dec 30, 2022 at 02:19:33PM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > Since we're moving away from freezing the filesystem for rmap repair, > remove the freeze/thaw race from this test to make it more interesting. > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> > --- Looks good to me, Reviewed-by: Zorro Lang <zlang@xxxxxxxxxx> > tests/xfs/422 | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > > diff --git a/tests/xfs/422 b/tests/xfs/422 > index 995f612166..339f12976a 100755 > --- a/tests/xfs/422 > +++ b/tests/xfs/422 > @@ -5,8 +5,6 @@ > # FS QA Test No. 422 > # > # Race fsstress and rmapbt repair for a while to see if we crash or livelock. > -# rmapbt repair requires us to freeze the filesystem to stop all filesystem > -# activity, so we can't have userspace wandering in and thawing it. > # > . ./common/preamble > _begin_fstest online_repair dangerous_fsstress_repair freeze > @@ -31,7 +29,7 @@ _require_xfs_stress_online_repair > _scratch_mkfs > "$seqres.full" 2>&1 > _scratch_mount > _require_xfs_has_feature "$SCRATCH_MNT" rmapbt > -_scratch_xfs_stress_online_repair -f -s "repair rmapbt %agno%" > +_scratch_xfs_stress_online_repair -s "repair rmapbt %agno%" > > # success, all done > echo Silence is golden >
