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
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
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
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
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
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
,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
,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
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
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
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 <
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).
>
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
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
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/
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
#
, 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(+),
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
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
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
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-
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
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
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
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
-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)
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.
-- 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
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
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
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
/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
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
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 @
' 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
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
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
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
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 -
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
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
> ---
>
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>
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
.
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
.
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
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
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
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
-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
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
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
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
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
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
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
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
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
: 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
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
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.
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
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)(
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
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
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
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
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,
201 - 267 of 267 matches
Mail list logo