[ovs-dev] [PATCH 3/3] datapath: compat: Block upstream ip_tunnels functions.
Pravin B Shelar
pshelar at nicira.com
Fri Dec 11 04:03:01 UTC 2015
Since upstream and compat ip_tunnel structures are not same, we can not
use exported upstream functions.
Following patch blocks definitions which used ip_tunnel internal
structure. Function which do not depend on these structures are
allows by explicitly by defining it in the header files. e.g.
iptunnel_handle_offloads(), iptunnel_pull_header(). etc.
Signed-off-by: Pravin B Shelar <pshelar at nicira.com>
---
datapath/linux/compat/include/net/gre.h | 3 ++-
datapath/linux/compat/include/net/ip_tunnels.h | 27 ++++++++++++++++++++------
2 files changed, 23 insertions(+), 7 deletions(-)
diff --git a/datapath/linux/compat/include/net/gre.h b/datapath/linux/compat/include/net/gre.h
index 09053b5..60618df 100644
--- a/datapath/linux/compat/include/net/gre.h
+++ b/datapath/linux/compat/include/net/gre.h
@@ -1,8 +1,10 @@
#ifndef __LINUX_GRE_WRAPPER_H
#define __LINUX_GRE_WRAPPER_H
+#include <linux/version.h>
#include <linux/skbuff.h>
#include <net/ip_tunnels.h>
+
#ifdef HAVE_METADATA_DST
#include_next <net/gre.h>
@@ -17,7 +19,6 @@ static inline void rpl_ipgre_fini(void)
#else
-#include <linux/version.h>
#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,37) || \
defined(HAVE_GRE_CISCO_REGISTER)
#include_next <net/gre.h>
diff --git a/datapath/linux/compat/include/net/ip_tunnels.h b/datapath/linux/compat/include/net/ip_tunnels.h
index 7083372..3d952d4 100644
--- a/datapath/linux/compat/include/net/ip_tunnels.h
+++ b/datapath/linux/compat/include/net/ip_tunnels.h
@@ -3,15 +3,14 @@
#include <linux/version.h>
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(3,10,0)
+#ifdef HAVE_METADATA_DST
+/* Block all ip_tunnel functions.
+ * Only function that do not depend on ip_tunnel structure can
+ * be used. Those needs to be explicitly defined in this header file. */
#include_next <net/ip_tunnels.h>
#endif
-#if LINUX_VERSION_CODE < KERNEL_VERSION(3,18,0)
-
#include <linux/if_tunnel.h>
-#include <linux/netdevice.h>
-#include <linux/skbuff.h>
#include <linux/types.h>
#include <net/dsfield.h>
#include <net/flow.h>
@@ -19,6 +18,7 @@
#include <net/ip.h>
#include <net/rtnetlink.h>
+#if LINUX_VERSION_CODE < KERNEL_VERSION(3,18,0)
struct sk_buff *ovs_iptunnel_handle_offloads(struct sk_buff *skb,
bool csum_help, int gso_type_mask,
void (*fix_segment)(struct sk_buff *));
@@ -36,8 +36,21 @@ int rpl_iptunnel_pull_header(struct sk_buff *skb, int hdr_len, __be16 inner_prot
#define ovs_iptunnel_handle_offloads(skb, csum_help, gso_type_mask, fix_segment) \
iptunnel_handle_offloads(skb, csum_help, gso_type_mask)
+/* This macro is to make OVS build happy about declared functions name. */
+#define rpl_iptunnel_pull_header iptunnel_pull_header
+int rpl_iptunnel_pull_header(struct sk_buff *skb, int hdr_len, __be16 inner_proto);
+
+#define rpl_iptunnel_xmit iptunnel_xmit
+int rpl_iptunnel_xmit(struct sock *sk, struct rtable *rt, struct sk_buff *skb,
+ __be32 src, __be32 dst, __u8 proto, __u8 tos, __u8 ttl,
+ __be16 df, bool xnet);
+
#endif /* 3.18 */
+#define rpl_iptunnel_handle_offloads iptunnel_handle_offloads
+struct sk_buff *rpl_iptunnel_handle_offloads(struct sk_buff *skb, bool gre_csum,
+ int gso_type_mask);
+
#ifndef TUNNEL_CSUM
#define TUNNEL_CSUM __cpu_to_be16(0x01)
#define TUNNEL_ROUTING __cpu_to_be16(0x02)
@@ -211,9 +224,11 @@ struct ip_tunnel_net {
#ifndef HAVE_PCPU_SW_NETSTATS
#define ip_tunnel_get_stats64 rpl_ip_tunnel_get_stats64
+#else
+#define rpl_ip_tunnel_get_stats64 ip_tunnel_get_stats64
+#endif
struct rtnl_link_stats64 *rpl_ip_tunnel_get_stats64(struct net_device *dev,
struct rtnl_link_stats64 *tot);
-#endif
#define ip_tunnel_get_dsfield rpl_ip_tunnel_get_dsfield
static inline u8 ip_tunnel_get_dsfield(const struct iphdr *iph,
--
1.8.3.1
More information about the dev
mailing list