• Eric Dumazet's avatar
    tcp: use tp->total_rto to track number of linear timeouts in SYN_SENT state · 14dd92d0
    Eric Dumazet authored
    In commit ccce324d ("tcp: make the first N SYN RTO backoffs linear")
    David used icsk->icsk_backoff field to track the number of linear timeouts.
    
    Since then, tp->total_rto has been added.
    
    This commit uses tp->total_rto instead of icsk->icsk_backoff
    so that tcp_ld_RTO_revert() no longer can trigger an overflow
    in inet_csk_rto_backoff(). Other than the potential UBSAN
    report, there was no issue because receiving an ICMP message
    currently aborts the connect().
    
    In the following patch, we want to adhere to RFC 6069
    and RFC 1122 4.2.3.9.
    Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
    Cc: David Morley <morleyd@google.com>
    Cc: Neal Cardwell <ncardwell@google.com>
    Cc: Yuchung Cheng <ycheng@google.com>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    14dd92d0
tcp_timer.c 25 KB