openvswitch: fix a compilation error when CONFIG_INET is not setW!
authorAndy Zhou <azhou@nicira.com>
Mon, 6 Oct 2014 22:15:14 +0000 (15:15 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 7 Oct 2014 04:10:49 +0000 (00:10 -0400)
Fix a openvswitch compilation error when CONFIG_INET is not set:

=====================================================
   In file included from include/net/geneve.h:4:0,
                       from net/openvswitch/flow_netlink.c:45:
          include/net/udp_tunnel.h: In function 'udp_tunnel_handle_offloads':
  >> include/net/udp_tunnel.h:100:2: error: implicit declaration of function 'iptunnel_handle_offloads' [-Werror=implicit-function-declaration]
  >>      return iptunnel_handle_offloads(skb, udp_csum, type);
  >>           ^
  >>           >> include/net/udp_tunnel.h:100:2: warning: return makes pointer from integer without a cast
  >>           >>    cc1: some warnings being treated as errors

=====================================================

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Andy Zhou <azhou@nicira.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/Kconfig
include/net/geneve.h
net/ipv4/Kconfig

index c6f6f69f8961a58f40771d3b80d200d8fd730967..4706386b7d34c2d719e30eb602defd49ee92a72d 100644 (file)
@@ -147,7 +147,6 @@ config MACVTAP
 config VXLAN
        tristate "Virtual eXtensible Local Area Network (VXLAN)"
        depends on INET
-       select NET_IP_TUNNEL
        select NET_UDP_TUNNEL
        ---help---
          This allows one to create vxlan virtual interfaces that provide
index ce98865318bf42c2408b00fdb5605b78b7535d39..112132cf8e2e878c0e4629c88e2065eadfda0f4f 100644 (file)
@@ -1,22 +1,10 @@
 #ifndef __NET_GENEVE_H
 #define __NET_GENEVE_H  1
 
+#ifdef CONFIG_INET
 #include <net/udp_tunnel.h>
+#endif
 
-struct geneve_sock;
-
-typedef void (geneve_rcv_t)(struct geneve_sock *gs, struct sk_buff *skb);
-
-struct geneve_sock {
-       struct hlist_node       hlist;
-       geneve_rcv_t            *rcv;
-       void                    *rcv_data;
-       struct work_struct      del_work;
-       struct socket           *sock;
-       struct rcu_head         rcu;
-       atomic_t                refcnt;
-       struct udp_offload      udp_offloads;
-};
 
 /* Geneve Header:
  *  +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
@@ -74,6 +62,22 @@ struct genevehdr {
        struct geneve_opt options[];
 };
 
+#ifdef CONFIG_INET
+struct geneve_sock;
+
+typedef void (geneve_rcv_t)(struct geneve_sock *gs, struct sk_buff *skb);
+
+struct geneve_sock {
+       struct hlist_node       hlist;
+       geneve_rcv_t            *rcv;
+       void                    *rcv_data;
+       struct work_struct      del_work;
+       struct socket           *sock;
+       struct rcu_head         rcu;
+       atomic_t                refcnt;
+       struct udp_offload      udp_offloads;
+};
+
 #define GENEVE_VER 0
 #define GENEVE_BASE_HLEN (sizeof(struct udphdr) + sizeof(struct genevehdr))
 
@@ -88,4 +92,6 @@ int geneve_xmit_skb(struct geneve_sock *gs, struct rtable *rt,
                    __u8 ttl, __be16 df, __be16 src_port, __be16 dst_port,
                    __be16 tun_flags, u8 vni[3], u8 opt_len, u8 *opt,
                    bool xnet);
-#endif
+#endif /*ifdef CONFIG_INET */
+
+#endif /*ifdef__NET_GENEVE_H */
index c2035447e649ffa39c4754346565531945d341a4..e682b48e070993e4221a4400a8c86c3fa0c02684 100644 (file)
@@ -309,6 +309,7 @@ config NET_IPVTI
 
 config NET_UDP_TUNNEL
        tristate
+       select NET_IP_TUNNEL
        default n
 
 config NET_FOU
@@ -321,6 +322,20 @@ config NET_FOU
          network mechanisms and optimizations for UDP (such as ECMP
          and RSS) can be leveraged to provide better service.
 
+config GENEVE
+       tristate "Generic Network Virtualization Encapsulation (Geneve)"
+       depends on INET
+       select NET_UDP_TUNNEL
+       ---help---
+       This allows one to create Geneve virtual interfaces that provide
+       Layer 2 Networks over Layer 3 Networks. Geneve is often used
+       to tunnel virtual network infrastructure in virtualized environments.
+       For more information see:
+         http://tools.ietf.org/html/draft-gross-geneve-01
+
+         To compile this driver as a module, choose M here: the module
+
+
 config INET_AH
        tristate "IP: AH transformation"
        select XFRM_ALGO
@@ -453,20 +468,6 @@ config TCP_CONG_BIC
        increase provides TCP friendliness.
        See http://www.csc.ncsu.edu/faculty/rhee/export/bitcp/
 
-config GENEVE
-       tristate "Generic Network Virtualization Encapsulation (Geneve)"
-       depends on INET
-       select NET_IP_TUNNEL
-       select NET_UDP_TUNNEL
-       ---help---
-       This allows one to create Geneve virtual interfaces that provide
-       Layer 2 Networks over Layer 3 Networks. Geneve is often used
-       to tunnel virtual network infrastructure in virtualized environments.
-       For more information see:
-         http://tools.ietf.org/html/draft-gross-geneve-01
-
-         To compile this driver as a module, choose M here: the module
-
 config TCP_CONG_CUBIC
        tristate "CUBIC TCP"
        default y