Re: [PATCH 0/4] Qgroup comment enhance and balance fix

2016-11-07 Thread David Sterba
On Tue, Oct 18, 2016 at 09:31:25AM +0800, Qu Wenruo wrote: > The patchset does the following things: > 1) Enhance comment for qgroup, rename 2 functions >Explain the how qgroup works, so new developers won't waste too much >time digging into the boring codes. > >The qgroup work flow

Re: [PATCH 0/4] Qgroup comment enhance and balance fix

2016-10-31 Thread David Sterba
On Fri, Oct 28, 2016 at 08:33:34AM +0800, Qu Wenruo wrote: > Any comment? > > Especially the final patch will fix a long standing bug. >From my perspective, review is required (but I appreciate Goldwyn's testing). Who's going to do that, I don't know. -- To unsubscribe from this list: send the

Re: [PATCH 0/4] Qgroup comment enhance and balance fix

2016-10-28 Thread Goldwyn Rodrigues
On 10/27/2016 07:33 PM, Qu Wenruo wrote: > Any comment? > > Especially the final patch will fix a long standing bug. > While I have tested it, and it works, I did not get time to review it. So, you can have my Tested-by: Goldwyn Rodrigues > Thanks, > Qu > > At

Re: [PATCH 0/4] Qgroup comment enhance and balance fix

2016-10-27 Thread Qu Wenruo
Any comment? Especially the final patch will fix a long standing bug. Thanks, Qu At 10/18/2016 09:31 AM, Qu Wenruo wrote: The patchset does the following things: 1) Enhance comment for qgroup, rename 2 functions Explain the how qgroup works, so new developers won't waste too much time

[PATCH 0/4] Qgroup comment enhance and balance fix

2016-10-17 Thread Qu Wenruo
The patchset does the following things: 1) Enhance comment for qgroup, rename 2 functions Explain the how qgroup works, so new developers won't waste too much time digging into the boring codes. The qgroup work flow is split into 3 main phrases: Reverse, Trace, Account. And rename