[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Douglas
Subject: [lwip-devel] [bug #52748] the bug in timeouts.c
Date: Fri, 5 Jan 2018 05:14:52 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0

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:


  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]