On 10/29/2016 11:50, Hans Petter Selasky wrote: > On 10/02/16 03:42, Eric van Gyzen wrote: >> + if (callout_reset(&ia->ia_garp_timer, (1 << ia->ia_garp_count) * hz, >> + garp_rexmit, ia) == 0) { >> + ifa_ref(ifa); >> + } > > Hi, > > Be careful when you MFC this, because the return values of callout_reset() and > callout_stop() are not the same in FreeBSD-9, -10 and 11/12.
Thanks for your keen eye, Hans. Indeed, the submitter (David Bright) discovered this The Hard Way when porting this forward from 10 to head. Luckily, though, that means we already have a version of this patch for 10. Thanks again, Eric (and David) _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"