All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anna Schumaker <Anna.Schumaker@netapp.com>
To: fstests@vger.kernel.org
Cc: Anna.Schumaker@netapp.com, linux-nfs@vger.kernel.org, hch@infradead.org
Subject: [PATCH 5/5] generic/346: Add a copy test for overwriting small amounts of data
Date: Fri, 13 May 2016 16:50:52 -0400	[thread overview]
Message-ID: <1463172652-22361-6-git-send-email-Anna.Schumaker@Netapp.com> (raw)
In-Reply-To: <1463172652-22361-1-git-send-email-Anna.Schumaker@Netapp.com>

This test is similar to 345, except that it copies one byte at a time to
make sure that this case works as expected.

Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
---
 tests/generic/346     | 94 +++++++++++++++++++++++++++++++++++++++++++++++++++
 tests/generic/346.out | 17 ++++++++++
 tests/generic/group   |  1 +
 3 files changed, 112 insertions(+)
 create mode 100644 tests/generic/346
 create mode 100644 tests/generic/346.out

diff --git a/tests/generic/346 b/tests/generic/346
new file mode 100644
index 0000000..4154bd5
--- /dev/null
+++ b/tests/generic/346
@@ -0,0 +1,94 @@
+#!/bin/bash
+# FS QA Test No. 346
+#
+# Tests vfs_copy_file_range():
+#   - Copy a small file
+#   - Use copy to swap data at beginning and end
+#   - Use copy to swap data in the middle
+#   - Use copy to swap data in a small file
+#-----------------------------------------------------------------------
+# Copyright (c) 2016 Netapp, 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 /
+	rm -rf $tmp.*
+}
+
+# get standard environment
+. common/rc
+. common/filter
+
+# real QA test starts here
+_supported_os Linux
+
+_require_copy_file_range
+_require_test
+
+testdir=$TEST_DIR/test-$seq
+rm -rf $testdir
+mkdir $testdir
+
+_checksum_files()
+{
+	for f in file copy; do
+		md5sum $testdir/$f | _filter_test_dir
+	done
+}
+
+rm -f $seqres.full
+
+echo "Create the original file and then copy"
+echo -n "abcde" > $testdir/file
+$COPY_FILE_RANGE_PROG $testdir/file $testdir/copy
+cmp $testdir/file $testdir/copy
+echo "Original md5sums:"
+_checksum_files
+
+echo "Swap beginning and end of original file"
+$COPY_FILE_RANGE_PROG -s 0 -d 4 -l 1 $testdir/file $testdir/copy
+$COPY_FILE_RANGE_PROG -s 4 -d 0 -l 1 $testdir/file $testdir/copy
+echo -n "ebcda" | cmp $testdir/copy
+echo "md5sums after swapping beginning and end:"
+_checksum_files
+
+echo "Swap middle parts of original file"
+$COPY_FILE_RANGE_PROG -s 1 -d 3 -l 1 $testdir/file $testdir/copy
+$COPY_FILE_RANGE_PROG -s 3 -d 1 -l 1 $testdir/file $testdir/copy
+echo -n "edcba" | cmp $testdir/copy
+echo "md5sums after swapping middle:"
+_checksum_files
+
+echo "Copy tail of original file onto copy"
+$COPY_FILE_RANGE_PROG -s 1 -d 3 -l 4 $testdir/file $testdir/copy
+echo -n "edcbcde" | cmp $testdir/copy
+echo "md5sums after copying tail:"
+_checksum_files
+
+#success, all done
+status=0
+exit
diff --git a/tests/generic/346.out b/tests/generic/346.out
new file mode 100644
index 0000000..0d37b62
--- /dev/null
+++ b/tests/generic/346.out
@@ -0,0 +1,17 @@
+QA output created by 346
+Create the original file and then copy
+Original md5sums:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/copy
+Swap beginning and end of original file
+md5sums after swapping beginning and end:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+32db1f6d06d15f7e38e1ab1c69da498a  TEST_DIR/test-346/copy
+Swap middle parts of original file
+md5sums after swapping middle:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+295228f3d82d344bbcf2f0030519c2ea  TEST_DIR/test-346/copy
+Copy tail of original file onto copy
+md5sums after copying tail:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+0c4aac952f72fa078e2f8419aca70b28  TEST_DIR/test-346/copy
diff --git a/tests/generic/group b/tests/generic/group
index 69696c7..8a8ba8d 100644
--- a/tests/generic/group
+++ b/tests/generic/group
@@ -348,3 +348,4 @@
 343 auto quick copy
 344 auto quick copy
 345 auto quick copy
+346 auto quick copy
-- 
2.8.2


WARNING: multiple messages have this Message-ID (diff)
From: Anna Schumaker <Anna.Schumaker@netapp.com>
To: <fstests@vger.kernel.org>
Cc: <Anna.Schumaker@netapp.com>, <linux-nfs@vger.kernel.org>,
	<hch@infradead.org>
Subject: [PATCH 5/5] generic/346: Add a copy test for overwriting small amounts of data
Date: Fri, 13 May 2016 16:50:52 -0400	[thread overview]
Message-ID: <1463172652-22361-6-git-send-email-Anna.Schumaker@Netapp.com> (raw)
In-Reply-To: <1463172652-22361-1-git-send-email-Anna.Schumaker@Netapp.com>

This test is similar to 345, except that it copies one byte at a time to
make sure that this case works as expected.

Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
---
 tests/generic/346     | 94 +++++++++++++++++++++++++++++++++++++++++++++++++++
 tests/generic/346.out | 17 ++++++++++
 tests/generic/group   |  1 +
 3 files changed, 112 insertions(+)
 create mode 100644 tests/generic/346
 create mode 100644 tests/generic/346.out

diff --git a/tests/generic/346 b/tests/generic/346
new file mode 100644
index 0000000..4154bd5
--- /dev/null
+++ b/tests/generic/346
@@ -0,0 +1,94 @@
+#!/bin/bash
+# FS QA Test No. 346
+#
+# Tests vfs_copy_file_range():
+#   - Copy a small file
+#   - Use copy to swap data at beginning and end
+#   - Use copy to swap data in the middle
+#   - Use copy to swap data in a small file
+#-----------------------------------------------------------------------
+# Copyright (c) 2016 Netapp, 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 /
+	rm -rf $tmp.*
+}
+
+# get standard environment
+. common/rc
+. common/filter
+
+# real QA test starts here
+_supported_os Linux
+
+_require_copy_file_range
+_require_test
+
+testdir=$TEST_DIR/test-$seq
+rm -rf $testdir
+mkdir $testdir
+
+_checksum_files()
+{
+	for f in file copy; do
+		md5sum $testdir/$f | _filter_test_dir
+	done
+}
+
+rm -f $seqres.full
+
+echo "Create the original file and then copy"
+echo -n "abcde" > $testdir/file
+$COPY_FILE_RANGE_PROG $testdir/file $testdir/copy
+cmp $testdir/file $testdir/copy
+echo "Original md5sums:"
+_checksum_files
+
+echo "Swap beginning and end of original file"
+$COPY_FILE_RANGE_PROG -s 0 -d 4 -l 1 $testdir/file $testdir/copy
+$COPY_FILE_RANGE_PROG -s 4 -d 0 -l 1 $testdir/file $testdir/copy
+echo -n "ebcda" | cmp $testdir/copy
+echo "md5sums after swapping beginning and end:"
+_checksum_files
+
+echo "Swap middle parts of original file"
+$COPY_FILE_RANGE_PROG -s 1 -d 3 -l 1 $testdir/file $testdir/copy
+$COPY_FILE_RANGE_PROG -s 3 -d 1 -l 1 $testdir/file $testdir/copy
+echo -n "edcba" | cmp $testdir/copy
+echo "md5sums after swapping middle:"
+_checksum_files
+
+echo "Copy tail of original file onto copy"
+$COPY_FILE_RANGE_PROG -s 1 -d 3 -l 4 $testdir/file $testdir/copy
+echo -n "edcbcde" | cmp $testdir/copy
+echo "md5sums after copying tail:"
+_checksum_files
+
+#success, all done
+status=0
+exit
diff --git a/tests/generic/346.out b/tests/generic/346.out
new file mode 100644
index 0000000..0d37b62
--- /dev/null
+++ b/tests/generic/346.out
@@ -0,0 +1,17 @@
+QA output created by 346
+Create the original file and then copy
+Original md5sums:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/copy
+Swap beginning and end of original file
+md5sums after swapping beginning and end:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+32db1f6d06d15f7e38e1ab1c69da498a  TEST_DIR/test-346/copy
+Swap middle parts of original file
+md5sums after swapping middle:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+295228f3d82d344bbcf2f0030519c2ea  TEST_DIR/test-346/copy
+Copy tail of original file onto copy
+md5sums after copying tail:
+ab56b4d92b40713acc5af89985d4b786  TEST_DIR/test-346/file
+0c4aac952f72fa078e2f8419aca70b28  TEST_DIR/test-346/copy
diff --git a/tests/generic/group b/tests/generic/group
index 69696c7..8a8ba8d 100644
--- a/tests/generic/group
+++ b/tests/generic/group
@@ -348,3 +348,4 @@
 343 auto quick copy
 344 auto quick copy
 345 auto quick copy
+346 auto quick copy
-- 
2.8.2


  parent reply	other threads:[~2016-05-13 20:51 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-13 20:50 [PATCH 0/5] Add copy_file_range() tests Anna Schumaker
2016-05-13 20:50 ` Anna Schumaker
2016-05-13 20:50 ` [PATCH 1/5] src/copy_file_range: Add a program for testing vfs_copy_file_range() Anna Schumaker
2016-05-13 20:50   ` Anna Schumaker
2016-05-16 23:20   ` Dave Chinner
2016-05-17 14:47     ` Anna Schumaker
2016-05-17 14:47       ` Anna Schumaker
2016-05-17 17:48       ` Darrick J. Wong
2016-05-17 17:52         ` Anna Schumaker
2016-05-17 17:52           ` Anna Schumaker
2016-05-13 20:50 ` [PATCH 2/5] generic/343: Add copy to new file test Anna Schumaker
2016-05-13 20:50   ` Anna Schumaker
2016-05-13 20:50 ` [PATCH 3/5] generic/344: Add small copies " Anna Schumaker
2016-05-13 20:50   ` Anna Schumaker
2016-05-13 20:50 ` [PATCH 4/5] generic/345: Add copy test that overwrites data Anna Schumaker
2016-05-13 20:50   ` Anna Schumaker
2016-05-13 20:50 ` Anna Schumaker [this message]
2016-05-13 20:50   ` [PATCH 5/5] generic/346: Add a copy test for overwriting small amounts of data Anna Schumaker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1463172652-22361-6-git-send-email-Anna.Schumaker@Netapp.com \
    --to=anna.schumaker@netapp.com \
    --cc=fstests@vger.kernel.org \
    --cc=hch@infradead.org \
    --cc=linux-nfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.