tun: rename generic_xdp to skb_xdp
authorJason Wang <jasowang@redhat.com>
Mon, 4 Sep 2017 03:36:09 +0000 (11:36 +0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 5 Sep 2017 21:26:41 +0000 (14:26 -0700)
Rename "generic_xdp" to "skb_xdp" to avoid confusing it with the
generic XDP which will be done at netif_receive_skb().

Cc: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/tun.c

index 80ac18f8b55f20ee0a8601dec03b4f0a2d07576d..3c9985f299503ea65dad7eb3b47e2ab3bef87800 100644 (file)
@@ -1267,7 +1267,7 @@ static struct sk_buff *tun_build_skb(struct tun_struct *tun,
                                     struct tun_file *tfile,
                                     struct iov_iter *from,
                                     struct virtio_net_hdr *hdr,
-                                    int len, int *generic_xdp)
+                                    int len, int *skb_xdp)
 {
        struct page_frag *alloc_frag = &current->task_frag;
        struct sk_buff *skb;
@@ -1301,13 +1301,13 @@ static struct sk_buff *tun_build_skb(struct tun_struct *tun,
         * we do XDP on skb in case the headroom is not enough.
         */
        if (hdr->gso_type || !xdp_prog)
-               *generic_xdp = 1;
+               *skb_xdp = 1;
        else
-               *generic_xdp = 0;
+               *skb_xdp = 0;
 
        rcu_read_lock();
        xdp_prog = rcu_dereference(tun->xdp_prog);
-       if (xdp_prog && !*generic_xdp) {
+       if (xdp_prog && !*skb_xdp) {
                struct xdp_buff xdp;
                void *orig_data;
                u32 act;
@@ -1389,7 +1389,7 @@ static ssize_t tun_get_user(struct tun_struct *tun, struct tun_file *tfile,
        bool zerocopy = false;
        int err;
        u32 rxhash;
-       int generic_xdp = 1;
+       int skb_xdp = 1;
 
        if (!(tun->dev->flags & IFF_UP))
                return -EIO;
@@ -1448,7 +1448,11 @@ static ssize_t tun_get_user(struct tun_struct *tun, struct tun_file *tfile,
        }
 
        if (tun_can_build_skb(tun, tfile, len, noblock, zerocopy)) {
-               skb = tun_build_skb(tun, tfile, from, &gso, len, &generic_xdp);
+               /* For the packet that is not easy to be processed
+                * (e.g gso or jumbo packet), we will do it at after
+                * skb was created with generic XDP routine.
+                */
+               skb = tun_build_skb(tun, tfile, from, &gso, len, &skb_xdp);
                if (IS_ERR(skb)) {
                        this_cpu_inc(tun->pcpu_stats->rx_dropped);
                        return PTR_ERR(skb);
@@ -1528,7 +1532,7 @@ static ssize_t tun_get_user(struct tun_struct *tun, struct tun_file *tfile,
        skb_reset_network_header(skb);
        skb_probe_transport_header(skb, 0);
 
-       if (generic_xdp) {
+       if (skb_xdp) {
                struct bpf_prog *xdp_prog;
                int ret;