Re: [PATCH 08/19] btrfs: qgroup: Introduce function to release/free reserved data range

2015-09-08 Thread Tsutomu Itoh
Hi, Qu, On 2015/09/08 18:08, Qu Wenruo wrote: > Introduce functions btrfs_qgroup_release/free_data() to release/free > reserved data range. > > Release means, just remove the data range from data rsv map, but doesn't > free the reserved space. > This is for normal buffered write case, when data

Re: [PATCH 10/19] btrfs: delayed_ref: release and free qgroup reserved at proper timing

2015-09-08 Thread Tsutomu Itoh
Hi, Qu, On 2015/09/08 18:08, Qu Wenruo wrote: > Qgroup reserved space needs to be released from inode dirty map and get > freed at different timing: > > 1) Release when the metadata is written into tree > After corresponding metadata is written into tree, any newer write will > be COWed(don't

Re: [PATCH 04/19] btrfs: qgroup: Introduce function to insert non-overlap reserve range

2015-09-08 Thread Tsutomu Itoh
Hi, Qu, On 2015/09/08 18:01, Qu Wenruo wrote: > New function insert_data_ranges() will insert non-overlap reserve ranges > into reserve map. > > It provides the basis for later qgroup reserve map implement. > > Signed-off-by: Qu Wenruo > --- > fs/btrfs/qgroup.c | 124

Re: [PATCH 13/19] btrfs: extent-tree: Add new verions of btrfs_check_data_free_space

2015-09-08 Thread Tsutomu Itoh
Hi, Qu, On 2015/09/08 18:22, Qu Wenruo wrote: > Add new function __btrfs_check_data_free_space() to do precious space > reservation. > > The new function will replace old btrfs_check_data_free_space(), but > until all the change is done, let's just use the new name. > > Also, export internal

Re: [PATCH 18/19] btrfs: qgroup: Cleanup old inaccurate facilities

2015-09-08 Thread Tsutomu Itoh
Hi, Qu, On 2015/09/08 18:25, Qu Wenruo wrote: > Cleanup the old facilities which use old btrfs_qgroup_reserve() function > call, replace them with the newer version, and remove the "__" prefix in > them. > > Also, make btrfs_qgroup_reserve/free() functions private, as they are > now only used

Re: [PATCH v2] fstests: generic: Check if a bull fallocate will change extent number

2015-09-29 Thread Tsutomu Itoh
On 2015/09/30 10:05, Qu Wenruo wrote: Dave Chinner wrote on 2015/09/30 07:51 +1000: On Tue, Sep 29, 2015 at 05:34:24PM +0800, Qu Wenruo wrote: Normally, a bull fallocate call on a fully written and synced file should not add an extent. Why not? Filesystems can do whatever they want with

[PATCH] Btrfs: fix output of compression message in btrfs_parse_options()

2015-12-15 Thread Tsutomu Itoh
,relatime,compress=zlib,space_cache,subvolid=5,subvol=/) Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- fs/btrfs/disk-io.c | 2 +- fs/btrfs/super.c | 21 ++--- 2 files changed, 15 insertions(+), 8 deletions(-) diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index 9

[PATCH V2] Btrfs: fix output of compression message in btrfs_parse_options()

2016-01-06 Thread Tsutomu Itoh
,space_cache,subvolid=5,subvol=/) Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- V1->V2: It is corrected that API doesn't change. fs/btrfs/super.c | 29 ++--- 1 file changed, 22 insertions(+), 7 deletions(-) diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c inde

Re: [PATCH] Btrfs: fix output of compression message in btrfs_parse_options()

2016-01-05 Thread Tsutomu Itoh
Hi, David, On 2016/01/05 23:12, David Sterba wrote: On Wed, Dec 16, 2015 at 11:57:38AM +0900, Tsutomu Itoh wrote: diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index 974be09..dcc1f15 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c @@ -2709,7 +2709,7 @@ int open_ctree(struct

Re: [PATCH] btrfs-progs: add the error message when open fails

2016-06-09 Thread Tsutomu Itoh
Hi, David, On 2016/06/09 22:13, David Sterba wrote: On Thu, Jun 09, 2016 at 10:23:15AM +0900, Tsutomu Itoh wrote: When open in btrfs_open_devices failed, only the following message is displayed. Therefore the user doesn't understand the reason why open failed. # btrfs check /dev/sdb8

[PATCH] btrfs-progs: add the error message when open fails

2016-06-08 Thread Tsutomu Itoh
When open in btrfs_open_devices failed, only the following message is displayed. Therefore the user doesn't understand the reason why open failed. # btrfs check /dev/sdb8 Couldn't open file system This patch adds the error message when open fails. Signed-off-by: Tsutomu Itoh <

Re: [PATCH 1/2] btrfs: use dynamic allocation for root item in create_subvol

2016-04-11 Thread Tsutomu Itoh
On 2016/04/12 3:04, David Sterba wrote: > The size of root item is more than 400 bytes, which is quite a lot of > stack space. As we do IO from inside the subvolume ioctls, we should > keep the stack usage low in case the filesystem is on top of other > layers (NFS, device mapper, iscsi, etc). >

Re: [PATCH] Btrfs: fix missing s_id setting

2016-04-05 Thread Tsutomu Itoh
On 2016/04/05 17:52, Tsutomu Itoh wrote: On 2016/04/05 16:56, Anand Jain wrote: On 04/05/2016 08:08 AM, Tsutomu Itoh wrote: When fs_devices->latest_bdev is deleted or is replaced, sb->s_id has not been updated. As a result, the deleted device name is displayed by btrfs_printk. [befo

[PATCH] btrfs-progs: send: fix handling of multiple snapshots

2016-03-24 Thread Tsutomu Itoh
We cannot send multiple snapshots at once. [before fix] # btrfs send ./snap[12] > snap12.data At subvol ./snap1 At subvol ./snap2 ERROR: parent determination failed for 0 # [after fix] # btrfs send ./snap[12] > snap12.data At subvol ./snap1 At subvol ./snap2 # Signed-off-by: Tsutomu Ito

[PATCH] Btrfs: fix missing s_id setting

2016-04-04 Thread Tsutomu Itoh
e sdc3): disk deleted /dev/sdc4 [ 84.401951] BTRFS info (device sdc3): disk added /dev/sdb6 Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- fs/btrfs/dev-replace.c | 5 - fs/btrfs/volumes.c | 11 +-- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/

Re: [PATCH] Btrfs: fix missing s_id setting

2016-04-05 Thread Tsutomu Itoh
On 2016/04/05 16:56, Anand Jain wrote: On 04/05/2016 08:08 AM, Tsutomu Itoh wrote: When fs_devices->latest_bdev is deleted or is replaced, sb->s_id has not been updated. As a result, the deleted device name is displayed by btrfs_printk. [before fix] # btrfs dev del /dev/sdc4 /mnt2 #

Re: [PATCH v2] btrfs: use dynamic allocation for root item in create_subvol

2016-04-25 Thread Tsutomu Itoh
, iscsi, etc). > > Signed-off-by: David Sterba <dste...@suse.com> Looks good to me. Reviewed-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> > --- > fs/btrfs/ioctl.c | 65 > > 1 file changed, 37 insertions(+),

[PATCH] btrfs-progs: image: fix compiler warning

2016-10-05 Thread Tsutomu Itoh
art); ^ btrfs-image.c:927:6: note: 'start' was declared here u64 start; ^ Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- btrfs-image.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/btrfs-image.c b/btrfs-image.c index 0d410f8..47f36b9 100644 --- a/btrfs-imag

[PATCH v2] btrfs-progs: send-test: add checking of clone-src option

2016-11-08 Thread Tsutomu Itoh
Sending stream size of clone-src(-c) option is checked. Fixed by "btrfs-progs: send: fix handling of -c option". Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- V2: old sending stream image is used --- .../016-send-clone-src/send-stream-v4.8.2.img | Bin 0 -> 6

Re: [PATCH] btrfs-progs: send-test: add checking of clone-src option

2016-11-08 Thread Tsutomu Itoh
On 2016/11/08 21:47, David Sterba wrote: > On Tue, Nov 08, 2016 at 08:53:04AM +0900, Tsutomu Itoh wrote: >> On 2016/11/08 0:16, David Sterba wrote: >>> On Fri, Nov 04, 2016 at 05:35:18PM +0900, Tsutomu Itoh wrote: >>>> +before_size=`ls -l "$here&q

Re: [PATCH v2] btrfs-progs: send-test: add checking of clone-src option

2016-11-08 Thread Tsutomu Itoh
On 2016/11/09 10:39, Qu Wenruo wrote: > > > At 11/09/2016 09:25 AM, Tsutomu Itoh wrote: >> Sending stream size of clone-src(-c) option is checked. >> Fixed by "btrfs-progs: send: fix handling of -c option". >> >> Signed-off-by: Tsutomu Itoh <t-

[PATCH v3] btrfs-progs: send-test: add checking of clone-src option

2016-11-08 Thread Tsutomu Itoh
Sending stream size of clone-src(-c) option is checked. Fixed by "btrfs-progs: send: fix handling of -c option". Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- V2: old sending stream image is used V3: image file has been compressed by gzip --- .../016-send-clone-src/se

Re: [PATCH] btrfs-progs: send-test: add checking of clone-src option

2016-11-07 Thread Tsutomu Itoh
On 2016/11/08 0:16, David Sterba wrote: > On Fri, Nov 04, 2016 at 05:35:18PM +0900, Tsutomu Itoh wrote: >> +before_size=`ls -l "$here"/send.stream.before | awk '{print $5}'` >> +after_size=`ls -l "$here"/send.stream.after | awk '{print $5}'` > > Thanks for

[PATCH v2 1/3] btrfs-progs: send: remove unnecessary code

2016-10-18 Thread Tsutomu Itoh
Some unnecessary codes are deleted. - the setting of subvol is double. - read only check was already done by previous loop. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- v2: description was changed --- cmds-send.c | 11 --- 1 file changed, 11 deletions(-) diff --git

[PATCH 1/3] btrfs-progs: send: remove unnecessary code

2016-10-18 Thread Tsutomu Itoh
Some unnecessary codes is deleted. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- cmds-send.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/cmds-send.c b/cmds-send.c index 74d0128..dfdfe01 100644 --- a/cmds-send.c +++ b/cmds-send.c @@ -564,8 +564,6 @@ int cmd_se

[PATCH 2/3] btrfs-progs: send: fix handling of multiple snapshots (-p option)

2016-10-18 Thread Tsutomu Itoh
-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- cmds-send.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/cmds-send.c b/cmds-send.c index dfdfe01..2a8a697 100644 --- a/cmds-send.c +++ b/cmds-send.c @@ -650,7 +650,7 @@ int cmd_send(int argc, char **argv)

[PATCH 3/3] btrfs-progs: send: fix handling of -c option

2016-10-18 Thread Tsutomu Itoh
19 10:37 /tmp/data1 # [after] # btrfs send -f /tmp/data1 -c Snap0 -c ../SnapX Snap[12] ../SnapY At subvol Snap1 At subvol Snap2 At subvol ../SnapY # ls -l /tmp/data1 -rw--- 1 root root 1492 Oct 19 10:39 /tmp/data1 # Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- cmds-send.

[PATCH] btrfs-progs: send: fix failure of xfstests btrfs/038

2016-11-15 Thread Tsutomu Itoh
-- btrfs | PLATFORM -- Linux/x86_64 luna 4.9.0-rc5 | MKFS_OPTIONS -- /dev/sdb3 | MOUNT_OPTIONS -- /dev/sdb3 /test6 | | btrfs/038 1s ... 1s | Ran: btrfs/038 | Passed all 1 tests Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- cmds-send.c | 4 ++-- 1 file changed, 2 inse

Re: [PATCH 2/3] btrfs-progs: send: fix handling of multiple snapshots (-p option)

2016-11-15 Thread Tsutomu Itoh
xfstests btrfs/038 fails when this patch is applied. Sorry for my fault. I posted the patch that corrected this problem. [PATCH] btrfs-progs: send: fix failure of xfstests btrfs/038 Thanks, Tsutomu On 2016/10/19 11:35, Tsutomu Itoh wrote: > We cannot send multiple snapshots at once by

[bug] btrfs-progs-v4.8.4: test of 'btrfs receive' in xfstests fails

2016-11-27 Thread Tsutomu Itoh
Hi, David, Many test of xfstests such as btrfs/007, btrfs/008 and btrfs/016 failed with the following patch. fefbab75 btrfs-progs: send-stream: check number of read bytes from stream This is because cmds-receive.c:do_receive() make a judgement of the end of stream by returning 1 from

[PATCH] btrfs-progs: qgroup: fix error in ASSERT condition expression

2016-11-16 Thread Tsutomu Itoh
Option -f, -F and --sort don't work because a conditional expression of ASSERT is wrong. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- qgroup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/qgroup.c b/qgroup.c index 9d10cb8..071d15e 100644 --- a/qgroup.c

[PATCH 2/3] btrfs-progs: test: expand size of test device of fsck-tests 013

2016-11-17 Thread Tsutomu Itoh
/btrfs-progs/tests/mnt test failed for case 013-extent-tree-rebuild Makefile:272: recipe for target 'test-fsck' failed make: *** [test-fsck] Error 1 # Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- tests/fsck-tests/013-extent-tree-rebuild/test.sh | 2 +- 1 file changed, 1 insertion

[PATCH 3/3] btrfs-progs: test: fix convert-tests 004 failure

2016-11-17 Thread Tsutomu Itoh
convert-tests 004 failed because the argument to check_all_images is not specified. # make test-convert [TEST] convert-tests.sh [TEST/conv] 004-ext2-backup-superblock-ranges find: '': No such file or directory # Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- tests/c

[PATCH 1/3] btrfs-progs: test: fix error of test target of Makefile

2016-11-17 Thread Tsutomu Itoh
Add test-cli to test target of Makefile. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- Makefile.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile.in b/Makefile.in index 4bd3e63..6c78841 100644 --- a/Makefile.in +++ b/Makefile.in @@ -298,7 +298,7 @

[PATCH v2 2/3] btrfs-progs: test: fix how to make test files in fsck-tests 013

2016-11-20 Thread Tsutomu Itoh
' to 'common'. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- v2: change how to make test files --- tests/common | 89 tests/common.convert | 89 tests/fsck-tests/013-exten

Re: [PATCH 2/3] btrfs-progs: test: expand size of test device of fsck-tests 013

2016-11-20 Thread Tsutomu Itoh
Hi, Qu and David, On 2016/11/19 3:28, David Sterba wrote: > On Fri, Nov 18, 2016 at 02:36:28PM +0800, Qu Wenruo wrote: >> >> >> On 11/18/2016 01:47 PM, Tsutomu Itoh wrote: >>> In my test environment, size of /lib/modules/`uname -r`/* is >>> larger than 1G

[PATCH v2 3/3] btrfs-progs: send: fix handling of -c option

2016-11-04 Thread Tsutomu Itoh
19 10:37 /tmp/data1 # [after] # btrfs send -f /tmp/data1 -c Snap0 -c ../SnapX Snap[12] ../SnapY At subvol Snap1 At subvol Snap2 At subvol ../SnapY # ls -l /tmp/data1 -rw--- 1 root root 1492 Oct 19 10:39 /tmp/data1 # Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- v2: make

[PATCH] btrfs-progs: send-test: add checking of clone-src option

2016-11-04 Thread Tsutomu Itoh
Sending stream size of clone-src(-c) option is checked. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- tests/misc-tests/016-send-clone-src/test.sh | 54 + 1 file changed, 54 insertions(+) create mode 100755 tests/misc-tests/016-send-clone-src/test.sh

Re: [PATCH 3/3] btrfs-progs: send: fix handling of -c option

2016-11-03 Thread Tsutomu Itoh
On 2016/11/02 21:22, David Sterba wrote: > On Wed, Oct 19, 2016 at 11:35:40AM +0900, Tsutomu Itoh wrote: >> When two or more -c options are specified, cannot find a suitable >> parent. So, output stream is bigger than correct one. >> >> [before] >> # btrfs send -

Re: [PATCH 2/3] btrfs-progs: send: fix handling of multiple snapshots (-p option)

2016-11-01 Thread Tsutomu Itoh
Sorry for the late reply. On 2016/10/29 0:10, David Sterba wrote: > On Wed, Oct 19, 2016 at 11:35:03AM +0900, Tsutomu Itoh wrote: >> We cannot send multiple snapshots at once by -p option. > > We cannot like that it's broken, or we cannot because it's not supposed > to work

Re: [PATCH 3/3] btrfs-progs: convert-test: trigger chunk allocation after convert

2016-12-13 Thread Tsutomu Itoh
On 2016/12/13 18:44, Qu Wenruo wrote: > Populate fs after convert so we can trigger data chunk allocation. > This can expose too restrict old rollback condition > > Reported-by: Chandan Rajendra > Signed-off-by: Qu Wenruo > --- >

Re: [PATCH v2] btrfs-progs: Fix NULL pointer when receive clone operation

2016-12-15 Thread Tsutomu Itoh
return int case, convert NULL pointer to -ENOENT. This patch also passed xfstests btrfs/{108,109,117}. Thanks for your work. Thanks, Tsutomu > > Reported-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> > Signed-off-by: Qu Wenruo <quwen...@cn.fujitsu.com>

Re: [PATCH] btrfs-progs: qgroup: add sync option to 'qgroup show'

2016-12-06 Thread Tsutomu Itoh
On 2016/12/07 13:59, Qu Wenruo wrote: > > > At 12/07/2016 12:31 PM, Tsutomu Itoh wrote: >> Hi Qu, >> >> Thanks for the review. >> >> On 2016/12/07 12:24, Qu Wenruo wrote: >>> >>> >>> At 12/07/2016 11:07 AM, Tsutomu Itoh wro

[PATCH v2] btrfs-progs: qgroup: add sync option to 'qgroup show'

2016-12-06 Thread Tsutomu Itoh
. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- v2: use getopt_long with enum instead of single letter (suggested by Qu) --- Documentation/btrfs-qgroup.asciidoc | 6 ++ cmds-qgroup.c | 33 + 2 files changed, 35 insertions

[PATCH] btrfs-progs: qgroup: add sync option to 'qgroup show'

2016-12-06 Thread Tsutomu Itoh
. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- Documentation/btrfs-qgroup.asciidoc | 5 + cmds-qgroup.c | 24 ++-- 2 files changed, 27 insertions(+), 2 deletions(-) diff --git a/Documentation/btrfs-qgroup.asciidoc b/Documentation

Re: [PATCH] btrfs-progs: qgroup: add sync option to 'qgroup show'

2016-12-06 Thread Tsutomu Itoh
Hi Qu, Thanks for the review. On 2016/12/07 12:24, Qu Wenruo wrote: > > > At 12/07/2016 11:07 AM, Tsutomu Itoh wrote: >> The 'qgroup show' command does not synchronize filesystem. >> Therefore, 'qgroup show' may not display the correct value unless >> synchro

Re: [PATCH v2] btrfs-progs: qgroup: add sync option to 'qgroup show'

2016-12-14 Thread Tsutomu Itoh
Hi David, Thanks for the review. On 2016/12/14 19:54, David Sterba wrote: > On Wed, Dec 07, 2016 at 04:55:15PM +0900, Tsutomu Itoh wrote: >> The 'qgroup show' command does not synchronize filesystem. >> Therefore, 'qgroup show' may not display the correct value unless

[PATCH v2] btrfs-progs: tests: add test for --sync option of qgroup show

2016-12-14 Thread Tsutomu Itoh
Simple test script for the following patch. btrfs-progs: qgroup: add sync option to 'qgroup show' Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- v2: dropped the test of --no-sync --- tests/cli-tests/005-qgroup-show-sync/test.sh | 30 1 file chang

[PATCH v3 1/2] btrfs-progs: qgroup: add sync option to 'qgroup show'

2016-12-14 Thread Tsutomu Itoh
-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- v2: use getopt_long with enum instead of single letter (suggested by Qu) v3: dropped the --no-sync option and separated the patch of sort option (suggested by David) --- Documentation/btrfs-qgroup.asciidoc | 4 cmds-qg

[PATCH v3 2/2] btrfs-progs: qgroup: change the value of sort option

2016-12-14 Thread Tsutomu Itoh
The value of sort option ('S') is not used for option letter. Therefore, I'll change the single letter to non-character. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- This patch is separated from patch of --sync option. --- cmds-qgroup.c | 7 --- 1 file changed, 4 insertions

Re: Btrfs progs pre-release 4.9-rc1

2016-12-14 Thread Tsutomu Itoh
On 2016/12/14 23:42, David Sterba wrote: > Hi, > > a pre-release has been tagged. Contains almost the entire devel branch from > today. There are small fixes, the lowmem mode of check gets more updates but > still does not work in the --repair mode and is considered experimental. > > ETA for 4.9

Re: Btrfs progs pre-release 4.9-rc1

2016-12-15 Thread Tsutomu Itoh
On 2016/12/15 15:45, Tsutomu Itoh wrote: > On 2016/12/14 23:42, David Sterba wrote: >> Hi, >> >> a pre-release has been tagged. Contains almost the entire devel branch from >> today. There are small fixes, the lowmem mode of check gets more updates but >> still d

Re: [PATCH] btrfs-progs: Fix NULL pointer when receive clone operation

2016-12-15 Thread Tsutomu Itoh
On 2016/12/15 16:28, Qu Wenruo wrote: > The subvol_info returned from subvol_uuid_search() can be NULL. > So the branch checking IS_ERR(si) should also check if it's NULL. > > Reported-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> > Signed-off-by: Qu Wenruo <quwen...@cn.fu

Re: [bug] btrfs-progs-v4.8.4: test of 'btrfs receive' in xfstests fails

2016-11-29 Thread Tsutomu Itoh
Hi David, On 2016/11/30 0:34, David Sterba wrote: > On Mon, Nov 28, 2016 at 04:27:06PM +0900, Tsutomu Itoh wrote: >> Many test of xfstests such as btrfs/007, btrfs/008 and btrfs/016 failed >> with the following patch. >> >> fefbab75 btrfs-progs: send-stream: chec

[PATCH] btrfs-progs: tests: add test for --sync option of qgroup show

2016-12-07 Thread Tsutomu Itoh
Simple test script for the following patch. btrfs-progs: qgroup: add sync option to 'qgroup show' Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- tests/cli-tests/005-qgroup-show-sync/test.sh | 30 1 file changed, 30 insertions(+) create mode 100755

Re: Btrfs progs pre-release 4.10-rc1

2017-03-06 Thread Tsutomu Itoh
eck: Output verbose error when fsck found a bug in any > tree > btrfs-progs: check: lowmem: Fix false alert in checking data extent > csums > btrfs-progs: check: lowmem: Fix extent item size false alert > btrfs-progs: check: lowmem: Fix false alert on inline compress

Re: Btrfs progs pre-release 4.10-rc1

2017-03-07 Thread Tsutomu Itoh
On 2017/03/07 20:54, David Sterba wrote: > On Tue, Mar 07, 2017 at 11:03:33AM +0900, Tsutomu Itoh wrote: >> Hi David, >> >> On 2017/03/02 23:59, David Sterba wrote: >>> Hi, >>> >>> a pre-release has been tagged. There are patches that have qu

[PATCH] btrfs-progs: tests: remove variable quotation from convert-tests

2017-05-15 Thread Tsutomu Itoh
: recipe for target 'test-convert' failed make: *** [test-convert] Error 1 So, remove quotes from $default_mke2fs. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- tests/convert-tests/008-readonly-image/test.sh | 2 +- tests/convert-tests/009-common-inode-flags/test.sh | 2 +- 2

Re: [PATCH] btrfs-progs: mkfs: Replace number with a macro

2017-06-26 Thread Tsutomu Itoh
On 2017/06/26 17:23, Gu Jinxiang wrote: > For code maintainability and scalability, > replace number with a macro of member blocks in btrfs_mkfs_config. > > Signed-off-by: Gu Jinxiang > --- > mkfs/common.c | 2 +- > mkfs/common.h | 5 - > 2 files changed, 5

Re: Btrfs progs release 4.11

2017-05-18 Thread Tsutomu Itoh
Hi David, Could you please apply the following patch? [PATCH] btrfs-progs: tests: remove variable quotation from convert-tests http://marc.info/?l=linux-btrfs=149490441323349=2 Thanks, Tsutomu On 2017/05/18 23:35, David Sterba wrote: > Hi, > > btrfs-progs version 4.11 have been released.

Re: btrfs-progs 4.11 broke snap-sync

2017-05-28 Thread Tsutomu Itoh
On 2017/05/29 12:41, Wael M. Nasreddine wrote: > Hello, > > I use Arch, the system is entirely on btrfs (on a LUKS partition). I > use snap-sync[0] to sync snapshots to an external drive. When 4.11 > came out, snap-sync errors out with the message: ERROR: unable to > resolve -c. > > Reverting

Re: [PATCH] btrfs: send: fix error code if an unknown inode type is found

2017-09-11 Thread Tsutomu Itoh
similar patch, you said "I'd rather make it EINVAL". https://marc.info/?l=linux-btrfs=145342170820262=2 But I think that EOPNOTSUPP is also good. So, anyway, Reviewed-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> > > The unknown type seems to be the block device (S_IFBLK)(

[PATCH v2] Btrfs: fix overlap of fs_info->flags values

2017-10-03 Thread Tsutomu Itoh
to enable quota override for CAP_SYS_RESOURCE") CC: sta...@vger.kernel.org # 4.13+ Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- v2: changed the value of BTRFS_FS_QUOTA_OVERRIDE instead of BTRFS_FS_EXCL_OP to 16. fs/btrfs/ctree.h | 7 +++ 1 file changed, 3 insertions(+), 4 deletio

[PATCH] Btrfs: fix fs_info->flags value

2017-10-02 Thread Tsutomu Itoh
Because the values of BTRFS_FS_QUOTA_OVERRIDE and BTRFS_FS_EXCL_OP overlap, we should change the value. Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- fs/btrfs/ctree.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h

Re: status of inline deduplication in btrfs

2017-08-23 Thread Tsutomu Itoh
On 2017/08/23 23:52, shally verma wrote: > HI > > Through btrfs wiki, I got to know about inline patch and this git > location https://github.com/adam900710/linux but I am not sure what's > progress and status on this. Could any one please confirm what is the > status of inline deduplication

Re: [PATCH] Remove unused dedupe argument btrfs_set_extent_delalloc()

2017-10-10 Thread Tsutomu Itoh
On 2017/10/11 11:29, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > 'int dedupe' was added to prepare for support of subpage sector size and in-band dedupe. commit ba8b04c1d4ad ("btrfs: extend btrfs_set_extent_delalloc and its friends to support in-band dedupe and

Re: [PATCH v14.8 12/14] btrfs: dedupe: Add ioctl for inband deduplication

2018-07-19 Thread Tsutomu Itoh
On 2018/07/12 10:25, Lu Fengqi wrote: > From: Wang Xiaoguang > > Add ioctl interface for inband deduplication, which includes: > 1) enable > 2) disable > 3) status > > And a pseudo RO compat flag, to imply that btrfs now supports inband > dedup. > However we don't add any ondisk format change,

<    1   2   3