On Wed, 7 Aug 2013 13:54:05 +0800, Wang Shilong wrote:
---
V1-V2: GPLv3-GPLv2
---
[...]
.SH COPYRIGHT
Copyright \(co 2013 Fujitsu, Inc.
-License GPLv2: GNU GPL version 2 http://gnu.org/licenses/gpl.html.
+License GPLv3: GNU GPL version 3 http://gnu.org/licenses/gpl.html.
...
--
To
On Tue, 06 Aug 2013 22:49:32 -0500, Eric Sandeen wrote:
On 8/6/13 8:05 PM, Eric Sandeen wrote:
Mark many functions as static, and remove any resulting dead code.
Signed-off-by: Eric Sandeen sand...@redhat.com
---
...
Actually, what the heck was this (note, this patch was against kdave's
On Wed, 7 Aug 2013 00:42:27 +0800
Tomasz Chmielewski man...@wpkg.org wrote:
What do I have to do to reproduce it here? How do you generate the
load? What is the disk setup, what the qgroups setup?
Unfortunately I don't have a way to reproduce, as the issue
happened to me only once.
I
Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com
Signed-off-by: Qu Wenruo quwen...@cn.fujitsu.com
---
V1-V2: GPLv3-GPLv2
Thanks to Stefan Behrens
---
man/Makefile | 2 +-
man/btrfs-find-root.8.in | 36
2 files changed, 37 insertions(+), 1
many thanks to Stefan,you are always so helpful.~_~
David, please ignore this patch, i will resend.
Thanks,
Wang
On 08/07/2013 02:46 PM, Stefan Behrens wrote:
On Wed, 7 Aug 2013 13:54:05 +0800, Wang Shilong wrote:
---
V1-V2: GPLv3-GPLv2
---
[...]
.SH COPYRIGHT
Copyright \(co 2013
On Tue, Aug 06, 2013 at 07:27:20PM +0100, Filipe David Borba Manana wrote:
This change allows for most mount options to be persisted in
the filesystem, and be applied when the filesystem is mounted.
If the same options are specified at mount time, the persisted
values for those options are
On Tue, Aug 06, 2013 at 04:05:50PM -0500, Eric Sandeen wrote:
I was thinking (and Josef just suggested too) that making a
dir flag, saying everything under this dir gets compressed might make
more sense for that scenario than adding a whole slew of
on-disk-persistent-mount-option code.
We
On Tue, Aug 06, 2013 at 11:01:37AM -0700, Zach Brown wrote:
My biggest worry about this is that it complicates the coordination of
automated testing, which is already in a terrible state for btrfs-progs.
It can't possibly motivate people to write tests if we make the process
more cumbersome
Currently the compression settings (algorithm and force mode) need
to be specified at mount time in order to have newly created files
compressed. If we mount a filesystem with the compress=lzo option
for example, create a directory, add the +c attribute to that directory,
unmount the file system,
On Wed, Aug 7, 2013 at 11:48 AM, David Sterba dste...@suse.cz wrote:
On Tue, Aug 06, 2013 at 04:05:50PM -0500, Eric Sandeen wrote:
I was thinking (and Josef just suggested too) that making a
dir flag, saying everything under this dir gets compressed might make
more sense for that scenario than
On Wed, Aug 7, 2013 at 11:40 AM, David Sterba dste...@suse.cz wrote:
On Tue, Aug 06, 2013 at 07:27:20PM +0100, Filipe David Borba Manana wrote:
This change allows for most mount options to be persisted in
the filesystem, and be applied when the filesystem is mounted.
If the same options are
test_dev_for_mkfs() is a common place where
we check if a device is fit for the btrfs use.
cmd_start_replace() should make use of test_dev_for_mkfs(),
and here the test_dev_for_mkfs() is further enhanced
to fit the cmd_start_replace() needs.
Thanks
Signed-off-by: Anand Jain anand.j...@oracle.com
This patch provides fix for the following bug,
When mkfs.btrfs fails the disks shouldn't be written.
btrfs fi show /dev/sdb
Label: none uuid: 60fb76f4-3b4d-4632-a7da-6a44dea5573d
Total devices 1 FS bytes used 24.00KiB
devid1 size 2.00GiB used 20.00MiB path
When the device disappear the path goes missing,
and that will be the one of the reason that user
will replace the device.
The devid of the missing btrfs device can be
obtained using the new cli option
btrfs fi show --kernel (coming soon)
And which can be used in the replace command.
---
Anand Jain (3):
btrfs-progs: let user know that devid can be used if path is missing
btrfs-progs: cmd_start_replace() to use test_dev_for_mkfs()
btrfs-progs: avoid write to the disk before sure to create fs
cmds-replace.c | 33 +++---
mkfs.c | 104
On Wed, Aug 07, 2013 at 01:26:58PM +0200, David Sterba wrote:
I have to re-organize integration branch(es) better, so there is eg. a
branch without unstable stuff, possibly always in a pullable state. On
top of that a bunch of topic branches with the _ features.
... but then there's no point
Am Dienstag, 6. August 2013, 16:05:50 schrieb Eric Sandeen:
On 8/6/13 3:45 PM, Filipe David Manana wrote:
On Tue, Aug 6, 2013 at 9:37 PM, Eric Sandeen sand...@redhat.com wrote:
On 8/6/13 1:27 PM, Filipe David Borba Manana wrote:
This change allows for most mount options to be persisted in
On 8/7/13 2:54 AM, Stefan Behrens wrote:
On Tue, 06 Aug 2013 22:49:32 -0500, Eric Sandeen wrote:
On 8/6/13 8:05 PM, Eric Sandeen wrote:
Mark many functions as static, and remove any resulting dead code.
Signed-off-by: Eric Sandeen sand...@redhat.com
---
...
Actually, what the heck was
There was a problem with send trying to overwrite a file that wasn't actually
the same. This is a test to check this particular case where receive fails when
it should succeed properly. I tested this to verify it fails without my fix and
passes with my fix. Thanks,
Signed-off-by: Josef Bacik
On Wed, Aug 07, 2013 at 02:38:12PM +0200, David Sterba wrote:
On Wed, Aug 07, 2013 at 01:26:58PM +0200, David Sterba wrote:
I have to re-organize integration branch(es) better, so there is eg. a
branch without unstable stuff, possibly always in a pullable state. On
top of that a bunch of
One of btrfs filesystems hanged on my server.
By hang, I mean it's not possible to finish any write operation, i.e.:
# cd /mnt/btrfs
# ls
some-file
# touch 1
[it never returns to shell here]
There are the following processes in D state:
root 2397 0.6 0.0 0 0 ?D
On Thu, Aug 08, 2013 at 02:17:05AM +0800, Tomasz Chmielewski wrote:
One of btrfs filesystems hanged on my server.
Yeah, that's the consequence of using BUG_ON() for error handling. It
forcefully tears down the task that executes it without returning back
up the stack. Any locks that are held
We can get ENOMEM trying to allocate dummy bufs for the rewind operation of the
tree mod log. Instead of BUG_ON()'ing in this case pass up ENOMEM. I looked
back through the callers and I'm pretty sure I got everybody who did BUG_ON(ret)
in this path. Thanks,
Signed-off-by: Josef Bacik
On Thu, Aug 08, 2013 at 02:17:05AM +0800, Tomasz Chmielewski wrote:
One of btrfs filesystems hanged on my server.
By hang, I mean it's not possible to finish any write operation, i.e.:
# cd /mnt/btrfs
# ls
some-file
# touch 1
[it never returns to shell here]
I've just posted a
On Wed, 7 Aug 2013 14:57:04 -0400
Josef Bacik jba...@fusionio.com wrote:
One of btrfs filesystems hanged on my server.
By hang, I mean it's not possible to finish any write operation,
i.e.:
# cd /mnt/btrfs
# ls
some-file
# touch 1
[it never returns to shell here]
We can get ENOMEM trying to allocate dummy bufs for the rewind operation of the
tree mod log. Instead of BUG_ON()'ing in this case pass up ENOMEM. I looked
back through the callers and I'm pretty sure I got everybody who did BUG_ON(ret)
in this path. Thanks,
Signed-off-by: Josef Bacik
stat(2) on btrfs returns a custom device, but proc uses s_dev from the super
block. This causes problems (abi breakage) because software (and users) are
not expecting the kernel to return different devices from these calls.
This patch fixes the problem by adding a new superblock flag,
--- a/fs/btrfs/ctree.c
+++ b/fs/btrfs/ctree.c
@@ -1211,7 +1211,8 @@ tree_mod_log_rewind(struct btrfs_fs_info *fs_info,
struct extent_buffer *eb,
BUG_ON(tm-slot != 0);
eb_rewin = alloc_dummy_extent_buffer(eb-start,
On Wed, Aug 07, 2013 at 01:02:30PM -0700, Zach Brown wrote:
--- a/fs/btrfs/ctree.c
+++ b/fs/btrfs/ctree.c
@@ -1211,7 +1211,8 @@ tree_mod_log_rewind(struct btrfs_fs_info *fs_info,
struct extent_buffer *eb,
BUG_ON(tm-slot != 0);
eb_rewin =
On Wed, Aug 07, 2013 at 12:57:18PM -0700, Mark Fasheh wrote:
stat(2) on btrfs returns a custom device, but proc uses s_dev from the super
block. This causes problems (abi breakage) because software (and users) are
not expecting the kernel to return different devices from these calls.
So fix
On Wed, Aug 07, 2013 at 01:18:26PM -0700, Christoph Hellwig wrote:
On Wed, Aug 07, 2013 at 12:57:18PM -0700, Mark Fasheh wrote:
stat(2) on btrfs returns a custom device, but proc uses s_dev from the super
block. This causes problems (abi breakage) because software (and users) are
not
We can get ENOMEM trying to allocate dummy bufs for the rewind operation of the
tree mod log. Instead of BUG_ON()'ing in this case pass up ENOMEM. I looked
back through the callers and I'm pretty sure I got everybody who did BUG_ON(ret)
in this path. Thanks,
Signed-off-by: Josef Bacik
There is no reason we can't just set the path to blocking and then do normal
GFP_NOFS allocations for these extent buffers. Thanks,
Signed-off-by: Josef Bacik jba...@fusionio.com
---
fs/btrfs/ctree.c | 16 ++--
fs/btrfs/extent_io.c |8
2 files changed, 14
From: Sergei Trofimovich sly...@gentoo.org
Found by uselex.rb:
set_state_private: [R]: exported from: fs/btrfs/disk-io.o fs/btrfs/built-in.o
fs/btrfs/btrfs.o
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
fs/btrfs/extent_io.c | 2 +-
fs/btrfs/extent_io.h | 1 -
2 files changed, 1
From: Sergei Trofimovich sly...@gentoo.org
Hi guys!
TL;DR:
the patches remove (currently or already) dead code
and localize symbol visibility used only in one module.
Thanks!
The Story:
Once upon a time I've got very simple idea to find dead code
in large C/C++ mixed project at work.
The
From: Sergei Trofimovich sly...@gentoo.org
Found by uselex.rb:
btrfs_lookup_fs_root: [R]: exported from: fs/btrfs/disk-io.o
fs/btrfs/built-in.o fs/btrfs/btrfs.o
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
fs/btrfs/disk-io.c | 4 ++--
1 file changed, 2 insertions(+), 2
From: Sergei Trofimovich sly...@gentoo.org
Found by uselex.rb:
btrfs_get_inode_ref_index: [R]: exported from: fs/btrfs/inode-item.o
fs/btrfs/btrfs.o fs/btrfs/built-in.o
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
fs/btrfs/ctree.h | 6 -
fs/btrfs/inode-item.c | 65
From: Sergei Trofimovich sly...@gentoo.org
Found by uselex.rb:
btrfs_start_transaction_lflush: [R]: exported from: fs/btrfs/btrfs.o
fs/btrfs/transaction.o fs/btrfs/built-in.o
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
fs/btrfs/transaction.c | 7 ---
fs/btrfs/transaction.h |
From: Sergei Trofimovich sly...@gentoo.org
Found by uselex.rb:
btrfs_write_and_wait_marked_extents: [R]: exported from: fs/btrfs/btrfs.o
fs/btrfs/transaction.o fs/btrfs/built-in.o
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
fs/btrfs/transaction.c | 4 ++--
fs/btrfs/transaction.h
From: Sergei Trofimovich sly...@gentoo.org
Found by uselex.rb:
btrfs_read_root_item: [R]: exported from: fs/btrfs/btrfs.o
fs/btrfs/built-in.o fs/btrfs/root-tree.o
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
fs/btrfs/ctree.h | 2 --
fs/btrfs/root-tree.c | 4 ++--
2 files
From: Sergei Trofimovich sly...@gentoo.org
Found by uselex.rb:
btrfs_reada_detach: [R]: exported from: fs/btrfs/btrfs.o fs/btrfs/built-in.o
fs/btrfs/reada.o
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
fs/btrfs/ctree.h | 1 -
fs/btrfs/reada.c | 9 +
2 files changed, 1
On 8/7/13 4:43 PM, Sergei Trofimovich wrote:
...
Meet uselex.rb: one-file script to parse 'nm' output:
https://github.com/trofi/uselex/blob/master/uselex.rb
Nice to meet you! I think I've met your close relative,
ftp://ftp.samba.org/pub/unpacked/junkcode/findstatic.pl :)
On Wed, 07 Aug 2013 17:02:23 -0500
Eric Sandeen sand...@redhat.com wrote:
On 8/7/13 4:43 PM, Sergei Trofimovich wrote:
...
Meet uselex.rb: one-file script to parse 'nm' output:
https://github.com/trofi/uselex/blob/master/uselex.rb
Nice to meet you! I think I've met your close
There is a lack of manpage of btrfs-image '-m' option.
Signed-off-by: Liu Bo bo.li@oracle.com
---
btrfs-image.c|1 +
man/btrfs-image.8.in |4
2 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/btrfs-image.c b/btrfs-image.c
index 82aa4f3..862f081 100644
---
I'm pretty I encountered the same leak over here, 15 days uptime and I
had consistent out of memory problems, I got angry and did go in
single user mode and noticed that with only systemd-screen-bash(2) I
had 2 GB of memory used on 4 GB total. This box mostly has random
reads and just before the
find_extent_in_eb() may return ENOMEM, catch this error return value.
Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com
Reviewed-by: Miao Xie mi...@cn.fujitsu.com
---
fs/btrfs/backref.c | 4
1 file changed, 4 insertions(+)
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c
index
Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com
Reviewed-by: Miao Xie mi...@cn.fujitsu.com
---
fs/btrfs/backref.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c
index cb73a12..54e7610 100644
--- a/fs/btrfs/backref.c
+++
struct __prelim_ref is allocated and freed frequently when
walking backref tree, using slab allocater can not only
speed up allocating but also detect memory leaks.
Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com
Reviewed-by: Miao Xie mi...@cn.fujitsu.com
---
fs/btrfs/backref.c | 30
On Thu, Aug 08, 2013 at 01:04:17PM +0800, Wang Shilong wrote:
Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com
Reviewed-by: Miao Xie mi...@cn.fujitsu.com
Sorry, I don't think I understand why it's a memory leak, some explanation
is needed here.
-liubo
---
fs/btrfs/backref.c | 2 +-
On Thu, Aug 08, 2013 at 01:04:17PM +0800, Wang Shilong wrote:
Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com
Reviewed-by: Miao Xie mi...@cn.fujitsu.com
I think I know the whys :p, but still a log is preferred.
-liubo
---
fs/btrfs/backref.c | 2 +-
1 file changed, 1 insertion(+), 1
On thu, 8 Aug 2013 13:22:12 +0800, Liu Bo wrote:
On Thu, Aug 08, 2013 at 01:04:17PM +0800, Wang Shilong wrote:
Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com
Reviewed-by: Miao Xie mi...@cn.fujitsu.com
I think I know the whys :p, but still a log is preferred.
Right, we need a
51 matches
Mail list logo