Re: callout_drain either broken or man page needs updating

From: Gleb Smirnoff <glebius_at_FreeBSD.org>
Date: Fri, 15 Jul 2016 01:43:03 -0700
On Thu, Jul 14, 2016 at 10:14:46PM -0700, Matthew Macy wrote:
M>  > On 07/15/16 05:45, Matthew Macy wrote: 
M>  > > glebius last commit needs some further re-work. 
M>  >  
M>  > Glebius commit needs to be backed out, at least the API change that  
M>  > changes the return value when calling callout_stop() when the callout is  
M>  > scheduled and being serviced. Simply because there is code out there,  
M>  > like Mattew and others have discovered that is "refcounting" on the  
M>  > callout_reset() and expecting that a subsequent callout_stop() will  
M>  > return 1 to "unref". 
M> 
M> Yes. This is the cause of the "refcnt 0 on LLE at boot..." regression.

No it isn't. The regression is caused by unintentional change of return
value for never scheduled callout. The fix is now being tested, see PR 210884.

The piece of ND6 code that Hans quotes isn't affected by change of return
value for scheduled+running callout, since ND6 doesn't create callouts in
this tricky state.


-- 
Totus tuus, Glebius.
Received on Fri Jul 15 2016 - 06:43:10 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:06 UTC