[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [patch #8737] tcp_timer_needed should use tcpip_timeout
From: |
Robert Szewczyk |
Subject: |
[lwip-devel] [patch #8737] tcp_timer_needed should use tcpip_timeout |
Date: |
Fri, 11 Sep 2015 21:31:07 +0000 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.85 Safari/537.36 |
Follow-up Comment #6, patch #8737 (project lwip):
There are two more things worth bringing up:
* Even with the warnings in rawapi.txt about thread safety, a programmer might
lull themselves into a (false) sense of security by thinking: "they warned us
about the thread safety, but they provided these helpful
LOCK_TCPIP_CORE/UNLOCK_TCPIP_CORE, so I'll use those in the 'right' places and
it will be great". That's the path we pursued. The particular problem with
tcpip timers was exposed when we tuned the system to minimize the wakeups.
* Some functions (as we've discovered recently) are not in the standard API,
and, yet, they do call into tcp_* methods -- the particular function that came
to bite us recently was the dhcp_release, which may ultimately close a
connection. Perhaps the "proper" solution here would have also been to go
through tcpip_callback.
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?8737>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/