Details

Reviewers

tuexen
hiren
lstewart
cc
rgrimes
rrs
pauamma_gundo.com

Group Reviewers

transport
manpages

Commits

rG4012ef7754cf: tcp: Functional implementation of Accurate ECN

Summary

This Diff adds initial support for AccECN, per IETF draft

https://tools.ietf.org/html/draft-ietf-tcpm-accurate-ecn-16

Only the AccECN handshake and TCP header flags are supported,
no support for the AccECN option. This minimalistic
implementation is sufficient to support DCTCP while
dramatically cutting the number of ACKs.

To enable AccECN support, sysctl net.inet.tcp.ecn.enable
was extended to include the value 3 (active AccECN support),
and 4 (passive AccECN support), similar to the Linux AccECN
patch.

Both AccECN modes will negotiate actively or passively for
legacy ECN (RFC3168) support, similar to a setting of 1 or
2, respectively, as per the other ends capabilities.

The (3 bit) ACE field, comprised of AE (formerly Nonce Sum,
CWR and ECE, reflects back the lowest 3 bits of the received
packet counter with CE set.

Test Plan

// Testing AccECN handshakes
// Both Client and Server, as well as parallel SYN

--tolerance_usecs=2000000

// Set sysctl defaults
0.0 `sysctl net.inet.tcp.cc.algorithm=newreno`
+0.1 `sysctl net.inet.tcp.functions_default=freebsd`
+0.1 `sysctl net.inet.tcp.initcwnd_segments=10`
+0.1 `sysctl net.inet.tcp.ecn.enable=4`
+0.1 `sysctl net.inet.tcp.hostcache.purgenow=1`
+0.1 `dmesg -c`

// Test Server side

// Establish a regular TCP ECN connection.
+0.0 `echo 'server: regular ECN'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.01 bind(3, ..., ...) = 0
+0.01 listen(3, 1) = 0
+0.04 <[noecn] SEW  0:0(0) win 65535 <mss 1000, wscale 10, eol >
+0.00 >[noecn] SE.  0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn]  .   1:1(0) ack 1 win 65535
+0.00 accept(3, ..., ...) = 4
+0.00 close(4) = 0
+0.00 >[noecn] R. 1:1(0) ack 1 win 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, non-ECT
+0.0 `echo 'server: AccECN non-ECT'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.01 bind(3, ..., ...) = 0
+0.01 listen(3, 1) = 0
+0.04 <[noecn] S7   0:0(0) win 65535 <mss 1000, wscale 10, eol >
+0.00 >[noecn] SW.  0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn]  .   1:1(0) ack 1 win 65535
+0.00 accept(3, ..., ...) = 5
+0.00 close(5) = 0
+0.00 >[noecn] R. 1:1(0) ack 1 win 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, ECT0
+0.0 `echo 'server: AccECN ECT0'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.01 bind(3, ..., ...) = 0
+0.01 listen(3, 1) = 0
+0.04 <[ect0]  S7   0:0(0) win 65535 <mss 1000, wscale 10, eol >
+0.00 >[noecn] SA.  0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn]  .   1:1(0) ack 1 win 65535
+0.00 accept(3, ..., ...) = 6
+0.00 close(6) = 0
+0.00 >[noecn] R. 1:1(0) ack 1 win 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, ECT1
+0.0 `echo 'server: AccECN ECT1'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.01 bind(3, ..., ...) = 0
+0.01 listen(3, 1) = 0
+0.04 <[ect1]  S7   0:0(0) win 65535 <mss 1000, wscale 10, eol >
+0.00 >[noecn] SEW. 0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn]  .   1:1(0) ack 1 win 65535
+0.00 accept(3, ..., ...) = 7
+0.00 close(7) = 0
+0.00 >[noecn] R. 1:1(0) ack 1 win 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, CE
+0.0 `echo 'server: AccECN CE'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.01 bind(3, ..., ...) = 0
+0.01 listen(3, 1) = 0
+0.04 <[ce]    S7   0:0(0) win 65535 <mss 1000, wscale 10, eol >
+0.00 >[noecn] SAW. 0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn]  .   1:1(0) ack 1 win 65535
+0.00 accept(3, ..., ...) = 8
+0.00 close(8) = 0
+0.00 >[noecn] R. 1:1(0) ack 1 win 0
+0.00 close(3) = 0


// Test Client Side

// Establish a regular TCP connection.
+0.0 `echo 'client: no ECN'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S   0:0(0) win 65535 <...>
+0.00 <[noecn] S.  0:0(0) ack 1 win 65535
+0.00 >[noecn]  .  1:1(0) ack 1 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

+0.0 `echo 'client: AccECN to non-ECN'`
+0.10 `sysctl net.inet.tcp.ecn.enable=3`
+0.40 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7  0:0(0) win 65535 <...>
+0.00 <[noecn] S.  0:0(0) ack 1 win 65535
+0.00 >[noecn]  .  1:1(0) ack 1 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

+0.0 `echo 'client: AccECN to RFC3168 ECN'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7  0:0(0) win 65535 <...>
+0.00 <[noecn] SE. 0:0(0) ack 1 win 65535
+0.00 >[noecn]  .  1:1(0) ack 1 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, non-ECT
+0.0 `echo 'client: AccECN non-ECT SYNACK'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7  0:0(0) win 65535 <...>
+0.00 <[noecn] SW. 0:0(0) ack 1 win 65535
+0.00 >[noecn]  .2  1:1(0) ack 1 win 65535
+0.00 <[noecn]  .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .5  1:1(0) ack 2 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, ECT0
+0.0 `echo 'client: AccECN ECT0 SYNACK'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7  0:0(0) win 65535 <...>
+0.00 <[ect0]  SW. 0:0(0) ack 1 win 65535
+0.00 >[noecn]  .4  1:1(0) ack 1 win 65535
+0.00 <[noecn]  .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .5  1:1(0) ack 2 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, ECT1
+0.0 `echo 'client: AccECN ECT1 SYNACK'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7  0:0(0) win 65535 <...>
+0.00 <[ect1]  SW. 0:0(0) ack 1 win 65535
+0.00 >[noecn]  .3  1:1(0) ack 1 win 65535
+0.00 <[noecn]  .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .5  1:1(0) ack 2 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Establish a AccECN TCP session, CE
+0.0 `echo 'client: AccECN CE SYNACK'`
+0.50 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7  0:0(0) win 65535 <...>
+0.00 <[ce]    SW. 0:0(0) ack 1 win 65535
+0.00 >[noecn]  .6  1:1(0) ack 1 win 65535
+0.00 <[noecn]  .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .6  1:1(0) ack 2 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Simultaneous Open

// Establish a regular TCP ECN connection.
+0.0 `echo 'parallel open: ECN'`
+0.10 `sysctl net.inet.tcp.ecn.enable=1`
+0.40 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] SEW  0:0(0) win 65535 <...>
+0.00 <[noecn] SEW  0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn] SE.  0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn] SE.  0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn] .    1:1(0) ack 1 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.05 write(3, ..., 1000) = 1000
+0.00 >[ect0]  PW.  1:1001(1000) ack 1 win 65535
+0.00 <[noecn] E.   1:1(0) ack 1001 win 65535
+0.05 write(3, ..., 1000) = 1000
+0.00 >[ect0]  PW.  1001:2001(1000) ack 1 win 65535
+0.00 <[ce]    .    1:2(1) ack 2001 win 65535
// rack: ect0??
+0.00 >[noecn] E.   2001:2001(0) ack 2 win 65535
+0.00 close(3) = 0

// Establish an AccECN TCP ECN without ECT
+0.0 `echo 'parallel open: AccECN non-ECT'`
+0.10 `sysctl net.inet.tcp.ecn.enable=3`
+0.40 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7   0:0(0) win 65535 <...>
+0.00 <[noecn] S7   0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn] SW.  0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn] SW.  0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn]  .5  1:1(0) ack 1 win 65535
+0.00 <[ce]     .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .6  1:1(0) ack 2 win 65535
+0.05 write(3, ..., 1000) = 1000
+0.00 >[ect0]  P.6  1:1001(1000) ack 2 win 65535
+0.00 <[ce]     .5  2:3(1) ack 1001 win 65535
+0.00 >[noecn]  .7  1001:1001(0) ack 3 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Establish an AccECN TCP ECN with ECT0
+0.0 `echo 'parallel open: AccECN ECT0'`
+0.40 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7   0:0(0) win 65535 <...>
+0.00 <[ect0]  S7   0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn] SA.  0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn] SW.  0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn]  .5  1:1(0) ack 1 win 65535
+0.00 <[ce]     .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .6  1:1(0) ack 2 win 65535
+0.05 write(3, ..., 1000) = 1000
+0.00 >[ect0]  P.6 1:1001(1000) ack 2 win 65535
+0.00 <[ce]     .5  2:3(1) ack 1001 win 65535
+0.00 >[noecn]  .7  1001:1001(0) ack 3 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Establish an AccECN TCP ECN with ECT1
+0.0 `echo 'parallel open: AccECN ECT1'`
+0.40 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7   0:0(0) win 65535 <...>
+0.00 <[ect1]  S7   0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn] SEW. 0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn] SW.  0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn]  .5  1:1(0) ack 1 win 65535
+0.00 <[ce]     .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .6  1:1(0) ack 2 win 65535
+0.05 write(3, ..., 1000) = 1000
+0.00 >[ect0]  P.6  1:1001(1000) ack 2 win 65535
+0.00 <[ce]     .5  2:3(1) ack 1001 win 65535
+0.00 >[noecn]  .7  1001:1001(0) ack 3 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

// Establish an AccECN TCP ECN with CE
+0.0 `echo 'parallel open: AccECN CE'`
+0.40 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3
+0.01 fcntl(3, F_SETFL, O_RDWR|O_NONBLOCK) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
+0.01 setsockopt(3, SOL_SOCKET, SO_LINGER, {onoff=1, linger=0}, 8) = 0
+0.04 connect(3, ..., ...) = -1 // EINPROGRESS
+0.00 >[noecn] S7   0:0(0) win 65535 <...>
+0.00 <[ce]    S7   0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn] SAW. 0:0(0) ack 1 win 65535 <...>
+0.00 <[noecn] SW.  0:0(0) ack 1 win 65535 <mss 1000>
+0.00 >[noecn]  .6  1:1(0) ack 1 win 65535
+0.00 <[ce]     .5  1:2(1) ack 1 win 65535
+0.04 >[noecn]  .7  1:1(0) ack 2 win 65535
+0.05 write(3, ..., 1000) = 1000
+0.00 >[ect0]  P.7  1:1001(1000) ack 2 win 65535
+0.00 <[ce]     .5  2:3(1) ack 1001 win 65535
+0.00 >[noecn]  .0  1001:1001(0) ack 3 win 65535
+0.01 getsockopt(3, SOL_SOCKET, SO_ERROR, [0], [4]) = 0
+0.01 fcntl(3, F_SETFL, O_RDWR) = 0
+0.00 close(3) = 0

`{
dmesg
sysctl net.inet.tcp.functions_default=freebsd;
sysctl net.inet.tcp.cc.algorithm=newreno;
sysctl net.inet.tcp.initcwnd_segments=10;
sysctl net.inet.tcp.rfc3465=1;
sysctl net.inet.tcp.rfc1323=1;
sysctl net.inet.tcp.sack.enable=1;
sysctl net.inet.tcp.rexmit_slop=200;
sysctl net.inet.tcp.rexmit_min=30;
sysctl net.inet.tcp.rexmit_initial=1000;
sysctl net.inet.tcp.ecn.enable=2;
}`

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25538
Build 24153: arc lint + arc unit

Event Timeline

rscheff created this revision.Jul 21 2019, 12:08 PM

Herald added a reviewer: transport. · View Herald TranscriptJul 21 2019, 12:08 PM

Herald added a subscriber: ae. · View Herald Transcript

Harbormaster completed remote builds in B25442: Diff 59984.Jul 21 2019, 12:08 PM

adding TH_AE to tcp_log_addr string
fixing initial cep couter on [ce]SYN to 6;
removal of debug code
remove debug code

Harbormaster completed remote builds in B25501: Diff 60105.Jul 24 2019, 6:33 PM

adding debug printout, fixing AccECN final ACK signaling
add ipv6 version check and #ifdef on ipv6 in t4_listen
final ACK of 3WHS working, proper initial s_cep, r_cep values
move all ECN related tcpcb flags to flags2
whitespace, remove debug logging code
missed trailing space
fix ipv6 version check

Harbormaster completed remote builds in B25511: Diff 60155.Jul 26 2019, 2:43 AM

adding CC reaction on <SYN(,ACK)> that are CE marked
limiting initwin to 2mss on a [CE] delivered for <SYN> or <SYN,ACK>
Set the initial window explicitly to 2*MSS

Harbormaster completed remote builds in B25518: Diff 60168.Jul 26 2019, 3:41 PM

adding AccECN settings of net.inet.tcp.ecn.enable to man page

Harbormaster completed remote builds in B25538: Diff 60195.Jul 27 2019, 3:37 PM

Adjust IW if SYN or SYN,ACK were delivered with [CE] set, as indicated by the SYN,ACK or final ACK of the 3WHS.

WIP: Refactor the negotiation of ECN in tcp_input.c and tcp_output.c into a sepatate file, so that rack can make use of identical utility function.

rscheff mentioned this in D24576: Update Rack and BBR to the latest from NF.Apr 27 2020, 12:34 PM

rgrimes added a reviewer: rgrimes.Apr 29 2020, 2:54 PM

Herald added a subscriber: melifaro. · View Herald TranscriptApr 29 2020, 2:54 PM

manual rebase to main

Herald added a reviewer: manpages. · View Herald TranscriptFeb 6 2022, 2:46 PM

Herald added subscribers: glebius, imp. · View Herald Transcript

Harbormaster completed remote builds in B44308: Diff 102432.Feb 6 2022, 2:46 PM

cleaning up a set of KMOD_ macros, and nicifying source code

Harbormaster completed remote builds in B44314: Diff 102438.Feb 7 2022, 8:20 AM

pauamma_gundo.com added a subscriber: pauamma_gundo.com.Feb 8 2022, 4:38 AM

pauamma_gundo.com added inline comments.

share/man/man4/tcp.4
567	Will the audience for this likely know what "accurate ECN" mean?
571

improve man wording

Harbormaster completed remote builds in B44329: Diff 102484.Feb 8 2022, 7:32 AM

rscheff marked 2 inline comments as done.Feb 8 2022, 7:32 AM

rscheff added inline comments.

share/man/man4/tcp.4
567	I would think so. Once adoped as an proposed Standard RFC, the RFC number can be added. In between, this is the current draft document: https://www.ietf.org/archive/id/draft-ietf-tcpm-accurate-ecn-16.html

Manual page LGTM now. Can't attest to code or consistency between them.

rrs accepted this revision.Feb 10 2022, 3:09 PM

This revision is now accepted and ready to land.Feb 10 2022, 3:09 PM

manual rebase to main

This revision now requires review to proceed.Feb 10 2022, 8:33 PM

Harbormaster completed remote builds in B44399: Diff 102634.Feb 10 2022, 8:33 PM

Can't see a manual page change since previous LGTM, so that and its disclaimers still apply.

rscheff edited the summary of this revision. (Show Details)Feb 11 2022, 11:34 AM

rscheff edited the test plan for this revision. (Show Details)

rename variables to match convention

Harbormaster completed remote builds in B44408: Diff 102652.Feb 11 2022, 12:16 PM

manual rebase to main

Harbormaster completed remote builds in B46515: Diff 108436.Jul 23 2022, 3:35 PM

fixed 2 oversights

Harbormaster completed remote builds in B46520: Diff 108449.Jul 24 2022, 12:48 PM

keep rack in sync

Harbormaster completed remote builds in B46523: Diff 108455.Jul 24 2022, 3:30 PM

This takes care of all my manual page edits as far as I can tell.

This revision is now accepted and ready to land.Jul 26 2022, 11:10 AM

rscheff mentioned this in D36303: Implement AccECN option (w/ early assigned option number).Aug 23 2022, 9:10 AM

Closed by commit rG4012ef7754cf: tcp: Functional implementation of Accurate ECN (authored by rscheff). · Explain WhyAug 31 2022, 1:24 PM

This revision was automatically updated to reflect the committed changes.

rscheff added a commit: rG4012ef7754cf: tcp: Functional implementation of Accurate ECN.

Functional implementation of Accurate ECN in FreeBSD
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 60195

share/man/man4/tcp.4

sys/dev/cxgbe/tom/t4_listen.c

sys/netinet/cc/cc_dctcp.c

sys/netinet/tcp.h

sys/netinet/tcp_input.c

sys/netinet/tcp_output.c

sys/netinet/tcp_stacks/rack.c

sys/netinet/tcp_subr.c

sys/netinet/tcp_syncache.h

sys/netinet/tcp_syncache.c

sys/netinet/tcp_usrreq.c

sys/netinet/tcp_var.h

sys/netinet/toecore.h

sys/netinet/toecore.c

usr.bin/netstat/inet.c

Functional implementation of Accurate ECN in FreeBSDClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 60195

share/man/man4/tcp.4

sys/dev/cxgbe/tom/t4_listen.c

sys/netinet/cc/cc_dctcp.c

sys/netinet/tcp.h

sys/netinet/tcp_input.c

sys/netinet/tcp_output.c

sys/netinet/tcp_stacks/rack.c

sys/netinet/tcp_subr.c

sys/netinet/tcp_syncache.h

sys/netinet/tcp_syncache.c

sys/netinet/tcp_usrreq.c

sys/netinet/tcp_var.h

sys/netinet/toecore.h

sys/netinet/toecore.c

usr.bin/netstat/inet.c

Functional implementation of Accurate ECN in FreeBSD
ClosedPublic
Actions

Revision Contents
Changeset List