[ovs-dev] Report

2014-10-13 Thread The Post Office
The message could not be delivered ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] Open Vswitch Feature Enhancement- Provider Bridging Feature

2014-10-13 Thread Hiteshi Madan
Hi Team, We are thinking to contibute in following feature ;    -IEEE 802.1ah (Provider Backbone Bridging). If anybody is already working on it please let us know. Thanks Regards, Hiteshi Madan -Hiteshi Madan/DEL/TCS wrote: - To: Ben Pfaff b...@nicira.com From: Hiteshi

[ovs-dev] Bug#763428: Bug#763428: openvswitch-switch: openvswitch doesn't work anymore since kernel 3.16 update

2014-10-13 Thread Laurent GUERBY
On Wed, 8 Oct 2014 13:02:42 -0700 Ben Pfaff b...@nicira.com wrote: I can't reproduce this problem with OVS master and upstream kernel 3.16, or with OVS branch-2.3 on the commit from which the Debian packages were made. Now I'm downloading a Debian ISO so that I can try it with the exact

[ovs-dev] Jobs Opportunities in France/Canada

2014-10-13 Thread EVANS HOTEL
Jobs Opportunities in France/Canada This letter is to inform you on behalf of the management of EVANS HOTEL Suites Paris France, that the hotel needs able men and women, married and not married who are willing to relocate to France in order to fill various slots in the available

[ovs-dev] Jobs Opportunities in France/Canada

2014-10-13 Thread EVANS HOTEL
Jobs Opportunities in France/Canada This letter is to inform you on behalf of the management of EVANS HOTEL Suites Paris France, that the hotel needs able men and women, married and not married who are willing to relocate to France in order to fill various slots in the available

Re: [ovs-dev] [PATCH 6/6] datapath-windows: loop iterator fixes in Vport.c

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com Tested-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com -Mesaj original- De la: dev [mailto:dev-boun...@openvswitch.org] În numele Nithin Raju Trimis: Monday, October 13, 2014 6:56 AM Către: dev@openvswitch.org

Re: [ovs-dev] [PATCH 1/6] datapath-windows: Add netlink command: vport new

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com Tested-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com -Mesaj original- De la: dev [mailto:dev-boun...@openvswitch.org] În numele Nithin Raju Trimis: Monday, October 13, 2014 6:56 AM Către: dev@openvswitch.org Cc:

Re: [ovs-dev] [PATCH 5/6] datapath-windows: delete ports from portIdHashArray during cleanup

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com Tested-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com -Mesaj original- De la: dev [mailto:dev-boun...@openvswitch.org] În numele Nithin Raju Trimis: Monday, October 13, 2014 6:56 AM Către: dev@openvswitch.org

Re: [ovs-dev] [PATCH 4/6] datpath-windows: pass NDIS_RWL_AT_DISPATCH_LEVEL instead of BOOLEAN

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com Tested-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com -Mesaj original- De la: dev [mailto:dev-boun...@openvswitch.org] În numele Nithin Raju Trimis: Monday, October 13, 2014 6:56 AM Către: dev@openvswitch.org

Re: [ovs-dev] [PATCH 2/6] datapath-windows: Add netlink command vport delete

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com Tested-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com -Mesaj original- De la: dev [mailto:dev-boun...@openvswitch.org] În numele Nithin Raju Trimis: Monday, October 13, 2014 6:56 AM Către: dev@openvswitch.org Cc:

Re: [ovs-dev] [PATCH 3/6] datapath-windows: remove vport from lists upon deletion

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com Tested-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com -Mesaj original- De la: dev [mailto:dev-boun...@openvswitch.org] În numele Nithin Raju Trimis: Monday, October 13, 2014 6:56 AM Către: dev@openvswitch.org

[ovs-dev] [PATCH 1/1] netdev-dpdk: Move to DPDK 1.7.1

2014-10-13 Thread maryam.tahhan
This patch updates the documentation to reflect that DPDK 1.7.1 is supported. Travis scripts have also been updated to reflect this. DPDK phy and ring ports were validated against DPDK 1.7.1. (note: ring ports were validated with the upcoming multi-queue patch fix). Reviewed-by: Mark D. Gray

Re: [ovs-dev] Open Vswitch Feature Enhancement- Provider Bridging Feature

2014-10-13 Thread Ben Pfaff
On Mon, Oct 13, 2014 at 02:22:00PM +0530, Hiteshi Madan wrote: We are thinking to contibute in following feature ;    -IEEE 802.1ah (Provider Backbone Bridging). If anybody is already working on it please let us know. A patch has already been posted:

[ovs-dev] Bug#764847: openvswitch-switch: tuntap issues (commands)

2014-10-13 Thread Ben Pfaff
On Sat, Oct 11, 2014 at 11:24:19AM -0400, westlake wrote: Package: openvswitch-switch Version: 2.3.0+git20140819-2 Severity: important (I gave a previous message containing a link to the thread http://openvswitch.org/pipermail/discuss/2011-October/005914.html , but over here in Debian,

[ovs-dev] Bug#764843: openvswitch-switch: tuntap creation failure

2014-10-13 Thread Ben Pfaff
On Sat, Oct 11, 2014 at 11:12:32AM -0400, westlake wrote: Package: openvswitch Version: 2.3.0+git20140819-2 Severity: important http://openvswitch.org/pipermail/discuss/2011-October/005922.html An old bug is lingering somewhere in this latest edition.. Either it is recursive or something

Re: [ovs-dev] [PATCH] datapath: compat: Fix compilation 3.11

2014-10-13 Thread Andy Zhou
Thanks for explaining. I don't think we necessarily need to tie the use of ip_tunnel kernel API to the existence on vxlan kernel API. On the other hand. It is not a big deal. Acked-by: Andy Zhou az...@nicira.com On Fri, Oct 10, 2014 at 7:38 PM, Pravin Shelar pshe...@nicira.com wrote: On Fri,

Re: [ovs-dev] [PATCH] datapath: compat: Fix compilation 3.11

2014-10-13 Thread Pravin Shelar
On Mon, Oct 13, 2014 at 9:22 AM, Andy Zhou az...@nicira.com wrote: Thanks for explaining. I don't think we necessarily need to tie the use of ip_tunnel kernel API to the existence on vxlan kernel API. On the other hand. It is not a big deal. We can do that later, as part of larger cleanup.

Re: [ovs-dev] [PATCH 0/2] V4 Add 802.1ad (qinq) Support to OVS

2014-10-13 Thread Jeff Peterson
Hey Tom, Let us know what you hear back on this stuff. Thanks On Sat, Oct 11, 2014 at 6:56 PM, Thomas F Herbert thomasfherb...@entpnt.com wrote: This patch adds 802.1ad support to OVS. It includes the user space and linux kernel portions. This effort is supported by Entry Point LLC. It

Re: [ovs-dev] [PATCH 0/2] V4 Add 802.1ad (qinq) Support to OVS

2014-10-13 Thread Ben Pfaff
Normally any reviews or feedback will be posted publicly anyhow. On Mon, Oct 13, 2014 at 11:22 AM, Jeff Peterson jpeter...@entpnt.com wrote: Hey Tom, Let us know what you hear back on this stuff. Thanks On Sat, Oct 11, 2014 at 6:56 PM, Thomas F Herbert thomasfherb...@entpnt.com wrote:

[ovs-dev] Bug#763428: Bug#763428: openvswitch-switch: openvswitch doesn't work anymore since kernel 3.16 update

2014-10-13 Thread Ben Pfaff
On Mon, Oct 13, 2014 at 10:47:19AM +0200, Laurent GUERBY wrote: On Wed, 8 Oct 2014 13:02:42 -0700 Ben Pfaff b...@nicira.com wrote: I can't reproduce this problem with OVS master and upstream kernel 3.16, or with OVS branch-2.3 on the commit from which the Debian packages were made. Now I'm

Re: [ovs-dev] [PATCH v2] lib/netlink-socket.c: always pass the output buffer in a transaction

2014-10-13 Thread Nithin Raju
On Oct 7, 2014, at 8:01 PM, Eitan Eliahu elia...@vmware.com wrote: Hi Nithin, Thanks for addressing the transactional error issue. Are we sure we want to make another copy for the Reply just for getting the transactional error on a different buffer? Can we use the stack buffer only if

Re: [ovs-dev] [PATCH v2] lib/netlink-socket.c: always pass the output buffer in a transaction

2014-10-13 Thread Ben Pfaff
On Tue, Oct 07, 2014 at 03:08:53PM -0700, Nithin Raju wrote: We need to pass down the output buffer so that the kernel can return transaction status - error or otherwise. Also, we were processing the output buffer only when when 'txn-reply != NULL' ie when the caller specified an ofpbuf for

Re: [ovs-dev] [PATCH/RFC repost 7/8] ofproto: translate datapath select group action

2014-10-13 Thread Ben Pfaff
On Thu, Oct 09, 2014 at 10:14:36AM +0900, Simon Horman wrote: On Fri, Sep 26, 2014 at 04:57:25PM -0700, Ben Pfaff wrote: On Thu, Sep 18, 2014 at 10:55:10AM +0900, Simon Horman wrote: This patch is a prototype and has several limitations: * It assumes that no actions follow a select

Re: [ovs-dev] [PATCH 0/2] Refinements on userspace tunneling pathces.

2014-10-13 Thread Pravin Shelar
On Fri, Oct 10, 2014 at 3:48 PM, Jarno Rajahalme jrajaha...@nicira.com wrote: I wanted to see that the recent classifier improvement was enough to remove the need to add locking when an classifier instance is used as a container. To that end I improved the relevant code in Pravin's userspace

Re: [ovs-dev] [PATCH] lib/classifier: Minimize critical section.

2014-10-13 Thread Ben Pfaff
On Fri, Oct 10, 2014 at 04:18:45PM -0700, Jarno Rajahalme wrote: classifier_find_rule_exactly() only needs the mutex to protect the list traversal. Subtables are already RCU protected. Locking here can be eliminated completely when RCU list is available. Signed-off-by: Jarno Rajahalme

[ovs-dev] Status of patch adding Auto Attach feature

2014-10-13 Thread Flynn, Dennis R (Dennis)
Hi, A while back (Oct. 2) we posted a series of patches offering support for the Auto Attach feature. A description of this feature can be found here. http://openvswitch.org/pipermail/dev/2014-October/046651.html I'm curious to know if anyone has had a chance to look at this. Thanks, Dennis

Re: [ovs-dev] [PATCH 0/6] datapath-windows: vport add/delete commands with fixes

2014-10-13 Thread Ben Pfaff
On Sun, Oct 12, 2014 at 08:56:13PM -0700, Nithin Raju wrote: In the first two patches of the series, I have rebased Samuel's patches that were submitted for review to tip-of-master. Only Samuel is listed as the Author just like in the original patch. In subsequent patches, we make fixes to

Re: [ovs-dev] Status of patch adding Auto Attach feature

2014-10-13 Thread Ben Pfaff
On Mon, Oct 13, 2014 at 09:00:54PM +, Flynn, Dennis R (Dennis) wrote: A while back (Oct. 2) we posted a series of patches offering support for the Auto Attach feature. A description of this feature can be found here. http://openvswitch.org/pipermail/dev/2014-October/046651.html I'm

Re: [ovs-dev] [PATCH v2 6/6] datapath-windows: Assert fix in Netlink.c

2014-10-13 Thread Ben Pfaff
On Sat, Oct 11, 2014 at 03:07:41PM -0700, Ankur Sharma wrote: NlBufAt should be called with valid boundary limits (within head and tail). Incorrect argument to NlBufAt was leading to assert hit, fixed the same. Signed-off-by: Ankur Sharma ankursha...@vmware.com Acked-by: Nithin Raju

Re: [ovs-dev] [PATCH RFC 2/2] openvswitch: Userspace tunneling.

2014-10-13 Thread Pravin Shelar
On Thu, Oct 9, 2014 at 2:53 PM, Jarno Rajahalme jrajaha...@nicira.com wrote: Pravin, Please find my comments below. I did not snip any code to make it easier for you to keep into context while reading this review. Thanks for detailed review. Jarno On Oct 3, 2014, at 8:24 PM, Pravin B

Re: [ovs-dev] [PATCH 0/3] auto-attach: Add support for IETF Auto Attach standard

2014-10-13 Thread Ben Pfaff
On Thu, Oct 02, 2014 at 06:42:10PM -0400, Ludovic Beliveau wrote: This patch sequence provides OVS support for the IETF Auto-Attach SPBM draft standard. This standard describes a compact method of using IEEE 802.1AB Link Layer Discovery Protocol (LLDP) together with a IEEE 802.1aq Shortest Path

Re: [ovs-dev] [PATCH 1/3] auto-attach: Initial support for Auto-Attach standard

2014-10-13 Thread Ben Pfaff
On Thu, Oct 02, 2014 at 06:42:24PM -0400, Ludovic Beliveau wrote: This commit provides the initial delivery of support for the Auto-Attach standard to Open vSwitch. This standard describes a compact method of using IEEE 802.1AB Link Layer Discovery Protocol (LLDP) with a IEEE 802.1aq Shortest

Re: [ovs-dev] [PATCH 2/3] auto-attach: Add auto-attach support to ofproto layer

2014-10-13 Thread Ben Pfaff
On Thu, Oct 02, 2014 at 06:42:33PM -0400, Ludovic Beliveau wrote: Signed-off-by: Ludovic Beliveau ludovic.beliv...@windriver.com Signed-off-by: Dennis Flynn drfl...@avaya.com This seems pretty reasonable except that I'd much prefer to delete all the #ifdefs.

Re: [ovs-dev] [PATCH 3/3] auto-attach: Add auto-attach support to bridge layer and command set

2014-10-13 Thread Ben Pfaff
On Thu, Oct 02, 2014 at 06:42:55PM -0400, Ludovic Beliveau wrote: This is the final commit in the series of commits that deliver initial support for Auto-Attach. Specifically this commit delivers auto-attach support to the OVS bridge layer as well as the new auto-attach commands. The OVSDB

Re: [ovs-dev] [PATCHv7 07/11] hash: Add 128-bit murmurhash.

2014-10-13 Thread Ben Pfaff
On Tue, Oct 07, 2014 at 12:23:34AM +1300, Joe Stringer wrote: Add the 128-bit murmurhash by Austin Appleby, r150 from: http://code.google.com/p/smhasher/source/browse/trunk/MurmurHash3.cpp Signed-off-by: Joe Stringer joestrin...@nicira.com Acked-by: Ben Pfaff b...@nicira.com Compiling on

Re: [ovs-dev] [PATCHv7 10/11] dpif: Index flows using unique identifiers.

2014-10-13 Thread Ben Pfaff
On Tue, Oct 07, 2014 at 12:23:37AM +1300, Joe Stringer wrote: This patch modifies the dpif interface to allow flows to be manipulated using a 128-bit identifier. This allows revalidator threads to perform datapath operations faster, as they do not need to serialise the entire flow key for

Re: [ovs-dev] [PATCH RFC 2/2] openvswitch: Userspace tunneling.

2014-10-13 Thread Jarno Rajahalme
On Oct 13, 2014, at 2:32 PM, Pravin Shelar pshe...@nicira.com wrote: +case OVS_ACTION_ATTR_TUNNEL_PUSH: +if (*depth MAX_RECIRC_DEPTH) { +struct dpif_packet *tnl_pkt[NETDEV_MAX_RX_BATCH]; +int err; + +if (may_steal) { +

[ovs-dev] Bug#764847: openvswitch-switch: tuntap issues (commands)

2014-10-13 Thread westlake
(the earlier report can be deleted) Have you been able to confirm/replicate this bug? The url I've given exhibits a more broken case, but what I've encountered after checking the created tap device almost gets properly created. On 13/10/14 11:21 AM, Ben Pfaff wrote: On Sat, Oct 11, 2014

Re: [ovs-dev] [PATCH v2] lib/dpif-netdev: Integrate megaflow classifier.

2014-10-13 Thread Alex Wang
Hey Jarno, I have some high-level comments/questions as follow: On Tue, Oct 7, 2014 at 2:43 PM, Jarno Rajahalme jrajaha...@nicira.com wrote: flow inserts and removals are simplified: - No need for classifier internal mutex, as dpif-netdev already has a 'flow_mutex'. I'm thinking maybe

[ovs-dev] [PATCH] nicira-ext.h: Update comments

2014-10-13 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi yamam...@valinux.co.jp --- include/openflow/nicira-ext.h | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h index 65ba950..b20bab3 100644 ---

Re: [ovs-dev] [PATCH/RFC repost 7/8] ofproto: translate datapath select group action

2014-10-13 Thread Simon Horman
On Mon, Oct 13, 2014 at 01:46:24PM -0700, Ben Pfaff wrote: On Thu, Oct 09, 2014 at 10:14:36AM +0900, Simon Horman wrote: On Fri, Sep 26, 2014 at 04:57:25PM -0700, Ben Pfaff wrote: On Thu, Sep 18, 2014 at 10:55:10AM +0900, Simon Horman wrote: This patch is a prototype and has several

[ovs-dev] [PATCH v1 1/2] datapath-windows: Fix CtrlLock acquire issue in OvsReadEventCmdHandler

2014-10-13 Thread Ankur Sharma
OvsReadEventCmdHandler was calling OvsRemoveEventEntry after acquiring CtrlLock. OvsRemoveEventEntry in turn also tries to acquire the same lock. Fixed the same. Also, in Event.c we removed the APIs OvsAcquireEventQueueLock and OvsReleaseEventQueueLock. These APIs were acquiring and releasing

[ovs-dev] [PATCH v1 2/2] datapath-windows: Transactional error fix for flow dump.

2014-10-13 Thread Ankur Sharma
My changes for trasacation error handling for not needed for dump commands. Fixed the same. Signed-off-by: Ankur Sharma ankursha...@vmware.com --- datapath-windows/ovsext/Flow.c | 50 +++--- 1 file changed, 13 insertions(+), 37 deletions(-) diff --git