netdev - Re: [PATCH net 1/3] udp_tunnel: Remove redundant udp_tunnel_gro

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <175616E2-FC50-4961-AB5F-43DF5820A833@ovn.org>
Date:	Fri, 29 Apr 2016 15:33:48 -0700
From:	Jarno Rajahalme <jarno@....org>
To:	netdev <netdev@...r.kernel.org>
Cc:	Jesse Gross <jesse@...nel.org>
Subject: Re: [PATCH net 1/3] udp_tunnel: Remove redundant udp_tunnel_gro_complete().

This is a two patch series.. I had a third patch which fixed a bug that was already fixed, so I dropped it but failed to generate messages again, hence the “[12]/3”. Sorry for the noise,

  Jarno

> On Apr 29, 2016, at 3:28 PM, Jarno Rajahalme <jarno@....org> wrote:
> 
> The setting of the UDP tunnel GSO type is already performed by
> udp[46]_gro_complete().
> 
> Signed-off-by: Jarno Rajahalme <jarno@....org>
> ---
> drivers/net/geneve.c     | 2 --
> drivers/net/vxlan.c      | 2 --
> include/net/udp_tunnel.h | 9 ---------
> net/ipv4/fou.c           | 2 --
> 4 files changed, 15 deletions(-)
> 
> diff --git a/drivers/net/geneve.c b/drivers/net/geneve.c
> index bc16889..98f1224 100644
> --- a/drivers/net/geneve.c
> +++ b/drivers/net/geneve.c
> @@ -504,8 +504,6 @@ static int geneve_gro_complete(struct sk_buff *skb, int nhoff,
> 	int gh_len;
> 	int err = -ENOSYS;
> 
> -	udp_tunnel_gro_complete(skb, nhoff);
> -
> 	gh = (struct genevehdr *)(skb->data + nhoff);
> 	gh_len = geneve_hlen(gh);
> 	type = gh->proto_type;
> diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c
> index 1c0fa36..dd2d032 100644
> --- a/drivers/net/vxlan.c
> +++ b/drivers/net/vxlan.c
> @@ -616,8 +616,6 @@ out:
> static int vxlan_gro_complete(struct sk_buff *skb, int nhoff,
> 			      struct udp_offload *uoff)
> {
> -	udp_tunnel_gro_complete(skb, nhoff);
> -
> 	return eth_gro_complete(skb, nhoff + sizeof(struct vxlanhdr));
> }
> 
> diff --git a/include/net/udp_tunnel.h b/include/net/udp_tunnel.h
> index b831140..a114024 100644
> --- a/include/net/udp_tunnel.h
> +++ b/include/net/udp_tunnel.h
> @@ -106,15 +106,6 @@ static inline struct sk_buff *udp_tunnel_handle_offloads(struct sk_buff *skb,
> 	return iptunnel_handle_offloads(skb, type);
> }
> 
> -static inline void udp_tunnel_gro_complete(struct sk_buff *skb, int nhoff)
> -{
> -	struct udphdr *uh;
> -
> -	uh = (struct udphdr *)(skb->data + nhoff - sizeof(struct udphdr));
> -	skb_shinfo(skb)->gso_type |= uh->check ?
> -				SKB_GSO_UDP_TUNNEL_CSUM : SKB_GSO_UDP_TUNNEL;
> -}
> -
> static inline void udp_tunnel_encap_enable(struct socket *sock)
> {
> #if IS_ENABLED(CONFIG_IPV6)
> diff --git a/net/ipv4/fou.c b/net/ipv4/fou.c
> index a39068b..305d9ac 100644
> --- a/net/ipv4/fou.c
> +++ b/net/ipv4/fou.c
> @@ -228,8 +228,6 @@ static int fou_gro_complete(struct sk_buff *skb, int nhoff,
> 	int err = -ENOSYS;
> 	const struct net_offload **offloads;
> 
> -	udp_tunnel_gro_complete(skb, nhoff);
> -
> 	rcu_read_lock();
> 	offloads = NAPI_GRO_CB(skb)->is_ipv6 ? inet6_offloads : inet_offloads;
> 	ops = rcu_dereference(offloads[proto]);
> -- 
> 2.7.4
>