[PATCH v2] xfstests: btrfs/316: cross-subvolume sparse copy

2013-07-01 Thread Koen De Wit
.html Two new common/rc functions used in this script (_require_cp_reflink and _verify_reflink) have been submitted recently: http://oss.sgi.com/archives/xfs/2013-05/msg00745.html Thanks to Eric Sandeen and Dave Chinner for the reviews. Signed-off-by: Koen De Wit koen.de@oracle.com --- tests

[PATCH v2] xfstests: btrfs/311: sparse copy between different filesystems/mountpoints

2013-07-01 Thread Koen De Wit
in error, # no file should be created. [sandeen: mostly cosmetic changes] Originally submitted as: xfstests: 301: sparse copy between different filesystems/mountpoints on btrfs Signed-off-by: Koen De Wit koen.de.wit@xx Signed-off-by: Eric Sandeen sandeen@xx --- tests/btrfs

[PATCH v3] xfstests: btrfs/316: cross-subvolume sparse copy

2013-07-02 Thread Koen De Wit
Two new common/rc functions used in this script (_require_cp_reflink and _verify_reflink) have been submitted recently: http://oss.sgi.com/archives/xfs/2013-05/msg00745.html Thanks to Eric Sandeen and Dave Chinner for the reviews. Version 3: fixing wrapped patch. Signed-off-by: Koen De Wit

[PATCH v3] xfstests: btrfs/311: sparse copy between different filesystems/mountpoints

2013-07-02 Thread Koen De Wit
in error, # no file should be created. [sandeen: mostly cosmetic changes] Originally submitted as: xfstests: 301: sparse copy between different filesystems/mountpoints on btrfs Version 3: fixing wrapped patch. Signed-off-by: Koen De Wit koen.de.wit@xx Signed-off-by: Eric Sandeen sandeen

Re: [PATCH v3] xfstests: btrfs/316: cross-subvolume sparse copy

2013-07-02 Thread Koen De Wit
idea to me, it makes debugging failures easier. Whose opinion should I follow? Koen. On 07/02/2013 12:15 PM, Dave Chinner wrote: On Tue, Jul 02, 2013 at 11:27:51AM +0200, Koen De Wit wrote: This testscript creates reflinks to files on different subvolumes, overwrites the original files

Re: [PATCH v3] xfstests: btrfs/316: cross-subvolume sparse copy

2013-07-03 Thread Koen De Wit
On 07/03/2013 08:37 AM, Dave Chinner wrote: On Tue, Jul 02, 2013 at 11:51:21AM -0400, Eric Sandeen wrote: On Jul 2, 2013, at 10:28 AM, Koen De Wit koen.de@oracle.com wrote: Dave, Thanks for the review. I will clean up the commit message and do a full mail-to-myself-and-test-patch

[PATCH v4] xfstests: btrfs/316: cross-subvolume sparse copy

2013-07-03 Thread Koen De Wit
This testscript creates reflinks to files on different subvolumes, overwrites the original files and reflinks, and moves reflinked files between subvolumes. Signed-off-by: Koen De Wit koen.de@oracle.com --- Two new common/rc functions used in this script (_require_cp_reflink

[PATCH v4] xfstests: btrfs/311: sparse copy between different filesystems/mountpoints

2013-07-03 Thread Koen De Wit
be created. - Copy a file with the reflink=always option. Should result in error, no file should be created. [sandeen: mostly cosmetic changes] Signed-off-by: Koen De Wit koen.de.wit@xx Signed-off-by: Eric Sandeen sandeen@xx --- Changelog: v3 - v4: Cleaning up commit message

[PATCH v2] xfstests: test for atime-related mount options

2014-02-17 Thread Koen De Wit
Tests the noatime, relatime, strictatime and nodiratime mount options. There is an extra check for Btrfs to ensure that the access time is never updated on read-only subvolumes. (Regression test for bug fixed with commit 93fd63c2f001ca6797c6b15b696a484b165b4800) Signed-off-by: Koen De Wit

Re: [PATCH] xfstests: test for atime-related mount options

2014-02-17 Thread Koen De Wit
Thanks for the review, Eric! Comments inline. On 02/13/2014 05:42 PM, Eric Sandeen wrote: On 2/13/14, 9:23 AM, Koen De Wit wrote: Tests the noatime, relatime, strictatime and nodiratime mount options. There is an extra check for Btrfs to ensure that the access time is never updated on read

Bug: trying to create reflink on different device results in empty file

2012-12-11 Thread Koen De Wit
When trying to create a clone (reflink) of a file on a different device, you'll get this error: Invalid cross-device link However, an empty file is created on the target location. An invalid clone operation should not result in the creation of a file. Example: # mount (...) /dev/sdb1

Adding a non-empty subvol to a qgroup

2013-03-22 Thread Koen De Wit
All, When adding a subvolume to a qgroup, pre-existing files in that subvolume are not counted in the referenced/exclusive space of the qgroup. Is this intended behavior ? I create a subvol with one file: # mkfs.btrfs /dev/sdg # mount /dev/sdg /mnt/fulldisk # cd /mnt/fulldisk #

The -c option of btrfs qgroup limit

2013-03-26 Thread Koen De Wit
All, The btrfs qgroup limit command has an option -c which means limit amount of data after compression. Whether this option is specified or not, I seem to be able to write more well-compressible data than the specified limit. I was expecting that omitting the -c option would never allow me to

[PATCH] btrfs-progs: add quota-related info to usage messages

2013-03-27 Thread Koen De Wit
after enabling quota on a non-empty filesystem. Signed-off-by: Koen De Wit koen.de@oracle.com --- cmds-qgroup.c|3 ++- cmds-quota.c |4 cmds-subvolume.c | 11 --- 3 files changed, 14 insertions(+), 4 deletions(-) diff --git a/cmds-qgroup.c b/cmds-qgroup.c index

[BUG?] Btrfs quota: overwritten space is counted twice

2013-04-05 Thread Koen De Wit
When data in a file is overwritten, starting somewhere in the middle of the file, the overwritten space is counted twice against the space usage numbers. Is this a bug, or did I something wrong? This is what I did: I create a subvolume and limit it to 4 MB, and create a 1000 KB file in the

[PATCH RESEND] xfstests: btrfs/027: simple sparse copy testcase for btrfs

2014-01-17 Thread Koen De Wit
Tests file clone functionality of btrfs (reflinks): - Reflink a file - Reflink the reflinked file - Modify the original file - Modify the reflinked file [sandeen: add helpers, make several mostly-cosmetic changes to the original testcase] Signed-off-by: Koen De Wit koen.de

[PATCH RESEND] xfstests: btrfs/028: sparse copy of a directory tree on btrfs

2014-01-17 Thread Koen De Wit
-by: Koen De Wit koen.de@oracle.com Signed-off-by: Eric Sandeen sand...@redhat.com --- Originally submitted as test 298, btrfs/309 The functions _require_cp_reflink() and _verify_reflink() in common/rc have been submitted with btrfs/027 diff --git a/tests/btrfs/028 b/tests/btrfs/028 new file mode

[PATCH RESEND] xfstests: btrfs/029: moving and deleting sparse copies on btrfs

2014-01-17 Thread Koen De Wit
Moving and deleting cloned (reflinked) files on btrfs: - Create a file and a reflink - Move both to a directory - Delete the original (moved) file, check that the copy still exists. [sandeen: mostly cosmetic changes] Signed-off-by: Koen De Wit koen.de@oracle.com Signed-off-by: Eric

[PATCH RESEND] xfstests: btrfs/030: sparse copy between different filesystems/mountpoints

2014-01-17 Thread Koen De Wit
be created. - Copy a file with the reflink=always option. Should result in error. [sandeen: mostly cosmetic changes] Signed-off-by: Koen De Wit koen.de@oracle.com Signed-off-by: Eric Sandeen sand...@redhat.com --- Originally submitted as test 301, btrfs/311 The functions

[PATCH RESEND] xfstests: btrfs: cross-subvolume sparse copy

2014-01-21 Thread Koen De Wit
This testscript creates reflinks to files on different subvolumes, overwrites the original files and reflinks, and moves reflinked files between subvolumes. Signed-off-by: Koen De Wit koen.de@oracle.com --- Originally submitted as test 302, btrfs/316 diff --git a/tests/btrfs/030 b/tests

[PATCH v2] xfstests: btrfs: cross-subvolume sparse copy

2014-01-22 Thread Koen De Wit
This testscript creates reflinks to files on different subvolumes, overwrites the original files and reflinks, and moves reflinked files between subvolumes. Signed-off-by: Koen De Wit koen.de@oracle.com Reviewed-by: David Sterba dste...@suse.cz --- v1: Resend (originally submitted as test 302

[PATCH] xfstests: Btrfs: add test for large metadata blocks

2014-02-07 Thread Koen De Wit
Tests Btrfs filesystems with all possible metadata block sizes, by setting large extended attributes on files. Signed-off-by: Koen De Wit koen.de@oracle.com --- tests/btrfs/036 | 128 +++ tests/btrfs/036.out |7 +++ tests/btrfs/group

Re: [PATCH] xfstests: Btrfs: add test for large metadata blocks

2014-02-08 Thread Koen De Wit
Thanks for the review, Dave! Comments inline. On 02/07/2014 11:49 PM, Dave Chinner wrote: On Fri, Feb 07, 2014 at 06:14:45PM +0100, Koen De Wit wrote: Tests Btrfs filesystems with all possible metadata block sizes, by setting large extended attributes on files. Signed-off-by: Koen De Wit

[PATCH v2] xfstests: Btrfs: add test for large metadata blocks

2014-02-08 Thread Koen De Wit
Tests Btrfs filesystems with all possible metadata block sizes, by setting large extended attributes on files. Signed-off-by: Koen De Wit koen.de@oracle.com --- v1-v2: - Fix indentation: 8 spaces instead of 4 - Move _scratch_unmount to end of loop, add _check_scratch_fs

[PATCH v3] xfstests: Btrfs: add test for large metadata blocks

2014-02-10 Thread Koen De Wit
Tests Btrfs filesystems with all possible metadata block sizes, by setting large extended attributes on files. Signed-off-by: Koen De Wit koen.de@oracle.com --- v1-v2: - Fix indentation: 8 spaces instead of 4 - Move _scratch_unmount to end of loop, add _check_scratch_fs - Sending

Re: [PATCH] xfstests: Btrfs: add test for large metadata blocks

2014-02-10 Thread Koen De Wit
On 02/10/2014 12:02 AM, Dave Chinner wrote: On Sat, Feb 08, 2014 at 09:30:51AM +0100, Koen De Wit wrote: On 02/07/2014 11:49 PM, Dave Chinner wrote: On Fri, Feb 07, 2014 at 06:14:45PM +0100, Koen De Wit wrote: echo -n $xattr_value | md5sum ${ATTR_PROG} -Lq -s attr_$char -V

[PATCH] xfstests: test for atime-related mount options

2014-02-13 Thread Koen De Wit
Tests the noatime, relatime, strictatime and nodiratime mount options. There is an extra check for Btrfs to ensure that the access time is never updated on read-only subvolumes. (Regression test for bug fixed with commit 93fd63c2f001ca6797c6b15b696a484b165b4800) Signed-off-by: Koen De Wit