acassis commented on a change in pull request #4070:
URL: https://github.com/apache/incubator-nuttx/pull/4070#discussion_r664091493



##########
File path: net/tcp/Kconfig
##########
@@ -106,6 +106,27 @@ config NET_TCP_FAST_RETRANSMIT_WATERMARK
                        missing segment, without waiting for a retransmission 
timer to
                        expire.
 
+config NET_TCP_WINDOW_SCALE
+       bool "Enable TCP/IP Window Scale Option"
+       default n
+       ---help---
+               RFC1323:
+               2. TCP WINDOW SCALE OPTION
+                       The window scale extension expands the definition of 
the TCP
+                       window to 32 bits and then uses a scale factor to carry 
this 32-
+                       bit value in the 16-bit Window field of the TCP header 
(SEG.WND in
+                       RFC-793).
+
+if NET_TCP_WINDOW_SCALE
+
+config NET_TCP_WINDOW_SCALE_FACTOR
+       int "TCP/IP Window Scale Factor"
+       default 7

Review comment:
       > I just noticed that most of the network configurations are 
CONFIG_IOB_BUFSIZE is greater than 128, so the scale factor is set to 7. is it 
better to set the default value to 0?
   
   @anchao did you make some performance test? Maybe it is better to test it do 
define a good value.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to