Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-11-01 Thread Wang Xiaoguang
hi, I have rebased these 2 patches against Linux 4.9-rc2, sorry for being late. After applying these patches, Stefan does not see any ENOSPC error :) If you have free time, please have a check, thanks. Regards, Xiaoguang Wang On 11/01/2016 06:18 PM, Wang Xiaoguang wrote: When testing btrfs

[PATCH 2/2] btrfs: fix false enospc for compression

2016-11-01 Thread Wang Xiaoguang
When testing btrfs compression, sometimes we got ENOSPC error, though fs still has much free space, xfstests generic/171, generic/172, generic/173, generic/174, generic/175 can reveal this bug in my test environment when compression is enabled. After some debuging work, we found that it's

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-25 Thread Wang Xiaoguang
hi, On 10/19/2016 10:23 PM, David Sterba wrote: On Mon, Oct 17, 2016 at 05:01:46PM +0800, Wang Xiaoguang wrote: [..] int btrfs_set_extent_delalloc(struct inode *inode, u64 start, u64 end, - struct extent_state **cached_state); +

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-19 Thread David Sterba
On Mon, Oct 17, 2016 at 05:01:46PM +0800, Wang Xiaoguang wrote: > > [..] > >> int btrfs_set_extent_delalloc(struct inode *inode, u64 start, u64 end, > >> -struct extent_state **cached_state); > >> +struct extent_state **cached_state, int flag); >

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-17 Thread David Sterba
On Wed, Oct 12, 2016 at 11:12:42AM +0800, Wang Xiaoguang wrote: > hi, > > Stefan often reports enospc error in his servers when having btrfs > compression > enabled. Now he has applied these 2 patches to run and no enospc error > occurs > for more than 6 days, it seems they are useful :) > >

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-17 Thread Wang Xiaoguang
hi, On 10/14/2016 09:59 PM, Holger Hoffstätte wrote: On 10/06/16 04:51, Wang Xiaoguang wrote: When testing btrfs compression, sometimes we got ENOSPC error, though fs still has much free space, xfstests generic/171, generic/172, generic/173, generic/174, generic/175 can reveal this bug in my

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-14 Thread Holger Hoffstätte
On 10/06/16 04:51, Wang Xiaoguang wrote: > When testing btrfs compression, sometimes we got ENOSPC error, though fs > still has much free space, xfstests generic/171, generic/172, generic/173, > generic/174, generic/175 can reveal this bug in my test environment when > compression is enabled. > >

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-14 Thread Stefan Priebe - Profihost AG
Am 06.10.2016 um 04:51 schrieb Wang Xiaoguang: > When testing btrfs compression, sometimes we got ENOSPC error, though fs > still has much free space, xfstests generic/171, generic/172, generic/173, > generic/174, generic/175 can reveal this bug in my test environment when > compression is

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-11 Thread Wang Xiaoguang
hi, Stefan often reports enospc error in his servers when having btrfs compression enabled. Now he has applied these 2 patches to run and no enospc error occurs for more than 6 days, it seems they are useful :) And these 2 patches are somewhat big, please check it, thanks. Regards,

Re: [PATCH 2/2] btrfs: fix false enospc for compression

2016-10-05 Thread Wang Xiaoguang
Hi, On 10/06/2016 10:51 AM, Wang Xiaoguang wrote: When testing btrfs compression, sometimes we got ENOSPC error, though fs still has much free space, xfstests generic/171, generic/172, generic/173, generic/174, generic/175 can reveal this bug in my test environment when compression is enabled.

[PATCH 2/2] btrfs: fix false enospc for compression

2016-10-05 Thread Wang Xiaoguang
When testing btrfs compression, sometimes we got ENOSPC error, though fs still has much free space, xfstests generic/171, generic/172, generic/173, generic/174, generic/175 can reveal this bug in my test environment when compression is enabled. After some debuging work, we found that it's