search

L4STeam / linux

Kernel tree containing patches for TCP Prague and the dualpi2 qdisc
Other
47 stars 16 forks source link

cwr_stamp not initialized might cause the first CWR to be skipped #6

Closed koen0607 closed 3 years ago

koen0607 commented 3 years ago

proposed to init cwr_stamp with a value that is 10 seconds ago:

ca->cwr_stamp = tp->tcp_mstamp - 10000000; // init 10 seconds ago
oliviertilmans commented 3 years ago

It is 0-initialized (like the rest of the icsk_ca_priv storage) when the socket is initialized/assigned a congestion control: https://github.com/L4STeam/linux/blob/testing/net/ipv4/tcp_cong.c#L170