svn commit: r306577 - head/sys/netinet

Eric van Gyzen vangyzen at FreeBSD.org
Sat Oct 29 19:16:27 UTC 2016


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)


More information about the svn-src-all mailing list