quinn icon indicating copy to clipboard operation
quinn copied to clipboard

Update BBR implementation

Open Ralith opened this issue 4 years ago • 9 comments

BBR2 has been formally drafted at https://datatracker.ietf.org/doc/html/draft-cardwell-iccrg-bbr-congestion-control. We should update our implementation to match the logic therein.

Ralith avatar Dec 01 '21 19:12 Ralith

Hi @Ralith curious whether this is still a priority for quinn?, the issue has been stale for quite a while.

mogendi avatar Jan 27 '23 16:01 mogendi

Contributions in this direction would be very welcome, but I'm not aware of anyone having immediate plans to take it on. BBR isn't used by default so it's not a critical issue, though congestion control is important and there's lots of room for improvement both here and with the default Cubic implementation.

Ralith avatar Jan 27 '23 21:01 Ralith

@Ralith I could look into handling the move to BBRv2 if it doesn't clash with anyone. Not sure of the laws of the land, should I reuse this issue ?

mogendi avatar Jan 28 '23 07:01 mogendi

Sure, would be great if you can work on that -- and it's fine to reference this issue.

djc avatar Jan 30 '23 13:01 djc

Shouldn't be any risk of conflicting efforts. Thanks for having a go at it!

Ralith avatar Jan 31 '23 08:01 Ralith

BBR v3 has been published: https://github.com/google/bbr/tree/v3

@mogendi, are you still interested in working on this?

Ralith avatar Aug 22 '23 02:08 Ralith

@Ralith yes!

mogendi avatar Aug 22 '23 03:08 mogendi

Great, please let us know if there's anything we can do to help you make progress! Feel free to reach out here or on discord.

Ralith avatar Aug 22 '23 20:08 Ralith

will do @Ralith

mogendi avatar Aug 23 '23 10:08 mogendi