V2ray-core: QUIC with BBR congestion control

Created on 14 May 2019 · 20Comments · Source: v2ray/v2ray-core

There is a (what seems to be) very experimental PR that implements BBR congestion control for quic-go: https://github.com/lucas-clemente/quic-go/pull/1888

It might be worth looking into it and see how it improves v2ray's performance.

Stale

Source

Mygod

👍8

All 20 comments

Worth trying

kslr on 14 May 2019

我有点看不懂了，拥塞控制是 TCP 协议才会实现的事情吧，为什么可以把拥塞控制的算法用在基于 UDP 的 QUIC 协议上？😂

IceCodeNew on 18 May 2019

👎1

我有点看不懂了，拥塞控制是 TCP 协议才会实现的事情吧，为什么可以把拥塞控制的算法用在基于 UDP 的 QUIC 协议上？😂

因为QUIC跟BBR都是谷歌家的，所以...

zhj9709 on 21 May 2019

我有点看不懂了，拥塞控制是 TCP 协议才会实现的事情吧，为什么可以把拥塞控制的算法用在基于 UDP 的 QUIC 协议上？😂

并不是，本身QUIC支持调用TCP的拥塞算法（或者说QUIC本身就是想在UDP实现一个用户态的类TCP），更何况BBR和QUIC都是同一家的产品。

Cwek on 2 Jun 2019

确实QUIC没有mKCP快（看YouTube），也许加入BBR可以有性能提升

blinkspark on 4 Jun 2019

试了一下，效果显著，性能可以超越同配置的tcp bbr。

Windendless on 7 Jun 2019

quic就是超慢的很怕丢包.
所以这个看起来是非常必要的.

k79e on 26 Jun 2019

刚试了latest https://github.com/fbzhong/quic-go
编译不通过
会报错：

_build/src/v2ray.com/core/external/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup.go:120:5: cannot use &cryptoSetup literal (type *cryptoSetup) as type qtls.RecordLayer in assignment:
        *cryptoSetup does not implement qtls.RecordLayer (wrong type for SetReadKey method)
                have SetReadKey(*qtls.CipherSuite, []byte)
                want SetReadKey(qtls.EncryptionLevel, *qtls.CipherSuite, []byte)
_build/src/v2ray.com/core/external/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup.go:220:35: cannot use cs (type *cryptoSetup) as type qtls.RecordLayer in argument to tlsConfigToQtlsConfig:
        *cryptoSetup does not implement qtls.RecordLayer (wrong type for SetReadKey method)
                have SetReadKey(*qtls.CipherSuite, []byte)
                want SetReadKey(qtls.EncryptionLevel, *qtls.CipherSuite, []byte)

然后去latest github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup.go 检查了一下，发现，latest的代码中，SetReadKey的参数都是错的。
qtls里面RecordLayer Interface里的SetReadKey多了一个参数qtls.EncryptionLevel，然而，在crypto_setup.go里面的实现，并没有这个参数。

所以，完全不知道上游开发者，提交上去的代码是什么样的东西。