Re: [PATCH] Btrfs: fix what bits we clear when erroring out from delalloc

2013-08-02 Thread Miao Xie
Hi, Josef On Tue, 30 Jul 2013 14:27:40 +0800, Miao Xie wrote: extent_clear_unlock_delalloc(inode, start, end, NULL, - EXTENT_DIRTY | - EXTENT_DELALLOC, +

Re: [PATCH] Btrfs: fix what bits we clear when erroring out from delalloc

2013-08-02 Thread Josef Bacik
On Fri, Aug 02, 2013 at 06:08:01PM +0800, Miao Xie wrote: Hi, Josef On Tue, 30 Jul 2013 14:27:40 +0800, Miao Xie wrote: extent_clear_unlock_delalloc(inode, start, end, NULL, - EXTENT_DIRTY | -

Re: [PATCH] Btrfs: fix what bits we clear when erroring out from delalloc

2013-07-30 Thread Miao Xie
On mon, 29 Jul 2013 13:24:22 -0400, Josef Bacik wrote: First of all we no longer set EXTENT_DIRTY when we dirty an extent so this patch removes the clearing of EXTENT_DIRTY since it confuses me. This patch also adds clearing EXTENT_DEFRAG and also doing EXTENT_DO_ACCOUNTING when we have

[PATCH] Btrfs: fix what bits we clear when erroring out from delalloc

2013-07-29 Thread Josef Bacik
First of all we no longer set EXTENT_DIRTY when we dirty an extent so this patch removes the clearing of EXTENT_DIRTY since it confuses me. This patch also adds clearing EXTENT_DEFRAG and also doing EXTENT_DO_ACCOUNTING when we have errors. This is because if we are clearing delalloc without

[PATCH] Btrfs: fix what bits we clear when erroring out from delalloc V2

2013-07-29 Thread Josef Bacik
First of all we no longer set EXTENT_DIRTY when we dirty an extent so this patch removes the clearing of EXTENT_DIRTY since it confuses me. This patch also adds clearing EXTENT_DEFRAG and also doing EXTENT_DO_ACCOUNTING when we have errors. This is because if we are clearing delalloc without