search for: btrfs_balanc

Hi all, Did someone have met this problem before. When doing the tests, I hit the WARN_ON. Is this log make sense or someone had fixed the problem. If needed, I can supply the detail log and the testcase source file. Version: the latest codes at linus git tree. [ 2140.981293] use_block_rsv: 336 callbacks suppressed [ 2140.981295] ------------[ cut here ]------------ [ 2140.981308]

...gt;] btrfs_alloc_block_rsv+0x39/0x70 [btrfs] [<ffffffffa04f8bad>] btrfs_orphan_add+0x13d/0x1b0 [btrfs] [<ffffffffa04e2b13>] btrfs_remove_block_group+0x143/0x500 [btrfs] [<ffffffffa0518158>] btrfs_relocate_chunk.isra.63+0x618/0x790 [btrfs] [<ffffffffa051bc27>] btrfs_balance+0x8f7/0xe90 [btrfs] [<ffffffffa05240a0>] btrfs_ioctl_balance+0x250/0x550 [btrfs] [<ffffffffa05269ca>] btrfs_ioctl+0xdfa/0x25f0 [btrfs] [<ffffffff8119c936>] do_vfs_ioctl+0x96/0x570 [<ffffffff8119cea1>] SyS_ioctl+0x91/0xb0 [<ffffffff81750242>] system...

....577537] [<ffffffffa0735ec7>] btrfs_relocate_block_group+0x14f/0x283 [btrfs] [18483.577634] [<ffffffffa0715d8a>] btrfs_relocate_chunk.isra.65+0x50/0x3bc [btrfs] [18483.577731] [<ffffffffa0711d09>] ? free_extent_buffer+0x77/0x7c [btrfs] [18483.577788] [<ffffffffa07190cc>] btrfs_balance+0x9c2/0xb9a [btrfs] [18483.577838] [<ffffffff810b9117>] ? __do_fault+0x3d0/0x40a [18483.577888] [<ffffffff810e5e02>] ? __sb_start_write+0xbd/0xf1 [18483.577947] [<ffffffffa071e778>] btrfs_ioctl_balance+0x22e/0x2ac [btrfs] [18483.578004] [<ffffffffa0721bac>] btrfs_ioctl+...

I have a 3 disk raid1 filesystem mounted with nodatacow. I have a folder in said filesystem with the ''C'' NOCOW & ''Z'' Not_Compressed flags set for good measure. I then copy in a large file and proceed to make random modifications. Filefrag shows no additional extents created, good so far. A big thank you to the those devs who got that working. However, after

...cks+0x5e3/0x610 [ 347.237743] [<ffffffff812849a4>] relocate_block_group+0x444/0x6c0 [ 347.237747] [<ffffffff81284dc9>] btrfs_relocate_block_group+0x1a9/0x2d0 [ 347.237751] [<ffffffff8125de36>] btrfs_relocate_chunk.isra.53+0x56/0x730 [ 347.237754] [<ffffffff812621fe>] btrfs_balance+0x82e/0xd50 [ 347.237758] [<ffffffff812627a0>] balance_kthread+0x80/0x90 [ 347.237762] [<ffffffff8105ec33>] kthread+0xb3/0xc0 [ 347.237766] [<ffffffff815a43fc>] ret_from_fork+0x7c/0xb0 [ 347.237770] ---[ end trace cf4d2bf19a87ec83 ]--- [ 347.239650] btrfs: block rsv retur...

...+++ b/fs/btrfs/volumes.c @@ -22,6 +22,7 @@ #include <linux/blkdev.h> #include <linux/random.h> #include <linux/iocontext.h> +#include <linux/capability.h> #include <asm/div64.h> #include "compat.h" #include "ctree.h" @@ -1905,6 +1906,9 @@ int btrfs_balance(struct btrfs_root *dev_root) if (dev_root->fs_info->sb->s_flags & MS_RDONLY) return -EROFS; + if (!capable(CAP_SYS_ADMIN)) + return -EPERM; + mutex_lock(&dev_root->fs_info->volume_mutex); dev_root = dev_root->fs_info->dev_root; -- 1.7.2.3 -- To unsubsc...

...-srv-03 kernel: [158311.402540] [<ffffffffa01e33e3>] ? btrfs_get_token_64+0x76/0xc6 [btrfs] Nov 18 06:59:46 hsad-srv-03 kernel: [158311.402551] [<ffffffffa01e5e29>] ? release_extent_buffer+0x90/0x97 [btrfs] Nov 18 06:59:46 hsad-srv-03 kernel: [158311.402561] [<ffffffffa01f0a8c>] ? btrfs_balance+0x980/0xb44 [btrfs] Nov 18 06:59:46 hsad-srv-03 kernel: [158311.402572] [<ffffffffa01f6ac9>] ? btrfs_ioctl_balance+0x21a/0x296 [btrfs] Nov 18 06:59:46 hsad-srv-03 kernel: [158311.402582] [<ffffffffa01f991f>] ? btrfs_ioctl+0xf3d/0x1eaa [btrfs] Nov 18 06:59:46 hsad-srv-03 kernel: [158311...

Hi, This series adds new APIs to support btrfs scrub, balance, rescue and inspect. Some of them don't have tests because: - btrfs_scrub and btrfs_balance completes too early before we can test btrfs_scrub_cancel, btrfs_scrub_resume, btrfs_scrub_status, btrfs_balance_pause, btrfs_balance_cancel, btrfs_balance_resume and btrfs_balance_status. - can't determine a valid logical address for test btrfs_inspect_logical_reso...

...address space subset filter Btrfs: save restripe parameters to disk Btrfs: recover restripe on mount Btrfs: allow for cancelling restriper Btrfs: allow for pausing restriper Btrfs: allow for resuming restriper after it was paused Btrfs: add skip_restripe mount option Btrfs: get rid of btrfs_balance() function Btrfs: add restripe progress reporting fs/btrfs/ctree.h | 156 ++++++++++- fs/btrfs/disk-io.c | 15 +- fs/btrfs/extent-tree.c | 118 ++++++-- fs/btrfs/ioctl.c | 214 ++++++++++++- fs/btrfs/ioctl.h | 44 +++ fs/btrfs/super.c | 8 +- fs/btrfs/volu...

...+- 8 files changed, 151 insertions(+), 1 deletion(-) diff --git a/daemon/btrfs.c b/daemon/btrfs.c index 5cab52a..5cebd91 100644 --- a/daemon/btrfs.c +++ b/daemon/btrfs.c @@ -1659,3 +1659,126 @@ do_btrfs_rescue_super_recover (const char *device) return 0; } + +guestfs_int_btrfsbalance * +do_btrfs_balance_status (const char *path) +{ + const size_t MAX_ARGS = 64; + const char *argv[MAX_ARGS]; + size_t i = 0; + CLEANUP_FREE char *path_buf = NULL; + CLEANUP_FREE char *err = NULL; + char *out; + int r; + guestfs_int_btrfsbalance *ret; + char **lines; + size_t nlines; + const char *errptr; +...

...+- 8 files changed, 154 insertions(+), 1 deletion(-) diff --git a/daemon/btrfs.c b/daemon/btrfs.c index 5cab52a..a979327 100644 --- a/daemon/btrfs.c +++ b/daemon/btrfs.c @@ -1659,3 +1659,129 @@ do_btrfs_rescue_super_recover (const char *device) return 0; } + +guestfs_int_btrfsbalance * +do_btrfs_balance_status (const char *path) +{ + const size_t MAX_ARGS = 64; + const char *argv[MAX_ARGS]; + size_t i = 0; + CLEANUP_FREE char *path_buf = NULL; + CLEANUP_FREE char *err = NULL; + char *out; + int r; + guestfs_int_btrfsbalance *ret; + char **lines; + size_t nlines; + const char *errptr; +...

...0825] [<ffffffffa02250b5>] btrfs_relocate_chunk.isra.54+0x75/0x730 [btrfs] [12451.490864] [<ffffffffa021b5e1>] ? release_extent_buffer.isra.26+0x81/0xf0 [btrfs] [12451.490904] [<ffffffffa0220b37>] ? free_extent_buffer+0x37/0x90 [btrfs] [12451.490945] [<ffffffffa02294d7>] btrfs_balance+0x8e7/0xe00 [btrfs] [12451.490988] [<ffffffffa023135f>] btrfs_ioctl_balance+0x22f/0x570 [btrfs] [12451.491030] [<ffffffffa0233cb0>] btrfs_ioctl+0xee0/0x1b70 [btrfs] [12451.491045] [<ffffffff811a66a3>] ? user_path_at_empty+0x63/0xa0 [12451.491058] [<ffffffff816d74a0>] ?...

v4: - add reply_with_error when nlines < 1 - remove `i == X' tests. v3: - rebase on upstream - fix some comments v2: - add check for the length of lines[] - the code parsing 'btrfs scrub -R status' output is changed into a loop Hu Tao (2): New API: btrfs_balance_status New API: btfs_scrub_status daemon/btrfs.c | 268 +++++++++++++++++++++++++++++++ generator/actions.ml | 26 +++ generator/structs.ml | 34 ++++ gobject/Makefile.inc | 4 + java/Makefile.inc...

...> diff --git a/daemon/btrfs.c b/daemon/btrfs.c > index 5cab52a..a979327 100644 > --- a/daemon/btrfs.c > +++ b/daemon/btrfs.c > @@ -1659,3 +1659,129 @@ do_btrfs_rescue_super_recover (const char *device) > > return 0; > } > + > +guestfs_int_btrfsbalance * > +do_btrfs_balance_status (const char *path) > +{ > + const size_t MAX_ARGS = 64; > + const char *argv[MAX_ARGS]; > + size_t i = 0; > + CLEANUP_FREE char *path_buf = NULL; > + CLEANUP_FREE char *err = NULL; > + char *out; > + int r; > + guestfs_int_btrfsbalance *ret; > + char *...

changes in v2: - add check for the length of lines[] - the code parsing 'btrfs scrub -R status' output is changed into a loop Hu Tao (2): New API: btrfs_balance_status New API: btfs_scrub_status. daemon/btrfs.c | 263 +++++++++++++++++++++++++++++++ generator/actions.ml | 26 +++ generator/structs.ml | 34 ++++ gobject/Makefile.inc | 4 + java/Makefile.inc...

...ix tests failure v4: - add reply_with_error when nlines < 1 - remove `i == X' tests. v3: - rebase on upstream - fix some comments v2: - add check for the length of lines[] - the code parsing 'btrfs scrub -R status' output is changed into a loop Hu Tao (2): New API: btrfs_balance_status New API: btfs_scrub_status daemon/btrfs.c | 270 +++++++++++++++++++++++++++++++ generator/actions.ml | 26 +++ generator/structs.ml | 34 ++++ gobject/Makefile.inc | 4 + java/Makefile.inc...

v3: - rebase on upstream - fix some comments v2: - add check for the length of lines[] - the code parsing 'btrfs scrub -R status' output is changed into a loop Hu Tao (2): New API: btrfs_balance_status New API: btfs_scrub_status daemon/btrfs.c | 263 +++++++++++++++++++++++++++++++ generator/actions.ml | 26 +++ generator/structs.ml | 34 ++++ gobject/Makefile.inc | 4 + java/Makefile.inc...

...+- 8 files changed, 137 insertions(+), 1 deletion(-) diff --git a/daemon/btrfs.c b/daemon/btrfs.c index 796eaff..bb82f32 100644 --- a/daemon/btrfs.c +++ b/daemon/btrfs.c @@ -1659,3 +1659,112 @@ do_btrfs_rescue_super_recover (const char *device) return 0; } + +guestfs_int_btrfsbalance * +do_btrfs_balance_status (const char *path) +{ + const size_t MAX_ARGS = 64; + const char *argv[MAX_ARGS]; + size_t i = 0; + CLEANUP_FREE char *path_buf = NULL; + CLEANUP_FREE char *err = NULL; + char *out; + int r; + guestfs_int_btrfsbalance *ret; + char **lines; + const char *errptr; + int erroffset; +...

The biggest one is a fix for fstrim, and there''s a fix for on-disk free space cache. Others are small fixes and cleanups. The last three have been sent weeks ago. The patchset is also available in this repo: git://repo.or.cz/linux-btrfs-devel.git for-chris Note there''s a small confict with Al Viro''s vfs changes. Li Zefan (11): Btrfs: add pinned extents to

This is a follow-up to the other generator changes in: https://www.redhat.com/archives/libguestfs/2017-February/msg00217.html Rich.