From 5cf8bb11001aebeed4af23e463ab77744af95058 Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Fri, 22 Jun 2018 11:42:14 -0400 Subject: [PATCH] tests: don't use a sparse test file If the TEST_BITS file is sparse, then the "debugfs -R write" command may skip holes in the file when copying it into the test image (depending on whether SEEK_HOLE/SEEK_DATA and/or FIEMAP are available in the copy_file() function). This was causing test failures on MacOS in the f_dup_resize and d_loaddump tests because the TEST_BITS file was the compiled "debugfs" binary, which apparently has holes when built on MacOS, and the number of blocks allocated in the test image was reduced as a result. This caused the expect output to differ in the summary line and resulted in failure. Instead of using the debugfs binary for TEST_BITS, generate a temporary file using /dev/urandom, if available. If not, fall back to the old behaviour or using debugfs. [ Fixed up use of non-POSIX /bin/sh constructs and add test_data.tmp to .gitignore and files to be deleted on a make clean. --tytso ] Signed-off-by: Andreas Dilger Signed-off-by: Theodore Ts'o --- .gitignore | 1 + tests/Makefile.in | 2 +- tests/test_config | 9 ++++++++- 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index ac5c2c1..df4021c 100644 --- a/.gitignore +++ b/.gitignore @@ -223,6 +223,7 @@ tests/*.failed tests/*.log tests/*.tmp tests/*.slow +tests/test_data.tmp tests/mke2fs.conf tests/test_script tests/test_one diff --git a/tests/Makefile.in b/tests/Makefile.in index 7b9df62..4a19758 100644 --- a/tests/Makefile.in +++ b/tests/Makefile.in @@ -94,7 +94,7 @@ testend: test_one ${TDIR}/image clean:: $(RM) -f *~ *.log *.new *.failed *.ok *.tmp *.slow - $(RM) -f test_one test_script mke2fs.conf + $(RM) -f test_one test_script mke2fs.conf test_data.tmp distclean:: clean $(RM) -f Makefile diff --git a/tests/test_config b/tests/test_config index c13aa74..595567f 100644 --- a/tests/test_config +++ b/tests/test_config @@ -13,7 +13,14 @@ E2IMAGE="$USE_VALGRIND ../misc/e2image" E2IMAGE_EXE="../misc/e2image" DEBUGFS="$USE_VALGRIND ../debugfs/debugfs" DEBUGFS_EXE="../debugfs/debugfs" -TEST_BITS="../debugfs/debugfs" +TEST_BITS="test_data.tmp" +if [ ! -s $TEST_BITS ]; then + # create a non-sparse test file if possible, since debugfs may be + # sparse and cause "debugfs write" (using copy_file()) to skip holes + # during testing if SEEK_DATA/SEEK_HOLE or FS_IOC_FIEMAP are available + dd if=/dev/urandom of=$TEST_BITS bs=128k count=1 > /dev/null 2>&1 || + TEST_BITS="$DEFBUGFS_EXE" +fi RESIZE2FS_EXE="../resize/resize2fs" RESIZE2FS="$USE_VALGRIND $RESIZE2FS_EXE" E2UNDO_EXE="../misc/e2undo" -- 1.8.3.1