mirror of
https://github.com/linux-apfs/apfstests.git
synced 2026-05-01 15:01:44 -07:00
xfstests: Decomplicate quota setup in 050
Only test 050 uses the _qsetup function, and it only uses the ID part of it in the test. The attempts to link output files is useless as there are no different output files used by the test. Hence kill the _qsetup function, and move the code into test 050, simplifying it down to the minimum needed. Signed-off-by: Dave Chinner <dchinner@redhat.com> Reviewed-by: Phil White <pwhite@sgi.com> Signed-off-by: Rich Johnston <rjohnston@sgi.com>
This commit is contained in:
committed by
Rich Johnston
parent
e5c7cd83c4
commit
9a3a7a1568
+27
-9
@@ -100,6 +100,25 @@ _filter_and_check_blks()
|
||||
' | _filter_report
|
||||
}
|
||||
|
||||
_qsetup()
|
||||
{
|
||||
opt=$1
|
||||
enforce=0
|
||||
if [ $opt = "u" -o $opt = "uno" ]; then
|
||||
type=u
|
||||
eval `_choose_uid`
|
||||
elif [ $opt = "g" -o $opt = "gno" ]; then
|
||||
type=g
|
||||
eval `_choose_gid`
|
||||
elif [ $opt = "p" -o $opt = "pno" ]; then
|
||||
type=p
|
||||
eval `_choose_prid`
|
||||
fi
|
||||
[ $opt = "u" -o $opt = "g" -o $opt = "p" ] && enforce=1
|
||||
|
||||
echo "Using type=$type id=$id" >> $seqres.full
|
||||
}
|
||||
|
||||
_exercise()
|
||||
{
|
||||
_scratch_mkfs_xfs | _filter_mkfs 2>$tmp.mkfs
|
||||
@@ -118,10 +137,9 @@ _exercise()
|
||||
[ $extsize -ge 512000 ] && \
|
||||
_notrun "Extent size hint is too large ($extsize bytes)"
|
||||
|
||||
_qsetup
|
||||
_qsetup $1
|
||||
|
||||
echo "Using output from '" `ls -l $seq.out` "'" >>$seqres.full
|
||||
echo "and using type=$type id=$id" >>$seqres.full
|
||||
echo "Using type=$type id=$id" >>$seqres.full
|
||||
|
||||
echo
|
||||
echo "*** report no quota settings" | tee -a $seqres.full
|
||||
@@ -206,11 +224,11 @@ then
|
||||
else
|
||||
_qmount_option "uquota"
|
||||
fi
|
||||
_exercise
|
||||
_exercise u
|
||||
|
||||
echo "*** group"
|
||||
_qmount_option "gquota"
|
||||
_exercise
|
||||
_exercise g
|
||||
|
||||
echo "*** uqnoenforce"
|
||||
if [ "$HOSTOS" == "IRIX" ]
|
||||
@@ -219,19 +237,19 @@ then
|
||||
else
|
||||
_qmount_option "uqnoenforce"
|
||||
fi
|
||||
_exercise
|
||||
_exercise uno
|
||||
|
||||
echo "*** gqnoenforce"
|
||||
_qmount_option "gqnoenforce"
|
||||
_exercise
|
||||
_exercise gno
|
||||
|
||||
echo "*** pquota"
|
||||
_qmount_option "pquota"
|
||||
_exercise
|
||||
_exercise p
|
||||
|
||||
echo "*** pqnoenforce"
|
||||
_qmount_option "pqnoenforce"
|
||||
_exercise
|
||||
_exercise pno
|
||||
|
||||
# success, all done
|
||||
status=0
|
||||
|
||||
Reference in New Issue
Block a user