[ovs-dev] Delivery reports about your e-mail

2014-10-13 Thread The Post Office
Your message was not delivered due to the following reason: Your message could not be delivered because the destination server was unreachable within the allowed queue period. The amount of time a message is queued before it is returned depends on local configura- tion parameters. Most likely the

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

2014-10-13 Thread Sorin Vinturis
Hi Ankur, In the Event.c the CtrlLock was used for event queue manipulation through the use of the OvsAcquireEventQueueLock and OvsReleaseEventQueueLock functions that you removed in this patch. But the CtrlLock should be used to guard only one shared resource, and is already used for guarding

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

2014-10-13 Thread YAMAMOTO Takashi
> On Tue, Oct 14, 2014 at 12:40:36PM +0900, YAMAMOTO Takashi wrote: >> Signed-off-by: YAMAMOTO Takashi > > Acked-by: Ben Pfaff thank you. applied. YAMAMOTO Takashi ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2014-10-13 Thread Ben Pfaff
On Tue, Oct 14, 2014 at 12:40:36PM +0900, YAMAMOTO Takashi wrote: > Signed-off-by: YAMAMOTO Takashi Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[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 Ctrl

[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 --- datapath-windows/ovsext/Flow.c | 50 +++--- 1 file changed, 13 insertions(+), 37 deletions(-) diff --git a/datapath-windows/ovsext/Flow

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 sev

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

2014-10-13 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- 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 --- a/include/openflow/nicira-ext.h +++ b/include/openflo

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 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 we should keep the

[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 RFC 2/2] openvswitch: Userspace tunneling.

2014-10-13 Thread Jarno Rajahalme
On Oct 13, 2014, at 2:32 PM, Pravin Shelar 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) { >>> +dp_n

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 o

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 > Acked-by: Ben Pfaff Compiling on i386, I get: ../lib/hash.c:77:

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 sc

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 > Signed-off-by: Dennis Flynn This seems pretty reasonable except that I'd much prefer to delete all the #ifdefs. ___ dev mailing list dev@openvswitch.o

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 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 Pa

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 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 Shelar wrote:

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 > Acked-by: Nithin Raju > Tested-by: Nithin Raju

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 > >

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

[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 F

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 Rajahalm

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 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 tunneling patch.

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

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

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 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 the caller doe

[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 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 download

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 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 > wrote: > >> This patch adds 802.1ad support to OV

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 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 has been tested in VMs an

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 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. > Acked-by: An

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 On Fri, Oct 10, 2014 at 7:38 PM, Pravin Shelar wrote: > On Fri, Oct 10, 2014 at 5:47 PM, Andy Zho

[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 so

[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 Debi

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: http://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 Sign

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 Tested-by: Alin Gabriel Serdean -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 Subiect: [ovs-dev] [PATCH 3/6] datapath-windows: remove vport from l

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

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean Tested-by: Alin Gabriel Serdean -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: Samuel Ghinet Subiect: [ovs-dev] [PATCH 2/6] datapath-windows: A

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 Tested-by: Alin Gabriel Serdean -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 Subiect: [ovs-dev] [PATCH 4/6] datpath-windows: pass NDIS_RWL_AT_DIS

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 Tested-by: Alin Gabriel Serdean -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 Subiect: [ovs-dev] [PATCH 5/6] datapath-windows: delete ports from p

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

2014-10-13 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean Tested-by: Alin Gabriel Serdean -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: Samuel Ghinet Subiect: [ovs-dev] [PATCH 1/6] datapath-windows: Ad

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 Tested-by: Alin Gabriel Serdean -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 Subiect: [ovs-dev] [PATCH 6/6] datapath-windows: loop iterator fixes

[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 jobs/vacancie

[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 jobs/vacancie

[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 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 kernel and OVS pa

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 From: Hiteshi Madan/DEL/TCS Date: 09/24/