Re: [PATCH v3] tcp: Use BPF timeout setting for SYN ACK RTO

From
Eric Dumazet <hidden>
Date
2021-11-03 20:54:43
Reply to
Akhmat Karakotov, 2021-11-03

On 11/3/21 1:46 PM, Akhmat Karakotov wrote:
When setting RTO through BPF program, some SYN ACK packets were unaffected
and continued to use TCP_TIMEOUT_INIT constant. This patch adds timeout
option to struct request_sock. Option is initialized with TCP_TIMEOUT_INIT
and is reassigned through BPF using tcp_timeout_init call. SYN ACK
retransmits now use newly added timeout option.

Signed-off-by: Akhmat Karakotov <redacted>
---
 include/net/request_sock.h      |  2 ++
 include/net/tcp.h               |  2 +-
 net/ipv4/inet_connection_sock.c |  4 +++-
 net/ipv4/tcp_input.c            |  8 +++++---
 net/ipv4/tcp_minisocks.c        | 12 +++++++++---
 5 files changed, 20 insertions(+), 8 deletions(-)
SGTM, thanks.

Reviewed-by: Eric Dumazet <redacted>
Thread (25 messages) 25 messages, 7 authors, last reply 2022-01-18
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help