* [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
@ 2015-04-06 2:19 David Miller
2015-04-07 6:02 ` Tom Herbert
0 siblings, 1 reply; 9+ messages in thread
From: David Miller @ 2015-04-06 2:19 UTC (permalink / raw)
To: netdev; +Cc: netfilter-devel, pablo, hannes, jiri
That was we can make sure the output path of ipv4/ipv6 operate on
the UDP socket rather than whatever random thing happens to be in
skb->sk.
Based upon a patch by Jiri Pirko.
Signed-off-by: David S. Miller <davem@davemloft.net>
---
drivers/net/vxlan.c | 14 ++++++++------
include/net/ip6_tunnel.h | 5 +++--
include/net/ipv6.h | 1 +
include/net/udp_tunnel.h | 5 +++--
include/net/vxlan.h | 2 +-
net/ipv4/geneve.c | 2 +-
net/ipv4/udp_tunnel.c | 4 ++--
net/ipv6/ip6_gre.c | 2 +-
net/ipv6/ip6_tunnel.c | 2 +-
net/ipv6/ip6_udp_tunnel.c | 5 +++--
net/ipv6/output_core.c | 21 ++++++++++++++++-----
net/openvswitch/vport-vxlan.c | 5 +++--
net/tipc/udp_media.c | 6 ++++--
13 files changed, 47 insertions(+), 27 deletions(-)
diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c
index b5fecb4..51baac7 100644
--- a/drivers/net/vxlan.c
+++ b/drivers/net/vxlan.c
@@ -1672,7 +1672,8 @@ static void vxlan_build_gbp_hdr(struct vxlanhdr *vxh, u32 vxflags,
}
#if IS_ENABLED(CONFIG_IPV6)
-static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
+static int vxlan6_xmit_skb(struct dst_entry *dst, struct sock *sk,
+ struct sk_buff *skb,
struct net_device *dev, struct in6_addr *saddr,
struct in6_addr *daddr, __u8 prio, __u8 ttl,
__be16 src_port, __be16 dst_port,
@@ -1748,7 +1749,7 @@ static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
skb_set_inner_protocol(skb, htons(ETH_P_TEB));
- udp_tunnel6_xmit_skb(dst, skb, dev, saddr, daddr, prio,
+ udp_tunnel6_xmit_skb(dst, sk, skb, dev, saddr, daddr, prio,
ttl, src_port, dst_port,
!!(vxflags & VXLAN_F_UDP_ZERO_CSUM6_TX));
return 0;
@@ -1758,7 +1759,7 @@ err:
}
#endif
-int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
+int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
__be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
__be16 src_port, __be16 dst_port,
struct vxlan_metadata *md, bool xnet, u32 vxflags)
@@ -1827,7 +1828,7 @@ int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
skb_set_inner_protocol(skb, htons(ETH_P_TEB));
- return udp_tunnel_xmit_skb(rt, skb, src, dst, tos,
+ return udp_tunnel_xmit_skb(rt, sk, skb, src, dst, tos,
ttl, df, src_port, dst_port, xnet,
!(vxflags & VXLAN_F_UDP_CSUM));
}
@@ -1882,6 +1883,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
struct vxlan_rdst *rdst, bool did_rsc)
{
struct vxlan_dev *vxlan = netdev_priv(dev);
+ struct sock *sk = vxlan->vn_sock->sock->sk;
struct rtable *rt = NULL;
const struct iphdr *old_iph;
struct flowi4 fl4;
@@ -1961,7 +1963,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
md.vni = htonl(vni << 8);
md.gbp = skb->mark;
- err = vxlan_xmit_skb(rt, skb, fl4.saddr,
+ err = vxlan_xmit_skb(rt, sk, skb, fl4.saddr,
dst->sin.sin_addr.s_addr, tos, ttl, df,
src_port, dst_port, &md,
!net_eq(vxlan->net, dev_net(vxlan->dev)),
@@ -2021,7 +2023,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
md.vni = htonl(vni << 8);
md.gbp = skb->mark;
- err = vxlan6_xmit_skb(ndst, skb, dev, &fl6.saddr, &fl6.daddr,
+ err = vxlan6_xmit_skb(ndst, sk, skb, dev, &fl6.saddr, &fl6.daddr,
0, ttl, src_port, dst_port, &md,
!net_eq(vxlan->net, dev_net(vxlan->dev)),
vxlan->flags);
diff --git a/include/net/ip6_tunnel.h b/include/net/ip6_tunnel.h
index 1668be5..b8529aa 100644
--- a/include/net/ip6_tunnel.h
+++ b/include/net/ip6_tunnel.h
@@ -73,13 +73,14 @@ __u32 ip6_tnl_get_cap(struct ip6_tnl *t, const struct in6_addr *laddr,
struct net *ip6_tnl_get_link_net(const struct net_device *dev);
int ip6_tnl_get_iflink(const struct net_device *dev);
-static inline void ip6tunnel_xmit(struct sk_buff *skb, struct net_device *dev)
+static inline void ip6tunnel_xmit(struct sock *sk, struct sk_buff *skb,
+ struct net_device *dev)
{
struct net_device_stats *stats = &dev->stats;
int pkt_len, err;
pkt_len = skb->len;
- err = ip6_local_out(skb);
+ err = ip6_local_out_sk(sk, skb);
if (net_xmit_eval(err) == 0) {
struct pcpu_sw_netstats *tstats = this_cpu_ptr(dev->tstats);
diff --git a/include/net/ipv6.h b/include/net/ipv6.h
index b6ae959..27470cd 100644
--- a/include/net/ipv6.h
+++ b/include/net/ipv6.h
@@ -827,6 +827,7 @@ int ip6_input(struct sk_buff *skb);
int ip6_mc_input(struct sk_buff *skb);
int __ip6_local_out(struct sk_buff *skb);
+int ip6_local_out_sk(struct sock *sk, struct sk_buff *skb);
int ip6_local_out(struct sk_buff *skb);
/*
diff --git a/include/net/udp_tunnel.h b/include/net/udp_tunnel.h
index 1a20d33..c491c12 100644
--- a/include/net/udp_tunnel.h
+++ b/include/net/udp_tunnel.h
@@ -77,13 +77,14 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
struct udp_tunnel_sock_cfg *sock_cfg);
/* Transmit the skb using UDP encapsulation. */
-int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
+int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
__be32 src, __be32 dst, __u8 tos, __u8 ttl,
__be16 df, __be16 src_port, __be16 dst_port,
bool xnet, bool nocheck);
#if IS_ENABLED(CONFIG_IPV6)
-int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
+int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sock *sk,
+ struct sk_buff *skb,
struct net_device *dev, struct in6_addr *saddr,
struct in6_addr *daddr,
__u8 prio, __u8 ttl, __be16 src_port,
diff --git a/include/net/vxlan.h b/include/net/vxlan.h
index 756e463..0082b5d 100644
--- a/include/net/vxlan.h
+++ b/include/net/vxlan.h
@@ -145,7 +145,7 @@ struct vxlan_sock *vxlan_sock_add(struct net *net, __be16 port,
void vxlan_sock_release(struct vxlan_sock *vs);
-int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
+int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
__be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
__be16 src_port, __be16 dst_port, struct vxlan_metadata *md,
bool xnet, u32 vxflags);
diff --git a/net/ipv4/geneve.c b/net/ipv4/geneve.c
index e64f8e9..b77f5e8 100644
--- a/net/ipv4/geneve.c
+++ b/net/ipv4/geneve.c
@@ -136,7 +136,7 @@ int geneve_xmit_skb(struct geneve_sock *gs, struct rtable *rt,
skb_set_inner_protocol(skb, htons(ETH_P_TEB));
- return udp_tunnel_xmit_skb(rt, skb, src, dst,
+ return udp_tunnel_xmit_skb(rt, gs->sock->sk, skb, src, dst,
tos, ttl, df, src_port, dst_port, xnet,
!csum);
}
diff --git a/net/ipv4/udp_tunnel.c b/net/ipv4/udp_tunnel.c
index c83b354..6bb98cc 100644
--- a/net/ipv4/udp_tunnel.c
+++ b/net/ipv4/udp_tunnel.c
@@ -75,7 +75,7 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
}
EXPORT_SYMBOL_GPL(setup_udp_tunnel_sock);
-int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
+int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
__be32 src, __be32 dst, __u8 tos, __u8 ttl,
__be16 df, __be16 src_port, __be16 dst_port,
bool xnet, bool nocheck)
@@ -92,7 +92,7 @@ int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
udp_set_csum(nocheck, skb, src, dst, skb->len);
- return iptunnel_xmit(skb->sk, rt, skb, src, dst, IPPROTO_UDP,
+ return iptunnel_xmit(sk, rt, skb, src, dst, IPPROTO_UDP,
tos, ttl, df, xnet);
}
EXPORT_SYMBOL_GPL(udp_tunnel_xmit_skb);
diff --git a/net/ipv6/ip6_gre.c b/net/ipv6/ip6_gre.c
index f724329..b5e6cc1 100644
--- a/net/ipv6/ip6_gre.c
+++ b/net/ipv6/ip6_gre.c
@@ -760,7 +760,7 @@ static netdev_tx_t ip6gre_xmit2(struct sk_buff *skb,
skb_set_inner_protocol(skb, protocol);
- ip6tunnel_xmit(skb, dev);
+ ip6tunnel_xmit(NULL, skb, dev);
if (ndst)
ip6_tnl_dst_store(tunnel, ndst);
return 0;
diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
index b6a211a..5cafd92 100644
--- a/net/ipv6/ip6_tunnel.c
+++ b/net/ipv6/ip6_tunnel.c
@@ -1100,7 +1100,7 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
ipv6h->nexthdr = proto;
ipv6h->saddr = fl6->saddr;
ipv6h->daddr = fl6->daddr;
- ip6tunnel_xmit(skb, dev);
+ ip6tunnel_xmit(NULL, skb, dev);
if (ndst)
ip6_tnl_dst_store(t, ndst);
return 0;
diff --git a/net/ipv6/ip6_udp_tunnel.c b/net/ipv6/ip6_udp_tunnel.c
index 32d9b26..bba8903 100644
--- a/net/ipv6/ip6_udp_tunnel.c
+++ b/net/ipv6/ip6_udp_tunnel.c
@@ -62,7 +62,8 @@ error:
}
EXPORT_SYMBOL_GPL(udp_sock_create6);
-int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
+int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sock *sk,
+ struct sk_buff *skb,
struct net_device *dev, struct in6_addr *saddr,
struct in6_addr *daddr,
__u8 prio, __u8 ttl, __be16 src_port,
@@ -97,7 +98,7 @@ int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
ip6h->daddr = *daddr;
ip6h->saddr = *saddr;
- ip6tunnel_xmit(skb, dev);
+ ip6tunnel_xmit(sk, skb, dev);
return 0;
}
EXPORT_SYMBOL_GPL(udp_tunnel6_xmit_skb);
diff --git a/net/ipv6/output_core.c b/net/ipv6/output_core.c
index 7d1131d..85892af 100644
--- a/net/ipv6/output_core.c
+++ b/net/ipv6/output_core.c
@@ -136,7 +136,7 @@ int ip6_dst_hoplimit(struct dst_entry *dst)
EXPORT_SYMBOL(ip6_dst_hoplimit);
#endif
-int __ip6_local_out(struct sk_buff *skb)
+static int __ip6_local_out_sk(struct sock *sk, struct sk_buff *skb)
{
int len;
@@ -146,19 +146,30 @@ int __ip6_local_out(struct sk_buff *skb)
ipv6_hdr(skb)->payload_len = htons(len);
IP6CB(skb)->nhoff = offsetof(struct ipv6hdr, nexthdr);
- return nf_hook(NFPROTO_IPV6, NF_INET_LOCAL_OUT, skb->sk, skb,
+ return nf_hook(NFPROTO_IPV6, NF_INET_LOCAL_OUT, sk, skb,
NULL, skb_dst(skb)->dev, dst_output_sk);
}
+
+int __ip6_local_out(struct sk_buff *skb)
+{
+ return __ip6_local_out_sk(skb->sk, skb);
+}
EXPORT_SYMBOL_GPL(__ip6_local_out);
-int ip6_local_out(struct sk_buff *skb)
+int ip6_local_out_sk(struct sock *sk, struct sk_buff *skb)
{
int err;
- err = __ip6_local_out(skb);
+ err = __ip6_local_out_sk(sk, skb);
if (likely(err == 1))
- err = dst_output(skb);
+ err = dst_output_sk(sk, skb);
return err;
}
+EXPORT_SYMBOL_GPL(ip6_local_out_sk);
+
+int ip6_local_out(struct sk_buff *skb)
+{
+ return ip6_local_out_sk(skb->sk, skb);
+}
EXPORT_SYMBOL_GPL(ip6_local_out);
diff --git a/net/openvswitch/vport-vxlan.c b/net/openvswitch/vport-vxlan.c
index 3277a75..6d39766 100644
--- a/net/openvswitch/vport-vxlan.c
+++ b/net/openvswitch/vport-vxlan.c
@@ -222,7 +222,8 @@ static int vxlan_tnl_send(struct vport *vport, struct sk_buff *skb)
{
struct net *net = ovs_dp_get_net(vport->dp);
struct vxlan_port *vxlan_port = vxlan_vport(vport);
- __be16 dst_port = inet_sk(vxlan_port->vs->sock->sk)->inet_sport;
+ struct sock *sk = vxlan_port->vs->sock->sk;
+ __be16 dst_port = inet_sk(sk)->inet_sport;
const struct ovs_key_ipv4_tunnel *tun_key;
struct vxlan_metadata md = {0};
struct rtable *rt;
@@ -255,7 +256,7 @@ static int vxlan_tnl_send(struct vport *vport, struct sk_buff *skb)
vxflags = vxlan_port->exts |
(tun_key->tun_flags & TUNNEL_CSUM ? VXLAN_F_UDP_CSUM : 0);
- err = vxlan_xmit_skb(rt, skb, fl.saddr, tun_key->ipv4_dst,
+ err = vxlan_xmit_skb(rt, sk, skb, fl.saddr, tun_key->ipv4_dst,
tun_key->ipv4_tos, tun_key->ipv4_ttl, df,
src_port, dst_port,
&md, false, vxflags);
diff --git a/net/tipc/udp_media.c b/net/tipc/udp_media.c
index ef3d7aa..66deebc 100644
--- a/net/tipc/udp_media.c
+++ b/net/tipc/udp_media.c
@@ -176,7 +176,8 @@ static int tipc_udp_send_msg(struct net *net, struct sk_buff *skb,
goto tx_error;
}
ttl = ip4_dst_hoplimit(&rt->dst);
- err = udp_tunnel_xmit_skb(rt, clone, src->ipv4.s_addr,
+ err = udp_tunnel_xmit_skb(rt, ub->ubsock->sk, clone,
+ src->ipv4.s_addr,
dst->ipv4.s_addr, 0, ttl, 0,
src->udp_port, dst->udp_port,
false, true);
@@ -197,7 +198,8 @@ static int tipc_udp_send_msg(struct net *net, struct sk_buff *skb,
if (err)
goto tx_error;
ttl = ip6_dst_hoplimit(ndst);
- err = udp_tunnel6_xmit_skb(ndst, clone, ndst->dev, &src->ipv6,
+ err = udp_tunnel6_xmit_skb(ndst, ub->ubsock->sk, clone,
+ ndst->dev, &src->ipv6,
&dst->ipv6, 0, ttl, src->udp_port,
dst->udp_port, false);
#endif
--
2.1.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-06 2:19 [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb() David Miller
@ 2015-04-07 6:02 ` Tom Herbert
2015-04-07 15:45 ` Hannes Frederic Sowa
0 siblings, 1 reply; 9+ messages in thread
From: Tom Herbert @ 2015-04-07 6:02 UTC (permalink / raw)
To: David Miller
Cc: netdev, netfilter-devel, pablo, hannes, Jiří Pírko
On Sun, Apr 5, 2015 at 7:19 PM, David Miller <davem@davemloft.net> wrote:
>
> That was we can make sure the output path of ipv4/ipv6 operate on
> the UDP socket rather than whatever random thing happens to be in
> skb->sk.
>
> Based upon a patch by Jiri Pirko.
>
> Signed-off-by: David S. Miller <davem@davemloft.net>
> ---
> drivers/net/vxlan.c | 14 ++++++++------
> include/net/ip6_tunnel.h | 5 +++--
> include/net/ipv6.h | 1 +
> include/net/udp_tunnel.h | 5 +++--
> include/net/vxlan.h | 2 +-
> net/ipv4/geneve.c | 2 +-
> net/ipv4/udp_tunnel.c | 4 ++--
> net/ipv6/ip6_gre.c | 2 +-
> net/ipv6/ip6_tunnel.c | 2 +-
> net/ipv6/ip6_udp_tunnel.c | 5 +++--
> net/ipv6/output_core.c | 21 ++++++++++++++++-----
> net/openvswitch/vport-vxlan.c | 5 +++--
> net/tipc/udp_media.c | 6 ++++--
> 13 files changed, 47 insertions(+), 27 deletions(-)
>
> diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c
> index b5fecb4..51baac7 100644
> --- a/drivers/net/vxlan.c
> +++ b/drivers/net/vxlan.c
> @@ -1672,7 +1672,8 @@ static void vxlan_build_gbp_hdr(struct vxlanhdr *vxh, u32 vxflags,
> }
>
> #if IS_ENABLED(CONFIG_IPV6)
> -static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
> +static int vxlan6_xmit_skb(struct dst_entry *dst, struct sock *sk,
> + struct sk_buff *skb,
> struct net_device *dev, struct in6_addr *saddr,
> struct in6_addr *daddr, __u8 prio, __u8 ttl,
> __be16 src_port, __be16 dst_port,
> @@ -1748,7 +1749,7 @@ static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
>
> skb_set_inner_protocol(skb, htons(ETH_P_TEB));
>
> - udp_tunnel6_xmit_skb(dst, skb, dev, saddr, daddr, prio,
> + udp_tunnel6_xmit_skb(dst, sk, skb, dev, saddr, daddr, prio,
> ttl, src_port, dst_port,
> !!(vxflags & VXLAN_F_UDP_ZERO_CSUM6_TX));
> return 0;
> @@ -1758,7 +1759,7 @@ err:
> }
> #endif
>
> -int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> +int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> __be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
> __be16 src_port, __be16 dst_port,
> struct vxlan_metadata *md, bool xnet, u32 vxflags)
> @@ -1827,7 +1828,7 @@ int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>
> skb_set_inner_protocol(skb, htons(ETH_P_TEB));
>
> - return udp_tunnel_xmit_skb(rt, skb, src, dst, tos,
> + return udp_tunnel_xmit_skb(rt, sk, skb, src, dst, tos,
> ttl, df, src_port, dst_port, xnet,
> !(vxflags & VXLAN_F_UDP_CSUM));
> }
> @@ -1882,6 +1883,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
> struct vxlan_rdst *rdst, bool did_rsc)
> {
> struct vxlan_dev *vxlan = netdev_priv(dev);
> + struct sock *sk = vxlan->vn_sock->sock->sk;
> struct rtable *rt = NULL;
> const struct iphdr *old_iph;
> struct flowi4 fl4;
> @@ -1961,7 +1963,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
> md.vni = htonl(vni << 8);
> md.gbp = skb->mark;
>
> - err = vxlan_xmit_skb(rt, skb, fl4.saddr,
> + err = vxlan_xmit_skb(rt, sk, skb, fl4.saddr,
> dst->sin.sin_addr.s_addr, tos, ttl, df,
> src_port, dst_port, &md,
> !net_eq(vxlan->net, dev_net(vxlan->dev)),
> @@ -2021,7 +2023,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
> md.vni = htonl(vni << 8);
> md.gbp = skb->mark;
>
> - err = vxlan6_xmit_skb(ndst, skb, dev, &fl6.saddr, &fl6.daddr,
> + err = vxlan6_xmit_skb(ndst, sk, skb, dev, &fl6.saddr, &fl6.daddr,
> 0, ttl, src_port, dst_port, &md,
> !net_eq(vxlan->net, dev_net(vxlan->dev)),
> vxlan->flags);
> diff --git a/include/net/ip6_tunnel.h b/include/net/ip6_tunnel.h
> index 1668be5..b8529aa 100644
> --- a/include/net/ip6_tunnel.h
> +++ b/include/net/ip6_tunnel.h
> @@ -73,13 +73,14 @@ __u32 ip6_tnl_get_cap(struct ip6_tnl *t, const struct in6_addr *laddr,
> struct net *ip6_tnl_get_link_net(const struct net_device *dev);
> int ip6_tnl_get_iflink(const struct net_device *dev);
>
> -static inline void ip6tunnel_xmit(struct sk_buff *skb, struct net_device *dev)
> +static inline void ip6tunnel_xmit(struct sock *sk, struct sk_buff *skb,
> + struct net_device *dev)
> {
> struct net_device_stats *stats = &dev->stats;
> int pkt_len, err;
>
> pkt_len = skb->len;
> - err = ip6_local_out(skb);
> + err = ip6_local_out_sk(sk, skb);
>
> if (net_xmit_eval(err) == 0) {
> struct pcpu_sw_netstats *tstats = this_cpu_ptr(dev->tstats);
> diff --git a/include/net/ipv6.h b/include/net/ipv6.h
> index b6ae959..27470cd 100644
> --- a/include/net/ipv6.h
> +++ b/include/net/ipv6.h
> @@ -827,6 +827,7 @@ int ip6_input(struct sk_buff *skb);
> int ip6_mc_input(struct sk_buff *skb);
>
> int __ip6_local_out(struct sk_buff *skb);
> +int ip6_local_out_sk(struct sock *sk, struct sk_buff *skb);
> int ip6_local_out(struct sk_buff *skb);
>
> /*
> diff --git a/include/net/udp_tunnel.h b/include/net/udp_tunnel.h
> index 1a20d33..c491c12 100644
> --- a/include/net/udp_tunnel.h
> +++ b/include/net/udp_tunnel.h
> @@ -77,13 +77,14 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
> struct udp_tunnel_sock_cfg *sock_cfg);
>
> /* Transmit the skb using UDP encapsulation. */
> -int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> +int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> __be32 src, __be32 dst, __u8 tos, __u8 ttl,
> __be16 df, __be16 src_port, __be16 dst_port,
> bool xnet, bool nocheck);
>
> #if IS_ENABLED(CONFIG_IPV6)
> -int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
> +int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sock *sk,
> + struct sk_buff *skb,
> struct net_device *dev, struct in6_addr *saddr,
> struct in6_addr *daddr,
> __u8 prio, __u8 ttl, __be16 src_port,
> diff --git a/include/net/vxlan.h b/include/net/vxlan.h
> index 756e463..0082b5d 100644
> --- a/include/net/vxlan.h
> +++ b/include/net/vxlan.h
> @@ -145,7 +145,7 @@ struct vxlan_sock *vxlan_sock_add(struct net *net, __be16 port,
>
> void vxlan_sock_release(struct vxlan_sock *vs);
>
> -int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> +int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> __be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
> __be16 src_port, __be16 dst_port, struct vxlan_metadata *md,
> bool xnet, u32 vxflags);
> diff --git a/net/ipv4/geneve.c b/net/ipv4/geneve.c
> index e64f8e9..b77f5e8 100644
> --- a/net/ipv4/geneve.c
> +++ b/net/ipv4/geneve.c
> @@ -136,7 +136,7 @@ int geneve_xmit_skb(struct geneve_sock *gs, struct rtable *rt,
>
> skb_set_inner_protocol(skb, htons(ETH_P_TEB));
>
> - return udp_tunnel_xmit_skb(rt, skb, src, dst,
> + return udp_tunnel_xmit_skb(rt, gs->sock->sk, skb, src, dst,
> tos, ttl, df, src_port, dst_port, xnet,
> !csum);
> }
> diff --git a/net/ipv4/udp_tunnel.c b/net/ipv4/udp_tunnel.c
> index c83b354..6bb98cc 100644
> --- a/net/ipv4/udp_tunnel.c
> +++ b/net/ipv4/udp_tunnel.c
> @@ -75,7 +75,7 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
> }
> EXPORT_SYMBOL_GPL(setup_udp_tunnel_sock);
>
> -int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> +int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> __be32 src, __be32 dst, __u8 tos, __u8 ttl,
> __be16 df, __be16 src_port, __be16 dst_port,
> bool xnet, bool nocheck)
> @@ -92,7 +92,7 @@ int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>
> udp_set_csum(nocheck, skb, src, dst, skb->len);
>
> - return iptunnel_xmit(skb->sk, rt, skb, src, dst, IPPROTO_UDP,
> + return iptunnel_xmit(sk, rt, skb, src, dst, IPPROTO_UDP,
> tos, ttl, df, xnet);
> }
> EXPORT_SYMBOL_GPL(udp_tunnel_xmit_skb);
> diff --git a/net/ipv6/ip6_gre.c b/net/ipv6/ip6_gre.c
> index f724329..b5e6cc1 100644
> --- a/net/ipv6/ip6_gre.c
> +++ b/net/ipv6/ip6_gre.c
> @@ -760,7 +760,7 @@ static netdev_tx_t ip6gre_xmit2(struct sk_buff *skb,
>
> skb_set_inner_protocol(skb, protocol);
>
> - ip6tunnel_xmit(skb, dev);
> + ip6tunnel_xmit(NULL, skb, dev);
> if (ndst)
> ip6_tnl_dst_store(tunnel, ndst);
> return 0;
> diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
> index b6a211a..5cafd92 100644
> --- a/net/ipv6/ip6_tunnel.c
> +++ b/net/ipv6/ip6_tunnel.c
> @@ -1100,7 +1100,7 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
> ipv6h->nexthdr = proto;
> ipv6h->saddr = fl6->saddr;
> ipv6h->daddr = fl6->daddr;
> - ip6tunnel_xmit(skb, dev);
> + ip6tunnel_xmit(NULL, skb, dev);
By same should logic iptunnel_xmit call in ip_tunnel_xmit should take
NULL arg for socket?
> if (ndst)
> ip6_tnl_dst_store(t, ndst);
> return 0;
> diff --git a/net/ipv6/ip6_udp_tunnel.c b/net/ipv6/ip6_udp_tunnel.c
> index 32d9b26..bba8903 100644
> --- a/net/ipv6/ip6_udp_tunnel.c
> +++ b/net/ipv6/ip6_udp_tunnel.c
> @@ -62,7 +62,8 @@ error:
> }
> EXPORT_SYMBOL_GPL(udp_sock_create6);
>
> -int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
> +int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sock *sk,
> + struct sk_buff *skb,
> struct net_device *dev, struct in6_addr *saddr,
> struct in6_addr *daddr,
> __u8 prio, __u8 ttl, __be16 src_port,
> @@ -97,7 +98,7 @@ int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
> ip6h->daddr = *daddr;
> ip6h->saddr = *saddr;
>
> - ip6tunnel_xmit(skb, dev);
> + ip6tunnel_xmit(sk, skb, dev);
> return 0;
> }
> EXPORT_SYMBOL_GPL(udp_tunnel6_xmit_skb);
> diff --git a/net/ipv6/output_core.c b/net/ipv6/output_core.c
> index 7d1131d..85892af 100644
> --- a/net/ipv6/output_core.c
> +++ b/net/ipv6/output_core.c
> @@ -136,7 +136,7 @@ int ip6_dst_hoplimit(struct dst_entry *dst)
> EXPORT_SYMBOL(ip6_dst_hoplimit);
> #endif
>
> -int __ip6_local_out(struct sk_buff *skb)
> +static int __ip6_local_out_sk(struct sock *sk, struct sk_buff *skb)
> {
> int len;
>
> @@ -146,19 +146,30 @@ int __ip6_local_out(struct sk_buff *skb)
> ipv6_hdr(skb)->payload_len = htons(len);
> IP6CB(skb)->nhoff = offsetof(struct ipv6hdr, nexthdr);
>
> - return nf_hook(NFPROTO_IPV6, NF_INET_LOCAL_OUT, skb->sk, skb,
> + return nf_hook(NFPROTO_IPV6, NF_INET_LOCAL_OUT, sk, skb,
> NULL, skb_dst(skb)->dev, dst_output_sk);
> }
> +
> +int __ip6_local_out(struct sk_buff *skb)
> +{
> + return __ip6_local_out_sk(skb->sk, skb);
> +}
> EXPORT_SYMBOL_GPL(__ip6_local_out);
>
> -int ip6_local_out(struct sk_buff *skb)
> +int ip6_local_out_sk(struct sock *sk, struct sk_buff *skb)
> {
> int err;
>
> - err = __ip6_local_out(skb);
> + err = __ip6_local_out_sk(sk, skb);
> if (likely(err == 1))
> - err = dst_output(skb);
> + err = dst_output_sk(sk, skb);
>
> return err;
> }
> +EXPORT_SYMBOL_GPL(ip6_local_out_sk);
> +
> +int ip6_local_out(struct sk_buff *skb)
> +{
> + return ip6_local_out_sk(skb->sk, skb);
> +}
> EXPORT_SYMBOL_GPL(ip6_local_out);
> diff --git a/net/openvswitch/vport-vxlan.c b/net/openvswitch/vport-vxlan.c
> index 3277a75..6d39766 100644
> --- a/net/openvswitch/vport-vxlan.c
> +++ b/net/openvswitch/vport-vxlan.c
> @@ -222,7 +222,8 @@ static int vxlan_tnl_send(struct vport *vport, struct sk_buff *skb)
> {
> struct net *net = ovs_dp_get_net(vport->dp);
> struct vxlan_port *vxlan_port = vxlan_vport(vport);
> - __be16 dst_port = inet_sk(vxlan_port->vs->sock->sk)->inet_sport;
> + struct sock *sk = vxlan_port->vs->sock->sk;
> + __be16 dst_port = inet_sk(sk)->inet_sport;
> const struct ovs_key_ipv4_tunnel *tun_key;
> struct vxlan_metadata md = {0};
> struct rtable *rt;
> @@ -255,7 +256,7 @@ static int vxlan_tnl_send(struct vport *vport, struct sk_buff *skb)
> vxflags = vxlan_port->exts |
> (tun_key->tun_flags & TUNNEL_CSUM ? VXLAN_F_UDP_CSUM : 0);
>
> - err = vxlan_xmit_skb(rt, skb, fl.saddr, tun_key->ipv4_dst,
> + err = vxlan_xmit_skb(rt, sk, skb, fl.saddr, tun_key->ipv4_dst,
> tun_key->ipv4_tos, tun_key->ipv4_ttl, df,
> src_port, dst_port,
> &md, false, vxflags);
> diff --git a/net/tipc/udp_media.c b/net/tipc/udp_media.c
> index ef3d7aa..66deebc 100644
> --- a/net/tipc/udp_media.c
> +++ b/net/tipc/udp_media.c
> @@ -176,7 +176,8 @@ static int tipc_udp_send_msg(struct net *net, struct sk_buff *skb,
> goto tx_error;
> }
> ttl = ip4_dst_hoplimit(&rt->dst);
> - err = udp_tunnel_xmit_skb(rt, clone, src->ipv4.s_addr,
> + err = udp_tunnel_xmit_skb(rt, ub->ubsock->sk, clone,
> + src->ipv4.s_addr,
> dst->ipv4.s_addr, 0, ttl, 0,
> src->udp_port, dst->udp_port,
> false, true);
> @@ -197,7 +198,8 @@ static int tipc_udp_send_msg(struct net *net, struct sk_buff *skb,
> if (err)
> goto tx_error;
> ttl = ip6_dst_hoplimit(ndst);
> - err = udp_tunnel6_xmit_skb(ndst, clone, ndst->dev, &src->ipv6,
> + err = udp_tunnel6_xmit_skb(ndst, ub->ubsock->sk, clone,
> + ndst->dev, &src->ipv6,
> &dst->ipv6, 0, ttl, src->udp_port,
> dst->udp_port, false);
> #endif
> --
> 2.1.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-07 6:02 ` Tom Herbert
@ 2015-04-07 15:45 ` Hannes Frederic Sowa
2015-04-07 18:56 ` Tom Herbert
2015-04-07 19:30 ` David Miller
0 siblings, 2 replies; 9+ messages in thread
From: Hannes Frederic Sowa @ 2015-04-07 15:45 UTC (permalink / raw)
To: Tom Herbert
Cc: David Miller, netdev, netfilter-devel, pablo, Jiří Pírko
On Mo, 2015-04-06 at 23:02 -0700, Tom Herbert wrote:
> On Sun, Apr 5, 2015 at 7:19 PM, David Miller <davem@davemloft.net> wrote:
> >
> > That was we can make sure the output path of ipv4/ipv6 operate on
> > the UDP socket rather than whatever random thing happens to be in
> > skb->sk.
> >
> > Based upon a patch by Jiri Pirko.
> >
> > Signed-off-by: David S. Miller <davem@davemloft.net>
> > ---
> > drivers/net/vxlan.c | 14 ++++++++------
> > include/net/ip6_tunnel.h | 5 +++--
> > include/net/ipv6.h | 1 +
> > include/net/udp_tunnel.h | 5 +++--
> > include/net/vxlan.h | 2 +-
> > net/ipv4/geneve.c | 2 +-
> > net/ipv4/udp_tunnel.c | 4 ++--
> > net/ipv6/ip6_gre.c | 2 +-
> > net/ipv6/ip6_tunnel.c | 2 +-
> > net/ipv6/ip6_udp_tunnel.c | 5 +++--
> > net/ipv6/output_core.c | 21 ++++++++++++++++-----
> > net/openvswitch/vport-vxlan.c | 5 +++--
> > net/tipc/udp_media.c | 6 ++++--
> > 13 files changed, 47 insertions(+), 27 deletions(-)
> >
> > diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c
> > index b5fecb4..51baac7 100644
> > --- a/drivers/net/vxlan.c
> > +++ b/drivers/net/vxlan.c
> > @@ -1672,7 +1672,8 @@ static void vxlan_build_gbp_hdr(struct vxlanhdr *vxh, u32 vxflags,
> > }
> >
> > #if IS_ENABLED(CONFIG_IPV6)
> > -static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
> > +static int vxlan6_xmit_skb(struct dst_entry *dst, struct sock *sk,
> > + struct sk_buff *skb,
> > struct net_device *dev, struct in6_addr *saddr,
> > struct in6_addr *daddr, __u8 prio, __u8 ttl,
> > __be16 src_port, __be16 dst_port,
> > @@ -1748,7 +1749,7 @@ static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
> >
> > skb_set_inner_protocol(skb, htons(ETH_P_TEB));
> >
> > - udp_tunnel6_xmit_skb(dst, skb, dev, saddr, daddr, prio,
> > + udp_tunnel6_xmit_skb(dst, sk, skb, dev, saddr, daddr, prio,
> > ttl, src_port, dst_port,
> > !!(vxflags & VXLAN_F_UDP_ZERO_CSUM6_TX));
> > return 0;
> > @@ -1758,7 +1759,7 @@ err:
> > }
> > #endif
> >
> > -int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> > +int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> > __be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
> > __be16 src_port, __be16 dst_port,
> > struct vxlan_metadata *md, bool xnet, u32 vxflags)
> > @@ -1827,7 +1828,7 @@ int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> >
> > skb_set_inner_protocol(skb, htons(ETH_P_TEB));
> >
> > - return udp_tunnel_xmit_skb(rt, skb, src, dst, tos,
> > + return udp_tunnel_xmit_skb(rt, sk, skb, src, dst, tos,
> > ttl, df, src_port, dst_port, xnet,
> > !(vxflags & VXLAN_F_UDP_CSUM));
> > }
> > @@ -1882,6 +1883,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
> > struct vxlan_rdst *rdst, bool did_rsc)
> > {
> > struct vxlan_dev *vxlan = netdev_priv(dev);
> > + struct sock *sk = vxlan->vn_sock->sock->sk;
> > struct rtable *rt = NULL;
> > const struct iphdr *old_iph;
> > struct flowi4 fl4;
> > @@ -1961,7 +1963,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
> > md.vni = htonl(vni << 8);
> > md.gbp = skb->mark;
> >
> > - err = vxlan_xmit_skb(rt, skb, fl4.saddr,
> > + err = vxlan_xmit_skb(rt, sk, skb, fl4.saddr,
> > dst->sin.sin_addr.s_addr, tos, ttl, df,
> > src_port, dst_port, &md,
> > !net_eq(vxlan->net, dev_net(vxlan->dev)),
> > @@ -2021,7 +2023,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
> > md.vni = htonl(vni << 8);
> > md.gbp = skb->mark;
> >
> > - err = vxlan6_xmit_skb(ndst, skb, dev, &fl6.saddr, &fl6.daddr,
> > + err = vxlan6_xmit_skb(ndst, sk, skb, dev, &fl6.saddr, &fl6.daddr,
> > 0, ttl, src_port, dst_port, &md,
> > !net_eq(vxlan->net, dev_net(vxlan->dev)),
> > vxlan->flags);
> > diff --git a/include/net/ip6_tunnel.h b/include/net/ip6_tunnel.h
> > index 1668be5..b8529aa 100644
> > --- a/include/net/ip6_tunnel.h
> > +++ b/include/net/ip6_tunnel.h
> > @@ -73,13 +73,14 @@ __u32 ip6_tnl_get_cap(struct ip6_tnl *t, const struct in6_addr *laddr,
> > struct net *ip6_tnl_get_link_net(const struct net_device *dev);
> > int ip6_tnl_get_iflink(const struct net_device *dev);
> >
> > -static inline void ip6tunnel_xmit(struct sk_buff *skb, struct net_device *dev)
> > +static inline void ip6tunnel_xmit(struct sock *sk, struct sk_buff *skb,
> > + struct net_device *dev)
> > {
> > struct net_device_stats *stats = &dev->stats;
> > int pkt_len, err;
> >
> > pkt_len = skb->len;
> > - err = ip6_local_out(skb);
> > + err = ip6_local_out_sk(sk, skb);
> >
> > if (net_xmit_eval(err) == 0) {
> > struct pcpu_sw_netstats *tstats = this_cpu_ptr(dev->tstats);
> > diff --git a/include/net/ipv6.h b/include/net/ipv6.h
> > index b6ae959..27470cd 100644
> > --- a/include/net/ipv6.h
> > +++ b/include/net/ipv6.h
> > @@ -827,6 +827,7 @@ int ip6_input(struct sk_buff *skb);
> > int ip6_mc_input(struct sk_buff *skb);
> >
> > int __ip6_local_out(struct sk_buff *skb);
> > +int ip6_local_out_sk(struct sock *sk, struct sk_buff *skb);
> > int ip6_local_out(struct sk_buff *skb);
> >
> > /*
> > diff --git a/include/net/udp_tunnel.h b/include/net/udp_tunnel.h
> > index 1a20d33..c491c12 100644
> > --- a/include/net/udp_tunnel.h
> > +++ b/include/net/udp_tunnel.h
> > @@ -77,13 +77,14 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
> > struct udp_tunnel_sock_cfg *sock_cfg);
> >
> > /* Transmit the skb using UDP encapsulation. */
> > -int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> > +int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> > __be32 src, __be32 dst, __u8 tos, __u8 ttl,
> > __be16 df, __be16 src_port, __be16 dst_port,
> > bool xnet, bool nocheck);
> >
> > #if IS_ENABLED(CONFIG_IPV6)
> > -int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
> > +int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sock *sk,
> > + struct sk_buff *skb,
> > struct net_device *dev, struct in6_addr *saddr,
> > struct in6_addr *daddr,
> > __u8 prio, __u8 ttl, __be16 src_port,
> > diff --git a/include/net/vxlan.h b/include/net/vxlan.h
> > index 756e463..0082b5d 100644
> > --- a/include/net/vxlan.h
> > +++ b/include/net/vxlan.h
> > @@ -145,7 +145,7 @@ struct vxlan_sock *vxlan_sock_add(struct net *net, __be16 port,
> >
> > void vxlan_sock_release(struct vxlan_sock *vs);
> >
> > -int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> > +int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> > __be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
> > __be16 src_port, __be16 dst_port, struct vxlan_metadata *md,
> > bool xnet, u32 vxflags);
> > diff --git a/net/ipv4/geneve.c b/net/ipv4/geneve.c
> > index e64f8e9..b77f5e8 100644
> > --- a/net/ipv4/geneve.c
> > +++ b/net/ipv4/geneve.c
> > @@ -136,7 +136,7 @@ int geneve_xmit_skb(struct geneve_sock *gs, struct rtable *rt,
> >
> > skb_set_inner_protocol(skb, htons(ETH_P_TEB));
> >
> > - return udp_tunnel_xmit_skb(rt, skb, src, dst,
> > + return udp_tunnel_xmit_skb(rt, gs->sock->sk, skb, src, dst,
> > tos, ttl, df, src_port, dst_port, xnet,
> > !csum);
> > }
> > diff --git a/net/ipv4/udp_tunnel.c b/net/ipv4/udp_tunnel.c
> > index c83b354..6bb98cc 100644
> > --- a/net/ipv4/udp_tunnel.c
> > +++ b/net/ipv4/udp_tunnel.c
> > @@ -75,7 +75,7 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
> > }
> > EXPORT_SYMBOL_GPL(setup_udp_tunnel_sock);
> >
> > -int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> > +int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
> > __be32 src, __be32 dst, __u8 tos, __u8 ttl,
> > __be16 df, __be16 src_port, __be16 dst_port,
> > bool xnet, bool nocheck)
> > @@ -92,7 +92,7 @@ int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
> >
> > udp_set_csum(nocheck, skb, src, dst, skb->len);
> >
> > - return iptunnel_xmit(skb->sk, rt, skb, src, dst, IPPROTO_UDP,
> > + return iptunnel_xmit(sk, rt, skb, src, dst, IPPROTO_UDP,
> > tos, ttl, df, xnet);
> > }
> > EXPORT_SYMBOL_GPL(udp_tunnel_xmit_skb);
> > diff --git a/net/ipv6/ip6_gre.c b/net/ipv6/ip6_gre.c
> > index f724329..b5e6cc1 100644
> > --- a/net/ipv6/ip6_gre.c
> > +++ b/net/ipv6/ip6_gre.c
> > @@ -760,7 +760,7 @@ static netdev_tx_t ip6gre_xmit2(struct sk_buff *skb,
> >
> > skb_set_inner_protocol(skb, protocol);
> >
> > - ip6tunnel_xmit(skb, dev);
> > + ip6tunnel_xmit(NULL, skb, dev);
> > if (ndst)
> > ip6_tnl_dst_store(tunnel, ndst);
> > return 0;
> > diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
> > index b6a211a..5cafd92 100644
> > --- a/net/ipv6/ip6_tunnel.c
> > +++ b/net/ipv6/ip6_tunnel.c
> > @@ -1100,7 +1100,7 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
> > ipv6h->nexthdr = proto;
> > ipv6h->saddr = fl6->saddr;
> > ipv6h->daddr = fl6->daddr;
> > - ip6tunnel_xmit(skb, dev);
> > + ip6tunnel_xmit(NULL, skb, dev);
>
> By same should logic iptunnel_xmit call in ip_tunnel_xmit should take
> NULL arg for socket?
Sure!
Thanks, otherwise the patches look good!
Acked-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-07 15:45 ` Hannes Frederic Sowa
@ 2015-04-07 18:56 ` Tom Herbert
2015-04-07 19:14 ` Hannes Frederic Sowa
2015-04-07 19:30 ` David Miller
1 sibling, 1 reply; 9+ messages in thread
From: Tom Herbert @ 2015-04-07 18:56 UTC (permalink / raw)
To: Hannes Frederic Sowa
Cc: David Miller, netdev, netfilter-devel, pablo, Jiří Pírko
On Tue, Apr 7, 2015 at 8:45 AM, Hannes Frederic Sowa
<hannes@stressinduktion.org> wrote:
> On Mo, 2015-04-06 at 23:02 -0700, Tom Herbert wrote:
>> On Sun, Apr 5, 2015 at 7:19 PM, David Miller <davem@davemloft.net> wrote:
>> >
>> > That was we can make sure the output path of ipv4/ipv6 operate on
>> > the UDP socket rather than whatever random thing happens to be in
>> > skb->sk.
>> >
>> > Based upon a patch by Jiri Pirko.
>> >
>> > Signed-off-by: David S. Miller <davem@davemloft.net>
>> > ---
>> > drivers/net/vxlan.c | 14 ++++++++------
>> > include/net/ip6_tunnel.h | 5 +++--
>> > include/net/ipv6.h | 1 +
>> > include/net/udp_tunnel.h | 5 +++--
>> > include/net/vxlan.h | 2 +-
>> > net/ipv4/geneve.c | 2 +-
>> > net/ipv4/udp_tunnel.c | 4 ++--
>> > net/ipv6/ip6_gre.c | 2 +-
>> > net/ipv6/ip6_tunnel.c | 2 +-
>> > net/ipv6/ip6_udp_tunnel.c | 5 +++--
>> > net/ipv6/output_core.c | 21 ++++++++++++++++-----
>> > net/openvswitch/vport-vxlan.c | 5 +++--
>> > net/tipc/udp_media.c | 6 ++++--
>> > 13 files changed, 47 insertions(+), 27 deletions(-)
>> >
>> > diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c
>> > index b5fecb4..51baac7 100644
>> > --- a/drivers/net/vxlan.c
>> > +++ b/drivers/net/vxlan.c
>> > @@ -1672,7 +1672,8 @@ static void vxlan_build_gbp_hdr(struct vxlanhdr *vxh, u32 vxflags,
>> > }
>> >
>> > #if IS_ENABLED(CONFIG_IPV6)
>> > -static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
>> > +static int vxlan6_xmit_skb(struct dst_entry *dst, struct sock *sk,
>> > + struct sk_buff *skb,
>> > struct net_device *dev, struct in6_addr *saddr,
>> > struct in6_addr *daddr, __u8 prio, __u8 ttl,
>> > __be16 src_port, __be16 dst_port,
>> > @@ -1748,7 +1749,7 @@ static int vxlan6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
>> >
>> > skb_set_inner_protocol(skb, htons(ETH_P_TEB));
>> >
>> > - udp_tunnel6_xmit_skb(dst, skb, dev, saddr, daddr, prio,
>> > + udp_tunnel6_xmit_skb(dst, sk, skb, dev, saddr, daddr, prio,
>> > ttl, src_port, dst_port,
>> > !!(vxflags & VXLAN_F_UDP_ZERO_CSUM6_TX));
>> > return 0;
>> > @@ -1758,7 +1759,7 @@ err:
>> > }
>> > #endif
>> >
>> > -int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>> > +int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
>> > __be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
>> > __be16 src_port, __be16 dst_port,
>> > struct vxlan_metadata *md, bool xnet, u32 vxflags)
>> > @@ -1827,7 +1828,7 @@ int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>> >
>> > skb_set_inner_protocol(skb, htons(ETH_P_TEB));
>> >
>> > - return udp_tunnel_xmit_skb(rt, skb, src, dst, tos,
>> > + return udp_tunnel_xmit_skb(rt, sk, skb, src, dst, tos,
>> > ttl, df, src_port, dst_port, xnet,
>> > !(vxflags & VXLAN_F_UDP_CSUM));
>> > }
>> > @@ -1882,6 +1883,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
>> > struct vxlan_rdst *rdst, bool did_rsc)
>> > {
>> > struct vxlan_dev *vxlan = netdev_priv(dev);
>> > + struct sock *sk = vxlan->vn_sock->sock->sk;
>> > struct rtable *rt = NULL;
>> > const struct iphdr *old_iph;
>> > struct flowi4 fl4;
>> > @@ -1961,7 +1963,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
>> > md.vni = htonl(vni << 8);
>> > md.gbp = skb->mark;
>> >
>> > - err = vxlan_xmit_skb(rt, skb, fl4.saddr,
>> > + err = vxlan_xmit_skb(rt, sk, skb, fl4.saddr,
>> > dst->sin.sin_addr.s_addr, tos, ttl, df,
>> > src_port, dst_port, &md,
>> > !net_eq(vxlan->net, dev_net(vxlan->dev)),
>> > @@ -2021,7 +2023,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
>> > md.vni = htonl(vni << 8);
>> > md.gbp = skb->mark;
>> >
>> > - err = vxlan6_xmit_skb(ndst, skb, dev, &fl6.saddr, &fl6.daddr,
>> > + err = vxlan6_xmit_skb(ndst, sk, skb, dev, &fl6.saddr, &fl6.daddr,
>> > 0, ttl, src_port, dst_port, &md,
>> > !net_eq(vxlan->net, dev_net(vxlan->dev)),
>> > vxlan->flags);
>> > diff --git a/include/net/ip6_tunnel.h b/include/net/ip6_tunnel.h
>> > index 1668be5..b8529aa 100644
>> > --- a/include/net/ip6_tunnel.h
>> > +++ b/include/net/ip6_tunnel.h
>> > @@ -73,13 +73,14 @@ __u32 ip6_tnl_get_cap(struct ip6_tnl *t, const struct in6_addr *laddr,
>> > struct net *ip6_tnl_get_link_net(const struct net_device *dev);
>> > int ip6_tnl_get_iflink(const struct net_device *dev);
>> >
>> > -static inline void ip6tunnel_xmit(struct sk_buff *skb, struct net_device *dev)
>> > +static inline void ip6tunnel_xmit(struct sock *sk, struct sk_buff *skb,
>> > + struct net_device *dev)
>> > {
>> > struct net_device_stats *stats = &dev->stats;
>> > int pkt_len, err;
>> >
>> > pkt_len = skb->len;
>> > - err = ip6_local_out(skb);
>> > + err = ip6_local_out_sk(sk, skb);
>> >
>> > if (net_xmit_eval(err) == 0) {
>> > struct pcpu_sw_netstats *tstats = this_cpu_ptr(dev->tstats);
>> > diff --git a/include/net/ipv6.h b/include/net/ipv6.h
>> > index b6ae959..27470cd 100644
>> > --- a/include/net/ipv6.h
>> > +++ b/include/net/ipv6.h
>> > @@ -827,6 +827,7 @@ int ip6_input(struct sk_buff *skb);
>> > int ip6_mc_input(struct sk_buff *skb);
>> >
>> > int __ip6_local_out(struct sk_buff *skb);
>> > +int ip6_local_out_sk(struct sock *sk, struct sk_buff *skb);
>> > int ip6_local_out(struct sk_buff *skb);
>> >
>> > /*
>> > diff --git a/include/net/udp_tunnel.h b/include/net/udp_tunnel.h
>> > index 1a20d33..c491c12 100644
>> > --- a/include/net/udp_tunnel.h
>> > +++ b/include/net/udp_tunnel.h
>> > @@ -77,13 +77,14 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
>> > struct udp_tunnel_sock_cfg *sock_cfg);
>> >
>> > /* Transmit the skb using UDP encapsulation. */
>> > -int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>> > +int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
>> > __be32 src, __be32 dst, __u8 tos, __u8 ttl,
>> > __be16 df, __be16 src_port, __be16 dst_port,
>> > bool xnet, bool nocheck);
>> >
>> > #if IS_ENABLED(CONFIG_IPV6)
>> > -int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sk_buff *skb,
>> > +int udp_tunnel6_xmit_skb(struct dst_entry *dst, struct sock *sk,
>> > + struct sk_buff *skb,
>> > struct net_device *dev, struct in6_addr *saddr,
>> > struct in6_addr *daddr,
>> > __u8 prio, __u8 ttl, __be16 src_port,
>> > diff --git a/include/net/vxlan.h b/include/net/vxlan.h
>> > index 756e463..0082b5d 100644
>> > --- a/include/net/vxlan.h
>> > +++ b/include/net/vxlan.h
>> > @@ -145,7 +145,7 @@ struct vxlan_sock *vxlan_sock_add(struct net *net, __be16 port,
>> >
>> > void vxlan_sock_release(struct vxlan_sock *vs);
>> >
>> > -int vxlan_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>> > +int vxlan_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
>> > __be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
>> > __be16 src_port, __be16 dst_port, struct vxlan_metadata *md,
>> > bool xnet, u32 vxflags);
>> > diff --git a/net/ipv4/geneve.c b/net/ipv4/geneve.c
>> > index e64f8e9..b77f5e8 100644
>> > --- a/net/ipv4/geneve.c
>> > +++ b/net/ipv4/geneve.c
>> > @@ -136,7 +136,7 @@ int geneve_xmit_skb(struct geneve_sock *gs, struct rtable *rt,
>> >
>> > skb_set_inner_protocol(skb, htons(ETH_P_TEB));
>> >
>> > - return udp_tunnel_xmit_skb(rt, skb, src, dst,
>> > + return udp_tunnel_xmit_skb(rt, gs->sock->sk, skb, src, dst,
>> > tos, ttl, df, src_port, dst_port, xnet,
>> > !csum);
>> > }
>> > diff --git a/net/ipv4/udp_tunnel.c b/net/ipv4/udp_tunnel.c
>> > index c83b354..6bb98cc 100644
>> > --- a/net/ipv4/udp_tunnel.c
>> > +++ b/net/ipv4/udp_tunnel.c
>> > @@ -75,7 +75,7 @@ void setup_udp_tunnel_sock(struct net *net, struct socket *sock,
>> > }
>> > EXPORT_SYMBOL_GPL(setup_udp_tunnel_sock);
>> >
>> > -int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>> > +int udp_tunnel_xmit_skb(struct rtable *rt, struct sock *sk, struct sk_buff *skb,
>> > __be32 src, __be32 dst, __u8 tos, __u8 ttl,
>> > __be16 df, __be16 src_port, __be16 dst_port,
>> > bool xnet, bool nocheck)
>> > @@ -92,7 +92,7 @@ int udp_tunnel_xmit_skb(struct rtable *rt, struct sk_buff *skb,
>> >
>> > udp_set_csum(nocheck, skb, src, dst, skb->len);
>> >
>> > - return iptunnel_xmit(skb->sk, rt, skb, src, dst, IPPROTO_UDP,
>> > + return iptunnel_xmit(sk, rt, skb, src, dst, IPPROTO_UDP,
>> > tos, ttl, df, xnet);
>> > }
>> > EXPORT_SYMBOL_GPL(udp_tunnel_xmit_skb);
>> > diff --git a/net/ipv6/ip6_gre.c b/net/ipv6/ip6_gre.c
>> > index f724329..b5e6cc1 100644
>> > --- a/net/ipv6/ip6_gre.c
>> > +++ b/net/ipv6/ip6_gre.c
>> > @@ -760,7 +760,7 @@ static netdev_tx_t ip6gre_xmit2(struct sk_buff *skb,
>> >
>> > skb_set_inner_protocol(skb, protocol);
>> >
>> > - ip6tunnel_xmit(skb, dev);
>> > + ip6tunnel_xmit(NULL, skb, dev);
>> > if (ndst)
>> > ip6_tnl_dst_store(tunnel, ndst);
>> > return 0;
>> > diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
>> > index b6a211a..5cafd92 100644
>> > --- a/net/ipv6/ip6_tunnel.c
>> > +++ b/net/ipv6/ip6_tunnel.c
>> > @@ -1100,7 +1100,7 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
>> > ipv6h->nexthdr = proto;
>> > ipv6h->saddr = fl6->saddr;
>> > ipv6h->daddr = fl6->daddr;
>> > - ip6tunnel_xmit(skb, dev);
>> > + ip6tunnel_xmit(NULL, skb, dev);
>>
>> By same should logic iptunnel_xmit call in ip_tunnel_xmit should take
>> NULL arg for socket?
>
> Sure!
>
Actually, for consistency amongst all the IP encapsulation protocols
and given Dave's comments on how this is supposed to work, maybe for
the v4 and v6 tunnels we could have a raw AF_INET{6} socket opened for
all the tunnels to use when ip_tunnel_xmit is called. If it makes a
difference for UDP encapsulation, a UDP socket could also be opened.
Tom
> Thanks, otherwise the patches look good!
>
> Acked-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
>
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-07 18:56 ` Tom Herbert
@ 2015-04-07 19:14 ` Hannes Frederic Sowa
0 siblings, 0 replies; 9+ messages in thread
From: Hannes Frederic Sowa @ 2015-04-07 19:14 UTC (permalink / raw)
To: Tom Herbert
Cc: David Miller, netdev, netfilter-devel, pablo, Jiří Pírko
On Tue, Apr 7, 2015, at 20:56, Tom Herbert wrote:
> On Tue, Apr 7, 2015 at 8:45 AM, Hannes Frederic Sowa
> <hannes@stressinduktion.org> wrote:
> >> > --- a/net/ipv6/ip6_tunnel.c
> >> > +++ b/net/ipv6/ip6_tunnel.c
> >> > @@ -1100,7 +1100,7 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
> >> > ipv6h->nexthdr = proto;
> >> > ipv6h->saddr = fl6->saddr;
> >> > ipv6h->daddr = fl6->daddr;
> >> > - ip6tunnel_xmit(skb, dev);
> >> > + ip6tunnel_xmit(NULL, skb, dev);
> >>
> >> By same should logic iptunnel_xmit call in ip_tunnel_xmit should take
> >> NULL arg for socket?
> >
> > Sure!
> >
> Actually, for consistency amongst all the IP encapsulation protocols
> and given Dave's comments on how this is supposed to work, maybe for
> the v4 and v6 tunnels we could have a raw AF_INET{6} socket opened for
> all the tunnels to use when ip_tunnel_xmit is called. If it makes a
> difference for UDP encapsulation, a UDP socket could also be opened.
Yes, agreed.
Currently the semantics seem fine to me. vxlan needs this because it is
communicating via multicast addresses and thus we depend on correct
mc_loop behaviour. Which other tunnel protocols could be affected by
this, too?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-07 15:45 ` Hannes Frederic Sowa
2015-04-07 18:56 ` Tom Herbert
@ 2015-04-07 19:30 ` David Miller
2015-04-07 23:24 ` Hannes Frederic Sowa
1 sibling, 1 reply; 9+ messages in thread
From: David Miller @ 2015-04-07 19:30 UTC (permalink / raw)
To: hannes; +Cc: tom, netdev, netfilter-devel, pablo, jiri
From: Hannes Frederic Sowa <hannes@stressinduktion.org>
Date: Tue, 07 Apr 2015 17:45:34 +0200
> On Mo, 2015-04-06 at 23:02 -0700, Tom Herbert wrote:
>> On Sun, Apr 5, 2015 at 7:19 PM, David Miller <davem@davemloft.net> wrote:
>> > diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
>> > index b6a211a..5cafd92 100644
>> > --- a/net/ipv6/ip6_tunnel.c
>> > +++ b/net/ipv6/ip6_tunnel.c
>> > @@ -1100,7 +1100,7 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
>> > ipv6h->nexthdr = proto;
>> > ipv6h->saddr = fl6->saddr;
>> > ipv6h->daddr = fl6->daddr;
>> > - ip6tunnel_xmit(skb, dev);
>> > + ip6tunnel_xmit(NULL, skb, dev);
>>
>> By same should logic iptunnel_xmit call in ip_tunnel_xmit should take
>> NULL arg for socket?
>
> Sure!
>
> Thanks, otherwise the patches look good!
>
> Acked-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Ok I've made this adjustment and will push the series out, thanks everyone.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-07 19:30 ` David Miller
@ 2015-04-07 23:24 ` Hannes Frederic Sowa
2015-04-08 2:13 ` David Miller
0 siblings, 1 reply; 9+ messages in thread
From: Hannes Frederic Sowa @ 2015-04-07 23:24 UTC (permalink / raw)
To: David Miller, Rick Jones, Tom Herbert
Cc: tom, netdev, netfilter-devel, pablo, jiri
Hi,
On Tue, Apr 7, 2015, at 21:30, David Miller wrote:
> From: Hannes Frederic Sowa <hannes@stressinduktion.org>
> Date: Tue, 07 Apr 2015 17:45:34 +0200
>
> > On Mo, 2015-04-06 at 23:02 -0700, Tom Herbert wrote:
> >> On Sun, Apr 5, 2015 at 7:19 PM, David Miller <davem@davemloft.net> wrote:
> >> > diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
> >> > index b6a211a..5cafd92 100644
> >> > --- a/net/ipv6/ip6_tunnel.c
> >> > +++ b/net/ipv6/ip6_tunnel.c
> >> > @@ -1100,7 +1100,7 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
> >> > ipv6h->nexthdr = proto;
> >> > ipv6h->saddr = fl6->saddr;
> >> > ipv6h->daddr = fl6->daddr;
> >> > - ip6tunnel_xmit(skb, dev);
> >> > + ip6tunnel_xmit(NULL, skb, dev);
> >>
> >> By same should logic iptunnel_xmit call in ip_tunnel_xmit should take
> >> NULL arg for socket?
> >
> > Sure!
> >
> > Thanks, otherwise the patches look good!
> >
> > Acked-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
>
> Ok I've made this adjustment and will push the series out, thanks
> everyone.
There is one small problem which already was present before this patch
because of my change b6a7719aedd7e5 ("ipv4: hash net ptr into
fragmentation bucket selection"). It uses sock_net in iptunnel_xmit,
which causes a NULL ptr panic. This patchset did make it more visible
because of the explicit NULL passing but the bug was introduced by my
patch.
Just wanted to let you know if reports are coming in - I fix it up
tomorrow. I will also explore the possibility to give each tunnel at
least a raw socket, what Tom proposed.
Rick Jones identified this, thanks!
Bye,
Hannes
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-07 23:24 ` Hannes Frederic Sowa
@ 2015-04-08 2:13 ` David Miller
2015-04-08 15:03 ` Hannes Frederic Sowa
0 siblings, 1 reply; 9+ messages in thread
From: David Miller @ 2015-04-08 2:13 UTC (permalink / raw)
To: hannes; +Cc: rick.jones2, tom, netdev, netfilter-devel, pablo, jiri
From: Hannes Frederic Sowa <hannes@stressinduktion.org>
Date: Wed, 08 Apr 2015 01:24:59 +0200
> There is one small problem which already was present before this patch
> because of my change b6a7719aedd7e5 ("ipv4: hash net ptr into
> fragmentation bucket selection"). It uses sock_net in iptunnel_xmit,
> which causes a NULL ptr panic. This patchset did make it more visible
> because of the explicit NULL passing but the bug was introduced by my
> patch.
>
> Just wanted to let you know if reports are coming in - I fix it up
> tomorrow. I will also explore the possibility to give each tunnel at
> least a raw socket, what Tom proposed.
>
> Rick Jones identified this, thanks!
Looking into this I also notice that ipip6_tunnel_xmit() passes
in skb->sk, that needs to be NULL too.
Want to take care of that along with your sock_net() fix?
Thanks!
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb().
2015-04-08 2:13 ` David Miller
@ 2015-04-08 15:03 ` Hannes Frederic Sowa
0 siblings, 0 replies; 9+ messages in thread
From: Hannes Frederic Sowa @ 2015-04-08 15:03 UTC (permalink / raw)
To: David Miller; +Cc: rick.jones2, tom, netdev, netfilter-devel, pablo, jiri
On Wed, Apr 8, 2015, at 04:13, David Miller wrote:
> From: Hannes Frederic Sowa <hannes@stressinduktion.org>
> Date: Wed, 08 Apr 2015 01:24:59 +0200
>
> > There is one small problem which already was present before this patch
> > because of my change b6a7719aedd7e5 ("ipv4: hash net ptr into
> > fragmentation bucket selection"). It uses sock_net in iptunnel_xmit,
> > which causes a NULL ptr panic. This patchset did make it more visible
> > because of the explicit NULL passing but the bug was introduced by my
> > patch.
> >
> > Just wanted to let you know if reports are coming in - I fix it up
> > tomorrow. I will also explore the possibility to give each tunnel at
> > least a raw socket, what Tom proposed.
> >
> > Rick Jones identified this, thanks!
>
> Looking into this I also notice that ipip6_tunnel_xmit() passes
> in skb->sk, that needs to be NULL too.
>
> Want to take care of that along with your sock_net() fix?
Yep, did so and used the simple solution for now.
I am currently trying to check if it makes sense to remove ignore_df and
always use the socket context.
Bye,
Hannes
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-04-08 15:03 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-06 2:19 [PATCH 4/4] udp_tunnel: Pass UDP socket down through udp_tunnel{,6}_xmit_skb() David Miller
2015-04-07 6:02 ` Tom Herbert
2015-04-07 15:45 ` Hannes Frederic Sowa
2015-04-07 18:56 ` Tom Herbert
2015-04-07 19:14 ` Hannes Frederic Sowa
2015-04-07 19:30 ` David Miller
2015-04-07 23:24 ` Hannes Frederic Sowa
2015-04-08 2:13 ` David Miller
2015-04-08 15:03 ` Hannes Frederic Sowa
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.