Re: btrfs scrub gives unable to find logical $hugenum len 16384

2013-04-20 Thread Martin Steigerwald
Am Samstag, 20. April 2013 schrieb Josef Bacik: On Fri, Apr 19, 2013 at 03:15:30AM -0600, Martin Steigerwald wrote: Am Dienstag, 16. April 2013 schrieb Martin Steigerwald: On Saturday 13 April 2013 17:48:31 Martin Steigerwald wrote: Hi! Please answer soon whether it would be a

Re: btrfs scrub gives unable to find logical $hugenum len 16384

2013-04-20 Thread Martin Steigerwald
Am Samstag, 20. April 2013 schrieb Josef Bacik: On Fri, Apr 19, 2013 at 03:15:30AM -0600, Martin Steigerwald wrote: Am Dienstag, 16. April 2013 schrieb Martin Steigerwald: On Saturday 13 April 2013 17:48:31 Martin Steigerwald wrote: Hi! Please answer soon whether it would be a

stale NFS file handle and corrupted btrfs

2013-04-20 Thread Chris Hoffmann
Hi, I have experienced an error message about stale NFS files when looking into a directory. ls -la manpages-dev.md5sums ls: cannot access manpages-dev.md5sums: Stale NFS file handle root@c3po:/media/revo/@/var/lib/dpkg/info# rm manpages-dev.md5sums rm: cannot remove ‘manpages-dev.md5sums’:

Re: minor patch to cmds-restore.c

2013-04-20 Thread Jon Nelson
On Fri, Apr 19, 2013 at 11:48 PM, Eric Sandeen sand...@redhat.com wrote: On 4/19/13 7:11 PM, Jon Nelson wrote: The following is a minor patch to cmds-restore.c Hi Jon - just a note - When sending a patch like this, it's best to follow the standard patch format, which closely mimics the

Re: corrupted filesystem. mounts with -o recovery,ro but not -o recovery or -o ro

2013-04-20 Thread Jon Nelson
Using 3.8.8, I tried mounting with -o recovery and -o recovery,nospace_cache (which shouldn't be any different, if I'm understanding the kernel sources properly) without any benefit. Then I tried btrfs-next ( bba653d1207646b17671c6cb9a0629736811848a as of this writing ) also without being able to

[PATCH] Btrfs: make ulist_{init,fin}() static and not exported as symbol

2013-04-20 Thread Wang Shilong
From: Wang Shilong wangsl-f...@cn.fujitsu.com ulist is a generic structure to iterate tree. we export them just in the hope it may be used by other modules. However, until now, it is just used by btrfs. The main point is that ulist_{init,fin}() should be only called by

Re: [PATCH] btrfs: make static code static remove dead code

2013-04-20 Thread Wang Shilong
Hello Eric, Big patch, but all it does is add statics to functions which are in fact static, then remove the associated dead-code fallout. removed functions: btrfs_iref_to_path() __btrfs_lookup_delayed_deletion_item() __btrfs_search_delayed_insertion_item()

Re: btrfs scrub gives unable to find logical $hugenum len 16384

2013-04-20 Thread Martin Steigerwald
Am Samstag, 20. April 2013 schrieb Josef Bacik: So I found your bug on the plane ride, as soon as I get home I'll email it. Thanks, I have redone my BTRFS now. But I have a dd image copy on my backup drive in case you have something to try out for me. Thanks, -- Martin 'Helios' Steigerwald

Re: [PATCH 0/4] [RFC] btrfs: offline dedupe

2013-04-20 Thread Gabriel de Perthuis
Hi, The following series of patches implements in btrfs an ioctl to do offline deduplication of file extents. I am a fan of this patch, the API is just right. I just have a few tweaks to suggest to the argument checking. At first the 1M limitation on the length was a bit inconvenient, but

Re: [PATCH] btrfs: make static code static remove dead code

2013-04-20 Thread Arne Jansen
On 04/19/13 21:21, Eric Sandeen wrote: Big patch, but all it does is add statics to functions which are in fact static, then remove the associated dead-code fallout. removed functions: btrfs_iref_to_path() __btrfs_lookup_delayed_deletion_item() __btrfs_search_delayed_insertion_item()

Re: [PATCH] Btrfs: make ulist_{init,fin}() static and not exported as symbol

2013-04-20 Thread Eric Sandeen
On 4/20/13 7:43 AM, Wang Shilong wrote: From: Wang Shilong wangsl-f...@cn.fujitsu.com ulist is a generic structure to iterate tree. we export them just in the hope it may be used by other modules. However, until now, it is just used by btrfs. The main point is that ulist_{init,fin}() should

Re: [PATCH] Btrfs: make ulist_{init,fin}() static and not exported as symbol

2013-04-20 Thread Arne Jansen
On 04/20/13 19:43, Eric Sandeen wrote: On 4/20/13 7:43 AM, Wang Shilong wrote: From: Wang Shilong wangsl-f...@cn.fujitsu.com ulist is a generic structure to iterate tree. we export them just in the hope it may be used by other modules. However, until now, it is just used by btrfs. The main

data DUP

2013-04-20 Thread Roger Binns
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Is there any particular reason why I can't use DUP for data? When I try to set it with balance there is a kernel message: btrfs: dup for data is not allowed The glossary at https://btrfs.wiki.kernel.org/index.php/Glossary says: Regular data

Re: data DUP

2013-04-20 Thread Hugo Mills
On Sat, Apr 20, 2013 at 01:17:06PM -0700, Roger Binns wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Is there any particular reason why I can't use DUP for data? Technically, no. Performance is likely to suck if you use rotational disks, and you may find some SSDs deduplicate

Re: data DUP

2013-04-20 Thread Roger Binns
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 20/04/13 13:48, Hugo Mills wrote: On Sat, Apr 20, 2013 at 01:17:06PM - -0700, Roger Binns wrote: Is there any particular reason why I can't use DUP for data? Technically, no. Performance is likely to suck if you use rotational disks, and you

Re: data DUP

2013-04-20 Thread Hugo Mills
On Sat, Apr 20, 2013 at 02:15:14PM -0700, Roger Binns wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 20/04/13 13:48, Hugo Mills wrote: On Sat, Apr 20, 2013 at 01:17:06PM - -0700, Roger Binns wrote: When I try to set it with balance there is a kernel message: btrfs: dup for

Re: data DUP

2013-04-20 Thread Roger Binns
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 20/04/13 14:23, Hugo Mills wrote: You should upgrade anyway -- there's been a number of serious bugs in btrfs fixed since then. 13.04 is imminent so I'll pick up a newer kernel as part of that anyway. (Also Tanglu which I hope to move to intends

Re: [PATCH] btrfs-progs: update mkfs.btrfs help info for raid5/6

2013-04-20 Thread Frank A. Kingswood
On 04/03/13 01:37, zwu.ker...@gmail.com wrote: From: Zhi Yong Wuwu...@linux.vnet.ibm.com Since raid5/6 support was introduced, we should update mkfs.btrfs help info. Signed-off-by: Zhi Yong Wuwu...@linux.vnet.ibm.com --- mkfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH] btrfs-progs: update mkfs.btrfs help info for raid5/6

2013-04-20 Thread Eric Sandeen
On 4/20/13 4:13 PM, Frank A. Kingswood wrote: On 04/03/13 01:37, zwu.ker...@gmail.com wrote: From: Zhi Yong Wuwu...@linux.vnet.ibm.com Since raid5/6 support was introduced, we should update mkfs.btrfs help info. Signed-off-by: Zhi Yong Wuwu...@linux.vnet.ibm.com --- mkfs.c | 2 +-