lkml.org 
[lkml]   [2018]   [Dec]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.19 53/97] net: clear skb->tstamp in forwarding paths
    Date
    From: Eric Dumazet <edumazet@google.com>

    [ Upstream commit 8203e2d844d34af247a151d8ebd68553a6e91785 ]

    Sergey reported that forwarding was no longer working
    if fq packet scheduler was used.

    This is caused by the recent switch to EDT model, since incoming
    packets might have been timestamped by __net_timestamp()

    __net_timestamp() uses ktime_get_real(), while fq expects packets
    using CLOCK_MONOTONIC base.

    The fix is to clear skb->tstamp in forwarding paths.

    Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time.")
    Fixes: fb420d5d91c1 ("tcp/fq: move back to CLOCK_MONOTONIC")
    Signed-off-by: Eric Dumazet <edumazet@google.com>
    Reported-by: Sergey Matyukevich <geomatsi@gmail.com>
    Tested-by: Sergey Matyukevich <geomatsi@gmail.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    net/ipv4/ip_forward.c | 1 +
    net/ipv6/ip6_output.c | 1 +
    2 files changed, 2 insertions(+)

    diff --git a/net/ipv4/ip_forward.c b/net/ipv4/ip_forward.c
    index 32662e9e5d21..d5984d31ab93 100644
    --- a/net/ipv4/ip_forward.c
    +++ b/net/ipv4/ip_forward.c
    @@ -72,6 +72,7 @@ static int ip_forward_finish(struct net *net, struct sock *sk, struct sk_buff *s
    if (unlikely(opt->optlen))
    ip_forward_options(skb);

    + skb->tstamp = 0;
    return dst_output(net, sk, skb);
    }

    diff --git a/net/ipv6/ip6_output.c b/net/ipv6/ip6_output.c
    index 2694def1e72c..0bb87f3a10c7 100644
    --- a/net/ipv6/ip6_output.c
    +++ b/net/ipv6/ip6_output.c
    @@ -378,6 +378,7 @@ static inline int ip6_forward_finish(struct net *net, struct sock *sk,
    __IP6_INC_STATS(net, ip6_dst_idev(dst), IPSTATS_MIB_OUTFORWDATAGRAMS);
    __IP6_ADD_STATS(net, ip6_dst_idev(dst), IPSTATS_MIB_OUTOCTETS, skb->len);

    + skb->tstamp = 0;
    return dst_output(net, sk, skb);
    }

    --
    2.19.1
    \
     
     \ /
      Last update: 2018-12-26 23:50    [W:4.085 / U:0.060 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site