Re: [PATCH v4 net-next 7/8] geneve: Consolidate Geneve functionality in single module.

2015-08-26 Thread Jesse Gross
On Wed, Aug 26, 2015 at 8:57 PM, Pravin Shelar wrote: > On Wed, Aug 26, 2015 at 8:08 PM, Jesse Gross wrote: >> On Wed, Aug 26, 2015 at 2:54 PM, Pravin B Shelar wrote: >>> @@ -293,15 +615,13 @@ static netdev_tx_t geneve_xmit(struct sk_buff *skb, >>> struct net_device *dev) >> [...] >>> +

Re: [PATCH v4 net-next 7/8] geneve: Consolidate Geneve functionality in single module.

2015-08-26 Thread Pravin Shelar
On Wed, Aug 26, 2015 at 8:08 PM, Jesse Gross wrote: > On Wed, Aug 26, 2015 at 2:54 PM, Pravin B Shelar wrote: >> diff --git a/drivers/net/geneve.c b/drivers/net/geneve.c >> index d05150c..a36a1de 100644 >> --- a/drivers/net/geneve.c >> +++ b/drivers/net/geneve.c >> @@ -138,16 +164,18 @@ static vo

Re: [PATCH v4 net-next 7/8] geneve: Consolidate Geneve functionality in single module.

2015-08-26 Thread Jesse Gross
On Wed, Aug 26, 2015 at 2:54 PM, Pravin B Shelar wrote: > diff --git a/drivers/net/geneve.c b/drivers/net/geneve.c > index d05150c..a36a1de 100644 > --- a/drivers/net/geneve.c > +++ b/drivers/net/geneve.c > @@ -138,16 +164,18 @@ static void geneve_rx(struct geneve_sock *gs, struct > sk_buff *skb)

[PATCH v4 net-next 7/8] geneve: Consolidate Geneve functionality in single module.

2015-08-26 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