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 <[email protected]> --- 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 _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
