From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Benc Subject: Re: [RFT] geneve: implement support for IPv6-based tunnels Date: Fri, 25 Sep 2015 14:08:44 +0200 Message-ID: <20150925140844.3e8faae2@griffin> References: <1443119682-14236-1-git-send-email-linville@tuxdriver.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, davem@davemloft.net, Pravin B Shelar , Jesse Gross To: "John W. Linville" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57963 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754412AbbIYMIs (ORCPT ); Fri, 25 Sep 2015 08:08:48 -0400 In-Reply-To: <1443119682-14236-1-git-send-email-linville@tuxdriver.com> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 24 Sep 2015 14:34:42 -0400, John W. Linville wrote: > +#if IS_ENABLED(CONFIG_IPV6) > +static netdev_tx_t geneve6_xmit_skb(struct sk_buff *skb, struct net_device *dev) > +{ > + struct geneve_dev *geneve = netdev_priv(dev); > + struct geneve_sock *gs = geneve->sock; > + struct ip_tunnel_info *info = NULL; > + struct dst_entry *dst = NULL; > + struct flowi6 fl6; > + __u8 ttl; > + __be16 sport; > + bool udp_csum; > + int err; > + bool xnet = !net_eq(geneve->net, dev_net(geneve->dev)); > + > + if (geneve->collect_md) { > + info = skb_tunnel_info(skb); > + if (unlikely(info && info->mode != IP_TUNNEL_INFO_TX)) { > + netdev_dbg(dev, "no tunnel metadata\n"); > + goto tx_error; > + } > + } You may get IPv4 tunnel info here. Either a check whether it's really IPv6 is needed or, better, decide whether to use IPv4 or IPv6 for xmit based on the tunnel info. See below. > +static netdev_tx_t geneve_xmit(struct sk_buff *skb, struct net_device *dev) > +{ > +#if IS_ENABLED(CONFIG_IPV6) > + struct geneve_dev *geneve = netdev_priv(dev); > + > + if (geneve->remote.sa.sa_family == AF_INET6) > + return geneve6_xmit_skb(skb, dev); > +#endif > + return geneve_xmit_skb(skb, dev); > +} For metadata based tunnels, there should be no requirement for the remote to be specified. As the consequence, you cannot decide based on the remote type. To be really useful, geneve should open both IPv4 and IPv6 socket when it's metadata based. Take a look at my recent patchset that does this for vxlan: http://thread.gmane.org/gmane.linux.network/379282 Jiri -- Jiri Benc