From 27d077ec0bdaaffd69e436236ec580bfb9f53385 Mon Sep 17 00:00:00 2001 From: Eric Sandeen Date: Mon, 21 Dec 2015 18:07:43 +1100 Subject: [PATCH] common: use mount/umount helpers everywhere Replace every explicit mount/umount of scratch or test devices with helper functions. This allows the next patch to add in hooks to these functions in order to set up & tear down overlayfs on every mount/umount (also adds _test_unmount(), which didn't exist prior) [Eryu Guan rebased the patch agains latest master and replaced more mount/umount with helpers] Signed-off-by: Eryu Guan Signed-off-by: Eric Sandeen Reviewed-by: Dave Chinner Signed-off-by: Dave Chinner --- check | 10 +++++----- common/attr | 2 +- common/dump | 2 +- common/filestreams | 2 +- common/log | 6 +++--- common/punch | 2 +- common/quota | 2 +- common/rc | 27 ++++++++++++++++----------- common/repair | 4 ++-- soak | 6 +++--- tests/btrfs/002 | 2 +- tests/btrfs/003 | 16 ++++++++-------- tests/btrfs/004 | 6 +++--- tests/btrfs/005 | 4 ++-- tests/btrfs/007 | 6 +++--- tests/btrfs/011 | 6 +++--- tests/btrfs/029 | 4 ++-- tests/btrfs/031 | 2 +- tests/ext4/002 | 2 +- tests/ext4/301 | 2 +- tests/ext4/302 | 2 +- tests/ext4/303 | 2 +- tests/ext4/304 | 2 +- tests/ext4/305 | 4 ++-- tests/generic/014 | 2 +- tests/generic/015 | 2 +- tests/generic/019 | 2 +- tests/generic/043 | 4 ++-- tests/generic/044 | 4 ++-- tests/generic/045 | 4 ++-- tests/generic/046 | 4 ++-- tests/generic/047 | 4 ++-- tests/generic/048 | 4 ++-- tests/generic/049 | 4 ++-- tests/generic/050 | 8 ++++---- tests/generic/050.out | 2 +- tests/generic/051 | 10 +++++----- tests/generic/052 | 4 ++-- tests/generic/053 | 2 +- tests/generic/054 | 6 +++--- tests/generic/055 | 6 +++--- tests/generic/062 | 2 +- tests/generic/067 | 2 +- tests/generic/068 | 2 +- tests/generic/069 | 2 +- tests/generic/076 | 4 ++-- tests/generic/077 | 4 ++-- tests/generic/079 | 2 +- tests/generic/083 | 4 ++-- tests/generic/105 | 2 +- tests/generic/117 | 4 ++-- tests/generic/120 | 2 +- tests/generic/124 | 2 +- tests/generic/128 | 2 +- tests/generic/135 | 2 +- tests/generic/157 | 2 +- tests/generic/158 | 2 +- tests/generic/166 | 2 +- tests/generic/167 | 2 +- tests/generic/169 | 4 ++-- tests/generic/192 | 2 +- tests/generic/219 | 8 ++++---- tests/generic/226 | 6 +++--- tests/generic/230 | 6 +++--- tests/generic/231 | 14 +++++++------- tests/generic/232 | 6 +++--- tests/generic/233 | 6 +++--- tests/generic/234 | 2 +- tests/generic/235 | 6 +++--- tests/generic/256 | 2 +- tests/generic/258 | 2 +- tests/generic/269 | 2 +- tests/generic/274 | 2 +- tests/generic/275 | 2 +- tests/generic/280 | 4 ++-- tests/generic/299 | 2 +- tests/generic/300 | 2 +- tests/generic/317 | 8 ++++---- tests/generic/318 | 8 ++++---- tests/shared/289 | 4 ++-- tests/udf/098 | 6 +++--- tests/udf/101 | 2 +- tests/xfs/004 | 4 ++-- tests/xfs/006 | 2 +- tests/xfs/007 | 4 ++-- tests/xfs/009 | 2 +- tests/xfs/010 | 4 ++-- tests/xfs/011 | 4 ++-- tests/xfs/013 | 4 ++-- tests/xfs/014 | 4 ++-- tests/xfs/016 | 4 ++-- tests/xfs/017 | 4 ++-- tests/xfs/018 | 4 ++-- tests/xfs/019 | 6 +++--- tests/xfs/021 | 6 +++--- tests/xfs/030 | 4 ++-- tests/xfs/033 | 4 ++-- tests/xfs/034 | 4 ++-- tests/xfs/041 | 6 +++--- tests/xfs/042 | 4 ++-- tests/xfs/044 | 2 +- tests/xfs/050 | 4 ++-- tests/xfs/052 | 4 ++-- tests/xfs/054 | 6 +++--- tests/xfs/065 | 4 ++-- tests/xfs/071 | 2 +- tests/xfs/072 | 4 ++-- tests/xfs/073 | 4 ++-- tests/xfs/076 | 2 +- tests/xfs/081 | 6 +++--- tests/xfs/082 | 4 ++-- tests/xfs/104 | 2 +- tests/xfs/106 | 2 +- tests/xfs/108 | 10 +++++----- tests/xfs/109 | 6 +++--- tests/xfs/110 | 2 +- tests/xfs/111 | 6 +++--- tests/xfs/116 | 10 +++++----- tests/xfs/121 | 4 ++-- tests/xfs/134 | 2 +- tests/xfs/136 | 6 +++--- tests/xfs/148 | 4 ++-- tests/xfs/175 | 2 +- tests/xfs/176 | 2 +- tests/xfs/177 | 4 ++-- tests/xfs/181 | 4 ++-- tests/xfs/189 | 12 ++++++------ tests/xfs/196 | 2 +- tests/xfs/199 | 6 +++--- tests/xfs/201 | 2 +- tests/xfs/220 | 6 +++--- tests/xfs/227 | 4 ++-- tests/xfs/244 | 4 ++-- tests/xfs/287 | 6 +++--- tests/xfs/299 | 6 +++--- tests/xfs/304 | 6 +++--- 136 files changed, 293 insertions(+), 288 deletions(-) diff --git a/check b/check index 8281322e..f0f2cdcb 100755 --- a/check +++ b/check @@ -439,7 +439,7 @@ for section in $HOST_OPTIONS_SECTIONS; do if $RECREATE_TEST_DEV || [ "$OLD_FSTYP" != "$FSTYP" ]; then echo "RECREATING -- $FSTYP on $TEST_DEV" - umount $TEST_DEV 2> /dev/null + _test_unmount 2> /dev/null if ! _test_mkfs >$tmp.err 2>&1 then echo "our local _test_mkfs routine ..." @@ -456,7 +456,7 @@ for section in $HOST_OPTIONS_SECTIONS; do fi _prepare_test_list elif [ "$OLD_MOUNT_OPTIONS" != "$MOUNT_OPTIONS" ]; then - umount $TEST_DEV 2> /dev/null + _test_unmount 2> /dev/null out=`_mount_or_remount_rw "$MOUNT_OPTIONS" $TEST_DEV $TEST_DIR` if [ $? -ne 1 ]; then echo $out @@ -486,7 +486,7 @@ for section in $HOST_OPTIONS_SECTIONS; do needwrap=true if [ ! -z "$SCRATCH_DEV" ]; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2> /dev/null # call the overridden mkfs - make sure the FS is built # the same as we'll create it later. @@ -679,8 +679,8 @@ for section in $HOST_OPTIONS_SECTIONS; do _wrapup echo - umount $TEST_DEV 2> /dev/null - umount $SCRATCH_DEV 2> /dev/null + _test_unmount 2> /dev/null + _scratch_unmount 2> /dev/null done interrupt=false diff --git a/common/attr b/common/attr index cc9cfdac..86fe1818 100644 --- a/common/attr +++ b/common/attr @@ -233,7 +233,7 @@ _require_noattr2() { _scratch_mkfs_xfs > /dev/null 2>&1 \ || _fail "_scratch_mkfs_xfs failed on $SCRATCH_DEV" - _mount -o noattr2 $SCRATCH_DEV > /dev/null 2>&1 \ + _scratch_mount -o noattr2 > /dev/null 2>&1 \ || _notrun "noattr2 mount option not supported on $SCRATCH_DEV" _scratch_unmount } diff --git a/common/dump b/common/dump index c42eb2c8..2db73789 100644 --- a/common/dump +++ b/common/dump @@ -288,7 +288,7 @@ _cleanup() _stable_fs() { _saveddir=`pwd`; cd / - umount $SCRATCH_MNT >>$seqres.full || _fail "unmount failed" + _scratch_unmount >>$seqres.full || _fail "unmount failed" _scratch_mount >>$seqres.full || _fail "mount failed" cd $_saveddir } diff --git a/common/filestreams b/common/filestreams index 590fa7f1..4d146648 100644 --- a/common/filestreams +++ b/common/filestreams @@ -225,5 +225,5 @@ _cleanup_streams_umount() { cd / rm -rf ${SCRATCH_MNT}/stream* - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null } diff --git a/common/log b/common/log index 5be5411a..cb687d2c 100644 --- a/common/log +++ b/common/log @@ -339,7 +339,7 @@ _create_log() # unmount the FS _full "umount" - umount $SCRATCH_DEV >>$seqres.full 2>&1 + _scratch_unmount >>$seqres.full 2>&1 if [ $? -ne 0 ] ; then _echofull "umount failed" return 1 @@ -370,7 +370,7 @@ _create_log_sync() # unmount the FS _full "umount" - umount $SCRATCH_DEV >>$seqres.full 2>&1 + _scratch_unmount >>$seqres.full 2>&1 if [ $? -ne 0 ] ; then _echofull "umount failed" return 1 @@ -482,7 +482,7 @@ _require_v2log() # check after unmount to see if it is clean # i.e. it is not a 6.5.25 buggy version checking kernel touch $SCRATCH_MNT/file - umount $SCRATCH_DEV >>$seqres.full 2>&1 + _scratch_unmount >>$seqres.full 2>&1 if _scratch_xfs_logprint -t | tee -a $seqres.full \ | head | grep -q ""; then _notrun "kernel does not support v2 logs" diff --git a/common/punch b/common/punch index be193d67..d9b8426f 100644 --- a/common/punch +++ b/common/punch @@ -115,7 +115,7 @@ _test_punch() { local filename=punch_test_file cd / - umount $SCRATCH_MNT >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 _scratch_mkfs_xfs -bsize=$blksize >/dev/null 2>&1 \ || _fail "mkfs failed" diff --git a/common/quota b/common/quota index 658b1c39..32777a91 100644 --- a/common/quota +++ b/common/quota @@ -167,7 +167,7 @@ _choose_prid() _qmount() { - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 _scratch_mount || _fail "qmount failed" chmod ugo+rwx $SCRATCH_MNT } diff --git a/common/rc b/common/rc index 2b39b0b1..5b9fbdc8 100644 --- a/common/rc +++ b/common/rc @@ -270,9 +270,14 @@ _test_mount() _mount -t $FSTYP $TEST_OPTIONS $TEST_FS_MOUNT_OPTS $SELINUX_MOUNT_OPTIONS $* $TEST_DEV $TEST_DIR } +_test_unmount() +{ + $UMOUNT_PROG $TEST_DEV +} + _test_remount() { - $UMOUNT_PROG $TEST_DEV + _test_unmount _test_mount } @@ -326,7 +331,7 @@ _setup_large_xfs_fs() $SCRATCH_MNT/.use_space 2>&1 > /dev/null export NUM_SPACE_FILES=1 status=$? - umount $SCRATCH_MNT + _scratch_unmount if [ $status -ne 0 ]; then echo "large file prealloc failed" cat $tmp_dir/mnt.err >&2 @@ -501,7 +506,7 @@ _setup_large_ext4_fs() fi export NUM_SPACE_FILES=$nfiles - umount $SCRATCH_MNT + _scratch_unmount if [ $status -ne 0 ]; then echo "large file prealloc failed" cat $tmp_dir/mnt.err >&2 @@ -1178,7 +1183,7 @@ _require_scratch_nocheck() exit 1 fi # and then unmount it - if ! $UMOUNT_PROG $SCRATCH_DEV + if ! _scratch_unmount then echo "failed to unmount $SCRATCH_DEV" exit 1 @@ -1422,7 +1427,7 @@ _require_xfs_crc() _scratch_mkfs_xfs -m crc=1 >/dev/null 2>&1 _scratch_mount >/dev/null 2>&1 \ || _notrun "Kernel doesn't support crc feature" - umount $SCRATCH_MNT + _scratch_unmount } # this test requires the ext4 kernel support crc feature on scratch device @@ -1433,7 +1438,7 @@ _require_scratch_ext4_crc() dumpe2fs -h $SCRATCH_DEV 2> /dev/null | grep -q metadata_csum || _notrun "metadata_csum not supported by this filesystem" _scratch_mount >/dev/null 2>&1 \ || _notrun "Kernel doesn't support metadata_csum feature" - umount $SCRATCH_MNT + _scratch_unmount } # this test requires the xfs kernel support crc feature on scratch device @@ -1444,7 +1449,7 @@ _require_scratch_xfs_crc() _scratch_mount >/dev/null 2>&1 \ || _notrun "Kernel doesn't support crc feature" xfs_info $SCRATCH_MNT | grep -q 'crc=1' || _notrun "crc feature not supported by this filesystem" - umount $SCRATCH_MNT + _scratch_unmount } # this test requires the bigalloc feature to be available in mkfs.ext4 @@ -1462,7 +1467,7 @@ _require_ext4_bigalloc() $MKFS_EXT4_PROG -F -O bigalloc $SCRATCH_DEV 512m >/dev/null 2>&1 _scratch_mount >/dev/null 2>&1 \ || _notrun "Ext4 kernel doesn't support bigalloc feature" - umount $SCRATCH_MNT + _scratch_unmount } # this test requires the finobt feature to be available in mkfs.xfs @@ -1480,7 +1485,7 @@ _require_xfs_finobt() _scratch_mkfs_xfs -m crc=1,finobt=1 >/dev/null 2>&1 _scratch_mount >/dev/null 2>&1 \ || _notrun "Kernel doesn't support finobt feature" - umount $SCRATCH_MNT + _scratch_unmount } # this test requires xfs sysfs attribute support @@ -1508,7 +1513,7 @@ _require_xfs_sparse_inodes() _scratch_mkfs_xfs -m crc=1 -i sparse > /dev/null 2>&1 _scratch_mount >/dev/null 2>&1 \ || _notrun "kernel does not support sparse inodes" - umount $SCRATCH_MNT + _scratch_unmount } # this test requires that external log/realtime devices are not in use @@ -2824,7 +2829,7 @@ _require_meta_uuid() _scratch_mount >/dev/null 2>&1 \ || _notrun "Kernel doesn't support meta_uuid feature" - umount $SCRATCH_MNT + _scratch_unmount } _require_btrfs_dev_del_by_devid() diff --git a/common/repair b/common/repair index ce6d2588..537d943f 100644 --- a/common/repair +++ b/common/repair @@ -114,7 +114,7 @@ _check_repair() dd if=/bin/bash of=$SCRATCH_MNT/sh 2>&1 |_filter_dd sync rm -f $SCRATCH_MNT/sh - umount $SCRATCH_MNT + _scratch_unmount _zero_position $value "$structure" _scratch_xfs_repair 2>&1 | _filter_repair @@ -127,7 +127,7 @@ _check_repair() POSIXLY_CORRECT=yes \ dd if=$SCRATCH_MNT/sh of=/dev/null 2>&1 |_filter_dd #read rm -f $SCRATCH_MNT/sh #unlink - umount $SCRATCH_MNT #umount + _scratch_unmount #umount } # make sure this script returns success diff --git a/soak b/soak index 7e9eea06..ec4fbb53 100755 --- a/soak +++ b/soak @@ -29,7 +29,7 @@ status=1 # failure is the default! _cleanup() { echo " *** umount" - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -85,7 +85,7 @@ _log "*** (`date`)" _log " *** init" _log " *** unmounting scratch device" -umount $SCRATCH_DEV 2>&1 | _fix_malloc >>$FULL +_scratch_unmount 2>&1 | _fix_malloc >>$FULL _log " *** clean scratch device" @@ -128,7 +128,7 @@ do _log " *** unmounting scratch device" - umount $SCRATCH_DEV 2>&1 | _logp \ + _scratch_unmount 2>&1 | _logp \ || _fail " !!! failed to umount" let "pass = pass + 1" diff --git a/tests/btrfs/002 b/tests/btrfs/002 index 19a62c6a..35e857b0 100755 --- a/tests/btrfs/002 +++ b/tests/btrfs/002 @@ -185,7 +185,7 @@ _save_checksum $SNAPNAME $tmp.$tname.sum \rm -rf $firstvol/* _verify_checksum $SNAPNAME $tmp.$tname.sum -umount $SCRATCH_DEV || _fail "unmount failed" +_scratch_unmount || _fail "unmount failed" echo "Silence is golden" status=0; exit diff --git a/tests/btrfs/003 b/tests/btrfs/003 index 9b60ea55..353cb48d 100755 --- a/tests/btrfs/003 +++ b/tests/btrfs/003 @@ -36,7 +36,7 @@ _cleanup() cd / rm -f $tmp.* if [ $dev_removed == 1 ]; then - umount $SCRATCH_MNT + _scratch_unmount _devmgt_add "${DEVHTL}" fi } @@ -63,7 +63,7 @@ _test_raid0() _scratch_mount dirp=`mktemp -duq $SCRATCH_MNT/dir.XXXXXX` _populate_fs -n 1 -f 20 -d 10 -r $dirp -s 10 - umount $SCRATCH_MNT + _scratch_unmount } _test_raid1() @@ -73,7 +73,7 @@ _test_raid1() _scratch_mount dirp=`mktemp -duq $SCRATCH_MNT/dir.XXXXXX` _populate_fs -n 1 -f 20 -d 10 -r $dirp -s 10 - umount $SCRATCH_MNT + _scratch_unmount } _test_raid10() @@ -83,7 +83,7 @@ _test_raid10() _scratch_mount dirp=`mktemp -duq $SCRATCH_MNT/dir.XXXXXX` _populate_fs -n 1 -f 20 -d 10 -r $dirp -s 10 - umount $SCRATCH_MNT + _scratch_unmount } _test_single() @@ -93,7 +93,7 @@ _test_single() _scratch_mount dirp=`mktemp -duq $SCRATCH_MNT/dir.XXXXXX` _populate_fs -n 1 -f 20 -d 10 -r $dirp -s 10 - umount $SCRATCH_MNT + _scratch_unmount } _test_add() @@ -115,7 +115,7 @@ _test_add() $BTRFS_UTIL_PROG device add ${devs[$i]} $SCRATCH_MNT >> $seqres.full 2>&1 || _fail "device add failed" done $BTRFS_UTIL_PROG filesystem balance $SCRATCH_MNT >> $seqres.full 2>&1 || _fail "balance failed" - umount $SCRATCH_MNT + _scratch_unmount } _test_replace() @@ -161,7 +161,7 @@ _test_replace() $BTRFS_UTIL_PROG filesystem balance $SCRATCH_MNT >> $seqres.full 2>&1 || _fail "dev balance failed" # cleaup. add the removed disk - umount $SCRATCH_MNT + _scratch_unmount _devmgt_add "${DEVHTL}" dev_removed=0 } @@ -177,7 +177,7 @@ _test_remove() dev_del=`echo ${SCRATCH_DEV_POOL} | awk '{print $NF}'` $BTRFS_UTIL_PROG device delete $dev_del $SCRATCH_MNT || _fail "btrfs device delete failed" $BTRFS_UTIL_PROG filesystem show $SCRATCH_DEV 2>&1 | grep $dev_del >> $seqres.full && _fail "btrfs still shows the deleted dev" - umount $SCRATCH_MNT + _scratch_unmount } _test_raid0 diff --git a/tests/btrfs/004 b/tests/btrfs/004 index b5489140..d588c5ba 100755 --- a/tests/btrfs/004 +++ b/tests/btrfs/004 @@ -180,7 +180,7 @@ workout() snap_name=$4 do_bg_noise=$5 - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 echo "*** mkfs -dsize=$fsz" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs_sized $fsz >>$seqres.full 2>&1 \ @@ -193,7 +193,7 @@ workout() _run_btrfs_util_prog subvolume snapshot $SCRATCH_MNT \ $SCRATCH_MNT/$snap_name - run_check umount $SCRATCH_DEV >/dev/null 2>&1 + run_check _scratch_unmount >/dev/null 2>&1 run_check _scratch_mount "-o compress=lzo" # make some noise but ensure we're not touching existing data @@ -207,7 +207,7 @@ workout() # now make more files to get a higher tree run_check $FSSTRESS_PROG -d $clean_dir -w -p $procs -n 2000 \ $FSSTRESS_AVOID - run_check umount $SCRATCH_DEV >/dev/null 2>&1 + run_check _scratch_unmount >/dev/null 2>&1 run_check _scratch_mount "-o atime" if [ $do_bg_noise -ne 0 ]; then diff --git a/tests/btrfs/005 b/tests/btrfs/005 index 76396358..c048ab57 100755 --- a/tests/btrfs/005 +++ b/tests/btrfs/005 @@ -106,12 +106,12 @@ _checksum() _cleanup_defrag() { - umount $SCRATCH_MNT > /dev/null 2>&1 + _scratch_unmount > /dev/null 2>&1 } _setup_defrag() { - umount $SCRATCH_MNT > /dev/null 2>&1 + _scratch_unmount > /dev/null 2>&1 _scratch_mkfs > /dev/null 2>&1 _scratch_mount _create_file $1 diff --git a/tests/btrfs/007 b/tests/btrfs/007 index f7f74235..82d093aa 100755 --- a/tests/btrfs/007 +++ b/tests/btrfs/007 @@ -37,7 +37,7 @@ status=1 _cleanup() { echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -61,7 +61,7 @@ workout() fsz=$1 ops=$2 - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 echo "*** mkfs -dsize=$fsz" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs_sized $fsz >>$seqres.full 2>&1 \ @@ -87,7 +87,7 @@ workout() run_check $FSSUM_PROG -A -f -w $tmp/incr.fssum -x $SCRATCH_MNT/incr/base \ $SCRATCH_MNT/incr - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 echo "*** mkfs -dsize=$fsz" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs_sized $fsz >>$seqres.full 2>&1 \ diff --git a/tests/btrfs/011 b/tests/btrfs/011 index c7d35fa4..72c53abf 100755 --- a/tests/btrfs/011 +++ b/tests/btrfs/011 @@ -151,7 +151,7 @@ workout() sync; sync btrfs_replace_test $source_dev $target_dev "" $with_cancel $quick - umount $SCRATCH_MNT > /dev/null 2>&1 + _scratch_unmount > /dev/null 2>&1 if echo $mkfs_options | egrep -qv "raid1|raid5|raid6|raid10" || \ [ "${with_cancel}Q" = "cancelQ" ]; then @@ -201,7 +201,7 @@ workout() fi btrfs_replace_test $source_dev $target_dev "-r" $with_cancel $quick - umount $SCRATCH_MNT > /dev/null 2>&1 + _scratch_unmount > /dev/null 2>&1 } btrfs_replace_test() @@ -264,7 +264,7 @@ btrfs_replace_test() # because in /proc/mounts the 2nd device of the filesystem is # shown after the replace operation. Let's just do the mount # test manually after _check_btrfs_filesystem is finished. - umount $SCRATCH_MNT > /dev/null 2>&1 + _scratch_unmount > /dev/null 2>&1 if [ "${with_cancel}Q" != "cancelQ" ]; then # after the replace operation, use the target_dev for everything _check_btrfs_filesystem $target_dev diff --git a/tests/btrfs/029 b/tests/btrfs/029 index 175317a8..cdce6e1b 100755 --- a/tests/btrfs/029 +++ b/tests/btrfs/029 @@ -39,7 +39,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15 _cleanup() { - umount $SCRATCH_MNT &>/dev/null + _scratch_unmount &>/dev/null cd / rm -f $tmp.* } @@ -104,7 +104,7 @@ _scratch_unmount echo "test reflinks across different mountpoints of same device" mount $TEST_DEV $SCRATCH_MNT || _fail "Couldn't double-mount $TEST_DEV" _create_reflinks_to $DUAL_MOUNT_DIR -umount $SCRATCH_MNT +_scratch_unmount # success, all done status=0 diff --git a/tests/btrfs/031 b/tests/btrfs/031 index c5763dab..0159c956 100755 --- a/tests/btrfs/031 +++ b/tests/btrfs/031 @@ -36,7 +36,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15 _cleanup() { - umount $SCRATCH_MNT + _scratch_unmount rm -rf $TESTDIR1 rm -rf $TESTDIR2 $BTRFS_UTIL_PROG subvolume delete $SUBVOL1 >> $seqres.full diff --git a/tests/ext4/002 b/tests/ext4/002 index 035ea7b5..255a18f8 100755 --- a/tests/ext4/002 +++ b/tests/ext4/002 @@ -78,7 +78,7 @@ _check_ext4_eof_flag() bit_set=1 # Unmount the ${TEST_DEV} - umount ${TEST_DEV} + _test_unmount # Run debugfs to gather file_parameters - specifically iflags. file_params=`debugfs ${TEST_DEV} -R "stat ${1}" 2>&1 | grep -e Flags:` diff --git a/tests/ext4/301 b/tests/ext4/301 index 568327d4..d6547c57 100755 --- a/tests/ext4/301 +++ b/tests/ext4/301 @@ -112,7 +112,7 @@ _scratch_mkfs >> $seqres.full 2>&1 _scratch_mount if ! _workout; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi diff --git a/tests/ext4/302 b/tests/ext4/302 index c604c8f6..45cd28c6 100755 --- a/tests/ext4/302 +++ b/tests/ext4/302 @@ -130,7 +130,7 @@ _scratch_mkfs >> $seqres.full 2>&1 _scratch_mount if ! _workout; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi diff --git a/tests/ext4/303 b/tests/ext4/303 index 2d01f0f0..64a64055 100755 --- a/tests/ext4/303 +++ b/tests/ext4/303 @@ -140,7 +140,7 @@ _scratch_mkfs >> $seqres.full 2>&1 _scratch_mount if ! _workout; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi diff --git a/tests/ext4/304 b/tests/ext4/304 index 01c51a40..cc03190a 100755 --- a/tests/ext4/304 +++ b/tests/ext4/304 @@ -117,7 +117,7 @@ _scratch_mkfs >> $seqres.full 2>&1 _scratch_mount if ! _workout; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi diff --git a/tests/ext4/305 b/tests/ext4/305 index 1711aae8..09db2e11 100755 --- a/tests/ext4/305 +++ b/tests/ext4/305 @@ -61,8 +61,8 @@ PIDS=$! # 30 loops is enough to crash a buggy kernel i=0 while [ $i -lt 30 ]; do - mount $SCRATCH_DEV $SCRATCH_MNT - umount $SCRATCH_DEV + _scratch_mount + _scratch_unmount let i=i+1 done diff --git a/tests/generic/014 b/tests/generic/014 index a144a474..d2dcc282 100755 --- a/tests/generic/014 +++ b/tests/generic/014 @@ -52,7 +52,7 @@ _require_sparse_files # by limiting the amount of preallocation and therefore the amount of blocks # zeroed during the truncfile test run. if [ "$FSTYP" == "xfs" ]; then - umount $TEST_DIR + _test_unmount _test_mount -o allocsize=64k fi diff --git a/tests/generic/015 b/tests/generic/015 index 8d3fd5f1..66c33c22 100755 --- a/tests/generic/015 +++ b/tests/generic/015 @@ -36,7 +36,7 @@ status=1 # success is the default! _cleanup() { - umount $SCRATCH_MNT + _scratch_unmount } trap "_cleanup; exit \$status" 0 1 2 3 15 diff --git a/tests/generic/019 b/tests/generic/019 index 76a09d5e..1137ffe9 100755 --- a/tests/generic/019 +++ b/tests/generic/019 @@ -160,7 +160,7 @@ _workout() wait $fio_pid # We expect that broken FS still can be umounted - run_check umount $SCRATCH_DEV + run_check _scratch_unmount # Once filesystem was umounted no one is able to write to block device # It is now safe to bring device back to normal state stop_fail_scratch_dev diff --git a/tests/generic/043 b/tests/generic/043 index 47bc7662..bd8eef8f 100755 --- a/tests/generic/043 +++ b/tests/generic/043 @@ -64,9 +64,9 @@ sleep 10 src/godown $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount if [ ! _check_scratch_fs ] then echo error detected in filesystem diff --git a/tests/generic/044 b/tests/generic/044 index c59aaf35..f46e8289 100755 --- a/tests/generic/044 +++ b/tests/generic/044 @@ -70,9 +70,9 @@ sleep 10 src/godown $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount if [ ! _check_scratch_fs ] then echo error detected in filesystem diff --git a/tests/generic/045 b/tests/generic/045 index 6c601a6d..4ec76509 100755 --- a/tests/generic/045 +++ b/tests/generic/045 @@ -70,9 +70,9 @@ sleep 10 src/godown $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount if [ ! _check_scratch_fs ] then echo error detected in filesystem diff --git a/tests/generic/046 b/tests/generic/046 index 76993d40..08f11370 100755 --- a/tests/generic/046 +++ b/tests/generic/046 @@ -70,9 +70,9 @@ sleep 10 src/godown $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount if [ ! _check_scratch_fs ] then echo error detected in filesystem diff --git a/tests/generic/047 b/tests/generic/047 index 0418a3bd..b894ee61 100755 --- a/tests/generic/047 +++ b/tests/generic/047 @@ -92,9 +92,9 @@ done # shutdown immediately after, then remount and test src/godown $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount if [ ! _check_scratch_fs ] then echo error detected in filesystem diff --git a/tests/generic/048 b/tests/generic/048 index 558fe5f6..6f5f4446 100755 --- a/tests/generic/048 +++ b/tests/generic/048 @@ -96,9 +96,9 @@ done # sync, then shutdown immediately after, then remount and test sync src/godown $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount if [ ! _check_scratch_fs ] then echo error detected in filesystem diff --git a/tests/generic/049 b/tests/generic/049 index 646d39e1..320318e6 100755 --- a/tests/generic/049 +++ b/tests/generic/049 @@ -93,9 +93,9 @@ done # sync, then shutdown immediately after, then remount and test sync src/godown $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount if [ ! _check_scratch_fs ] then echo error detected in filesystem diff --git a/tests/generic/050 b/tests/generic/050 index a32e049a..5fa28a76 100755 --- a/tests/generic/050 +++ b/tests/generic/050 @@ -70,7 +70,7 @@ touch $SCRATCH_MNT/foo 2>&1 | _filter_scratch # http://oss.sgi.com/bugzilla/show_bug.cgi?id=807 # echo "unmounting read-only filesystem" -umount $SCRATCH_MNT 2>&1 | _filter_scratch +_scratch_unmount 2>&1 | _filter_scratch echo "setting device read-write" blockdev --setrw $SCRATCH_DEV @@ -85,7 +85,7 @@ echo "going down:" src/godown -f $SCRATCH_MNT echo "unmounting shutdown filesystem:" -umount $SCRATCH_MNT 2>&1 | _filter_scratch +_scratch_unmount 2>&1 | _filter_scratch echo "setting device read-only" blockdev --setro $SCRATCH_DEV @@ -98,7 +98,7 @@ echo "mounting filesystem that needs recovery on a read-only device:" _scratch_mount 2>&1 | _filter_scratch | _filter_ro_mount echo "unmounting read-only filesystem" -umount $SCRATCH_MNT 2>&1 | _filter_scratch +_scratch_unmount 2>&1 | _filter_scratch # # This is the way out if the underlying device really is read-only. @@ -109,7 +109,7 @@ echo "mounting filesystem with -o norecovery on a read-only device:" _scratch_mount -o norecovery 2>&1 | _filter_scratch | _filter_ro_mount echo "unmounting read-only filesystem" -umount $SCRATCH_MNT 2>&1 | _filter_scratch +_scratch_unmount 2>&1 | _filter_scratch echo "setting device read-write" blockdev --setrw $SCRATCH_DEV diff --git a/tests/generic/050.out b/tests/generic/050.out index f8dbb7a4..fb90f6ea 100644 --- a/tests/generic/050.out +++ b/tests/generic/050.out @@ -15,7 +15,7 @@ mounting filesystem that needs recovery on a read-only device: mount: SCRATCH_DEV is write-protected, mounting read-only mount: cannot mount SCRATCH_DEV read-only unmounting read-only filesystem -umount: SCRATCH_MNT: not mounted +umount: SCRATCH_DEV: not mounted mounting filesystem with -o norecovery on a read-only device: mount: SCRATCH_DEV is write-protected, mounting read-only unmounting read-only filesystem diff --git a/tests/generic/051 b/tests/generic/051 index 614939f3..d6208079 100755 --- a/tests/generic/051 +++ b/tests/generic/051 @@ -39,7 +39,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -67,7 +67,7 @@ sleep $SLEEP_TIME killall -q $FSSTRESS_PROG wait sync -umount $SCRATCH_MNT +_scratch_unmount # now mount again, run the load again, this time with a shutdown. _scratch_mount @@ -84,11 +84,11 @@ wait # for some reason fsstress processes manage to live on beyond the wait? sleep 5 -umount $SCRATCH_MNT +_scratch_unmount # now recover, check the filesystem for consistency _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount _check_scratch_fs # now clean up. @@ -97,7 +97,7 @@ for d in $load_dir/*; do rm -rf $d > /dev/null 2>&1 & done wait -umount $SCRATCH_MNT +_scratch_unmount echo "No output is good. Failures are loud." diff --git a/tests/generic/052 b/tests/generic/052 index 3430b655..a81885b4 100755 --- a/tests/generic/052 +++ b/tests/generic/052 @@ -65,7 +65,7 @@ echo "godown" src/godown -v -f $SCRATCH_MNT >> $seqres.full echo "unmount" -umount $SCRATCH_MNT +_scratch_unmount echo "logprint after going down..." _print_logstate @@ -85,7 +85,7 @@ echo "ls SCRATCH_MNT" ls $SCRATCH_MNT echo "unmount" -umount $SCRATCH_MNT +_scratch_unmount echo "logprint after mount and replay..." _print_logstate diff --git a/tests/generic/053 b/tests/generic/053 index b8bd9e69..a0e7280b 100755 --- a/tests/generic/053 +++ b/tests/generic/053 @@ -81,7 +81,7 @@ list_acls() echo "acls before repair:" list_acls -_do 'unmount $SCRATCH_DEV' 'umount $SCRATCH_DEV' +_do 'unmount $SCRATCH_DEV' '_scratch_unmount' _do 'repair filesystem' '_check_scratch_fs' _do 'mount filesytem' '_scratch_mount' echo "acls after repair: " diff --git a/tests/generic/054 b/tests/generic/054 index 6cb0122b..238741c1 100755 --- a/tests/generic/054 +++ b/tests/generic/054 @@ -48,7 +48,7 @@ _require_scratch_shutdown _require_logstate echo "*** init FS" -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 _get_log_configs > $tmp.seq.params @@ -108,7 +108,7 @@ for s in sync nosync ; do src/godown -v -f $SCRATCH_MNT >> $seqres.full _echofull "unmount" - umount $SCRATCH_DEV >>$seqres.full 2>&1 \ + _scratch_unmount >>$seqres.full 2>&1 \ || _fail "umount failed" _echofull "logprint after going down..." @@ -123,7 +123,7 @@ for s in sync nosync ; do ls $SCRATCH_MNT _echofull "unmount" - umount $SCRATCH_MNT + _scratch_unmount _echofull "logprint after mount and replay..." _print_logstate diff --git a/tests/generic/055 b/tests/generic/055 index 82ec3319..207c007f 100755 --- a/tests/generic/055 +++ b/tests/generic/055 @@ -77,7 +77,7 @@ _require_logstate QUOTA_OPTION=`_get_quota_option` echo "*** init FS" -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 _get_log_configs > $tmp.seq.params @@ -118,7 +118,7 @@ do src/godown -v -f $SCRATCH_MNT >> $seqres.full _echofull "unmount" - umount $SCRATCH_DEV >>$seqres.full 2>&1 \ + _scratch_unmount >>$seqres.full 2>&1 \ || _fail "umount failed" _echofull "logprint after going down..." @@ -139,7 +139,7 @@ do diff -us $tmp.ls1 $tmp.ls2 | sed "s#$tmp#TMP#g" _echofull "unmount" - umount $SCRATCH_MNT + _scratch_unmount _echofull "logprint after mount and replay..." _print_logstate diff --git a/tests/generic/062 b/tests/generic/062 index 047c9304..90fcbdaf 100755 --- a/tests/generic/062 +++ b/tests/generic/062 @@ -40,7 +40,7 @@ _cleanup() { cd / echo; echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 diff --git a/tests/generic/067 b/tests/generic/067 index 8322ef84..8a7a55c1 100755 --- a/tests/generic/067 +++ b/tests/generic/067 @@ -126,7 +126,7 @@ lazy_umount_symlink() ln -s $SCRATCH_MNT/testdir $symlink $UMOUNT_PROG -l $symlink >>$seqres.full 2>&1 - # umount $SCRATCH_MNT should not be blocked + # _scratch_unmount should not be blocked _scratch_unmount 2>&1 | tee -a $seqres.full } diff --git a/tests/generic/068 b/tests/generic/068 index 77943265..f9f5f5aa 100755 --- a/tests/generic/068 +++ b/tests/generic/068 @@ -57,7 +57,7 @@ _require_freeze echo "*** init FS" rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs >>$seqres.full 2>&1 \ diff --git a/tests/generic/069 b/tests/generic/069 index c3fa862c..aaf6bbf5 100755 --- a/tests/generic/069 +++ b/tests/generic/069 @@ -42,7 +42,7 @@ _supported_os IRIX Linux _require_scratch rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** mkfs" _scratch_mkfs >/dev/null 2>&1 || _fail "mkfs failed" diff --git a/tests/generic/076 b/tests/generic/076 index f3e5b6f7..64e69583 100755 --- a/tests/generic/076 +++ b/tests/generic/076 @@ -42,7 +42,7 @@ _lets_get_pidst() _cleanup() { echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null _lets_get_pidst } trap "_cleanup; rm -f $tmp.*; exit \$status" 0 1 2 3 15 @@ -61,7 +61,7 @@ _require_block_device $SCRATCH_DEV echo "*** init fs" rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs >>$seqres.full 2>&1 \ diff --git a/tests/generic/077 b/tests/generic/077 index 8405b02d..add16ea8 100755 --- a/tests/generic/077 +++ b/tests/generic/077 @@ -39,7 +39,7 @@ _cleanup() { cd / echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } trap "_cleanup; rm -f $tmp.*; exit \$status" 0 1 2 3 15 @@ -62,7 +62,7 @@ _require_user echo "*** create filesystem" rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full SIZE=`expr 50 \* 1024 \* 1024` diff --git a/tests/generic/079 b/tests/generic/079 index a533484f..939c817f 100755 --- a/tests/generic/079 +++ b/tests/generic/079 @@ -37,7 +37,7 @@ _cleanup() cd / echo "*** cleaning up" $timmutable -r $SCRATCH_MNT/$seq - umount $SCRATCH_MNT + _scratch_unmount } # get standard environment, filters and checks diff --git a/tests/generic/083 b/tests/generic/083 index 61316bff..2f8c4a0d 100755 --- a/tests/generic/083 +++ b/tests/generic/083 @@ -42,7 +42,7 @@ status=1 _cleanup() { echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -67,7 +67,7 @@ workout() procs=$3 nops=$4 - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 echo "*** mkfs -dsize=$fsz,agcount=$ags" >>$seqres.full echo "" >>$seqres.full if [ $FSTYP = xfs ] diff --git a/tests/generic/105 b/tests/generic/105 index 2f99401f..9b7beb4d 100755 --- a/tests/generic/105 +++ b/tests/generic/105 @@ -57,7 +57,7 @@ _require_acls _acl_setup_ids -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs >>$seqres.full 2>&1 \ diff --git a/tests/generic/117 b/tests/generic/117 index 935dcafb..b634861d 100755 --- a/tests/generic/117 +++ b/tests/generic/117 @@ -82,7 +82,7 @@ _require_scratch _require_attrs rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs >>$seqres.full 2>&1 \ @@ -105,7 +105,7 @@ while [ $i -lt $ITERATIONS ]; do done cd / -umount $SCRATCH_MNT +_scratch_unmount echo echo Checking filesystem diff --git a/tests/generic/120 b/tests/generic/120 index fb7c8cb9..42713444 100755 --- a/tests/generic/120 +++ b/tests/generic/120 @@ -92,7 +92,7 @@ echo "*** writing to file ***" echo "asdf" >> $SCRATCH_MNT/testfile2 _compare_access_times $SCRATCH_MNT/testfile2 "writing file" -umount $SCRATCH_MNT +_scratch_unmount # success, all done status=0 diff --git a/tests/generic/124 b/tests/generic/124 index 888bbc0b..b78c1891 100755 --- a/tests/generic/124 +++ b/tests/generic/124 @@ -75,6 +75,6 @@ if (test $? -eq 0 ) then status=0 fi -umount $SCRATCH_DEV +_scratch_unmount exit diff --git a/tests/generic/128 b/tests/generic/128 index 22e4f712..39fc807e 100755 --- a/tests/generic/128 +++ b/tests/generic/128 @@ -54,7 +54,7 @@ su -s/bin/bash - $qa_user -c "$SCRATCH_MNT/ls $SCRATCH_MNT/nosuid >/dev/null 2>& if [ $? -eq 0 ] ; then echo "Error: we shouldn't be able to ls the directory" fi -umount $SCRATCH_DEV +_scratch_unmount status=0 exit diff --git a/tests/generic/135 b/tests/generic/135 index 24448032..44006726 100755 --- a/tests/generic/135 +++ b/tests/generic/135 @@ -48,7 +48,7 @@ _umount_mount() CWD=`pwd` cd / # pipe error into /dev/null, in case not mounted (after _require_scratch) - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null _scratch_mount cd "$CWD" } diff --git a/tests/generic/157 b/tests/generic/157 index a43fb0d4..2fe06d1c 100755 --- a/tests/generic/157 +++ b/tests/generic/157 @@ -114,7 +114,7 @@ _reflink_range "$TESTDIR1/file1" 0 "$TESTDIR1/file2" 0 $BLKSZ >> "$seqres.full" _reflink_range "$TESTDIR2/file1" 0 "$TESTDIR2/file2" 0 $BLKSZ >> "$seqres.full" echo "Check scratch fs" -umount $SCRATCH_MNT +_scratch_unmount _check_scratch_fs # success, all done diff --git a/tests/generic/158 b/tests/generic/158 index a499b214..91cc041b 100755 --- a/tests/generic/158 +++ b/tests/generic/158 @@ -115,7 +115,7 @@ _dedupe_range "$TESTDIR1/file1" 0 "$TESTDIR1/file2" 0 $BLKSZ >> "$seqres.full" _dedupe_range "$TESTDIR2/file1" 0 "$TESTDIR2/file2" 0 $BLKSZ >> "$seqres.full" echo "Check scratch fs" -umount $SCRATCH_MNT +_scratch_unmount _check_scratch_fs # success, all done diff --git a/tests/generic/166 b/tests/generic/166 index 71eb2aba..93c4bc9a 100755 --- a/tests/generic/166 +++ b/tests/generic/166 @@ -83,7 +83,7 @@ touch $TESTDIR/finished wait echo "Check for damage" -umount $SCRATCH_MNT +_scratch_unmount _check_scratch_fs echo "Done" diff --git a/tests/generic/167 b/tests/generic/167 index c4e6ce80..6d707371 100755 --- a/tests/generic/167 +++ b/tests/generic/167 @@ -83,7 +83,7 @@ touch $TESTDIR/finished wait echo "Check for damage" -umount $SCRATCH_MNT +_scratch_unmount _check_scratch_fs echo "Done" diff --git a/tests/generic/169 b/tests/generic/169 index ebeffa76..839ff9d8 100755 --- a/tests/generic/169 +++ b/tests/generic/169 @@ -74,7 +74,7 @@ $XFS_IO_PROG -a -c "pwrite 0 5k" -c "fsync" \ | _show_wrote_and_stat_only echo "# unmounting scratch" -umount $SCRATCH_MNT>>$seqres.full 2>&1 \ +_scratch_unmount>>$seqres.full 2>&1 \ || _fail "unmount failed" echo "# mounting scratch" @@ -91,7 +91,7 @@ $XFS_IO_PROG -f -c "pwrite 0 5" -c s -c "pwrite 5 5" \ | _show_wrote_and_stat_only echo "# unmounting scratch" -umount $SCRATCH_MNT>>$seqres.full 2>&1 \ +_scratch_unmount>>$seqres.full 2>&1 \ || _fail "unmount failed" echo "# mounting scratch" diff --git a/tests/generic/192 b/tests/generic/192 index 5b6cfbc9..ebabea27 100755 --- a/tests/generic/192 +++ b/tests/generic/192 @@ -78,7 +78,7 @@ cat $testfile time2=`_access_time $testfile | tee -a $seqres.full` cd / -umount $TEST_DIR +_test_unmount _test_mount time3=`_access_time $testfile | tee -a $seqres.full` diff --git a/tests/generic/219 b/tests/generic/219 index 2b75b894..bdeb0334 100755 --- a/tests/generic/219 +++ b/tests/generic/219 @@ -106,12 +106,12 @@ test_accounting() # real QA test starts here rm -f $seqres.full -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mkfs >> $seqres.full 2>&1 _scratch_mount "-o usrquota,grpquota" quotacheck -u -g $SCRATCH_MNT 2>/dev/null quotaon $SCRATCH_MNT 2>/dev/null -umount $SCRATCH_DEV +_scratch_unmount echo; echo "### test user accounting" export MOUNT_OPTIONS="-o usrquota" @@ -120,7 +120,7 @@ quotaon $SCRATCH_MNT 2>/dev/null type=u test_files test_accounting -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null echo; echo "### test group accounting" export MOUNT_OPTIONS="-o grpquota" @@ -129,7 +129,7 @@ quotaon $SCRATCH_MNT 2>/dev/null type=g test_files test_accounting -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null status=0 exit diff --git a/tests/generic/226 b/tests/generic/226 index 4ad56a50..b12965a0 100755 --- a/tests/generic/226 +++ b/tests/generic/226 @@ -42,7 +42,7 @@ _require_odirect # real QA test starts here rm -f $seqres.full -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null echo "--> mkfs 256m filesystem" _scratch_mkfs_sized `expr 256 \* 1024 \* 1024` >> $seqres.full 2>&1 _scratch_mount @@ -61,7 +61,7 @@ for I in `seq 1 $loops`; do done echo -umount $SCRATCH_DEV +_scratch_unmount _scratch_mount echo "--> $loops direct 64m writes in a loop" @@ -73,7 +73,7 @@ for I in `seq 1 $loops`; do done echo -umount $SCRATCH_DEV +_scratch_unmount status=0 exit diff --git a/tests/generic/230 b/tests/generic/230 index b77210f6..0b1b3bfb 100755 --- a/tests/generic/230 +++ b/tests/generic/230 @@ -122,7 +122,7 @@ setquota -u $qa_user 1000 2000 3 5 $SCRATCH_MNT setquota -u -t $grace $grace $SCRATCH_MNT setquota -g $qa_user 1000 2000 3 5 $SCRATCH_MNT setquota -g -t $grace $grace $SCRATCH_MNT -umount $SCRATCH_DEV +_scratch_unmount echo; echo "### test user limit enforcement" _scratch_mount "-o usrquota" @@ -131,7 +131,7 @@ type=u test_files test_enforcement cleanup_files -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null echo; echo "### test group limit enforcement" _scratch_mount "-o grpquota" @@ -140,7 +140,7 @@ type=g test_files test_enforcement cleanup_files -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null status=0 exit diff --git a/tests/generic/231 b/tests/generic/231 index df7d76c6..feda0c16 100755 --- a/tests/generic/231 +++ b/tests/generic/231 @@ -78,25 +78,25 @@ quotacheck -u -g $SCRATCH_MNT 2>/dev/null quotaon -u -g $SCRATCH_MNT 2>/dev/null if ! _fsx 1; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi if ! _check_quota_usage; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi if ! _fsx 4; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi if ! _check_quota_usage; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi @@ -104,17 +104,17 @@ fi #flush cache after write FSX_ARGS="-f $FSX_ARGS" if ! _fsx 1; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi if ! _check_quota_usage; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null status=0 exit diff --git a/tests/generic/232 b/tests/generic/232 index 8ada8aac..b3f6b7e1 100755 --- a/tests/generic/232 +++ b/tests/generic/232 @@ -78,16 +78,16 @@ quotacheck -u -g $SCRATCH_MNT 2>/dev/null quotaon -u -g $SCRATCH_MNT 2>/dev/null if ! _fsstress; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi if ! _check_quota_usage; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null status=0 exit diff --git a/tests/generic/233 b/tests/generic/233 index 58b3672d..95255f8a 100755 --- a/tests/generic/233 +++ b/tests/generic/233 @@ -87,16 +87,16 @@ quotaon -u -g $SCRATCH_MNT 2>/dev/null setquota -u $qa_user 32000 32000 1000 1000 $SCRATCH_MNT 2>/dev/null if ! _fsstress; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi if ! _check_quota_usage; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null status=1 exit fi -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null status=0 exit diff --git a/tests/generic/234 b/tests/generic/234 index eb0871f1..30be0e91 100755 --- a/tests/generic/234 +++ b/tests/generic/234 @@ -101,7 +101,7 @@ _scratch_mount "-o usrquota,grpquota" quotacheck -u -g $SCRATCH_MNT 2>/dev/null quotaon -u -g $SCRATCH_MNT 2>/dev/null test_setting -umount $SCRATCH_DEV +_scratch_unmount status=0 exit diff --git a/tests/generic/235 b/tests/generic/235 index bbc99e0c..e27fd3df 100755 --- a/tests/generic/235 +++ b/tests/generic/235 @@ -65,15 +65,15 @@ repquota -u -g $SCRATCH_MNT | grep -v "^root" | _filter_scratch # https://bugzilla.redhat.com/show_bug.cgi?id=563267 # # then you need a more recent mount binary. -mount -o remount,ro $SCRATCH_DEV 2>&1 | tee -a $seqres.full | _filter_scratch +_scratch_mount "-o remount,ro" 2>&1 | tee -a $seqres.full | _filter_scratch touch $SCRATCH_MNT/failed 2>&1 | tee -a $seqres.full | _filter_scratch -mount -o remount,rw $SCRATCH_DEV 2>&1 | tee -a $seqres.full | _filter_scratch +_scratch_mount "-o remount,rw" 2>&1 | tee -a $seqres.full | _filter_scratch $XFS_IO_PROG -c 'pwrite 0 8k' -c 'fsync' \ $SCRATCH_MNT/testfile >>$seqres.full 2>&1 repquota -u -g $SCRATCH_MNT | grep -v "^root" | _filter_scratch -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null status=0 exit diff --git a/tests/generic/256 b/tests/generic/256 index 086c00a1..cfbf7909 100755 --- a/tests/generic/256 +++ b/tests/generic/256 @@ -165,7 +165,7 @@ _test_full_fs_punch() } # Make a small file system to fill -umount $SCRATCH_DEV &> /dev/null +_scratch_unmount &> /dev/null _scratch_mkfs_sized $(( 1536 * 1024 * 1024 )) &> /dev/null _scratch_mount # Test must be able to write files with non-root permissions diff --git a/tests/generic/258 b/tests/generic/258 index b9b5cf9b..285a4229 100755 --- a/tests/generic/258 +++ b/tests/generic/258 @@ -62,7 +62,7 @@ fi # unmount, remount, and check the timestamp echo "Remounting to flush cache" -umount $TEST_DEV +_test_unmount _test_mount # Should yield -315593940 (prior to epoch) diff --git a/tests/generic/269 b/tests/generic/269 index 09582220..d2043d4e 100755 --- a/tests/generic/269 +++ b/tests/generic/269 @@ -72,7 +72,7 @@ _scratch_mkfs_sized $((512 * 1024 * 1024)) >> $seqres.full 2>&1 _scratch_mount if ! _workout; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi diff --git a/tests/generic/274 b/tests/generic/274 index 2d8d4962..61ab2209 100755 --- a/tests/generic/274 +++ b/tests/generic/274 @@ -56,7 +56,7 @@ echo "------------------------------" rm -f $seqres.full -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mkfs_sized $((2 * 1024 * 1024 * 1024)) >>$seqres.full 2>&1 _scratch_mount diff --git a/tests/generic/275 b/tests/generic/275 index f1963d88..955c4aca 100755 --- a/tests/generic/275 +++ b/tests/generic/275 @@ -53,7 +53,7 @@ echo "------------------------------" rm -f $seqres.full -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mkfs_sized $((2 * 1024 * 1024 * 1024)) >>$seqres.full 2>&1 _scratch_mount diff --git a/tests/generic/280 b/tests/generic/280 index 7a2d1d01..069d46d3 100755 --- a/tests/generic/280 +++ b/tests/generic/280 @@ -54,7 +54,7 @@ _supported_fs generic rm -f $seqres.full -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mkfs >> $seqres.full 2>&1 _scratch_mount "-o usrquota,grpquota" quotacheck -u -g $SCRATCH_MNT 2>/dev/null @@ -65,7 +65,7 @@ pid=$! sleep 1 xfs_freeze -u $SCRATCH_MNT wait $pid -umount $SCRATCH_DEV +_scratch_unmount # Failure comes in the form of a deadlock. diff --git a/tests/generic/299 b/tests/generic/299 index 0459f42b..4174de0d 100755 --- a/tests/generic/299 +++ b/tests/generic/299 @@ -148,7 +148,7 @@ _scratch_mkfs >> $seqres.full 2>&1 _scratch_mount if ! _workout; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi diff --git a/tests/generic/300 b/tests/generic/300 index d7523aae..a12aba61 100755 --- a/tests/generic/300 +++ b/tests/generic/300 @@ -147,7 +147,7 @@ _scratch_mkfs_sized $FS_SIZE >> $seqres.full 2>&1 _scratch_mount if ! _workout; then - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null exit fi diff --git a/tests/generic/317 b/tests/generic/317 index efa37c76..68f231c7 100755 --- a/tests/generic/317 +++ b/tests/generic/317 @@ -37,7 +37,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -80,7 +80,7 @@ _print_numeric_uid() src/nsexec -s -U -M "0 $qa_user_id 1000" -G "0 $qa_user_id 1000" src/lstat64 $file |head -3 |_filter_output } -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs >>$seqres.full 2>&1 || _fail "mkfs failed" @@ -96,11 +96,11 @@ echo "" echo "*** Remounting ***" echo "" sync -umount $SCRATCH_MNT >>$seqres.full 2>&1 +_scratch_unmount >>$seqres.full 2>&1 _scratch_mount >>$seqres.full 2>&1 || _fail "mount failed" _print_numeric_uid -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 status=0 exit diff --git a/tests/generic/318 b/tests/generic/318 index 8237434e..c730b501 100755 --- a/tests/generic/318 +++ b/tests/generic/318 @@ -38,7 +38,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -90,7 +90,7 @@ _print_getfacls() $nsexec -U -M "0 $acl1 1000" -G "0 $acl1 1000" getfacl --absolute-names -n $file 2>/dev/null | _filter_scratch | _getfacl_filter_nsid } -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs >>$seqres.full 2>&1 || _fail "mkfs failed" @@ -109,11 +109,11 @@ _print_getfacls echo "*** Remounting ***" echo "" sync -umount $SCRATCH_MNT >>$seqres.full 2>&1 +_scratch_unmount >>$seqres.full 2>&1 _scratch_mount >>$seqres.full 2>&1 || _fail "mount failed" _print_getfacls -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 status=0 exit diff --git a/tests/shared/289 b/tests/shared/289 index 96c949d4..4a2e04c5 100755 --- a/tests/shared/289 +++ b/tests/shared/289 @@ -78,11 +78,11 @@ OVERHEAD=$(($TOTAL_BLOCKS-$FREE_BLOCKS)) _scratch_mount "-o minixdf" MINIX_F_BLOCKS=`stat -f $SCRATCH_MNT | awk '/^Blocks/{print $3}'` -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount "-o bsddf" BSD_F_BLOCKS=`stat -f $SCRATCH_MNT | awk '/^Blocks/{print $3}'` -umount $SCRATCH_MNT +_scratch_unmount # Echo data to $seqres.full for analysis echo "Overhead is $OVERHEAD blocks out of $TOTAL_BLOCKS ($FREE_BLOCKS free)" >> $seqres.full diff --git a/tests/udf/098 b/tests/udf/098 index 3a5db395..09792a76 100755 --- a/tests/udf/098 +++ b/tests/udf/098 @@ -66,7 +66,7 @@ _db_filter() _umount_check_mount() { cd / - umount $SCRATCH_MNT + _scratch_unmount udf_db -f $SCRATCH_DEV | _db_filter @@ -244,7 +244,7 @@ echo "list root EAs: none" ${ATTR_PROG} -R -l foo cd / -umount $SCRATCH_MNT +_scratch_unmount udf_db -f $SCRATCH_DEV | _db_filter @@ -258,7 +258,7 @@ echo "delete the file foo - which will delete the associated streams" rm foo cd / -umount $SCRATCH_MNT +_scratch_unmount udf_db -f $SCRATCH_DEV | _db_filter diff --git a/tests/udf/101 b/tests/udf/101 index 0f5fe874..9609c072 100755 --- a/tests/udf/101 +++ b/tests/udf/101 @@ -49,7 +49,7 @@ MKFS_OPTIONS="-s $FS_SIZE" _require_scratch _setup_udf_scratchdir -umount $SCRATCH_MNT +_scratch_unmount # Inspect the space bitmap echo bitmap | udf_db -c - $SCRATCH_DEV diff --git a/tests/xfs/004 b/tests/xfs/004 index 978943d1..cf172867 100755 --- a/tests/xfs/004 +++ b/tests/xfs/004 @@ -32,7 +32,7 @@ status=0 _cleanup() { - umount $SCRATCH_MNT + _scratch_unmount rm -f $tmp.* exit $status } @@ -48,7 +48,7 @@ _populate_scratch() dd if=/dev/zero of=$SCRATCH_MNT/goo count=400 bs=4096 >/dev/null 2>&1 & dd if=/dev/zero of=$SCRATCH_MNT/moo count=800 bs=4096 >/dev/null 2>&1 & wait - umount $SCRATCH_MNT # flush everything + _scratch_unmount # flush everything _scratch_mount # and then remount } diff --git a/tests/xfs/006 b/tests/xfs/006 index fa84e40e..13631be1 100755 --- a/tests/xfs/006 +++ b/tests/xfs/006 @@ -54,7 +54,7 @@ _scratch_mount | _filter_scratch mkdir $SCRATCH_MNT/x $here/src/dirhash_collide -d -n 10000 $SCRATCH_MNT/x -umount $SCRATCH_MNT 2>&1 | _filter_scratch +_scratch_unmount 2>&1 | _filter_scratch _scratch_xfs_repair 2>&1 | _filter_repair diff --git a/tests/xfs/007 b/tests/xfs/007 index 76c100ff..f2e35849 100755 --- a/tests/xfs/007 +++ b/tests/xfs/007 @@ -60,7 +60,7 @@ do_test() off_opts=$3 echo "*** umount" - umount $SCRATCH_MNT + _scratch_unmount QINO_1=`xfs_db -c "sb 0" -c "p" $SCRATCH_DEV | \ grep $qino_1 | awk '{print $NF}'` @@ -76,7 +76,7 @@ do_test() xfs_quota -x -c "off -$off_opts" $SCRATCH_MNT xfs_quota -x -c "remove -$off_opts" $SCRATCH_MNT echo "*** umount" - umount $SCRATCH_MNT + _scratch_unmount echo "*** Usage after quotarm ***" xfs_db -c "inode $QINO_1" -c "p core.nblocks" $SCRATCH_DEV diff --git a/tests/xfs/009 b/tests/xfs/009 index d0850c61..abef2e05 100755 --- a/tests/xfs/009 +++ b/tests/xfs/009 @@ -37,7 +37,7 @@ bsize=4096 _cleanup() { echo "*** unmount" - umount $SCRATCH_MNT + _scratch_unmount } _block_filter() diff --git a/tests/xfs/010 b/tests/xfs/010 index fd2e27cd..d350d641 100755 --- a/tests/xfs/010 +++ b/tests/xfs/010 @@ -40,7 +40,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -114,7 +114,7 @@ _scratch_mkfs_xfs "-m crc=1,finobt=1 -d agcount=2" | _filter_mkfs 2>$seqres.full # sparsely populate the fs such that we create records with free inodes _scratch_mount _sparse_inode_populate $SCRATCH_MNT 999 -umount $SCRATCH_MNT +_scratch_unmount # corrupt some finobt records _corrupt_finobt_records $SCRATCH_DEV diff --git a/tests/xfs/011 b/tests/xfs/011 index 197752c7..e84faa5b 100755 --- a/tests/xfs/011 +++ b/tests/xfs/011 @@ -40,7 +40,7 @@ _cleanup() killall -9 fsstress 2>/dev/null wait cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -109,7 +109,7 @@ done killall $FSSTRESS_PROG wait -umount $SCRATCH_MNT +_scratch_unmount status=0 exit diff --git a/tests/xfs/013 b/tests/xfs/013 index dea3d081..3a96fea1 100755 --- a/tests/xfs/013 +++ b/tests/xfs/013 @@ -43,7 +43,7 @@ _cleanup() killall -9 fsstress 2>/dev/null wait cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -149,7 +149,7 @@ wait rm -rf $SCRATCH_MNT/fsstress rm -rf $SCRATCH_MNT/dir* -umount $SCRATCH_MNT +_scratch_unmount status=0 exit diff --git a/tests/xfs/014 b/tests/xfs/014 index ab3d85b2..22ac4a81 100755 --- a/tests/xfs/014 +++ b/tests/xfs/014 @@ -44,7 +44,7 @@ _cleanup() { cd / umount $LOOP_MNT 2>/dev/null - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -193,7 +193,7 @@ umount $LOOP_MNT echo $orig_sp_time > /proc/sys/fs/xfs/speculative_prealloc_lifetime -umount $SCRATCH_MNT +_scratch_unmount status=0 exit diff --git a/tests/xfs/016 b/tests/xfs/016 index e52f80c2..01637160 100755 --- a/tests/xfs/016 +++ b/tests/xfs/016 @@ -50,7 +50,7 @@ _cleanup() cd / rm -f $tmp.* echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } _block_filter() @@ -116,7 +116,7 @@ _log_traffic() done echo " *** unmount" - if ! umount $SCRATCH_DEV + if ! _scratch_unmount then echo "failed to unmount $SCRATCH_DEV" exit 1 diff --git a/tests/xfs/017 b/tests/xfs/017 index 9fc16c24..b576cd15 100755 --- a/tests/xfs/017 +++ b/tests/xfs/017 @@ -34,7 +34,7 @@ trap "_cleanup; rm -f $tmp.*; exit \$status" 0 1 2 3 15 _cleanup() { echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } # get standard environment, filters and checks @@ -54,7 +54,7 @@ _require_no_large_scratch_dev echo "*** init FS" rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs_xfs >>$seqres.full 2>&1 \ diff --git a/tests/xfs/018 b/tests/xfs/018 index 848981f8..451735fd 100755 --- a/tests/xfs/018 +++ b/tests/xfs/018 @@ -41,7 +41,7 @@ _cleanup() cd / _cleanup_logfiles rm -f $tmp.* - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -57,7 +57,7 @@ _require_v2log _link_out_file_named $seqfull.op $(_get_os_name) echo "*** init FS" -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 cat >$tmp.seq.params </dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -111,7 +111,7 @@ _verify_fs() VERSION="-n version=$1" rm -f $seqfull - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 _full "mkfs" _scratch_mkfs_xfs $VERSION -p $protofile >>$seqfull 2>&1 \ @@ -134,7 +134,7 @@ _verify_fs() echo "*** unmount FS" _full "umount" - umount $SCRATCH_DEV >>$seqfull 2>&1 \ + _scratch_unmount >>$seqfull 2>&1 \ || _fail "umount failed" } diff --git a/tests/xfs/021 b/tests/xfs/021 index a5480927..795b71cc 100755 --- a/tests/xfs/021 +++ b/tests/xfs/021 @@ -37,7 +37,7 @@ status=0 # success is the default! _cleanup() { echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -80,7 +80,7 @@ _require_scratch _require_attrs rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** mkfs" _scratch_mkfs_xfs >/dev/null \ @@ -125,7 +125,7 @@ echo "" inum_2=`ls -li $testfile.2 | $AWK_PROG '{ print $1 }'` echo "*** unmount FS" -umount $SCRATCH_DEV >>$seqres.full 2>&1 \ +_scratch_unmount >>$seqres.full 2>&1 \ || _fail "umount failed" echo "*** dump attributes (1)" diff --git a/tests/xfs/030 b/tests/xfs/030 index d2f5ed19..4cb1524e 100755 --- a/tests/xfs/030 +++ b/tests/xfs/030 @@ -34,7 +34,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } @@ -87,7 +87,7 @@ else _notrun "GQuota are enabled, test needs controlled sb recovery" src/feature -P $SCRATCH_DEV && \ _notrun "PQuota are enabled, test needs controlled sb recovery" - umount $SCRATCH_DEV + _scratch_unmount fi clear="" eval `xfs_db -r -c "sb 1" -c stack $SCRATCH_DEV | perl -ne ' diff --git a/tests/xfs/033 b/tests/xfs/033 index dab111aa..9342f0c9 100755 --- a/tests/xfs/033 +++ b/tests/xfs/033 @@ -34,7 +34,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } @@ -101,7 +101,7 @@ src/feature -G $SCRATCH_DEV && \ _notrun "GQuota are enabled, test needs controlled sb recovery" src/feature -P $SCRATCH_DEV && \ _notrun "PQuota are enabled, test needs controlled sb recovery" -umount $SCRATCH_DEV +_scratch_unmount # rootino, rbmino, and rsumino are now set (lets blow em away!) _check_root_inos 0 diff --git a/tests/xfs/034 b/tests/xfs/034 index 6beeb561..2bf319bf 100755 --- a/tests/xfs/034 +++ b/tests/xfs/034 @@ -35,7 +35,7 @@ _cleanup() cd / rm -f $tmp.* echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -52,7 +52,7 @@ _require_scratch echo "*** init FS" rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 echo "*** MKFS ***" >>$seqres.full echo "" >>$seqres.full _scratch_mkfs_xfs >>$seqres.full 2>&1 \ diff --git a/tests/xfs/041 b/tests/xfs/041 index 8b109fcc..50385288 100755 --- a/tests/xfs/041 +++ b/tests/xfs/041 @@ -35,7 +35,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT + _scratch_unmount rm -f $tmp.* } trap "_cleanup ; exit \$status" 0 1 2 3 15 @@ -50,7 +50,7 @@ _supported_os IRIX Linux _require_scratch _require_no_large_scratch_dev -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _fill() { @@ -80,7 +80,7 @@ do _fill $SCRATCH_MNT/fill_$size _do "Grow filesystem to ${size}m" "xfs_growfs -D $grow_size $SCRATCH_MNT" echo -n "Flush filesystem... " - _do "umount $SCRATCH_MNT" + _do "_scratch_unmount" _do "_scratch_mount" echo "done" echo -n "Check files... " diff --git a/tests/xfs/042 b/tests/xfs/042 index 446fa801..ba5eed32 100755 --- a/tests/xfs/042 +++ b/tests/xfs/042 @@ -35,7 +35,7 @@ status=1 # failure is the default! _cleanup() { - umount $SCRATCH_MNT + _scratch_unmount rm -f $tmp.* } trap "_cleanup ; exit \$status" 0 1 2 3 15 @@ -96,7 +96,7 @@ done _do "xfs_bmap -vp $SCRATCH_MNT/fill" echo "done" # flush the filesystem - make sure there is no space "lost" to pre-allocation -_do "umount $SCRATCH_MNT" +_do "_scratch_unmount" _do "_scratch_mount" echo -n "Use up any further available space... " _do "$XFS_IO_PROG -f -c \"falloc 0 1m\" $SCRATCH_MNT/pad" diff --git a/tests/xfs/044 b/tests/xfs/044 index a84af227..2ffd74f0 100755 --- a/tests/xfs/044 +++ b/tests/xfs/044 @@ -68,7 +68,7 @@ _check_mount() fi echo " *** umount" - if ! umount $SCRATCH_DEV + if ! _scratch_unmount then echo " !!! umount failed (expecting success)" status=1 diff --git a/tests/xfs/050 b/tests/xfs/050 index 6641e7d9..130df77e 100755 --- a/tests/xfs/050 +++ b/tests/xfs/050 @@ -39,7 +39,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -197,7 +197,7 @@ _exercise() echo echo "*** unmount" - umount $SCRATCH_MNT + _scratch_unmount } diff --git a/tests/xfs/052 b/tests/xfs/052 index c2cf5961..18cf037a 100755 --- a/tests/xfs/052 +++ b/tests/xfs/052 @@ -41,7 +41,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -107,7 +107,7 @@ echo ===quota output >> $seqres.full cat $tmp.quota >> $seqres.full [ ! -s $tmp.quota ] && echo "warning: quota output file is empty" -umount $SCRATCH_MNT +_scratch_unmount # note - does (insitu) conversion from fs blocks to 1K blocks xfs_db -rc "dquot -$type $id" -c p $SCRATCH_DEV | tee -a $seqres.full | perl -ne ' diff --git a/tests/xfs/054 b/tests/xfs/054 index 8c180508..ea0f9f86 100755 --- a/tests/xfs/054 +++ b/tests/xfs/054 @@ -38,7 +38,7 @@ status=1 # failure is the default! _cleanup() { - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -96,7 +96,7 @@ _exercise() src/lstat64 $SCRATCH_MNT/testfile | _filter_stat echo - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } _scratch_mkfs_xfs >> $seqres.full 2>&1 || _fail "mkfs failed!" @@ -108,7 +108,7 @@ if src/feature -G $SCRATCH_DEV ; then else _notrun "No quota support at mount time" fi -umount $SCRATCH_MNT 2>/dev/null +_scratch_unmount 2>/dev/null _qmount_option "rw" # no quota options echo "*** Default mount options" diff --git a/tests/xfs/065 b/tests/xfs/065 index a4faa824..6b2139f3 100755 --- a/tests/xfs/065 +++ b/tests/xfs/065 @@ -64,7 +64,7 @@ _supported_os IRIX Linux # too much hassle to get output matching with quotas turned on # so don't run it # -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mount $here/src/feature -U $SCRATCH_DEV && \ _notrun "UQuota enabled, test needs controlled xfsdump output" @@ -72,7 +72,7 @@ $here/src/feature -G $SCRATCH_DEV && \ _notrun "GQuota enabled, test needs controlled xfsdump output" $here/src/feature -P $SCRATCH_DEV && \ _notrun "PQuota enabled, test needs controlled xfsdump output" -umount $SCRATCH_DEV +_scratch_unmount # # adding - touch/echo, mkdir diff --git a/tests/xfs/071 b/tests/xfs/071 index 86c6899d..12874548 100755 --- a/tests/xfs/071 +++ b/tests/xfs/071 @@ -35,7 +35,7 @@ _cleanup() { cd / rm -f $tmp.* - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null } trap "_cleanup; exit \$status" 0 1 2 3 15 diff --git a/tests/xfs/072 b/tests/xfs/072 index 9949749e..43aa4211 100755 --- a/tests/xfs/072 +++ b/tests/xfs/072 @@ -30,7 +30,7 @@ _cleanup() { cd / rm -f $tmp.* - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } here=`pwd` @@ -51,7 +51,7 @@ _supported_os IRIX Linux _require_scratch rm -f $seqres.full -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 _scratch_mkfs_xfs >/dev/null || _fail "mkfs failed" _scratch_mount >/dev/null || _fail "mount failed" diff --git a/tests/xfs/073 b/tests/xfs/073 index 9def3c6f..e1335df9 100755 --- a/tests/xfs/073 +++ b/tests/xfs/073 @@ -41,7 +41,7 @@ imgs=$TEST_DIR/$$ _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null umount $imgs.loop 2>/dev/null [ -d $imgs.loop ] && rmdir $imgs.loop umount $imgs.source_dir 2>/dev/null @@ -144,7 +144,7 @@ _scratch_mount 2>/dev/null || _fail "initial scratch mount failed" echo echo === populating scratch device _populate_scratch -umount $SCRATCH_MNT 2>/dev/null +_scratch_unmount 2>/dev/null echo echo === copying scratch device to single target diff --git a/tests/xfs/076 b/tests/xfs/076 index 1ecfca6f..86f9a056 100755 --- a/tests/xfs/076 +++ b/tests/xfs/076 @@ -48,7 +48,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 diff --git a/tests/xfs/081 b/tests/xfs/081 index 1acb85d9..6f6f8256 100755 --- a/tests/xfs/081 +++ b/tests/xfs/081 @@ -46,7 +46,7 @@ _cleanup() _cleanup_logfiles rm -f $tmp.* echo "*** unmount" - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -57,7 +57,7 @@ _require_xfs_quota _require_v2log echo "*** init FS" -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 # do a simple quota test to ensure DQUOT data is happening export MOUNT_OPTIONS="-o quota,gquota" @@ -72,7 +72,7 @@ fi _scratch_mkfs_xfs "-l version=1" 1>/dev/null 2>&1 _qmount src/feature -u $SCRATCH_DEV || _notrun "No quota support at mount time" -umount $SCRATCH_MNT +_scratch_unmount _mkfs_log "-l version=1" _create_log diff --git a/tests/xfs/082 b/tests/xfs/082 index f1178fdc..0c0b98af 100755 --- a/tests/xfs/082 +++ b/tests/xfs/082 @@ -41,7 +41,7 @@ _cleanup() cd / _cleanup_logfiles rm -f $tmp.* - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -58,7 +58,7 @@ _require_v2log _link_out_file_named $seqfull.op $(_get_os_name) echo "*** init FS" -umount $SCRATCH_DEV >/dev/null 2>&1 +_scratch_unmount >/dev/null 2>&1 cat >$tmp.seq.params </dev/null +_scratch_unmount 2>/dev/null _scratch_mkfs_xfs | _filter_mkfs 2>$tmp.mkfs cat $tmp.mkfs >>$seqres.full _scratch_mount @@ -91,7 +91,7 @@ uid=1 gid=2 prid=3 export QARGS="-x -D /dev/null -P /dev/null $SCRATCH_MNT" -umount $SCRATCH_DEV +_scratch_unmount echo; echo "### test user accounting" export MOUNT_OPTIONS="-ouquota" @@ -100,7 +100,7 @@ type=u id=$uid test_files test_accounting -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null echo; echo "### test group accounting" export MOUNT_OPTIONS="-ogquota" @@ -109,7 +109,7 @@ type=g id=$gid test_files test_accounting -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null #echo; echo "### test project accounting" export MOUNT_OPTIONS="-opquota" @@ -118,7 +118,7 @@ type=p id=$prid test_files test_accounting -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null status=0 exit diff --git a/tests/xfs/109 b/tests/xfs/109 index 5578c909..4d5450ec 100755 --- a/tests/xfs/109 +++ b/tests/xfs/109 @@ -63,7 +63,7 @@ populate() done echo "flushing changes via umount/mount." - umount $SCRATCH_DEV + _scratch_unmount _scratch_mount } @@ -91,7 +91,7 @@ allocate() } # real QA test starts here -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mount # see if faststart is possible (and requested) @@ -100,7 +100,7 @@ faststart="" if [ -n "$FASTSTART" -a -f $SCRATCH_MNT/f0 ]; then faststart="-N" # causes us to skip the mkfs step fi -umount $SCRATCH_DEV +_scratch_unmount _scratch_mkfs_xfs -dsize=160m,agcount=4 $faststart | _filter_mkfs 2>$tmp.mkfs cat $tmp.mkfs >>$seqres.full diff --git a/tests/xfs/110 b/tests/xfs/110 index 7428faf6..19eddafb 100755 --- a/tests/xfs/110 +++ b/tests/xfs/110 @@ -41,7 +41,7 @@ _supported_os Linux _require_scratch # real QA test starts here -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mkfs_xfs | _filter_mkfs 2>$tmp.mkfs STR1=AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA diff --git a/tests/xfs/111 b/tests/xfs/111 index 7c8f2638..f6a41a8f 100755 --- a/tests/xfs/111 +++ b/tests/xfs/111 @@ -45,7 +45,7 @@ rm -f $seqres.full [ "$XFS_FSR_PROG" = "" ] && _notrun "xfs_fsr not found" # real QA test starts here -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null MKFS_OPTIONS="-bsize=4096" MOUNT_OPTIONS="-o noatime" _scratch_mkfs_xfs | _filter_mkfs 2>$tmp.mkfs @@ -61,7 +61,7 @@ done # Roughly the middle of the created inodes INO=`ls -i $SCRATCH_MNT/512 | awk '{print $1}'` -umount $SCRATCH_DEV +_scratch_unmount # Figure out where that middle inode starts BYTE=`$XFS_DB_PROG -c "convert inode $INO byte" $SCRATCH_DEV \ @@ -77,7 +77,7 @@ echo Attempting bulkstat $XFS_FSR_PROG $SCRATCH_MNT | _filter_scratch # Fix up intentional corruption so test can pass -umount $SCRATCH_DEV +_scratch_unmount $XFS_REPAIR_PROG $SCRATCH_DEV >> $seqres.full 2>&1 status=0 diff --git a/tests/xfs/116 b/tests/xfs/116 index b6924405..2dabde98 100755 --- a/tests/xfs/116 +++ b/tests/xfs/116 @@ -52,12 +52,12 @@ _supported_os IRIX Linux _require_scratch _require_xfs_quota -mkfs.xfs -f $SCRATCH_DEV >/dev/null 2>&1 -mount -o uquota $SCRATCH_DEV $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_mkfs >/dev/null 2>&1 +_scratch_mount "-o uquota" +_scratch_unmount xfs_db -r -c sb -c print $SCRATCH_DEV | grep qflags -mount $SCRATCH_DEV $SCRATCH_MNT -umount $SCRATCH_MNT +_scratch_mount +_scratch_unmount xfs_db -r -c sb -c print $SCRATCH_DEV | grep qflags # success, all done diff --git a/tests/xfs/121 b/tests/xfs/121 index 1457ed1e..d301808b 100755 --- a/tests/xfs/121 +++ b/tests/xfs/121 @@ -76,7 +76,7 @@ src/godown -v -f $SCRATCH_MNT >> $seqres.full wait 2>/dev/null echo "unmount" -umount $SCRATCH_MNT +_scratch_unmount echo "logprint after going down..." _print_logstate @@ -89,7 +89,7 @@ echo "godown" src/godown -v -f $SCRATCH_MNT >> $seqres.full echo "unmount" -umount $SCRATCH_MNT +_scratch_unmount echo "logprint after going down..." _print_logstate diff --git a/tests/xfs/134 b/tests/xfs/134 index adafa396..be18ee84 100755 --- a/tests/xfs/134 +++ b/tests/xfs/134 @@ -39,7 +39,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 diff --git a/tests/xfs/136 b/tests/xfs/136 index e7994305..679fa9f5 100755 --- a/tests/xfs/136 +++ b/tests/xfs/136 @@ -102,7 +102,7 @@ do_extents() _print_inode() { - umount $SCRATCH_MNT + _scratch_unmount xfs_db -r -c "inode $inum" -c "print" $SCRATCH_DEV |\ awk ' /nextents/ { print; next } @@ -118,14 +118,14 @@ _print_inode() _print_inode_u() { - umount $SCRATCH_MNT + _scratch_unmount xfs_db -r -c "inode $inum" -c "print u" $SCRATCH_DEV _scratch_mount } _print_inode_a() { - umount $SCRATCH_MNT + _scratch_unmount xfs_db -r -c "inode $inum" -c "print a" $SCRATCH_DEV _scratch_mount } diff --git a/tests/xfs/148 b/tests/xfs/148 index 5dc8df64..42c04810 100755 --- a/tests/xfs/148 +++ b/tests/xfs/148 @@ -34,7 +34,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_DEV 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } @@ -87,7 +87,7 @@ else _notrun "GQuota are enabled, test needs controlled sb recovery" src/feature -P $SCRATCH_DEV && \ _notrun "PQuota are enabled, test needs controlled sb recovery" - umount $SCRATCH_DEV + _scratch_unmount fi clear="" eval `xfs_db -r -c "sb 1" -c stack $SCRATCH_DEV | $PERL_PROG -ne ' diff --git a/tests/xfs/175 b/tests/xfs/175 index 16fcfccf..ad18e421 100755 --- a/tests/xfs/175 +++ b/tests/xfs/175 @@ -54,7 +54,7 @@ _require_scratch # test that we have DMAPI support _dmapi_scratch_mount -umount $SCRATCH_MNT > /dev/null 2>&1 +_scratch_unmount > /dev/null 2>&1 # _test_punch args: blksize extsize_hint filesize "pp1 pp2 pp3" \ # "ps1 ps2 ps3" "pt1 pt2 pt3" "wp1 wp2" "ws1 ws2" \ diff --git a/tests/xfs/176 b/tests/xfs/176 index ea07e00b..035d3b45 100755 --- a/tests/xfs/176 +++ b/tests/xfs/176 @@ -51,7 +51,7 @@ _require_scratch # test that we have DMAPI support _dmapi_scratch_mount -umount $SCRATCH_MNT > /dev/null 2>&1 +_scratch_unmount > /dev/null 2>&1 # _test_punch args: blksize extsize_hint filesize "pp1 pp2 pp3" \ # "ps1 ps2 ps3" "pt1 pt2 pt3" "wp1 wp2" "ws1 ws2" \ diff --git a/tests/xfs/177 b/tests/xfs/177 index 9a4fd382..3957b49a 100755 --- a/tests/xfs/177 +++ b/tests/xfs/177 @@ -54,7 +54,7 @@ _require_scratch # test that we have DMAPI support _dmapi_scratch_mount -umount $SCRATCH_MNT > /dev/null 2>&1 +_scratch_unmount > /dev/null 2>&1 _scratch_mkfs_xfs >/dev/null 2>&1 \ || _fail "mkfs failed" @@ -72,7 +72,7 @@ $here/src/bulkstat_unlink_test 15 100 6 $SCRATCH_MNT/bulkstat $here/src/bulkstat_unlink_test 20 1000 3 $SCRATCH_MNT/bulkstat # Mount scratch with ikeep enabled -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount "-o dmapi,mtpt=$SCRATCH_MNT" \ || _fail "mount failed" diff --git a/tests/xfs/181 b/tests/xfs/181 index 4e7ac3a9..a2010455 100755 --- a/tests/xfs/181 +++ b/tests/xfs/181 @@ -90,7 +90,7 @@ wait $pid 2>/dev/null pid="" echo "unmount" -umount $SCRATCH_MNT +_scratch_unmount echo "logprint after going down..." _print_logstate @@ -103,7 +103,7 @@ echo "godown" src/godown -v -f $SCRATCH_MNT >> $seqres.full echo "unmount" -umount $SCRATCH_MNT +_scratch_unmount echo "logprint after going down..." _print_logstate diff --git a/tests/xfs/189 b/tests/xfs/189 index a56bbdec..540a0d7b 100755 --- a/tests/xfs/189 +++ b/tests/xfs/189 @@ -64,7 +64,7 @@ rm -f $seqres.full _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null _putback_scratch_fstab rm -f $tmp.* } @@ -119,7 +119,7 @@ _test_remount_rw() _check_mount rw filestreams done - umount $SCRATCH_MNT + _scratch_unmount # remount ignores attr2, and noattr2 mount option does does not result # in any "attr2" specific option in /proc/mounts, so we can only check @@ -138,7 +138,7 @@ _test_remount_rw() _check_mount rw done - umount $SCRATCH_MNT + _scratch_unmount } # @@ -155,7 +155,7 @@ _test_remount_write() touch $SCRATCH_MNT/foobar [ $? -eq 0 ] || echo "touch (1) failed unexpectedly" - umount $SCRATCH_MNT + _scratch_unmount _scratch_mount -o ro [ $? -eq 0 ] || echo "mount (2) failed unexpectedly" @@ -166,7 +166,7 @@ _test_remount_write() touch $SCRATCH_MNT/foobar [ $? -eq 0 ] || echo "touch (2) failed unexpectedly" - umount $SCRATCH_MNT + _scratch_unmount } # @@ -193,7 +193,7 @@ _test_remount_barrier() [ $? -eq 0 ] || _fail "remount barrier failed" _check_mount rw - umount $SCRATCH_MNT + _scratch_unmount } # diff --git a/tests/xfs/196 b/tests/xfs/196 index 189f4ce9..3bff8f62 100755 --- a/tests/xfs/196 +++ b/tests/xfs/196 @@ -35,7 +35,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT + _scratch_unmount rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 diff --git a/tests/xfs/199 b/tests/xfs/199 index c168fec8..31338932 100755 --- a/tests/xfs/199 +++ b/tests/xfs/199 @@ -35,7 +35,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -73,7 +73,7 @@ echo "Clearing features2:" xfs_db -x $SCRATCH_DEV -c 'sb' -c 'write features2 0' _scratch_mount -umount $SCRATCH_MNT +_scratch_unmount rwf2=`get_features features2` # @@ -87,7 +87,7 @@ xfs_db -x $SCRATCH_DEV -c 'sb' -c 'write features2 0' # _scratch_mount -o ro _scratch_mount -o remount,rw -umount $SCRATCH_MNT +_scratch_unmount rof2=`get_features features2` [ "$f2" != "$bf2" ] && echo "mkfs: features2 $f2 != bad_features2 $bf2" diff --git a/tests/xfs/201 b/tests/xfs/201 index 921d2a3f..ac8abf49 100755 --- a/tests/xfs/201 +++ b/tests/xfs/201 @@ -34,7 +34,7 @@ status=1 # failure is the default! _cleanup() { - umount $SCRATCH_MNT + _scratch_unmount } trap "_cleanup; exit \$status" 0 1 2 3 15 diff --git a/tests/xfs/220 b/tests/xfs/220 index 780aa447..69b539cc 100755 --- a/tests/xfs/220 +++ b/tests/xfs/220 @@ -36,7 +36,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_DEV >/dev/null 2>&1 + _scratch_unmount >/dev/null 2>&1 } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -64,7 +64,7 @@ _scratch_mount -o uquota xfs_quota -x -c off $SCRATCH_DEV # and unmount (this used to crash) -umount $SCRATCH_DEV +_scratch_unmount # create scratch filesystem _scratch_mkfs_xfs >/dev/null 2>&1 @@ -77,7 +77,7 @@ _scratch_mount -o uquota xfs_quota -x -c off -c remove $SCRATCH_DEV # and unmount again -umount $SCRATCH_DEV +_scratch_unmount status=0 exit $status diff --git a/tests/xfs/227 b/tests/xfs/227 index 3816e9fc..64e8f147 100755 --- a/tests/xfs/227 +++ b/tests/xfs/227 @@ -161,7 +161,7 @@ _scratch_mount fragment_freespace # unmount and remount to reset all allocator indexes -umount $SCRATCH_MNT +_scratch_unmount _scratch_mount # create a range of source files, then fsr them to a known size @@ -205,6 +205,6 @@ for n in `seq 4 1 12`; do done done -umount $SCRATCH_MNT +_scratch_unmount echo "--- silence is golden ---" status=0 ; exit diff --git a/tests/xfs/244 b/tests/xfs/244 index 6dd8224e..257d121a 100755 --- a/tests/xfs/244 +++ b/tests/xfs/244 @@ -39,7 +39,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15 _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } @@ -96,7 +96,7 @@ if $XFS_QUOTA_PROG -x -c "project -s -p $dir/over32bit 5344967296" $SCRATCH_DEV fi # Do testing on filesystem with projid32bit feature enabled -umount $SCRATCH_DEV 2>/dev/null +_scratch_unmount 2>/dev/null _scratch_mkfs_xfs -i projid32bit=1 >> $seqres.full || _fail "mkfs failed" _qmount mkdir $dir diff --git a/tests/xfs/287 b/tests/xfs/287 index b9276a72..aca410ab 100755 --- a/tests/xfs/287 +++ b/tests/xfs/287 @@ -40,7 +40,7 @@ rm -f $seqres.full _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -rf $tmp.* } @@ -84,7 +84,7 @@ echo "No 32bit project quotas:" $XFS_IO_PROG -r -c "lsproj" $dir/16bit $XFS_IO_PROG -r -c "lsproj" $dir/32bit -umount $SCRATCH_MNT +_scratch_unmount # Now, enable projid32bit support by xfs_admin xfs_admin -p $SCRATCH_DEV >> $seqres.full 2>&1 || _fail "xfs_admin failed" @@ -129,7 +129,7 @@ $XFS_IO_PROG -r -c "lsproj" $restore_dir/16bit $XFS_IO_PROG -r -c "lsproj" $restore_dir/32bit $XFS_IO_PROG -r -c "lsproj" $restore_dir/32bitv2 -umount $SCRATCH_MNT +_scratch_unmount # Now, we can examine the file systems with xfs_db echo "These two values of 16bit project quota ids shall be the same" diff --git a/tests/xfs/299 b/tests/xfs/299 index d1909743..1f61eea4 100755 --- a/tests/xfs/299 +++ b/tests/xfs/299 @@ -40,7 +40,7 @@ status=1 # failure is the default! _cleanup() { cd / - umount $SCRATCH_MNT 2>/dev/null + _scratch_unmount 2>/dev/null rm -f $tmp.* } trap "_cleanup; exit \$status" 0 1 2 3 15 @@ -225,7 +225,7 @@ _exercise g _exercise u echo "*** unmount" -umount $SCRATCH_MNT +_scratch_unmount _scratch_mkfs_xfs -m crc=1 2>/dev/null | _filter_mkfs 2>$tmp.mkfs @@ -241,7 +241,7 @@ _exercise gno _exercise pno echo "*** unmount" -umount $SCRATCH_MNT +_scratch_unmount # success, all done status=0 diff --git a/tests/xfs/304 b/tests/xfs/304 index ef8643a1..0f902fc5 100755 --- a/tests/xfs/304 +++ b/tests/xfs/304 @@ -63,7 +63,7 @@ echo "*** turn off group quotas" xfs_quota -x -c 'off -g' $SCRATCH_MNT rmdir $QUOTA_DIR echo "*** umount" -umount $SCRATCH_MNT +_scratch_unmount _qmount mkdir -p $QUOTA_DIR @@ -71,7 +71,7 @@ echo "*** turn off project quotas" xfs_quota -x -c 'off -p' $SCRATCH_MNT rmdir $QUOTA_DIR echo "*** umount" -umount $SCRATCH_MNT +_scratch_unmount _qmount mkdir -p $QUOTA_DIR @@ -79,7 +79,7 @@ echo "*** turn off group/project quotas" xfs_quota -x -c 'off -gp' $SCRATCH_MNT rmdir $QUOTA_DIR echo "*** umount" -umount $SCRATCH_MNT +_scratch_unmount # success, all done status=0 -- 2.30.2