commit 4269098e14603e38936ea7351705373ad55c08a7
author Florin Coras <fcoras@cisco.com> Tue May 02 12:40:39 2017 -0700
committer Dave Barach <openvpp@barachs.net> Tue May 02 21:52:56 2017 +0000
tree 52fd936551278b3eacc51d4022c4e3f753119a77
parent f3bcdbf071c98ed676591bd22c3d3f8601009fa8

Fix TCP tx when snd_wnd < smss

Fixes VPP-728, VPP-729, VPP-730

Change-Id: Ie8c6c0dd006f98527525e87d19b508bb8d39db69
Signed-off-by: Florin Coras <fcoras@cisco.com>

src/vnet/tcp/tcp.c

diff --git a/src/vnet/tcp/tcp.c b/src/vnet/tcp/tcp.c
index 245a35a..de4edfa 100644
--- a/src/vnet/tcp/tcp.c
+++ b/src/vnet/tcp/tcp.c
@@ -573,7 +573,7 @@
 u32
 tcp_session_send_space (transport_connection_t * trans_conn)
 {
-  u32 snd_space;
+  u32 snd_space, chunk;
   tcp_connection_t *tc = (tcp_connection_t *) trans_conn;
 
   /* If we haven't gotten dupacks or if we did and have gotten sacked bytes
@@ -582,14 +582,15 @@
 		    && (tc->rcv_dupacks == 0
 			|| tc->sack_sb.last_sacked_bytes)))
     {
+      chunk = tc->snd_wnd > tc->snd_mss ? tc->snd_mss : tc->snd_wnd;
       snd_space = tcp_available_snd_space (tc);
 
       /* If we can't write at least a segment, don't try at all */
-      if (snd_space < tc->snd_mss)
+      if (chunk == 0 || snd_space < chunk)
 	return 0;
 
       /* round down to mss multiple */
-      return snd_space - (snd_space % tc->snd_mss);
+      return snd_space - (snd_space % chunk);
     }
 
   /* If in fast recovery, send 1 SMSS if wnd allows */