diff --git a/ChangeLog b/ChangeLog index 93150d588a4f0923c8e3fd36e2fe7a5c2880f087..7269302d63bc6e88e5fd67cb4f0d3d377ddeecd0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -863,3 +863,6 @@ * net/net_close.c. Fixed another important TCP/IP race condition bug: If the host closes the TCP connection just before the target calls close(), then the close operation may hang indefinitely! + * net/net_tcppoll.c. Removed an unnecessary check for outstanding, un-ACKed + data. The NuttX socket layer keeps track of ACKs and doesn't need this check; + removing the check should improve write throughput diff --git a/Documentation/NuttX.html b/Documentation/NuttX.html index 9c7269825ff20d2b3ff527aa82a8464c8ac8fb81..fdc628d49c5bb850d2e0e5473f18badfdf62acff 100644 --- a/Documentation/NuttX.html +++ b/Documentation/NuttX.html @@ -1524,6 +1524,9 @@ nuttx-0.4.11 2009-xx-xx Gregory Nutt <spudmonkey@racsa.co.cr> * net/net_close.c. Fixed another important TCP/IP race condition bug: If the host closes the TCP connection just before the target calls close(), then the close operation may hang indefinitely! + * net/net_tcppoll.c. Removed an unnecessary check for outstanding, un-ACKed + data. The NuttX socket layer keeps track of ACKs and doesn't need this check; + removing the check should improve write throughput pascal-0.1.3 2009-xx-xx Gregory Nutt <spudmonkey@racsa.co.cr> diff --git a/net/uip/uip_tcppoll.c b/net/uip/uip_tcppoll.c index b851ba1672eac65a10e32fdb7ef2dacf3d696614..e64ea2935a6da48fb8cbc05d6f16f6445f973571 100644 --- a/net/uip/uip_tcppoll.c +++ b/net/uip/uip_tcppoll.c @@ -96,12 +96,9 @@ void uip_tcppoll(struct uip_driver_s *dev, struct uip_conn *conn) { uint8 result; - /* Verify that the connection is established and if the connection has - * no outstanding (unacknowledged) sent data. - */ + /* Verify that the connection is established */ - if ((conn->tcpstateflags & UIP_TS_MASK) == UIP_ESTABLISHED && - !uip_outstanding(conn)) + if ((conn->tcpstateflags & UIP_TS_MASK) == UIP_ESTABLISHED) { /* Set up for the callback */