Message ID | 20210805154328.GB3601392@magnolia |
---|---|
State | Accepted |
Headers | show |
Series | [RESEND] tests: skip u_direct_io if losetup fails | expand |
On Thu, Aug 05, 2021 at 08:43:28AM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@kernel.org> > > This new test requires a loop device to run testing. While it checks > for some "obvious" parameters that might cause the test to fail such as > not being root and no losetup executable, it doesn't actually check that > the losetup -a call succeeds. This causes a test regression in my > package building container (where there is only a minimal /dev with no > loop devices available) so I can't build debian packages. > > Fix the test to skip out if we can't create a loop device. > > Signed-off-by: Darrick J. Wong <djwong@kernel.org> Applied, thanks. - Ted
diff --git a/tests/u_direct_io/script b/tests/u_direct_io/script index 0b5d7083..b4f07752 100644 --- a/tests/u_direct_io/script +++ b/tests/u_direct_io/script @@ -9,6 +9,11 @@ elif test ! -x $DEBUGFS_EXE; then else dd if=/dev/zero of=$TMPFILE bs=1M count=128 > /dev/null 2>&1 LOOP=$(losetup --show --sector-size 4096 -f $TMPFILE) + if [ ! -b "$LOOP" ]; then + echo "$test_name: $DESCRIPTION: skipped (no loop devices)" + rm -f $TMPFILE + exit 0 + fi echo mke2fs -F -o Linux -t ext4 -O ^metadata_csum,^uninit_bg -D \$LOOP > $OUT $MKE2FS -F -o Linux -t ext4 -O ^metadata_csum,^uninit_bg -D $LOOP 2>&1 | \ sed -f $cmd_dir/filter.sed >> $OUT