-
Notifications
You must be signed in to change notification settings - Fork 114
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
transport: default packet pacer #1000
Conversation
Benchmark report |
Benchmark report |
Benchmark report |
Benchmark report |
Benchmark report |
}; | ||
use core::ops::Div; | ||
|
||
struct Fraction(u32, u32); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice this is even better than what I was thinking
Issue #, if available: #146
Description of changes: This change introduces a packet pacer to be used by the Cubic congestion controller. This pacer calculates an interval at which bursts of packets should be distributed across an RTT using the congestion window, smoothed RTT, and MTU. Following Linux, the interval is smaller during Slow Start, to account for the higher rate of change of the congestion window during that time.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.