Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-08 Thread Qu Wenruo
At 08/09/2016 10:24 AM, Goldwyn Rodrigues wrote: On 08/08/2016 08:12 PM, Qu Wenruo wrote: At 08/09/2016 09:01 AM, Goldwyn Rodrigues wrote: On 08/08/2016 07:26 PM, Qu Wenruo wrote: At 08/08/2016 10:54 AM, Goldwyn Rodrigues wrote: On 08/07/2016 07:39 PM, Qu Wenruo wrote: At 08/07

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-08 Thread Goldwyn Rodrigues
On 08/08/2016 08:12 PM, Qu Wenruo wrote: > > > At 08/09/2016 09:01 AM, Goldwyn Rodrigues wrote: >> >> >> On 08/08/2016 07:26 PM, Qu Wenruo wrote: >>> >>> >>> At 08/08/2016 10:54 AM, Goldwyn Rodrigues wrote: On 08/07/2016 07:39 PM, Qu Wenruo wrote: > > > At 08/07/2016

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-08 Thread Qu Wenruo
At 08/09/2016 09:01 AM, Goldwyn Rodrigues wrote: On 08/08/2016 07:26 PM, Qu Wenruo wrote: At 08/08/2016 10:54 AM, Goldwyn Rodrigues wrote: On 08/07/2016 07:39 PM, Qu Wenruo wrote: At 08/07/2016 01:19 AM, Goldwyn Rodrigues wrote: Thanks Qu, This patch set fixes all the reported prob

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-08 Thread Goldwyn Rodrigues
On 08/08/2016 07:26 PM, Qu Wenruo wrote: > > > At 08/08/2016 10:54 AM, Goldwyn Rodrigues wrote: >> >> >> On 08/07/2016 07:39 PM, Qu Wenruo wrote: >>> >>> >>> At 08/07/2016 01:19 AM, Goldwyn Rodrigues wrote: Thanks Qu, This patch set fixes all the reported problems. However, I hav

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-08 Thread Qu Wenruo
At 08/08/2016 10:54 AM, Goldwyn Rodrigues wrote: On 08/07/2016 07:39 PM, Qu Wenruo wrote: At 08/07/2016 01:19 AM, Goldwyn Rodrigues wrote: Thanks Qu, This patch set fixes all the reported problems. However, I have some minor issues with coding style. Thanks for the test. On 08/04/2

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-07 Thread Goldwyn Rodrigues
On 08/07/2016 07:39 PM, Qu Wenruo wrote: > > > At 08/07/2016 01:19 AM, Goldwyn Rodrigues wrote: >> Thanks Qu, >> >> This patch set fixes all the reported problems. However, I have some >> minor issues with coding style. >> > > Thanks for the test. > >> >> On 08/04/2016 09:29 PM, Qu Wenruo wro

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-07 Thread Qu Wenruo
At 08/07/2016 01:19 AM, Goldwyn Rodrigues wrote: Thanks Qu, This patch set fixes all the reported problems. However, I have some minor issues with coding style. Thanks for the test. On 08/04/2016 09:29 PM, Qu Wenruo wrote: Refactor btrfs_qgroup_insert_dirty_extent() function, to two fun

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-06 Thread Goldwyn Rodrigues
Thanks Qu, This patch set fixes all the reported problems. However, I have some minor issues with coding style. On 08/04/2016 09:29 PM, Qu Wenruo wrote: > Refactor btrfs_qgroup_insert_dirty_extent() function, to two functions: > 1. _btrfs_qgroup_insert_dirty_extent() >Almost the same with or

Re: [PATCH v2 1/3] btrfs: qgroup: Refactor btrfs_qgroup_insert_dirty_extent()

2016-08-06 Thread Goldwyn Rodrigues
Thanks Qu, This patch set fixes all the reported problems. However, I have some minor issues with coding style. On 08/04/2016 09:29 PM, Qu Wenruo wrote: > Refactor btrfs_qgroup_insert_dirty_extent() function, to two functions: > 1. _btrfs_qgroup_insert_dirty_extent() >Almost the same with or