Re: [GIT PULL] Btrfs pull request

2011-11-09 Thread Christoph Hellwig
On Wed, Nov 09, 2011 at 10:06:55AM +0100, Arne Jansen wrote: That's... unexpected. I guess this won't change with 3.3? For 3.3 I have to rework that part from scrub to account for Chris' bigblocks. If it can wait that long I'd prefer to fix both at once. Chris? device mapper has always been

Re: [GIT PULL] Btrfs pull request

2011-11-09 Thread Jan Schmidt
Am 09.11.2011 08:48, schrieb Christoph Hellwig: On Tue, Nov 08, 2011 at 08:07:01PM -0500, Chris Mason wrote: Looks like bio_add_page() is failing and we're getting the enomem from there. LVM is only letting us put one page in each bio. Yes, at the moment all bio based DM targets only allow

Re: [GIT PULL] Btrfs pull request

2011-11-09 Thread Christoph Hellwig
On Wed, Nov 09, 2011 at 11:29:31AM +0100, Jan Schmidt wrote: Am 09.11.2011 08:48, schrieb Christoph Hellwig: On Tue, Nov 08, 2011 at 08:07:01PM -0500, Chris Mason wrote: Looks like bio_add_page() is failing and we're getting the enomem from there. LVM is only letting us put one page in each

Re: [GIT PULL] Btrfs pull request

2011-11-09 Thread Jan Schmidt
On 09.11.2011 11:29, Jan Schmidt wrote: grep bio_add_page fs/btrfs/*.c will make you unhappy. Phew. Actually, not that unhappy. Many of the lines seeming to have no return value check are in fact part of a long or-ed if-statement. Seems okay after a closer look. -Jan -- To unsubscribe from

Re: [GIT PULL] Btrfs pull request

2011-11-09 Thread Chris Mason
On Wed, Nov 09, 2011 at 10:06:55AM +0100, Arne Jansen wrote: On 09.11.2011 08:48, Christoph Hellwig wrote: On Tue, Nov 08, 2011 at 08:07:01PM -0500, Chris Mason wrote: Looks like bio_add_page() is failing and we're getting the enomem from there. LVM is only letting us put one page in each

Re: [GIT PULL] Btrfs pull request

2011-11-09 Thread Chris Mason
On Wed, Nov 09, 2011 at 11:29:31AM +0100, Jan Schmidt wrote: Am 09.11.2011 08:48, schrieb Christoph Hellwig: On Tue, Nov 08, 2011 at 08:07:01PM -0500, Chris Mason wrote: Looks like bio_add_page() is failing and we're getting the enomem from there. LVM is only letting us put one page in each

Re: [GIT PULL] Btrfs pull request

2011-11-08 Thread Dan Merillat
On Sun, Nov 6, 2011 at 1:38 PM, Chris Mason chris.ma...@oracle.com wrote: Hi everyone, This pull request is pretty beefy, it ended up merging a number of long running projects and cleanup queues.  I've got btrfs patches in the new kernel.org btrfs repo.  There are two different branches with

Re: [GIT PULL] Btrfs pull request

2011-11-08 Thread Chris Mason
On Tue, Nov 08, 2011 at 12:55:40PM -0500, Dan Merillat wrote: On Sun, Nov 6, 2011 at 1:38 PM, Chris Mason chris.ma...@oracle.com wrote: Hi everyone, This pull request is pretty beefy, it ended up merging a number of long running projects and cleanup queues.  I've got btrfs patches in the

Re: [GIT PULL] Btrfs pull request

2011-11-08 Thread Chris Mason
On Tue, Nov 08, 2011 at 01:27:28PM -0500, Chris Mason wrote: On Tue, Nov 08, 2011 at 12:55:40PM -0500, Dan Merillat wrote: On Sun, Nov 6, 2011 at 1:38 PM, Chris Mason chris.ma...@oracle.com wrote: Hi everyone, This pull request is pretty beefy, it ended up merging a number of long

Re: [GIT PULL] Btrfs pull request

2011-11-08 Thread Chris Mason
On Tue, Nov 08, 2011 at 08:47:56AM +0100, Arne Jansen wrote: On 07.11.2011 13:50, Andrea Gelmini wrote: 2011/11/7 Arne Jansen sensi...@gmx.net: is it 32 or 64 bit? 64bit. Please take a look at my other reply. Can you please have a look with strace to make sure it's really the ioctl

Re: [GIT PULL] Btrfs pull request

2011-11-08 Thread Dan Merillat
On Tue, Nov 8, 2011 at 3:17 PM, Chris Mason chris.ma...@oracle.com wrote: On Tue, Nov 08, 2011 at 01:27:28PM -0500, Chris Mason wrote: On Tue, Nov 08, 2011 at 12:55:40PM -0500, Dan Merillat wrote: On Sun, Nov 6, 2011 at 1:38 PM, Chris Mason chris.ma...@oracle.com wrote: Hi everyone,

Re: [GIT PULL] Btrfs pull request

2011-11-08 Thread Christoph Hellwig
On Tue, Nov 08, 2011 at 08:07:01PM -0500, Chris Mason wrote: Looks like bio_add_page() is failing and we're getting the enomem from there. LVM is only letting us put one page in each bio. Yes, at the moment all bio based DM targets only allow single page I/O. -- To unsubscribe from this list:

Re: [GIT PULL] Btrfs pull request

2011-11-07 Thread Arne Jansen
On 06.11.2011 20:29, Andrea Gelmini wrote: 2011/11/6 Chris Mason chris.ma...@oracle.com: Hi Chris, and thanks a lot for your work. Arne Jansen and Jan Schmidt have improved the scrubber and provided utilities to walk btrfs' many backrefs. The scrubber is much faster thanks to

Re: [GIT PULL] Btrfs pull request

2011-11-07 Thread Arne Jansen
On 07.11.2011 10:49, Roman Mamedov wrote: On Mon, 07 Nov 2011 10:37:16 +0100 Arne Jansen sensi...@gmx.net wrote: I've got this: root@Q45:/home/gelma/dev/prg/btrfs# ./btrfs scrub start -Br /dev/md126 ERROR: scrubbing /dev/md126 failed for device id 1 (Cannot allocate memory) scrub

Re: [GIT PULL] Btrfs pull request

2011-11-07 Thread Andrea Gelmini
2011/11/7 Arne Jansen sensi...@gmx.net: On what platform are you running this? Can you please try this after a fresh boot? Maybe there's an allocation that can't be served with a badly fragmented memory. Hi Arne, and thanks a lot for your reply. So: a) it's a fresh Ubuntu 11.04, with

Re: [GIT PULL] Btrfs pull request

2011-11-07 Thread Arne Jansen
Hi Andrea, On 07.11.2011 13:42, Andrea Gelmini wrote: 2011/11/7 Arne Jansen sensi...@gmx.net: On what platform are you running this? Can you please try this after a fresh boot? Maybe there's an allocation that can't be served with a badly fragmented memory. Hi Arne, and thanks a lot

Re: [GIT PULL] Btrfs pull request

2011-11-07 Thread Andrea Gelmini
2011/11/7 Roman Mamedov r...@romanrm.ru: If so, shouldn't there also be a corresponding dmesg warning about Unable to allocate, which would confirm or rule this out? So before following the did you try turning it off and on again advice (and throwing away useful debug info), I'd suggest

Re: [GIT PULL] Btrfs pull request

2011-11-07 Thread Andrea Gelmini
2011/11/7 Arne Jansen sensi...@gmx.net: is it 32 or 64 bit? 64bit. Please take a look at my other reply. Thanks a lot for your time, Andrea -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [GIT PULL] Btrfs pull request

2011-11-07 Thread Arne Jansen
On 07.11.2011 13:50, Andrea Gelmini wrote: 2011/11/7 Arne Jansen sensi...@gmx.net: is it 32 or 64 bit? 64bit. Please take a look at my other reply. Can you please have a look with strace to make sure it's really the ioctl the ENOMEM originates froM? Thanks, Arne Thanks a lot for your

[GIT PULL] Btrfs pull request

2011-11-06 Thread Chris Mason
Hi everyone, This pull request is pretty beefy, it ended up merging a number of long running projects and cleanup queues. I've got btrfs patches in the new kernel.org btrfs repo. There are two different branches with the same changes. for-linus is against 3.1 and has also been tested against

Re: [GIT PULL] Btrfs pull request

2011-11-06 Thread Andrea Gelmini
2011/11/6 Chris Mason chris.ma...@oracle.com: Hi Chris, and thanks a lot for your work. Arne Jansen and Jan Schmidt have improved the scrubber and provided utilities to walk btrfs' many backrefs.  The scrubber is much faster thanks to extensive btree readahead and instead of just telling