Re: [ovs-dev] [PATCH v2 2/3] datapath-windows: Support for multiple VXLAN tunnels

2015-03-16 Thread Eitan Eliahu
Acked-by: Eitan Eliahu Thanks, Eitan -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Sorin Vinturis Sent: Thursday, March 12, 2015 5:43 PM To: dev@openvswitch.org Subject: [ovs-dev] [PATCH v2 2/3] datapath-windows: Support for multiple VXLAN tunnels At

Re: [ovs-dev] [PATCH v2 2/3] datapath-windows: Support for multiple VXLAN tunnels

2015-03-16 Thread Sorin Vinturis
, Sorin -Original Message- From: Nithin Raju [mailto:nit...@vmware.com] Sent: Sunday, 15 March, 2015 18:23 To: Sorin Vinturis Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH v2 2/3] datapath-windows: Support for multiple VXLAN tunnels hi Sorin, This change looks good. Have you tested

Re: [ovs-dev] [PATCH v2 2/3] datapath-windows: Support for multiple VXLAN tunnels

2015-03-15 Thread Nithin Raju
hi Sorin, This change looks good. Have you tested the scenario of unloading the OVS extension while the VXLAN ports exist in the kernel? I had a minor comment as follows, but looks good otherwise. > static __inline BOOLEAN > OvsIsInternalVportType(OVS_VPORT_TYPE ovsType) > { > diff --git a/data

[ovs-dev] [PATCH v2 2/3] datapath-windows: Support for multiple VXLAN tunnels

2015-03-12 Thread Sorin Vinturis
At the moment the OVS extension supports only one VXLAN tunnel that is cached in the extension switch context. Replaced the latter cached pointer with an array list that contains all VXLAN tunnel vports. Signed-off-by: Sorin Vinturis Reported-by: Alin Gabriel Serdean Reported-at: https://github.