tcp: add data bytes sent stats
authorWei Wang <weiwan@google.com>
Wed, 1 Aug 2018 00:46:21 +0000 (17:46 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 1 Aug 2018 16:56:10 +0000 (09:56 -0700)
Introduce a new TCP stat to record the number of bytes sent
(RFC4898 tcpEStatsPerfHCDataOctetsOut) and expose it in both tcp_info
(TCP_INFO) and opt_stats (SOF_TIMESTAMPING_OPT_STATS).

Signed-off-by: Wei Wang <weiwan@google.com>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Acked-by: Neal Cardwell <ncardwell@google.com>
Acked-by: Soheil Hassas Yeganeh <soheil@google.com>
Acked-by: Yuchung Cheng <ycheng@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/tcp.h
include/uapi/linux/tcp.h
net/ipv4/tcp.c
net/ipv4/tcp_output.c

index 58a8d7d713549bee88113a0a3373dd8cc055d72d..d0798dcd2cabd436333837a916ccdd8c486b34ec 100644 (file)
@@ -181,6 +181,9 @@ struct tcp_sock {
        u32     data_segs_out;  /* RFC4898 tcpEStatsPerfDataSegsOut
                                 * total number of data segments sent.
                                 */
+       u64     bytes_sent;     /* RFC4898 tcpEStatsPerfHCDataOctetsOut
+                                * total number of data bytes sent.
+                                */
        u64     bytes_acked;    /* RFC4898 tcpEStatsAppHCThruOctetsAcked
                                 * sum(delta(snd_una)), or how many bytes
                                 * were acked.
index e3f6ed8a7064f9276ca2b57ed5ecff3364786e9d..1c70ed287c3b74a557b194bb66c242d8ab039949 100644 (file)
@@ -235,6 +235,8 @@ struct tcp_info {
 
        __u32   tcpi_delivered;
        __u32   tcpi_delivered_ce;
+
+       __u64   tcpi_bytes_sent;     /* RFC4898 tcpEStatsPerfHCDataOctetsOut */
 };
 
 /* netlink attributes types for SCM_TIMESTAMPING_OPT_STATS */
@@ -257,7 +259,7 @@ enum {
        TCP_NLA_SND_SSTHRESH,   /* Slow start size threshold */
        TCP_NLA_DELIVERED,      /* Data pkts delivered incl. out-of-order */
        TCP_NLA_DELIVERED_CE,   /* Like above but only ones w/ CE marks */
-
+       TCP_NLA_BYTES_SENT,     /* Data bytes sent including retransmission */
 };
 
 /* for TCP_MD5SIG socket option */
index 27bbe6a792b7151df084275fdf8a8a3e9c75b6b5..873cb9968ff547cf7c80fb68b017637e193c304f 100644 (file)
@@ -2594,6 +2594,7 @@ int tcp_disconnect(struct sock *sk, int flags)
        sk->sk_rx_dst = NULL;
        tcp_saved_syn_free(tp);
        tp->compressed_ack = 0;
+       tp->bytes_sent = 0;
 
        /* Clean up fastopen related fields */
        tcp_free_fastopen_req(tp);
@@ -3201,6 +3202,7 @@ void tcp_get_info(struct sock *sk, struct tcp_info *info)
                info->tcpi_delivery_rate = rate64;
        info->tcpi_delivered = tp->delivered;
        info->tcpi_delivered_ce = tp->delivered_ce;
+       info->tcpi_bytes_sent = tp->bytes_sent;
        unlock_sock_fast(sk, slow);
 }
 EXPORT_SYMBOL_GPL(tcp_get_info);
@@ -3225,6 +3227,7 @@ static size_t tcp_opt_stats_get_size(void)
                nla_total_size(sizeof(u32)) + /* TCP_NLA_SND_SSTHRESH */
                nla_total_size(sizeof(u32)) + /* TCP_NLA_DELIVERED */
                nla_total_size(sizeof(u32)) + /* TCP_NLA_DELIVERED_CE */
+               nla_total_size_64bit(sizeof(u64)) + /* TCP_NLA_BYTES_SENT */
                0;
 }
 
@@ -3272,6 +3275,9 @@ struct sk_buff *tcp_get_timestamping_opt_stats(const struct sock *sk)
        nla_put_u32(stats, TCP_NLA_SNDQ_SIZE, tp->write_seq - tp->snd_una);
        nla_put_u8(stats, TCP_NLA_CA_STATE, inet_csk(sk)->icsk_ca_state);
 
+       nla_put_u64_64bit(stats, TCP_NLA_BYTES_SENT, tp->bytes_sent,
+                         TCP_NLA_PAD);
+
        return stats;
 }
 
index 490df62f26d47e918818859aa2cd9718dd3ec658..861531fe0e97b3da5b1ed79df93cebdc5e23ce91 100644 (file)
@@ -1136,6 +1136,7 @@ static int __tcp_transmit_skb(struct sock *sk, struct sk_buff *skb,
        if (skb->len != tcp_header_size) {
                tcp_event_data_sent(tp, sk);
                tp->data_segs_out += tcp_skb_pcount(skb);
+               tp->bytes_sent += skb->len - tcp_header_size;
                tcp_internal_pacing(sk, skb);
        }