diff options
| author | Yuchung Cheng <ycheng@google.com> | 2013-07-12 11:33:04 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-07-12 16:15:56 -0700 |
| commit | 24ab6bec80861d0c55263047e8bf97e460a32e7b (patch) | |
| tree | 251608b48a332a1cc7b0fa0c72fd2e4679dcaf01 | |
| parent | e76385240ee5c812267319c9084e5a0d62bfbf90 (diff) | |
| download | op-kernel-dev-24ab6bec80861d0c55263047e8bf97e460a32e7b.zip op-kernel-dev-24ab6bec80861d0c55263047e8bf97e460a32e7b.tar.gz | |
tcp: account all retransmit failures
Change snmp RETRANSFAILS stat to include timeout retransmit failures
in addition to other loss recoveries.
Signed-off-by: Yuchung Cheng <ycheng@google.com>
Acked-by: Neal Cardwell <ncardwell@google.com>
Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/ipv4/tcp_output.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index 3d60949..92fde8d 100644 --- a/net/ipv4/tcp_output.c +++ b/net/ipv4/tcp_output.c @@ -2407,6 +2407,8 @@ int tcp_retransmit_skb(struct sock *sk, struct sk_buff *skb) * see tcp_input.c tcp_sacktag_write_queue(). */ TCP_SKB_CB(skb)->ack_seq = tp->snd_nxt; + } else { + NET_INC_STATS_BH(sock_net(sk), LINUX_MIB_TCPRETRANSFAIL); } return err; } @@ -2528,10 +2530,9 @@ begin_fwd: if (sacked & (TCPCB_SACKED_ACKED|TCPCB_SACKED_RETRANS)) continue; - if (tcp_retransmit_skb(sk, skb)) { - NET_INC_STATS_BH(sock_net(sk), LINUX_MIB_TCPRETRANSFAIL); + if (tcp_retransmit_skb(sk, skb)) return; - } + NET_INC_STATS_BH(sock_net(sk), mib_idx); if (tcp_in_cwnd_reduction(sk)) |
