lwIP - A Lightweight TCP/IP stack - Bugs: bug #27905, FIN isn't combined with data on...
You are not allowed to post comments on this tracker with your current authentication level.
bug #27905: FIN isn't combined with data on unsent
Submitter: | Simon Goldschmidt <goldsimon> | ||
Submitted: | Sun 01 Nov 2009 05:20:41 PM UTC | ||
Category: | TCP | Severity: | 3 - Normal |
Item Group: | Change Request | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | CVS Head |
Sun 22 Nov 2009 05:58:58 PM UTC, comment #3: |
Simon Goldschmidt <goldsimon> |
Mon 02 Nov 2009 10:33:53 AM UTC, comment #2: I spotted this while working on the raw api httpd: with HTTP/1.1, the server currently sends "Connection: close" because it closes the connection after each request/response. However, since the actual FIN needs a while, it happens often that the FINs are sent at the same time, resulting in duplicate FINs and ACKs.
|
Simon Goldschmidt <goldsimon> |
Mon 02 Nov 2009 09:10:46 AM UTC, comment #1: It was probably just simpler that way. I don't think this is a big deal. Whichever results in smaller code is best. |
Kieran Mansley <kieranm> |
Sun 01 Nov 2009 05:20:41 PM UTC, original submission:
In tcp_enqueue, we currently don't combine FIN segments to the last unsent. Does anybody know why? I would have thought that this would be a good idea to keep the number of segments sent low.
|
Simon Goldschmidt <goldsimon> |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2009-11-22 | goldsimon | Open/Closed | Open | Closed | |
Assigned to | None | goldsimon | |||
Status | Need Info | Fixed | |||
2009-11-01 | goldsimon | Summary | FIN isn\'t combined with data sent | FIN isn't combined with data on unsent |
Powered by Savane 3.13-02a9.
Corresponding source code
Fixed since the code change is rather small and the code size doesn't change much, too.