]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
veth: use skb_orphan_partial instead of skb_orphan
authorPaolo Abeni <pabeni@redhat.com>
Fri, 9 Apr 2021 11:04:37 +0000 (13:04 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 11 Apr 2021 23:39:28 +0000 (16:39 -0700)
As described by commit 9c4c325252c5 ("skbuff: preserve sock
reference when scrubbing the skb."), orphaning a skb
in the TX path will cause OoO.

Let's use skb_orphan_partial() instead of skb_orphan(), so
that we keep the sk around for queue's selection sake and we
still avoid the problem fixed with commit 4bf9ffa0fb57 ("veth:
Orphan skb before GRO")

Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/veth.c

index 9e525646df1d9fa061df7048596d8399b77eff8e..ce085659d55fdd8211506d22f1385d3f6727b7dd 100644 (file)
@@ -686,7 +686,7 @@ static struct sk_buff *veth_xdp_rcv_skb(struct veth_rq *rq,
        int mac_len, delta, off;
        struct xdp_buff xdp;
 
-       skb_orphan(skb);
+       skb_orphan_partial(skb);
 
        rcu_read_lock();
        xdp_prog = rcu_dereference(rq->xdp_prog);