lkml.org 
[lkml]   [2016]   [Nov]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[PATCH 3.16 215/346] tun: fix transmit timestamp support
    3.16.39-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Soheil Hassas Yeganeh <soheil@google.com>

    commit 7b996243fab46092fb3a29c773c54be8152366e4 upstream.

    Instead of using sock_tx_timestamp, use skb_tx_timestamp to record
    software transmit timestamp of a packet.

    sock_tx_timestamp resets and overrides the tx_flags of the skb.
    The function is intended to be called from within the protocol
    layer when creating the skb, not from a device driver. This is
    inconsistent with other drivers and will cause issues for TCP.

    In TCP, we intend to sample the timestamps for the last byte
    for each sendmsg/sendpage. For that reason, tcp_sendmsg calls
    tcp_tx_timestamp only with the last skb that it generates.
    For example, if a 128KB message is split into two 64KB packets
    we want to sample the SND timestamp of the last packet. The current
    code in the tun driver, however, will result in sampling the SND
    timestamp for both packets.

    Also, when the last packet is split into smaller packets for
    retranmission (see tcp_fragment), the tun driver will record
    timestamps for all of the retransmitted packets and not only the
    last packet.

    Fixes: eda297729171 (tun: Support software transmit time stamping.)
    Signed-off-by: Soheil Hassas Yeganeh <soheil@google.com>
    Signed-off-by: Francis Yan <francisyyan@google.com>
    Acked-by: Eric Dumazet <edumazet@google.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    [bwh: Backported to 3.16: call to sock_tx_timestamp() was different]
    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    ---
    drivers/net/tun.c | 6 +-----
    1 file changed, 1 insertion(+), 5 deletions(-)

    --- a/drivers/net/tun.c
    +++ b/drivers/net/tun.c
    @@ -794,10 +794,7 @@ static netdev_tx_t tun_net_xmit(struct s
    if (unlikely(skb_orphan_frags(skb, GFP_ATOMIC)))
    goto drop;

    - if (skb->sk) {
    - sock_tx_timestamp(skb->sk, &skb_shinfo(skb)->tx_flags);
    - sw_tx_timestamp(skb);
    - }
    + skb_tx_timestamp(skb);

    /* Orphan the skb - required as we might hang on to it
    * for indefinite time.
    \
     
     \ /
      Last update: 2016-11-14 03:52    [W:3.094 / U:0.272 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site