From 1cfe6e9351c256ec6aebcf64223aa21dc7654257 Mon Sep 17 00:00:00 2001 From: Jason Wang Date: Mon, 4 Sep 2017 11:36:09 +0800 Subject: [PATCH] tun: rename generic_xdp to skb_xdp 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 Signed-off-by: Jason Wang Signed-off-by: David S. Miller --- drivers/net/tun.c | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/drivers/net/tun.c b/drivers/net/tun.c index 80ac18f8b55f..3c9985f29950 100644 --- a/drivers/net/tun.c +++ b/drivers/net/tun.c @@ -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 = ¤t->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; -- 2.39.2