generic: test partial block device failure

Calls like fsync() should report failure on partial I/O failure, e.g. a
single failed disk in a raid 0 stripe.

This test is motivated by an XFS bug, and this commit fixed the issue
xfs: return errors from partial I/O failures to files

This case is written by David Jeffery <djeffery@redhat.com> originally.

Signed-off-by: Eryu Guan <eguan@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>

diff --git a/tests/generic/108 b/tests/generic/108
new file mode 100755 (executable)
index 0000000..ff3d3e6
--- /dev/null
+++ b/tests/generic/108
@@ -0,0 +1,98 @@
+#! /bin/bash
+# FS QA Test 108
+#
+# Test partial block device failure. Calls like fsync() should report failure
+# on partial I/O failure, e.g. a single failed disk in a raid 0 stripe.
+#
+# Test motivated by an XFS bug, and this commit fixed the issue
+# xfs: return errors from partial I/O failures to files
+#
+#-----------------------------------------------------------------------
+# Copyright (c) 2015 Red Hat Inc. All Rights Reserved.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it would be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write the Free Software Foundation,
+# Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+#-----------------------------------------------------------------------
+#
+
+seq=`basename $0`
+seqres=$RESULT_DIR/$seq
+echo "QA output created by $seq"
+
+here=`pwd`
+tmp=/tmp/$$
+status=1       # failure is the default!
+trap "_cleanup; exit \$status" 0 1 2 3 15
+
+_cleanup()
+{
+       cd /
+       echo running > /sys/block/`_short_dev $SCSI_DEBUG_DEV`/device/state
+       $UMOUNT_PROG $SCRATCH_MNT >>$seqres.full 2>&1
+       $LVM_PROG vgremove -f $vgname >>$seqres.full 2>&1
+       $LVM_PROG pvremove -f $SCRATCH_DEV $SCSI_DEBUG_DEV >>$seqres.full 2>&1
+       _put_scsi_debug_dev
+       rm -f $tmp.*
+}
+
+# get standard environment, filters and checks
+. ./common/rc
+. ./common/filter
+. ./common/scsi_debug
+
+# real QA test starts here
+_supported_fs generic
+_supported_os Linux
+_require_scratch_nocheck
+_require_block_device $SCRATCH_DEV
+_require_scsi_debug
+_require_command "$LVM_PROG" lvm
+
+lvname=lv_$seq
+vgname=vg_$seq
+rm -f $seqres.full
+
+# _get_scsi_debug_dev returns a scsi debug device with 128M in size by default
+SCSI_DEBUG_DEV=`_get_scsi_debug_dev`
+if [ "$SCSI_DEBUG_DEV" == "" ]; then
+       _fail "Failed to initialize scsi debug device"
+fi
+echo "SCSI debug device $SCSI_DEBUG_DEV" >>$seqres.full
+
+# create striped volume with 4MB stripe size
+$LVM_PROG pvcreate -f $SCSI_DEBUG_DEV $SCRATCH_DEV >>$seqres.full 2>&1
+$LVM_PROG vgcreate -f $vgname $SCSI_DEBUG_DEV $SCRATCH_DEV >>$seqres.full 2>&1
+$LVM_PROG lvcreate --yes -i 2 -I 4m -l 100%FREE -n $lvname $vgname \
+       >>$seqres.full 2>&1
+# wait for lv creation to fully complete
+$UDEV_SETTLE_PROG >>$seqres.full 2>&1
+
+# _mkfs_dev exits the test on failure, this makes sure test lv is created by
+# above vgcreate/lvcreate operations
+_mkfs_dev /dev/mapper/$vgname-$lvname
+
+_mount /dev/mapper/$vgname-$lvname $SCRATCH_MNT
+
+# create a test file with contiguous blocks which will span across the 2 disks
+$XFS_IO_PROG -f -c "pwrite 0 16M" -c fsync $SCRATCH_MNT/testfile >>$seqres.full
+
+# offline the scsi debug device
+echo offline > /sys/block/`_short_dev $SCSI_DEBUG_DEV`/device/state
+
+# write to an allocated area of the test file with writes which spans both disks
+# and call fsync, the fsync should report failure
+$XFS_IO_PROG -c "pwrite -b 1M 0 6M" -c fsync $SCRATCH_MNT/testfile \
+       >>$seqres.full
+
+status=0
+exit

diff --git a/tests/generic/108.out b/tests/generic/108.out
new file mode 100644 (file)
index 0000000..7f60868
--- /dev/null
+++ b/tests/generic/108.out
@@ -0,0 +1,2 @@
+QA output created by 108
+fsync: Input/output error

diff --git a/tests/generic/group b/tests/generic/group
index 123e8d4c301a92bf5562c321f67e7e9eaa82adca..4ae256fc54f915d65af0aaf72a16a9aba09decb9 100644 (file)
--- a/tests/generic/group
+++ b/tests/generic/group
@@ -110,6 +110,7 @@
 105 acl auto quick
 106 auto quick metadata
 107 auto quick metadata
+108 auto quick rw
 109 auto metadata dir
 112 rw aio auto quick
 113 rw aio auto quick