On Mon, May 06, 2013 at 11:24:13AM +0800, Cong Wang wrote:

On Fri, 2013-05-03 at 17:30 +0800, Herbert Xu wrote:

quoted

On Fri, May 03, 2013 at 03:08:46PM +0800, Cong Wang wrote:

quoted

On Fri, 2013-05-03 at 12:01 +0800, Herbert Xu wrote:

quoted

I think I quoted the wrong hunk in the patch, I meant the code
that arms the timer should no longer be in the leave_group function
unless we just sent a query ourselves (and in that case the expiration
should also be adjusted accordingly).

Is the following patch what you meant?

Right, but also add the corresponding timers in case we send a
group-specific query since that won't loop back to ourselves.

Ok, then something like below , it is a delta patch, the first piece
should go to patch 3/3.

Yes this looks about right.  BTW, for the groups-epcific queries
you should use the leave timeout values and not the startup timeout
values.

Thanks,
-- 
Email: Herbert Xu [off-list ref]
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt