lwIP - A Lightweight TCP/IP stack - Bugs: bug #21181, Initial congestion window
You are not allowed to post comments on this tracker with your current authentication level.
bug #21181: Initial congestion window
Submitter: | Oleg Tyshev <olegreen> | ||
Submitted: | Fri 28 Sep 2007 12:46:39 PM UTC | ||
Category: | TCP | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | None |
Jump to the original submission
Thu 01 Nov 2007 03:54:51 PM UTC, comment #7: |
Simon Goldschmidt <goldsimon> |
Thu 01 Nov 2007 02:29:57 PM UTC, comment #6:
|
Simon Goldschmidt <goldsimon> |
Thu 01 Nov 2007 12:32:09 PM UTC, comment #5: Let's stick with the 2MSS initial congestion window, but the other change that this bug suggests looks like a good one to me. |
Kieran Mansley <kieranm> |
Mon 29 Oct 2007 03:53:33 PM UTC, comment #4:
min(4MSS, max (2MSS, 4380 bytes)) is from RFC 3390.
|
Oleg Tyshev <olegreen> |
Sun 28 Oct 2007 06:19:06 PM UTC, comment #3:
|
Simon Goldschmidt <goldsimon> |
Mon 08 Oct 2007 07:58:18 AM UTC, comment #2: In state SYN_SENT we have already sent our maximum segment size in first TCP segment (MSS in TCP options).
|
Oleg Tyshev <olegreen> |
Thu 04 Oct 2007 06:39:42 PM UTC, comment #1: Could you maybe explain this a little more detailed (e.g. why do you think the order should be reversed) for those of us not currently having the time looking through the code? |
Simon Goldschmidt <goldsimon> |
Fri 28 Sep 2007 12:46:39 PM UTC, original submission:
If we see tcp_process() function
|
Oleg Tyshev <olegreen> |
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 3 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2007-11-01 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed |
Powered by Savane 3.13-02a9.
Corresponding source code
OK, fixed this.
What remains is that the sent MSS option is based on pcb->mss (the one received from the remote host) instead of TCP_MSS (the one configured).
What also is missing is the outgoing MSS to be automatically configured smaller than the MTU (see RFC 1122, chapter 4.2.2.6).
I'll open a task for these.