[lwip-devel] [bug #52748] the bug in timeouts.c

[Douglas]

Follow-up Comment #5, bug #52748 (project lwip):

fwiw it might still be beneficial to adopt the change from patch #9533 to
sample sys_now() each time around the sys_check_timeouts loop. Consider the
case that the code in the handler installs a new timeout and if that is
already expired on return from the handler (in the limit the delay might be
zero). Consider the case of a slow handler, it might be best to bring forward
the handling of some pending timeouts if they have already expired.

Could it now do better with the cyclic timers, so note the last expired time,
and add the period to that, rather than to sys_now(), to calculate then next
expired time. This way the times for a cyclic timer should note drift wrt the
system time.

So perhaps create a separate version of sys_timeout that takes an absolute
expiry time, and have sys_timeout add sys_now to the msec and then call this
new function to install the timeout. This new entry point could be used to
reload the cyclic timeouts.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?52748>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/