On Wed, Mar 13, 2024 at 11:45 PM Boris Burkov <boris@xxxxxx> wrote: > > btrfs/316 is broken on the squota configuration because it uses a raw > rescan call which fails, instead of using the rescan wrapper. The test > passes with squota, so run it (instead of requiring rescan) though I > suspect it isn't the most meaningful test. > > Signed-off-by: Boris Burkov <boris@xxxxxx> > --- > tests/btrfs/316 | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/btrfs/316 b/tests/btrfs/316 > index 07a94334a..36fcad7f8 100755 > --- a/tests/btrfs/316 > +++ b/tests/btrfs/316 > @@ -24,7 +24,8 @@ _scratch_mkfs >> $seqres.full > _scratch_mount > > $BTRFS_UTIL_PROG quota enable $SCRATCH_MNT > -$BTRFS_UTIL_PROG quota rescan -w $SCRATCH_MNT >> $seqres.full > +#$BTRFS_UTIL_PROG quota rescan -w $SCRATCH_MNT >> $seqres.full Why the comment and not remove the line? Seems like unintentional. > +_qgroup_rescan $SCRATCH_MNT >> $seqres.full With that fixed: Reviewed-by: Filipe Manana <fdmanana@xxxxxxxx> > > $BTRFS_UTIL_PROG qgroup create 1/0 $SCRATCH_MNT >> $seqres.full > $BTRFS_UTIL_PROG subvolume create $SCRATCH_MNT/subv1 >> $seqres.full > -- > 2.43.0 > >
