btrfs sync(2) performance

2012-04-02 Thread Andy Spencer
I've been testing (using) btrfs for a little while and noticed some performance issues a couple months ago whenever I called sync(2). After some googling it seem some other people have had similar issues: http://comments.gmane.org/gmane.comp.file-systems.btrfs/13511 https://bugs.launchpad.net/

[PATCH] Btrfs: do not mount when we have a sectorsize unequal to PAGE_SIZE

2012-04-02 Thread Liu Bo
Our code is not ready to cope with a sectorsize that's not equal to PAGE_SIZE. It will lead to hanging-on while writing something. Signed-off-by: Liu Bo --- fs/btrfs/disk-io.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c in

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Dave Jones
On Mon, Apr 02, 2012 at 07:50:50PM -0400, Chris Mason wrote: > > I'll start a bisect later to see if I can narrow it down at least. > > Ok, a directed bisect of the major suspects. Josef changed the extent > buffer eio code in this commit (jump to the commit before it): I had already starte

Re: [PATCH] Btrfs: do not mount when we have a sectorsize larger than PAGE_SIZE

2012-04-02 Thread Liu Bo
On 04/02/2012 08:17 PM, David Sterba wrote: > On Mon, Apr 02, 2012 at 07:28:18PM +0800, Liu Bo wrote: >> --- a/fs/btrfs/disk-io.c >> +++ b/fs/btrfs/disk-io.c >> @@ -2104,6 +2104,14 @@ int open_ctree(struct super_block *sb, >> err = -EINVAL; >> goto fail_alloc; >> } >>

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Chris Mason
On Mon, Apr 02, 2012 at 06:51:31PM -0400, Dave Jones wrote: > On Mon, Apr 02, 2012 at 06:39:19PM -0400, Chris Mason wrote: > > On Mon, Apr 02, 2012 at 06:33:50PM -0400, Dave Jones wrote: > > > On Mon, Apr 02, 2012 at 06:28:02PM -0400, Chris Mason wrote: > > > > > > > > x86-64. > > > > > >

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Dave Jones
On Mon, Apr 02, 2012 at 06:39:19PM -0400, Chris Mason wrote: > On Mon, Apr 02, 2012 at 06:33:50PM -0400, Dave Jones wrote: > > On Mon, Apr 02, 2012 at 06:28:02PM -0400, Chris Mason wrote: > > > > > > x86-64. > > > > > > > > dmesg below. (ignore the rpc oops, reported elsewhere, it's

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Chris Mason
On Mon, Apr 02, 2012 at 06:33:50PM -0400, Dave Jones wrote: > On Mon, Apr 02, 2012 at 06:28:02PM -0400, Chris Mason wrote: > > > > x86-64. > > > > > > dmesg below. (ignore the rpc oops, reported elsewhere, it's unrelated) > > > > Well, there really are no btrfs messages in there at all.

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Dave Jones
On Mon, Apr 02, 2012 at 06:28:02PM -0400, Chris Mason wrote: > > x86-64. > > > > dmesg below. (ignore the rpc oops, reported elsewhere, it's unrelated) > > Well, there really are no btrfs messages in there at all. Do you have > free space for a clean copy of the btrfs partition? Trying

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Chris Mason
On Mon, Apr 02, 2012 at 05:40:51PM -0400, Dave Jones wrote: > On Mon, Apr 02, 2012 at 05:26:08PM -0400, Chris Mason wrote: > > On Mon, Apr 02, 2012 at 05:16:22PM -0400, Dave Jones wrote: > > > On Mon, Apr 02, 2012 at 03:48:14PM -0400, Chris Mason wrote: > > > > On Mon, Apr 02, 2012 at 02:02:14P

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Dave Jones
On Mon, Apr 02, 2012 at 05:26:08PM -0400, Chris Mason wrote: > On Mon, Apr 02, 2012 at 05:16:22PM -0400, Dave Jones wrote: > > On Mon, Apr 02, 2012 at 03:48:14PM -0400, Chris Mason wrote: > > > On Mon, Apr 02, 2012 at 02:02:14PM -0400, Dave Jones wrote: > > > > Updated to rc1 this morning, an

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Chris Mason
On Mon, Apr 02, 2012 at 05:16:22PM -0400, Dave Jones wrote: > On Mon, Apr 02, 2012 at 03:48:14PM -0400, Chris Mason wrote: > > On Mon, Apr 02, 2012 at 02:02:14PM -0400, Dave Jones wrote: > > > Updated to rc1 this morning, and my machines with btrfs are all freaking > out.. > > > > > > I got t

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Dave Jones
On Mon, Apr 02, 2012 at 03:48:14PM -0400, Chris Mason wrote: > On Mon, Apr 02, 2012 at 02:02:14PM -0400, Dave Jones wrote: > > Updated to rc1 this morning, and my machines with btrfs are all freaking > > out.. > > > > I got this from my cron email .. > > > > /etc/cron.daily/prelink: line

WARNING: at fs/btrfs/extent-tree.c:6170 btrfs_alloc_free_block+0x354/0x360()

2012-04-02 Thread Alexey Shvetsov
Hi! I get this traces under high parallel io load on btrfs btrfs: block rsv returned -28 [ cut here ] WARNING: at fs/btrfs/extent-tree.c:6170 btrfs_alloc_free_block+0x354/0x360() Hardware name: EP45-DS4 Modules linked in: tun ath5k ath i2c_i801 Pid: 20480, comm: btrfs-endi

Re: btrfs io errors on 3.4rc1

2012-04-02 Thread Chris Mason
On Mon, Apr 02, 2012 at 02:02:14PM -0400, Dave Jones wrote: > Updated to rc1 this morning, and my machines with btrfs are all freaking out.. > > I got this from my cron email .. > > /etc/cron.daily/prelink: line 41: /var/lib/prelink/full: Input/output error > cp: cannot create regular file `/var/

kernel BUG at /home/apw/COD/linux/fs/btrfs/inode.c:3962!

2012-04-02 Thread Bob McElrath
A bug you all might be interested in. If I can provide any more information, please let me know. The command that crashed was find, being run by updatedb (daily cron job). I can't tell from below which partition it was looking at, but I have two raid1 (both data and metadata) btrfs filesystems:

btrfs io errors on 3.4rc1

2012-04-02 Thread Dave Jones
Updated to rc1 this morning, and my machines with btrfs are all freaking out.. I got this from my cron email .. /etc/cron.daily/prelink: line 41: /var/lib/prelink/full: Input/output error cp: cannot create regular file `/var/lib/prelink/quick': Input/output error /etc/cron.daily/prelink: line 45:

3.4rc1 btrfs IO problem ?

2012-04-02 Thread Dave Jones
I noticed something odd when I booted rc1 on my btrfs test box.. First I tried updating my git tree.. $ gp error: cannot open .git/FETCH_HEAD: Input/output error $ gp remote: Counting objects: 5101, done. remote: Compressing objects: 100% (546/546), done. remote: Total 3764 (delta 3233), reused 3

[PATCH] btrfs: add missing unlocks to transaction abort paths

2012-04-02 Thread David Sterba
Added in commit 49b25e0540904be0bf558b84475c69d72e4de66e ("btrfs: enhance transaction abort infrastructure") Reported-by: Dan Carpenter Signed-off-by: David Sterba --- fs/btrfs/transaction.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/fs/btrfs/transaction.c b/f

[btrfs-progs: PATCH 2/2] Makefile: use $(MAKE) instead of hardcoded 'make'

2012-04-02 Thread Sergei Trofimovich
From: Sergei Trofimovich CC: Chris Mason Signed-off-by: Sergei Trofimovich --- Makefile |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index ebfd5c8..969 100644 --- a/Makefile +++ b/Makefile @@ -101,15 +101,15 @@ ioctl-test: $(objects) ioc

[btrfs-progs: PATCH 1/2] Makefile: use $(CC) as a compilers instead of $(CC)/gcc

2012-04-02 Thread Sergei Trofimovich
From: Sergei Trofimovich CC: Chris Mason Signed-off-by: Sergei Trofimovich --- Makefile |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 79818e6..ebfd5c8 100644 --- a/Makefile +++ b/Makefile @@ -44,13 +44,13 @@ btrfs: $(objects) btrfs.o he

Re: btrfs: enhance transaction abort infrastructure

2012-04-02 Thread Dan Carpenter
On Mon, Apr 02, 2012 at 12:46:44PM +0200, David Sterba wrote: > > Thanks for cathing it. Are you going to send a patch or are you fine > with a Reported-by: ? > You're welcome. :) Reported-by is fine. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-

Re: [PATCH] Btrfs: do not mount when we have a sectorsize larger than PAGE_SIZE

2012-04-02 Thread David Sterba
On Mon, Apr 02, 2012 at 07:28:18PM +0800, Liu Bo wrote: > --- a/fs/btrfs/disk-io.c > +++ b/fs/btrfs/disk-io.c > @@ -2104,6 +2104,14 @@ int open_ctree(struct super_block *sb, > err = -EINVAL; > goto fail_alloc; > } > + if (btrfs_super_sectorsize(disk_super) > PA

[PATCH] Btrfs: do not mount when we have a sectorsize larger than PAGE_SIZE

2012-04-02 Thread Liu Bo
Our code is not ready to cope with a sectorsize that's larger than PAGE_SIZE. Signed-off-by: Liu Bo --- fs/btrfs/disk-io.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index 20196f4..08e49d2 100644 --- a/fs/btrfs/disk-io.

Re: [GIT PULL] Btrfs fixes and features

2012-04-02 Thread Liu Bo
On 03/31/2012 01:51 AM, Chris Mason wrote: > Hi everyone, > > This pull request is pretty big, picking up patches that have been under > development for some time. I have it in two branches: > > # against 3.3 > # > git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus >

Re: btrfs: enhance transaction abort infrastructure

2012-04-02 Thread David Sterba
Hi, On Mon, Apr 02, 2012 at 12:08:54PM +0300, Dan Carpenter wrote: > The patch 49b25e054090: "btrfs: enhance transaction abort > infrastructure" from Mar 1, 2012, leads to the following warning: > fs/btrfs/transaction.c:137 join_transaction() > warn: inconsistent returns spin_lock:&root->fs

re: btrfs: enhance transaction abort infrastructure

2012-04-02 Thread Dan Carpenter
Hi Jeff, The patch 49b25e054090: "btrfs: enhance transaction abort infrastructure" from Mar 1, 2012, leads to the following warning: fs/btrfs/transaction.c:137 join_transaction() warn: inconsistent returns spin_lock:&root->fs_info->trans_lock: locked (77) unlocked (64,70,82,88,13

Re: cross-subvolume cp --reflink

2012-04-02 Thread David Sterba
On Sun, Apr 01, 2012 at 10:27:38PM +0300, Konstantinos Skarlatos wrote: > thats true, cp --reflink is much better. Also am I wrong that btrfs has > a limitation on the number of hard links that can only be fixed with a > disk format change? There's patch in development to lift the limit and it d