Re: [PATCH] Btrfs: disable snapshot aware defrag for now

2014-02-04 Thread Josef Bacik
On 02/03/2014 10:19 PM, Roger Binns wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/02/14 09:27, Josef Bacik wrote: It is so totally broken that I don't want it being turned on by anybody who can't edit this and change it themselves. The symptoms I saw are huge amounts of kernel

Re: [PATCH] Btrfs: disable snapshot aware defrag for now

2014-02-03 Thread David Sterba
On Wed, Jan 29, 2014 at 04:05:30PM -0500, Josef Bacik wrote: It's just broken and it's taking a lot of effort to fix it, so for now just disable it so people can defrag in peace. Thanks, Cc: sta...@vger.kernel.org Signed-off-by: Josef Bacik jba...@fb.com --- fs/btrfs/inode.c | 2 +- 1

Re: [PATCH] Btrfs: disable snapshot aware defrag for now

2014-02-03 Thread Josef Bacik
On 02/03/2014 09:48 AM, David Sterba wrote: On Wed, Jan 29, 2014 at 04:05:30PM -0500, Josef Bacik wrote: It's just broken and it's taking a lot of effort to fix it, so for now just disable it so people can defrag in peace. Thanks, Cc: sta...@vger.kernel.org Signed-off-by: Josef Bacik

Re: [PATCH] Btrfs: disable snapshot aware defrag for now

2014-02-03 Thread Roger Binns
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/02/14 09:27, Josef Bacik wrote: It is so totally broken that I don't want it being turned on by anybody who can't edit this and change it themselves. The symptoms I saw are huge amounts of kernel memory consumption, possibly till exhaustion

[PATCH] Btrfs: disable snapshot aware defrag for now

2014-01-29 Thread Josef Bacik
It's just broken and it's taking a lot of effort to fix it, so for now just disable it so people can defrag in peace. Thanks, Cc: sta...@vger.kernel.org Signed-off-by: Josef Bacik jba...@fb.com --- fs/btrfs/inode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git