-
Notifications
You must be signed in to change notification settings - Fork 201
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ack Delay and TLP #1796
Merged
Merged
Ack Delay and TLP #1796
Commits on Sep 25, 2018
-
Fixes some remaining issues now that QUIC recommends sending an ACK every two packets, mostly involving reverting to the TCP style TLP calculation. I wrote this based on 'two full-sized packets' equalling a fixed 2400 bytes. QUIC could send immediate acks for non-full-sized packets, but that's not what TCP does. I could base it on the connection's packet size, but that requires the receiver to monitor the sender's max packet size, which made me a bit concerned? Advice on how to ensure sender and receiver agree on packet size would be appreciated.
Configuration menu - View commit details
-
Copy full SHA for 1245368 - Browse repository at this point
Copy the full SHA 1245368View commit details -
Configuration menu - View commit details
-
Copy full SHA for 33a6b56 - Browse repository at this point
Copy the full SHA 33a6b56View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0c4826f - Browse repository at this point
Copy the full SHA 0c4826fView commit details
Commits on Oct 2, 2018
-
Configuration menu - View commit details
-
Copy full SHA for 4fc23d7 - Browse repository at this point
Copy the full SHA 4fc23d7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 037bb0e - Browse repository at this point
Copy the full SHA 037bb0eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 01018b9 - Browse repository at this point
Copy the full SHA 01018b9View commit details -
Configuration menu - View commit details
-
Copy full SHA for b0d513d - Browse repository at this point
Copy the full SHA b0d513dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4ce2b80 - Browse repository at this point
Copy the full SHA 4ce2b80View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8cf0183 - Browse repository at this point
Copy the full SHA 8cf0183View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2202c69 - Browse repository at this point
Copy the full SHA 2202c69View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6117a2d - Browse repository at this point
Copy the full SHA 6117a2dView commit details -
Configuration menu - View commit details
-
Copy full SHA for f199e21 - Browse repository at this point
Copy the full SHA f199e21View commit details -
Configuration menu - View commit details
-
Copy full SHA for 058c0f2 - Browse repository at this point
Copy the full SHA 058c0f2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6e3f1e8 - Browse repository at this point
Copy the full SHA 6e3f1e8View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.