Re: btrfs: failed to read chunk tree on vde1 (f18 anaconda crash)

2013-01-20 Thread Hugo Mills
On Sat, Jan 19, 2013 at 07:53:08PM -0300, Reartes Guillermo wrote: Hi, I reported a bug to anaconda (rhbz 901905), but it might be btfs related. Anaconda does this: 18:49:10,317 INFO program: Running... mkfs.btrfs --data=single --label=fedora /dev/vdb1 /dev/vdc1 /dev/vdd1 /dev/vde1

[PATCH 2/5] Btrfs: fix mutually exclusive op is running error code

2013-01-20 Thread Ilya Dryomov
The error code that is returned in response to starting a mutually exclusive operation when there is one already running got silently changed from EINVAL to EINPROGRESS by 5ac00add. Returning EINPROGRESS to, say, add_dev, when rm_dev is running is misleading. Furthermore, the operation itself

[PATCH 3/5] Btrfs: fix unlock order in btrfs_ioctl_resize

2013-01-20 Thread Ilya Dryomov
Fix unlock order in btrfs_ioctl_resize(). Signed-off-by: Ilya Dryomov idryo...@gmail.com --- fs/btrfs/ioctl.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index 3aebb28..b9a0190 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c

[PATCH 4/5] Btrfs: fix unlock order in btrfs_ioctl_rm_dev

2013-01-20 Thread Ilya Dryomov
Fix unlock order in btrfs_ioctl_rm_dev(). Signed-off-by: Ilya Dryomov idryo...@gmail.com --- fs/btrfs/ioctl.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index b9a0190..9c079dd 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c

[PATCH 0/5] Mutually exclusive ops fixes

2013-01-20 Thread Ilya Dryomov
Hello, This is a set of fixes for mutually exclusive ops stuff added by Stefan as part of the dev-replace patchset. The main one here is the first patch, it fixes a regression in balancing interface. The second patch brings EINVAL error code back. The rest are relatively minor ordering fixes.

Re: [PATCH 0/5] Mutually exclusive ops fixes

2013-01-20 Thread Ilya Dryomov
On Sun, Jan 20, 2013 at 05:22:27PM +0200, Ilya Dryomov wrote: Hello, This is a set of fixes for mutually exclusive ops stuff added by Stefan as part of the dev-replace patchset. The main one here is the first patch, it fixes a regression in balancing interface. The second patch brings

[PATCH 1/5] Btrfs: bring back balance pause/resume logic

2013-01-20 Thread Ilya Dryomov
Balance pause/resume logic got broken by 5ac00add (went in into 3.8-rc1 as part of dev-replace merge). Offending commit took a stab at making mutually exclusive volume operations (add_dev, rm_dev, resize, balance, replace_dev) not block behind volume_mutex if another such operation is in progress

scrub fails, any way to recover?

2013-01-20 Thread Neil Schemenauer
I have a ~350 GB Btrfs filesystem that is corrupted. I think the damage was caused by a bad SATA cable. I can mount the filesystem and read most of the data (I already have backups of most everything). The scrub is aborted after a few seconds with the following error in the kernel log:

[PATCH 00/13] Btrfs-progs: more patches for integration

2013-01-20 Thread Gene Czarcinski
I have done some additional scraping of the mailing list to identify some low hanging fruit which I consider should be merged into the btrfs-progs repository. All of the patches below were applied on top of the 19 bugfix patches pulled together by David Sterba and the 6 patches now part of the

[PATCH 01/13] btrfs-progs: btrfs-image.c: Added NULL pointer check.

2013-01-20 Thread Gene Czarcinski
From: Nageswara R Sastry nasas...@in.ibm.com Check for the return value of 'open_ctree()' before dereferencing it. Signed-off-by: Nageswara R Sastry nasas...@in.ibm.com Signed-off-by: Gene Czarcinski g...@czarc.net --- btrfs-image.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 06/13] btrfs-show-super.c

2013-01-20 Thread Gene Czarcinski
remove extra blank line at EOF Signed-off-by: Gene Czarcinski g...@czarc.net --- btrfs-show-super.c | 1 - man/Makefile | 14 +++--- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/btrfs-show-super.c b/btrfs-show-super.c index a9e2524..7c66d84 100644 ---

[PATCH 05/13] Add btrfs-show-super

2013-01-20 Thread Gene Czarcinski
From: Goffredo Baroncelli kreij...@inwind.it Signed-off-by: Goffredo Baroncelli kreij...@inwind.it Signed-off-by: Gene Czarcinski g...@czarc.net --- Makefile | 5 +- btrfs-show-super.c | 284 + 2 files changed, 288 insertions(+), 1

[PATCH 12/13] Btrfs-progs: disable qgroupid 0 for quota_tree

2013-01-20 Thread Gene Czarcinski
From: Wang Shilong wangsl-f...@cn.fujitsu.com In kernel, qgroupid 0 is a special number when we run the quota group limit command. So, we should not be able to create a quota group whose id is 0, otherwise the kernel can't deal with it. Fix it. Signed-off-by: Wang Shilong

[PATCH 13/13] trivial patch for btrfs-progs

2013-01-20 Thread Gene Czarcinski
From: Arvin Schnell aschn...@suse.de Hi, please find attached a trivial patch for btrfs-progs. Likely not strictly needed but I noticed valgrind complaining about uninitialised memory in the ioctl call. Regards, Arvin Signed-off-by: Gene Czarcinski g...@czarc.net --- cmds-send.c | 1 + 1

[PATCH 11/13] Btrfs-progs: check the relation of two group by real level numbers

2013-01-20 Thread Gene Czarcinski
From: Wang Shilong wangsl-f...@cn.fujitsu.com Comparing qgroupid is not good way to check the relationship of two groups, the right way is to compare the real level numbers. Signed-off-by: Wang Shilong wangsl-f...@cn.fujitsu.com Signed-off-by: Miao Xie mi...@cn.fujitsu.com Signed-off-by: Gene

[PATCH 09/13] Btrfs-progs: fix arg parsing for btrfs qgroup limit commands

2013-01-20 Thread Gene Czarcinski
From: Wang Shilong wangsl-f...@cn.fujitsu.com We can use this command in two ways. 1. btrfs qgroup limit size qgroupid path 2. btrfs qgroup limit size path Before applying this patch, we differentiate them by check the parsing result of the second argument. It is not so good because it may make

[PATCH 10/13] Btrfs-progs: clean up reduplicate parse_qgroupid() and replace atoi with strtoull

2013-01-20 Thread Gene Czarcinski
From: Wang Shilong wangsl-f...@cn.fujitsu.com 1. parse_qgroupid() is implemented twice, clean up the reduplicate code. 2. atoi() can not detect errors, so use strtoull() instead of it. Signed-off-by: Wang Shilong wangsl-f...@cn.fujitsu.com Signed-off-by: Miao Xie mi...@cn.fujitsu.com

[PATCH 08/13] Btrfs-progs: bugfix for subvolume parent determination in btrfs send

2013-01-20 Thread Gene Czarcinski
From: Jan Schmidt list.bt...@jan-o-sch.net We missed to add the default subvolume, because it has no ROOT_BACKREF_ITEM. This made get_parent always fail for direct decendants of the default subvolume, resulting in lots of full streams where incremental streams were requested. Signed-off-by: Jan

[PATCH 04/13] Btrfs-progs: Fix a segmentation fault in btrfstune when device is invalid.

2013-01-20 Thread Gene Czarcinski
From: Chen Yang chenyang.f...@cn.fujitsu.com When open_ctree(device, ...) failed, the return value should be checked. Signed-off-by: Cheng Yang chenyang.f...@cn.fujitsu.com Signed-off-by: Gene Czarcinski g...@czarc.net --- btrfstune.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 07/13] Btrfs-progs: correcting misnamed parameter options for btrfs send

2013-01-20 Thread Gene Czarcinski
From: Jan Schmidt list.bt...@jan-o-sch.net Unfortunately, the command line options for btrfs send were misnamed. The -i option should not be used to give clone sources, we'll be using -c instead. Compatibily note: -i option was broken anyway, which makes it less critical renaming it. For

[PATCH 03/13] Btrfs-progs: correct btrfs receive usage string

2013-01-20 Thread Gene Czarcinski
From: Ulrik ulrik.sverd...@gmail.com Fix the usage string to specify the correct '-f' option for input file, not '-i'. Signed-off-by: Ulrik Sverdrup ulrik.sverd...@gmail.com Signed-off-by: Gene Czarcinski g...@czarc.net --- cmds-receive.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 02/13] Btrfs-progs: fix resolving of loop devices

2013-01-20 Thread Gene Czarcinski
From: Nirbheek Chauhan nirbheek.chau...@collabora.co.uk The LOOP_GET_STATUS ioctl truncates filenames to 64 characters. We should get the backing file for a given loop device from /sys/. This is how losetup does it as well. Signed-off-by: Gene Czarcinski g...@czarc.net --- utils.c | 26

Re: [PATCH 07/13] Btrfs-progs: correcting misnamed parameter options for btrfs send

2013-01-20 Thread Gene Czarcinski
Bad patch ... working on fix On 01/20/2013 04:04 PM, Gene Czarcinski wrote: From: Jan Schmidt list.bt...@jan-o-sch.net Unfortunately, the command line options for btrfs send were misnamed. The -i option should not be used to give clone sources, we'll be using -c instead. Compatibily note: -i

[PATCH] Btrfs-progs: correcting misnamed parameter options for btrfs send

2013-01-20 Thread Gene Czarcinski
This is a hand refit of the patch but otherwise the same. Unfortunately, the command line options for btrfs send were misnamed. The -i option should not be used to give clone sources, we'll be using -c instead. Compatibily note: -i option was broken anyway, which makes it less critical renaming

Re: [PATCH 00/11 V3] add show command to the subvol sub command

2013-01-20 Thread Gene Czarcinski
On 01/10/2013 06:41 AM, Anand Jain wrote: This is an attempt to make btrfs cli more end user friendly. And adds show subcommand to display all known (as of now) information of the given subvol including its snapshot(s). The below patch: Btrfs-progs: accommodate different layout for printing

A couple bugs with btrfs and 3.5.0 kernel

2013-01-20 Thread Elladan
I upgraded to Ubuntu 12.10 and thought, Hey, that 3.5 kernel is relatively recent. And they seem to finally have implemented restriping. Maybe it's time to try btrfs again! So, first off, I backed up all my data. Next, I decided I would attempt to use btrfs's features for my benefit.

Re: A couple bugs with btrfs and 3.5.0 kernel

2013-01-20 Thread Liu Bo
On Sun, Jan 20, 2013 at 05:39:57PM -0800, Elladan wrote: I upgraded to Ubuntu 12.10 and thought, Hey, that 3.5 kernel is relatively recent. And they seem to finally have implemented restriping. Maybe it's time to try btrfs again! So, first off, I backed up all my data. Next, I decided I

BTRFS dmesg error

2013-01-20 Thread Skender
Hi there, I noticed in the dmesg logs that there seems to be a bug/issue with btrfs. Here is the trace/message from dmesg. Im going by the instructions from the https://btrfs.wiki.kernel.org/index.php/Problem_FAQ#How_do_I_report_bugs_and_issues.3F . Let me know if you need anything else.

Re: A couple bugs with btrfs and 3.5.0 kernel

2013-01-20 Thread Elladan
On Sun, Jan 20, 2013 at 5:51 PM, Liu Bo bo.li@oracle.com wrote: On Sun, Jan 20, 2013 at 05:39:57PM -0800, Elladan wrote: Any ideas? I guess I could try to mount in degraded mode or try a 3.6 kernel or something, but this all seems like I should probably just restore from backups and move