overlay/065: adjust test to expect EBUSY only with index=on

This is needed to support the kernel regression fix commit 0be0bfd2de9d
("ovl: fix regression caused by overlapping layers detection").

Overlayfs mount is not supposed to fail due to upper/work dir in-use
by other mount unless option index=on is enabled.

Add test variants for index=on and index=off.

Fix some wrong comments and clean noise lines in golden output.

Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Reviewed-by: Eryu Guan <guaneryu@gmail.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
This commit is contained in:
Amir Goldstein
2019-07-28 13:21:24 +03:00
committed by Eryu Guan
parent 6a04d0f74e
commit 04d08996e9
2 changed files with 44 additions and 24 deletions
+38 -18
View File
@@ -13,9 +13,14 @@
#
# Overlapping layers on mount or lookup results in ELOOP.
# Overlapping lowerdir with other mount upperdir/workdir
# result in EBUSY.
# result in EBUSY (only if index=on is used).
#
# Kernel patch "ovl: detect overlapping layers" is needed to pass the test.
# This is a regression test for kernel commit:
#
# 146d62e5a586 "ovl: detect overlapping layers"
#
# and its followup fix commit:
# 0be0bfd2de9d "ovl: fix regression caused by overlapping layers detection"
#
seq=`basename $0`
seqres=$RESULT_DIR/$seq
@@ -46,6 +51,7 @@ _supported_os Linux
# Use non-default scratch underlying overlay dirs, we need to check
# them explicity after test.
_require_scratch_nocheck
_require_scratch_feature index
# Remove all files from previous tests
_scratch_mkfs
@@ -61,10 +67,10 @@ mnt2=$basedir/mnt.2
mkdir -p $lowerdir/lower $upperdir $workdir
# Try to mount an overlay with the same upperdir/lowerdir - expect EBUSY
# Try to mount an overlay with the same upperdir/lowerdir - expect ELOOP
echo Conflicting upperdir/lowerdir
_overlay_scratch_mount_dirs $upperdir $upperdir $workdir \
2>&1 | _filter_busy_mount
2>&1 | _filter_error_mount
$UMOUNT_PROG $SCRATCH_MNT 2>/dev/null
# Use new upper/work dirs for each test to avoid ESTALE errors
@@ -72,11 +78,11 @@ $UMOUNT_PROG $SCRATCH_MNT 2>/dev/null
rm -rf $upperdir $workdir
mkdir $upperdir $workdir
# Try to mount an overlay with the same workdir/lowerdir - expect EBUSY
# Try to mount an overlay with the same workdir/lowerdir - expect ELOOP
# because $workdir/work overlaps with lowerdir
echo Conflicting workdir/lowerdir
_overlay_scratch_mount_dirs $workdir $upperdir $workdir \
2>&1 | _filter_busy_mount
-oindex=off 2>&1 | _filter_error_mount
$UMOUNT_PROG $SCRATCH_MNT 2>/dev/null
rm -rf $upperdir $workdir
@@ -91,7 +97,7 @@ $UMOUNT_PROG $SCRATCH_MNT 2>/dev/null
rm -rf $upperdir $workdir
mkdir $upperdir $workdir
# Try to mount an overlay with the same lower layers - expect EEXIST
# Try to mount an overlay with the same lower layers - expect ELOOP
echo Conflicting lower layers
_overlay_scratch_mount_dirs $lowerdir:$lowerdir $upperdir $workdir \
2>&1 | _filter_error_mount
@@ -118,36 +124,50 @@ $UMOUNT_PROG $SCRATCH_MNT 2>/dev/null
rm -rf $upperdir $workdir
mkdir -p $upperdir/upper $workdir $mnt2
# Mount overlay with non overlapping lowerdir, upperdir, workdir - expect
# success
# Mount overlay with non overlapping lowerdir, upperdir, workdir -
# expect success
_overlay_mount_dirs $lowerdir $upperdir $workdir overlay $mnt2
rm -rf $upperdir2 $workdir2
mkdir -p $upperdir2 $workdir2 $mnt2
# Try to mount an overlay with layers overlapping with another overlayfs
# upperdir - expect EBUSY
echo Overlapping with upperdir of another instance
# upperdir - expect EBUSY with index=on and success with index=off
echo "Overlapping with upperdir of another instance (index=on)"
_overlay_scratch_mount_dirs $upperdir/upper $upperdir2 $workdir2 \
2>&1 | _filter_busy_mount
-oindex=on 2>&1 | _filter_busy_mount
$UMOUNT_PROG $SCRATCH_MNT 2>/dev/null
rm -rf $upperdir2 $workdir2
mkdir -p $upperdir2 $workdir2
echo "Overlapping with upperdir of another instance (index=off)"
_overlay_scratch_mount_dirs $upperdir/upper $upperdir2 $workdir2 \
-oindex=off && $UMOUNT_PROG $SCRATCH_MNT
rm -rf $upperdir2 $workdir2
mkdir -p $upperdir2 $workdir2
# Try to mount an overlay with layers overlapping with another overlayfs
# workdir - expect EBUSY
echo Overlapping with workdir of another instance
# workdir - expect EBUSY with index=on and success with index=off
echo "Overlapping with workdir of another instance (index=on)"
_overlay_scratch_mount_dirs $workdir/work $upperdir2 $workdir2 \
2>&1 | _filter_busy_mount
-oindex=on 2>&1 | _filter_busy_mount
$UMOUNT_PROG $SCRATCH_MNT 2>/dev/null
rm -rf $upperdir2 $workdir2
mkdir -p $upperdir2 $workdir2
echo "Overlapping with workdir of another instance (index=off)"
_overlay_scratch_mount_dirs $workdir/work $upperdir2 $workdir2 \
-oindex=off && $UMOUNT_PROG $SCRATCH_MNT
# Move upper layer root into lower layer after mount
echo Overlapping upperdir/lowerdir after mount
mv $upperdir $lowerdir/
# Lookup files in overlay mount with overlapping layers - expect ELOOP
# when overlay merge dir is found
find $mnt2 2>&1 | _filter_scratch
# Lookup files in overlay mount with overlapping layers -
# expect ELOOP when upper layer root is found in lower layer
find $mnt2/upper 2>&1 | _filter_scratch
# success, all done
status=0
+6 -6
View File
@@ -1,8 +1,8 @@
QA output created by 065
Conflicting upperdir/lowerdir
mount: device already mounted or mount point busy
mount: Too many levels of symbolic links
Conflicting workdir/lowerdir
mount: device already mounted or mount point busy
mount: Too many levels of symbolic links
Overlapping upperdir/lowerdir
mount: Too many levels of symbolic links
Conflicting lower layers
@@ -11,11 +11,11 @@ Overlapping lower layers below
mount: Too many levels of symbolic links
Overlapping lower layers above
mount: Too many levels of symbolic links
Overlapping with upperdir of another instance
Overlapping with upperdir of another instance (index=on)
mount: device already mounted or mount point busy
Overlapping with workdir of another instance
Overlapping with upperdir of another instance (index=off)
Overlapping with workdir of another instance (index=on)
mount: device already mounted or mount point busy
Overlapping with workdir of another instance (index=off)
Overlapping upperdir/lowerdir after mount
SCRATCH_DEV/mnt.2
SCRATCH_DEV/mnt.2/lower
find: 'SCRATCH_DEV/mnt.2/upper': Too many levels of symbolic links