[GitHub] [incubator-nuttx] a-lunev edited a comment on pull request #4659: net/tcp(unbuffered): retransmit only one the earliest not acknowledged segment

[GitBox <gi...@apache.org>]

a-lunev edited a comment on pull request #4659:
URL: https://github.com/apache/incubator-nuttx/pull/4659#issuecomment-941886030


   >     * i suspect this is something should be fixed with cwnd, which nuttx doesn't have right now.
   > 
   >     * doesn't the "buffered" version behave in a similar way? it basically moves segments on unacked_q back to write_q.
   > 
   > 
   > how do you think?
   
   I think it would be reasonable to implement congestion window control as an option. Also I see that Fast Retransmit algorithm is not supported in unbuffered tcp mode of NuttX that is possibly even more important.
   
   Concerning "buffered" version I can not say at the moment as I have not yet analyzed the source code in that part.
   So far I'm using "unbuffered" mode as in my case it surprisingly provides twice as high throughput as "buffered" one. Though, I expected that "buffered" mode should have better performance compared to "unbuffered" one.


-- 
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-unsubscribe@nuttx.apache.org

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