More tests for the NULL files problem

Merge of master-melb:xfs-cmds:27632a by kenmcd.

  Fix test number and simplify test case

8 files changed, 253 insertions, 11 deletions

diff --git a/137 b/137
index 3921078a..28352ebe 100755
--- a/137
+++ b/137
@@ -1,5 +1,5 @@
 #! /bin/sh
-# FSQA Test No. 133
+# FSQA Test No. 137
 #
 # Test for NULL files problem
 #
@@ -71,16 +71,6 @@ do
 			elif xfs_bmap $file | grep 'hole' > /dev/null
 			then
 				echo corrupt file $file - contains holes
-			else
-				for byte in `od -td1 -w16 -v $file | tr -s ' ' | cut -d ' ' -f 2-17`
-				do
-					# each byte in the file must not be 0
-					if [ $byte -eq 0 ]
-					then
-						echo corrupt file $file - NULLs found
-						break;
-					fi
-				done
 			fi
 		fi
 		rm -f $file
diff --git a/138 b/138
new file mode 100755
index 00000000..c0c37719
--- /dev/null
+++ b/138
@@ -0,0 +1,83 @@
+#! /bin/sh
+# FSQA Test No. 138
+#
+# Test for NULL files problem
+#
+#-----------------------------------------------------------------------
+#  Copyright (c) 2006 Silicon Graphics, Inc.  All Rights Reserved.
+#-----------------------------------------------------------------------
+#
+# creator
+owner=lachlan@sgi.com
+
+seq=`basename $0`
+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()
+{
+    _cleanup_testdir
+}
+
+# get standard environment, filters and checks
+. ./common.rc
+. ./common.filter
+
+# real QA test starts here
+_supported_fs xfs
+_supported_os Linux IRIX
+
+_setup_testdir
+_require_scratch
+_scratch_mkfs_xfs >/dev/null 2>&1
+_scratch_mount
+
+# create files
+i=1;
+while [ $i -lt 1000 ]
+do
+	file=$SCRATCH_MNT/$i
+	xfs_io -f -c "pwrite -b 64k -S 0xff 0 64k" $file > /dev/null
+	xfs_io -c "truncate 64k" $file > /dev/null
+	i=`expr $i + 1`
+done
+
+# give the system a chance to write something out
+sleep 10
+
+src/godown $SCRATCH_MNT
+
+umount $SCRATCH_MNT
+_scratch_mount
+
+# check file size and contents
+i=1;
+while [ $i -lt 1000 ]
+do
+	file=$SCRATCH_MNT/$i
+	# if file does not exist, the create was not logged, skip it
+	if [ -e $file ]
+	then
+		# if file size is zero it cannot be corrupt, skip it
+		if [ -s $file ]
+		then
+			# if file has non-zero size but no extents then it's contents will be NULLs, bad.
+			if xfs_bmap $file | grep 'no extents' > /dev/null
+			then
+				echo corrupt file $file - non-zero size but no extents
+			elif xfs_bmap $file | grep 'hole' > /dev/null
+			then
+				echo corrupt file $file - contains holes
+			fi
+		fi
+		rm -f $file
+	fi
+	i=`expr $i + 1`
+done
+
+status=0
+exit 
diff --git a/138.out b/138.out
new file mode 100644
index 00000000..f3eb4116
--- /dev/null
+++ b/138.out
@@ -0,0 +1 @@
+QA output created by 138
diff --git a/139 b/139
new file mode 100755
index 00000000..386dcbc4
--- /dev/null
+++ b/139
@@ -0,0 +1,83 @@
+#! /bin/sh
+# FSQA Test No. 139
+#
+# Test for NULL files problem
+#
+#-----------------------------------------------------------------------
+#  Copyright (c) 2006 Silicon Graphics, Inc.  All Rights Reserved.
+#-----------------------------------------------------------------------
+#
+# creator
+owner=lachlan@sgi.com
+
+seq=`basename $0`
+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()
+{
+    _cleanup_testdir
+}
+
+# get standard environment, filters and checks
+. ./common.rc
+. ./common.filter
+
+# real QA test starts here
+_supported_fs xfs
+_supported_os Linux IRIX
+
+_setup_testdir
+_require_scratch
+_scratch_mkfs_xfs >/dev/null 2>&1
+_scratch_mount
+
+# create files
+i=1;
+while [ $i -lt 1000 ]
+do
+	file=$SCRATCH_MNT/$i
+	xfs_io -f -c "pwrite -b 64k -S 0xff 0 64k" $file > /dev/null
+	xfs_io -f -c "truncate 32k" $file > /dev/null
+	i=`expr $i + 1`
+done
+
+# give the system a chance to write something out
+sleep 10
+
+src/godown $SCRATCH_MNT
+
+umount $SCRATCH_MNT
+_scratch_mount
+
+# check file size and contents
+i=1;
+while [ $i -lt 1000 ]
+do
+	file=$SCRATCH_MNT/$i
+	# if file does not exist, the create was not logged, skip it
+	if [ -e $file ]
+	then
+		# if file size is zero it cannot be corrupt, skip it
+		if [ -s $file ]
+		then
+			# if file has non-zero size but no extents then it's contents will be NULLs, bad.
+			if xfs_bmap $file | grep 'no extents' > /dev/null
+			then
+				echo corrupt file $file - non-zero size but no extents
+			elif xfs_bmap $file | grep 'hole' > /dev/null
+			then
+				echo corrupt file $file - contains holes
+			fi
+		fi
+		rm -f $file
+	fi
+	i=`expr $i + 1`
+done
+
+status=0
+exit 
diff --git a/139.out b/139.out
new file mode 100644
index 00000000..be0ebc21
--- /dev/null
+++ b/139.out
@@ -0,0 +1 @@
+QA output created by 139
diff --git a/140 b/140
new file mode 100755
index 00000000..5c4bbf1d
--- /dev/null
+++ b/140
@@ -0,0 +1,80 @@
+#! /bin/sh
+# FSQA Test No. 140
+#
+# Test for NULL files problem
+#
+#-----------------------------------------------------------------------
+#  Copyright (c) 2006 Silicon Graphics, Inc.  All Rights Reserved.
+#-----------------------------------------------------------------------
+#
+# creator
+owner=lachlan@sgi.com
+
+seq=`basename $0`
+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()
+{
+    _cleanup_testdir
+}
+
+# get standard environment, filters and checks
+. ./common.rc
+. ./common.filter
+
+# real QA test starts here
+_supported_fs xfs
+_supported_os Linux IRIX
+
+_setup_testdir
+_require_scratch
+_scratch_mkfs_xfs >/dev/null 2>&1
+_scratch_mount
+
+# create files
+i=1;
+while [ $i -lt 1000 ]
+do
+	file=$SCRATCH_MNT/$i
+	xfs_io -f -c "pwrite -b 32k -S 0xff 0 32k" $file > /dev/null
+	xfs_io -f -c "truncate 64k" $file > /dev/null
+	i=`expr $i + 1`
+done
+
+# give the system a chance to write something out
+sleep 10
+
+src/godown $SCRATCH_MNT
+
+umount $SCRATCH_MNT
+_scratch_mount
+
+# check file size and contents
+i=1;
+while [ $i -lt 1000 ]
+do
+	file=$SCRATCH_MNT/$i
+	# if file does not exist, the create was not logged, skip it
+	if [ -e $file ]
+	then
+		# if file size is zero it cannot be corrupt, skip it
+		if [ -s $file ]
+		then
+			# if file has non-zero size but no extents then it's contents will be NULLs, bad.
+			if xfs_bmap $file | grep 'no extents' > /dev/null
+			then
+				echo corrupt file $file - non-zero size but no extents
+			fi
+		fi
+		rm -f $file
+	fi
+	i=`expr $i + 1`
+done
+
+status=0
+exit 
diff --git a/140.out b/140.out
new file mode 100644
index 00000000..e6506b89
--- /dev/null
+++ b/140.out
@@ -0,0 +1 @@
+QA output created by 140
diff --git a/group b/group
index a51fad60..38c0bc04 100644
--- a/group
+++ b/group
@@ -217,3 +217,6 @@ pattern         ajones@sgi.com
 135 metadata auto
 136 attr2 
 137 metadata log
+138 metadata log
+139 metadata log
+140 metadata log