virtio-net: correctly handle XDP_PASS for linearized packets
authorJason Wang <jasowang@redhat.com>
Fri, 23 Dec 2016 14:37:27 +0000 (22:37 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 23 Dec 2016 18:48:54 +0000 (13:48 -0500)
When XDP_PASS were determined for linearized packets, we try to get
new buffers in the virtqueue and build skbs from them. This is wrong,
we should create skbs based on existed buffers instead. Fixing them by
creating skb based on xdp_page.

With this patch "ping 192.168.100.4 -s 3900 -M do" works for XDP_PASS.

Cc: John Fastabend <john.r.fastabend@intel.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Acked-by: John Fastabend <john.r.fastabend@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/virtio_net.c

index 58ad40e17a74fb455dd956ab06e83601c8c2310e..470293e2b84d4cf6bf168943ed1f0f0baabae363 100644 (file)
@@ -578,8 +578,14 @@ static struct sk_buff *receive_mergeable(struct net_device *dev,
                act = do_xdp_prog(vi, rq, xdp_prog, xdp_page, offset, len);
                switch (act) {
                case XDP_PASS:
-                       if (unlikely(xdp_page != page))
-                               __free_pages(xdp_page, 0);
+                       /* We can only create skb based on xdp_page. */
+                       if (unlikely(xdp_page != page)) {
+                               rcu_read_unlock();
+                               put_page(page);
+                               head_skb = page_to_skb(vi, rq, xdp_page,
+                                                      0, len, PAGE_SIZE);
+                               return head_skb;
+                       }
                        break;
                case XDP_TX:
                        if (unlikely(xdp_page != page))