Re: [PATCH net-next 5/6] geneve: Consolidate Geneve functionality in single module.

2015-08-13 Thread Pravin Shelar
On Thu, Aug 13, 2015 at 2:24 PM, Jesse Gross wrote: > On Wed, Aug 12, 2015 at 4:33 PM, Pravin Shelar wrote: >> On Wed, Aug 12, 2015 at 2:55 PM, Jesse Gross wrote: >>> The farther I go in the series, the more that I hope that we can avoid >>> the use of collect_md_tun. It really seems to add a lo

Re: [PATCH net-next 5/6] geneve: Consolidate Geneve functionality in single module.

2015-08-13 Thread Jesse Gross
On Wed, Aug 12, 2015 at 4:33 PM, Pravin Shelar wrote: > On Wed, Aug 12, 2015 at 2:55 PM, Jesse Gross wrote: >> The farther I go in the series, the more that I hope that we can avoid >> the use of collect_md_tun. It really seems to add a lot of special >> cases. > > Use of collect_md_tun allows us

Re: [PATCH net-next 5/6] geneve: Consolidate Geneve functionality in single module.

2015-08-12 Thread Pravin Shelar
On Wed, Aug 12, 2015 at 2:55 PM, Jesse Gross wrote: > On Tue, Aug 11, 2015 at 10:17 PM, Pravin B Shelar wrote: >> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig >> index c18f9e6..0002ab7 100644 >> --- a/drivers/net/Kconfig >> +++ b/drivers/net/Kconfig >> @@ -181,7 +181,7 @@ config VXLAN >

Re: [PATCH net-next 5/6] geneve: Consolidate Geneve functionality in single module.

2015-08-12 Thread Jesse Gross
On Tue, Aug 11, 2015 at 10:17 PM, Pravin B Shelar wrote: > diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig > index c18f9e6..0002ab7 100644 > --- a/drivers/net/Kconfig > +++ b/drivers/net/Kconfig > @@ -181,7 +181,7 @@ config VXLAN > > config GENEVE > tristate "Generic Network Virtua

[PATCH net-next 5/6] geneve: Consolidate Geneve functionality in single module.

2015-08-11 Thread Pravin B Shelar
geneve_core module handles send and receive functionality. This way OVS could use the Geneve API. Now with use of tunnel meatadata mode OVS can directly use Geneve netdevice. So there is no need for separate module for Geneve. Following patch consolidates Geneve protocol processing in single module