[ovs-dev] [PATCH v2] openvswitch: Add Kconfig dependency on GRE-DEMUX.
Randy Dunlap
rdunlap at infradead.org
Fri Jun 28 21:00:41 UTC 2013
On 06/28/13 12:39, Pravin B Shelar wrote:
> Openvswitch uses function from NET_IPGRE_DEMUX module.
> Add Kconfig dependency to fix following compilation errors:
> http://marc.info/?l=linux-netdev&m=137244035226634
>
> CC: Jesse Gross <jesse at nicira.com>
> Reported-by: Randy Dunlap <rdunlap at infradead.org>
> Signed-off-by: Pravin Shelar <pshelar at nicira.com>
Well, that works if CONFIG_INET is enabled, but if I disable that
kconfig symbol and then rebuild, this is what happens:
net/openvswitch/vport-gre.c: In function 'gre_tnl_send':
net/openvswitch/vport-gre.c:179:2: error: implicit declaration of function 'iptunnel_xmit' [-Werror=implicit-function-declaration]
> ---
> v1-v2:
> - use select to resolve dependency.
> ---
> net/openvswitch/Kconfig | 1 +
> net/openvswitch/vport-gre.c | 2 --
> net/openvswitch/vport.c | 3 ---
> 3 files changed, 1 insertions(+), 5 deletions(-)
>
> diff --git a/net/openvswitch/Kconfig b/net/openvswitch/Kconfig
> index 9fbc04a..deaa9bb 100644
> --- a/net/openvswitch/Kconfig
> +++ b/net/openvswitch/Kconfig
> @@ -4,6 +4,7 @@
>
> config OPENVSWITCH
> tristate "Open vSwitch"
> + select NET_IPGRE_DEMUX
> ---help---
> Open vSwitch is a multilayer Ethernet switch targeted at virtualized
> environments. In addition to supporting a variety of features
> diff --git a/net/openvswitch/vport-gre.c b/net/openvswitch/vport-gre.c
> index 943e5c4..21d5073 100644
> --- a/net/openvswitch/vport-gre.c
> +++ b/net/openvswitch/vport-gre.c
> @@ -16,7 +16,6 @@
> * 02110-1301, USA
> */
>
> -#if IS_ENABLED(CONFIG_NET_IPGRE_DEMUX)
> #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
>
> #include <linux/if.h>
> @@ -271,4 +270,3 @@ const struct vport_ops ovs_gre_vport_ops = {
> .get_name = gre_get_name,
> .send = gre_tnl_send,
> };
> -#endif
> diff --git a/net/openvswitch/vport.c b/net/openvswitch/vport.c
> index ba81294..ba240a8 100644
> --- a/net/openvswitch/vport.c
> +++ b/net/openvswitch/vport.c
> @@ -38,10 +38,7 @@
> static const struct vport_ops *vport_ops_list[] = {
> &ovs_netdev_vport_ops,
> &ovs_internal_vport_ops,
> -
> -#if IS_ENABLED(CONFIG_NET_IPGRE_DEMUX)
> &ovs_gre_vport_ops,
> -#endif
> };
>
> /* Protected by RCU read lock for reading, ovs_mutex for writing. */
>
--
~Randy
More information about the dev
mailing list