Page MenuHomeFreeBSD
Differential D21011

Functional implementation of Accurate ECN in FreeBSD
ClosedPublic
Actions

Authored by rscheff on Jul 21 2019, 12:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 23, 1:43 PM
Unknown Object (File)
Sun, Feb 16, 8:05 PM
Unknown Object (File)
Sun, Feb 16, 1:55 PM
Unknown Object (File)
Wed, Feb 12, 10:57 PM
Unknown Object (File)
Sun, Feb 9, 2:32 PM
Unknown Object (File)
Jan 28 2025, 9:19 PM
Unknown Object (File)
Jan 27 2025, 12:56 AM
Unknown Object (File)
Jan 27 2025, 12:42 AM
View All Files
Subscribers
ae
glebius
imp
melifaro
pauamma_gundo.com

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 25501
Build 24120: 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
rscheff updated this revision to Diff 60105.Jul 24 2019, 6:33 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
rscheff updated this revision to Diff 60155.Jul 26 2019, 2:43 AM
  • 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
rscheff updated this revision to Diff 60168.Jul 26 2019, 3:40 PM
  • 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
rscheff updated this revision to Diff 60195.Jul 27 2019, 3:36 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
rscheff added a comment.Jul 27 2019, 3:41 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
rscheff updated this revision to Diff 102432.Feb 6 2022, 2:46 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
rscheff updated this revision to Diff 102438.Feb 7 2022, 8:20 AM
  • 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
709 ↗(On Diff #102438)

Will the audience for this likely know what "accurate ECN" mean?

713 ↗(On Diff #102438)
rscheff updated this revision to Diff 102484.Feb 8 2022, 7:32 AM
  • 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
709 ↗(On Diff #102438)

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

pauamma_gundo.com added a comment.Feb 9 2022, 4:54 AM

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
rscheff updated this revision to Diff 102634.Feb 10 2022, 8:33 PM
rscheff marked an inline comment as done.
  • 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
pauamma_gundo.com added a comment.Feb 11 2022, 4:11 AM

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)
rscheff updated this revision to Diff 102652.Feb 11 2022, 12:16 PM
  • rename variables to match convention
Harbormaster completed remote builds in B44408: Diff 102652.Feb 11 2022, 12:16 PM
rscheff updated this revision to Diff 108436.Jul 23 2022, 3:35 PM
  • manual rebase to main
Harbormaster completed remote builds in B46515: Diff 108436.Jul 23 2022, 3:35 PM
rscheff updated this revision to Diff 108449.Jul 24 2022, 12:48 PM
  • fixed 2 oversights
Harbormaster completed remote builds in B46520: Diff 108449.Jul 24 2022, 12:48 PM
rscheff updated this revision to Diff 108455.Jul 24 2022, 3:30 PM
  • keep rack in sync
Harbormaster completed remote builds in B46523: Diff 108455.Jul 24 2022, 3:30 PM
pauamma_gundo.com accepted this revision.Jul 26 2022, 11:10 AM

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.

Revision Contents
Changeset List

PathSize
sys/
dev/
cxgbe/
tom/
21 lines
netinet/
4 lines
74 lines
45 lines
15 lines
6 lines
96 lines
13 lines
2 lines
4 lines
usr.bin/
netstat/
9 lines

Diff 60105

View Options

sys/dev/cxgbe/tom/t4_listen.c

Loading...
View Options

sys/netinet/tcp.h

Loading...
View Options

sys/netinet/tcp_input.c

Loading...
View Options

sys/netinet/tcp_output.c

Loading...
View Options

sys/netinet/tcp_subr.c

Loading...
View Options

sys/netinet/tcp_syncache.h

Loading...
View Options

sys/netinet/tcp_syncache.c

Loading...
View Options

sys/netinet/tcp_var.h

Loading...
View Options

sys/netinet/toecore.h

Loading...
View Options

sys/netinet/toecore.c

Loading...
View Options

usr.bin/netstat/inet.c

Loading...