btrfs: inline enable_quota helper in test 301

The enable_quota helper in btrfs/301 takes a mode argument ('n', 's',
or anything else) to switch between "no quota", "simple quota", and
"full quota" enable.  Only the simple-quota path is ever exercised by
this test, so the dispatch is dead weight and the indirection just
obscures the actual btrfs command being run.

Inline the two call sites to call 'btrfs quota enable --simple'
directly and drop the helper.

Signed-off-by: Boris Burkov <boris@bur.io>
Reviewed-by: Anand Jain <asj@kernel.org>
Signed-off-by: Zorro Lang <zlang@kernel.org>

diff --git a/tests/btrfs/301 b/tests/btrfs/301
index 1f72a97b31bcf3e99237397518e45a94d1800277..31243aab02e4fa4d330619a5a89a3c7ac563dbec 100755 (executable)
--- a/tests/btrfs/301
+++ b/tests/btrfs/301
@@ -169,16 +169,6 @@ do_enospc_falloc()
        do_falloc $file $sz
 }
 
-enable_quota()
-{
-       local mode=$1
-
-       [ $mode == "n" ] && return
-       arg=$([ $mode == "s" ] && echo "--simple")
-
-       $BTRFS_UTIL_PROG quota enable $arg $SCRATCH_MNT
-}
-
 get_subvid()
 {
        _btrfs_get_subvolid $SCRATCH_MNT subv
@@ -198,7 +188,7 @@ prepare()
 {
        _scratch_mkfs >> $seqres.full
        _scratch_mount
-       enable_quota "s"
+       $BTRFS_UTIL_PROG quota enable --simple $SCRATCH_MNT
        $BTRFS_UTIL_PROG subvolume create $subv >> $seqres.full
        local subvid=$(get_subvid)
        set_subvol_limit $subvid $limit
@@ -421,7 +411,7 @@ enable_mature()
        # Sync before enabling squotas to reliably *not* count the writes
        # we did before enabling.
        sync
-       enable_quota "s"
+       $BTRFS_UTIL_PROG quota enable --simple $SCRATCH_MNT
        set_subvol_limit $subvid $limit
        _scratch_cycle_mount
        usage=$(get_subvol_usage $subvid)