Message ID | e2fbed200fa09a4850f3ac6ba0dfef9f302d1184.1546969349.git.pabeni@redhat.com |
---|---|
State | Accepted |
Delegated to: | David Miller |
Headers | show |
Series | [net] net: clear skb->tstamp in bridge forwarding path | expand |
On 08/01/2019 19:45, Paolo Abeni wrote: > Matteo reported forwarding issues inside the linux bridge, > if the enslaved interfaces use the fq qdisc. > > Similar to commit 8203e2d844d3 ("net: clear skb->tstamp in > forwarding paths"), we need to clear the tstamp field in > the bridge forwarding path. > > Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time.") > Fixes: fb420d5d91c1 ("tcp/fq: move back to CLOCK_MONOTONIC") > Reported-and-tested-by: Matteo Croce <mcroce@redhat.com> > Signed-off-by: Paolo Abeni <pabeni@redhat.com> > --- > net/bridge/br_forward.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c > index 5372e2042adf..2cb8da465b98 100644 > --- a/net/bridge/br_forward.c > +++ b/net/bridge/br_forward.c > @@ -65,6 +65,7 @@ EXPORT_SYMBOL_GPL(br_dev_queue_push_xmit); > > int br_forward_finish(struct net *net, struct sock *sk, struct sk_buff *skb) > { > + skb->tstamp = 0; > return NF_HOOK(NFPROTO_BRIDGE, NF_BR_POST_ROUTING, > net, sk, skb, NULL, skb->dev, > br_dev_queue_push_xmit); > Acked-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
On Tue, Jan 8, 2019 at 9:45 AM Paolo Abeni <pabeni@redhat.com> wrote: > > Matteo reported forwarding issues inside the linux bridge, > if the enslaved interfaces use the fq qdisc. > > Similar to commit 8203e2d844d3 ("net: clear skb->tstamp in > forwarding paths"), we need to clear the tstamp field in > the bridge forwarding path. > > Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time.") > Fixes: fb420d5d91c1 ("tcp/fq: move back to CLOCK_MONOTONIC") > Reported-and-tested-by: Matteo Croce <mcroce@redhat.com> > Signed-off-by: Paolo Abeni <pabeni@redhat.com> Acked-by: Roopa Prabhu <roopa@cumulusnetworks.com>
On 01/08/2019 09:45 AM, Paolo Abeni wrote: > Matteo reported forwarding issues inside the linux bridge, > if the enslaved interfaces use the fq qdisc. > > Similar to commit 8203e2d844d3 ("net: clear skb->tstamp in > forwarding paths"), we need to clear the tstamp field in > the bridge forwarding path. > > Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time.") > Fixes: fb420d5d91c1 ("tcp/fq: move back to CLOCK_MONOTONIC") > Reported-and-tested-by: Matteo Croce <mcroce@redhat.com> > Signed-off-by: Paolo Abeni <pabeni@redhat.com> > --- Reviewed-by: Eric Dumazet <edumazet@google.com> Sorry for the delay, I have been traveling last days.
From: Paolo Abeni <pabeni@redhat.com> Date: Tue, 8 Jan 2019 18:45:05 +0100 > Matteo reported forwarding issues inside the linux bridge, > if the enslaved interfaces use the fq qdisc. > > Similar to commit 8203e2d844d3 ("net: clear skb->tstamp in > forwarding paths"), we need to clear the tstamp field in > the bridge forwarding path. > > Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time.") > Fixes: fb420d5d91c1 ("tcp/fq: move back to CLOCK_MONOTONIC") > Reported-and-tested-by: Matteo Croce <mcroce@redhat.com> > Signed-off-by: Paolo Abeni <pabeni@redhat.com> Applied and queued up for -stable.
diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c index 5372e2042adf..2cb8da465b98 100644 --- a/net/bridge/br_forward.c +++ b/net/bridge/br_forward.c @@ -65,6 +65,7 @@ EXPORT_SYMBOL_GPL(br_dev_queue_push_xmit); int br_forward_finish(struct net *net, struct sock *sk, struct sk_buff *skb) { + skb->tstamp = 0; return NF_HOOK(NFPROTO_BRIDGE, NF_BR_POST_ROUTING, net, sk, skb, NULL, skb->dev, br_dev_queue_push_xmit);
Matteo reported forwarding issues inside the linux bridge, if the enslaved interfaces use the fq qdisc. Similar to commit 8203e2d844d3 ("net: clear skb->tstamp in forwarding paths"), we need to clear the tstamp field in the bridge forwarding path. Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time.") Fixes: fb420d5d91c1 ("tcp/fq: move back to CLOCK_MONOTONIC") Reported-and-tested-by: Matteo Croce <mcroce@redhat.com> Signed-off-by: Paolo Abeni <pabeni@redhat.com> --- net/bridge/br_forward.c | 1 + 1 file changed, 1 insertion(+)