Skip to content
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.

Sign up for GitHub

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

Jump to bottom

implement BBR congestion control #341

Closed
marten-seemann opened this issue Nov 1, 2016 · 13 comments
Closed

implement BBR congestion control #341

marten-seemann opened this issue Nov 1, 2016 · 13 comments
Labels
feature

Comments

@marten-seemann
Copy link
Member

No description provided.

@marten-seemann marten-seemann added this to the 0.6 milestone Nov 1, 2016
@marten-seemann marten-seemann self-assigned this Nov 2, 2016
@marten-seemann marten-seemann removed their assignment Dec 19, 2016
@lucas-clemente lucas-clemente modified the milestone: 0.6 Apr 11, 2017
@glen-dai
Copy link

glen-dai commented May 6, 2017

When will be quic-go support BBR congestion control?

@marten-seemann
Copy link
Member Author

It's still on our list, but we have a lot of higher-priority issues at the moment.

@marten-seemann marten-seemann mentioned this issue Jul 24, 2017
Closed
@lucas-clemente lucas-clemente marked this as a duplicate of #752 Jul 24, 2017
@lucas-clemente lucas-clemente mentioned this issue Aug 1, 2017
Open
@marten-seemann marten-seemann mentioned this issue May 8, 2018
Closed
@vegertar vegertar mentioned this issue Jun 19, 2018
Closed
@marten-seemann marten-seemann mentioned this issue Aug 20, 2018
Closed
@LibinLiu0189
Copy link

is BBR implemented in any quic-go branch now? Thanks a lot.

@marten-seemann
Copy link
Member Author

No, and we're currently not planning to implement it anytime soon.

@LibinLiu0189
Copy link

No, and we're currently not planning to implement it anytime soon.

Got it. Thanks.

@jinleileiking
Copy link

I do not know why this thing is not on the top list. Bbr is so critical in quic cc

@ghost
Copy link

ghost commented Jan 20, 2020

any news on add it to quic-go？

@marten-seemann
Copy link
Member Author

There’s no need to regularly ping us on this issue. It will be updated once we start working on it.

@klzgrad klzgrad mentioned this issue Oct 3, 2020
Closed
@marten-seemann marten-seemann mentioned this issue May 16, 2022
Closed
@c1ngular
Copy link

Any news on this ?

@moranno
Copy link

moranno commented Oct 30, 2022

It's been almost 3 years...

@bt90
Copy link
Contributor

bt90 commented Oct 30, 2022

#3417 (comment) still stands

@marten-seemann
Copy link
Member Author

Update on BBR: We're not planning to implement BBR in quic-go. Over the years, BBR has been shown to not be the silver bullet to solve all of our performance problems. BBR remains a highly complex algorithm with an enormous set of magic configuration values. It has also not seen the widespread adoption that one might have expected after BBR's initial introduction.

In the mean time, different congestion controller algorithms have been explored by the congestion control community, with promising results. quic-go might experiment with those at some point in the future. Until then, there are lots of improvements outside of congestion control that will have a significant performance impact, for example #3526, #2482, #3563.

@marten-seemann marten-seemann closed this as not planned Won't fix, can't repro, duplicate, stale Oct 30, 2022
@aggresss aggresss mentioned this issue Aug 27, 2023
Open
@kixelated kixelated mentioned this issue Aug 31, 2023
Closed
@marten-seemann
Copy link
Member Author

BBRv3 support is now tracked in #4565.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@jinleileiking @lucas-clemente @marten-seemann @bt90 @glen-dai @c1ngular @LibinLiu0189 @moranno