4.9-rc7 regression: scrub status blocks

2016-12-02 Thread M G Berberich
Hello, on a system with 4.9.0-rc7 I have the problem, that ‘scrub status’ blocks when a scru is running. Example. I startet a scrub and then (in another shell) ‘scrub status’ was startet while. It blocked and returned only after the scrub finished (hours later) with: WARNING: failed to read

Re: [PATCH] fstests: btrfs, add missing umount for raid5 tests 124 and 125

2016-12-02 Thread Filipe Manana
On Fri, Dec 2, 2016 at 1:14 AM, Anand Jain wrote: > > Hi, > > I didn't add umount at end of the test because... > _check_btrfs_filesystem() does it, which gets called as this test does not Nop it doesn't. See common/check: _check_filesystems() { if [ -f ${RESULT_DIR}/require_test ]; then

Re: [PATCH] fstests: btrfs, add missing umount for raid5 tests 124 and 125

2016-12-02 Thread Anand Jain
On 12/02/16 18:54, Filipe Manana wrote: On Fri, Dec 2, 2016 at 1:14 AM, Anand Jain wrote: Hi, I didn't add umount at end of the test because... _check_btrfs_filesystem() does it, which gets called as this test does not Nop it doesn't. See common/check: _check_filesystems() { if [ -f

Re: resend: Re: Btrfs: adjust len of writes if following a preallocated extent

2016-12-02 Thread Filipe Manana
On Fri, Dec 2, 2016 at 1:41 AM, Liu Bo wrote: > On Thu, Nov 24, 2016 at 11:13:37AM +, Filipe Manana wrote: >> On Wed, Nov 23, 2016 at 9:22 PM, Liu Bo wrote: >> > Hi, >> > >> > On Wed, Nov 23, 2016 at 06:21:35PM +0100, Stefan Priebe - Profihost AG >> > wrote: >> >> Hi, >> >> >> >> sorry last

Re: [PATCH] btrfs-progs: add dev stats returncode option

2016-12-02 Thread Austin S. Hemmelgarn
On 2016-12-01 15:32, Mike Fleetwood wrote: On 1 December 2016 at 18:43, Austin S. Hemmelgarn wrote: Currently, `btrfs device stats` returns non-zero only when there was an error getting the counter values. This is fine for when it gets run by a user directly, but is a serious pain when trying

[PATCH v2] btrfs-progs: add dev stats returncode option

2016-12-02 Thread Austin S. Hemmelgarn
Currently, `btrfs device stats` returns non-zero only when there was an error getting the counter values. This is fine for when it gets run by a user directly, but is a serious pain when trying to use it in a script or for monitoring since you need to parse the (not at all machine friendly) output

missing checksums on reboot

2016-12-02 Thread Blake Lewis
Hi, all, this is my first posting to the mailing list. I am a long-time file system guy who is just starting to take a serious interest in btrfs. My company's product uses btrfs for its backing storage. We maintain a log file to let us synchronize after reboots. In testing, we find that when th

Re: missing checksums on reboot

2016-12-02 Thread Lionel Bouton
Hi, Le 02/12/2016 à 20:07, Blake Lewis a écrit : > Hi, all, this is my first posting to the mailing list. I am a > long-time file system guy who is just starting to take a serious > interest in btrfs. > > My company's product uses btrfs for its backing storage. We > maintain a log file to let us

Re: missing checksums on reboot

2016-12-02 Thread Blake Lewis
Well, 3.10 is what you get with the RHEL7.x distributions, so that's why people are running it. Apparently, it is "good enough" for many purposes. My real goal here is to understand the scope of the bug and whether any mitigation is possible. Of course, I don't expect anyone else to make a patch

Re: [PATCH 01/18] btrfs: call functions that overwrite their root parameter with fs_info

2016-12-02 Thread Omar Sandoval
On Fri, Dec 02, 2016 at 12:07:21AM -0500, je...@suse.com wrote: > From: Jeff Mahoney > > There are 11 functions that accept a root parameter and immediately > overwrite it. We can pass those an fs_info pointer instead. > > Signed-off-by: Jeff Mahoney > --- > fs/btrfs/ctree.h| 4 +

Re: Metadata balance fails ENOSPC

2016-12-02 Thread Andrei Borzenkov
01.12.2016 18:48, Chris Murphy пишет: > On Thu, Dec 1, 2016 at 7:10 AM, Stefan Priebe - Profihost AG > wrote: >> >> Am 01.12.2016 um 14:51 schrieb Hans van Kranenburg: >>> On 12/01/2016 09:12 AM, Andrei Borzenkov wrote: On Thu, Dec 1, 2016 at 10:49 AM, Stefan Priebe - Profihost AG wrote

Re: missing checksums on reboot

2016-12-02 Thread Duncan
Blake Lewis posted on Fri, 02 Dec 2016 12:36:29 -0800 as excerpted: > Well, 3.10 is what you get with the RHEL7.x distributions, so that's why > people are running it. > Apparently, it is "good enough" for many purposes. > > My real goal here is to understand the scope of the bug and whether any

[PATCH] btrfs-progs: utils: negative numbers are more plausible than sizes over 8 EiB

2016-12-02 Thread Zygo Blaxell
I got tired of seeing "16.00EiB" whenever btrfs-progs encounters a negative size value. e.g. during filesystem shrink we see: Unallocated: /dev/mapper/testvol0 16.00EiB Interpreting this as a signed quantity is much more useful: Unallocated: /dev/mapper/testvol0 -26.29GiB Signed-off-b

Re: missing checksums on reboot

2016-12-02 Thread Blake Lewis
Thanks, Duncan. Your points are very reasonable and I will be doing a bisect, as you suggest (though I don't know that the problem is fixed in later releases, just that we couldn't reproduce it readily). Just to be clear, I am not looking for support. I just hoped that perhaps this had been a co