overlay/065: adjust test to expect EBUSY only with index=on
PATCH v2 1/1 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 <[email protected]>
---
Eryu,
I piggy backed two minor cleanups in this update on top of your requested change.
Thanks,
Amir.
Changes since v1:
- Add test variants with index=on/off
- Fix wring comment about expect EEXIST
- Make golden output deterministic by changing 'find $mnt2' to find '$mnt2/upper'
tests/overlay/065 | 56 +++++++++++++++++++++++++++++-------------- tests/overlay/065.out | 12 +++++----- 2 files changed, 44 insertions(+), 24 deletions(-)
tests/overlay/065 +38 -18
@@ -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
tests/overlay/065.out +6 -6
@@ -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
Signature · 1 line
2.17.1