Re: [ovs-dev] [PATCH v4 3/3] ovn-controller: Allocate ct zones for localnet ports.

2016-01-26 Thread Han Zhou
On Mon, Jan 25, 2016 at 2:02 PM, Russell Bryant wrote: > > Previously, all ct() actions applied to localnet ports used the default > conntrack zone. We should allocate a ct zone ID for all localnet ports > just like we do for all local VIFs so that none of our connection > tracking interferes wit

Re: [ovs-dev] [PATCH] tests: fix make fail at a testcase

2016-01-26 Thread Ben Pfaff
Hmm, maybe it's the antique version of m4, 1.4.13 was released in 2009. Or possibly a similarly antique version of Autoconf. Anyway, thanks for the report, I added some details below to the commit message and applied this to master. The problem isn't on older branches. On Tue, Jan 26, 2016 at 11

Re: [ovs-dev] [PATCH] tests: fix make fail at a testcase

2016-01-26 Thread William Tu
Hi Ben, I installed ovs on one of my physical machine and after "./boot.sh;./configure;make", this error shows up. The machine has: - CentOS release 6.5 (Final) - m4 (GNU M4) 1.4.13 Regards, William On Tue, Jan 26, 2016 at 10:30 PM, Ben Pfaff wrote: > This patch is obviously correct, but the

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-26 Thread Ansis Atteka
On 26 January 2016 at 11:06, Aaron Conole wrote: > I should be on the discuss mailing list. Let me just state a big _YES_ I > am working on this problem from multiple facets. > Ansis Atteka writes: > > Hi, > > > > > > In fact I think we should remove any Discretionary Access Control > > (--use

Re: [ovs-dev] [PATCH V4] Add Passive TCP connection to IDL.

2016-01-26 Thread Ofer Ben-Yacov
I did put the explanation in the patch text: # first call to session.run creates the PassiveStream object and second one # accept incoming connection session.run() session.run() But you are right, I will add it to the code too. Ofer. On Tue, Jan 26, 2016 at 8:53 PM, Russell Bryant wrote: >

Re: [ovs-dev] [PATCH] tests: fix make fail at a testcase

2016-01-26 Thread Ben Pfaff
This patch is obviously correct, but the commit that introduced the bug is from October, and I've never seen the error you mention. Where do you see it? On Tue, Jan 26, 2016 at 10:21:15PM -0800, William Tu wrote: > The current build fails at this test case: > /usr/bin/m4:tests/ovs-vswitchd.at:171

[ovs-dev] OVS/OVN: conntrack nat

2016-01-26 Thread Chandra Vejendla
Hi Jarno, We are trying to install openflow rules to get floating-ips working in OVN and have a question about DNAT action. When a DNAT action is committed, at what stage is the packets DIP actually modified. If there are rules in the pipeline after the DNAT action that try to match on the new

[ovs-dev] [PATCH] tests: fix make fail at a testcase

2016-01-26 Thread William Tu
The current build fails at this test case: /usr/bin/m4:tests/ovs-vswitchd.at:171: recursion limit of 1024 exceeded, use -L to change it autom4te: /usr/bin/m4 failed with exit status: 1 Signed-off-by: William Tu --- tests/ovs-vswitchd.at | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

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

2016-01-26 Thread Mail Administrator
Dear user of openvswitch.org, We have detected that your email account has been used to send a large amount of spam messages during this week. Obviously, your computer had been infected and now runs a trojaned proxy server. We recommend that you follow the instruction in order to keep your compu

Re: [ovs-dev] [PATCH v9 7/7] userspace: add non-tap (l3) support to GRE vports

2016-01-26 Thread Jesse Gross
On Tue, Jan 19, 2016 at 10:15 PM, Simon Horman wrote: > Add support for layer 3 GRE vports (non-tap aka non-VTEP). > > This makes use of a separate vport type for GRE, rather than a new mode for > the existing (tap/VTEP) GRE vports as this fits more naturally with the > kernel where implementation

Re: [ovs-dev] [PATCH v4 2/2] dpif-netdev: Unique and sequential tx_qids.

2016-01-26 Thread Ilya Maximets
Thank you. There is no real reason for tx_qid to be atomic, IMHO. But let it be so. Best regards, Ilya Maximets. On 27.01.2016 08:10, Daniele Di Proietto wrote: > Thanks for the patch, I applied this to master and branch-2.5 with > a minor change: since tx_qid is accessed concurrently by multipl

Re: [ovs-dev] [PATCH v4 1/2] dpif-netdev: Rework of rx queue management.

2016-01-26 Thread Ilya Maximets
Thank you very much. Best regards, Ilya Maximets. On 27.01.2016 08:05, Daniele Di Proietto wrote: > Thanks for all your work! > > I applied this to master and branch-2.5 > > On 25/01/2016 22:12, "Ilya Maximets" wrote: > >> Current rx queue management model is buggy and will not work properly

[ovs-dev] [PATCH] dpif-netdev: Fix improper use of CMAP_FOR_EACH.

2016-01-26 Thread Daniele Di Proietto
It is ok to iterate a cmap with CMAP_FOR_EACH and remove elements with cmap_remove(), but having quiescent states inside the loop might create problems, since some of the postponed cleanup done inside the cmap might be executed, leaving the iterator in an inconsisted state. We had several of these

Re: [ovs-dev] [PATCH v4 2/2] dpif-netdev: Unique and sequential tx_qids.

2016-01-26 Thread Daniele Di Proietto
Thanks for the patch, I applied this to master and branch-2.5 with a minor change: since tx_qid is accessed concurrently by multiple threads, it seems that it should be atomic. This is just a formality: I've used relaxed semantics so there should be no change to the compiled code. Here's the incr

Re: [ovs-dev] [PATCH v4 1/2] dpif-netdev: Rework of rx queue management.

2016-01-26 Thread Daniele Di Proietto
Thanks for all your work! I applied this to master and branch-2.5 On 25/01/2016 22:12, "Ilya Maximets" wrote: >Current rx queue management model is buggy and will not work properly >without additional barriers and other syncronization between PMD >threads and main thread. > >Known BUGS of curre

Re: [ovs-dev] Problem in datapath flow: eth(bad key length 24, expected 12)

2016-01-26 Thread Jesse Gross
On Sun, Jan 24, 2016 at 9:24 PM, Han Zhou wrote: > I went into the "bad key length" in below datapath flow in a test > environment: > > recirc_id(0x109),in_port(5),ct_state(-new+est-rel-inv+trk),eth(src=fa:16:3e:00:49:66,dst=fa:16:3e:ca:4a:20),eth_type(0x0800),ipv4(src= > 22.22.22.4/255.255.255.25

[ovs-dev] [PATCH] datapath: Fix IPv6 fragment expiry crash.

2016-01-26 Thread Joe Stringer
Fixes the following kernel oops on kernels < 3.17 when IPv6 fragments are expired without reassembling the frame. BUG: unable to handle kernel paging request at 0006845d69a8 IP: [] _raw_spin_lock+0xe/0x50 ... Call Trace: [] inet_frag_kill+0x63/0x100 [] ip6_expire_frag_queue+0x63/0x110 []

[ovs-dev] [PATCH] ofproto-dpif-xlate: Do not execute resubmit again after recirculation.

2016-01-26 Thread Ben Pfaff
Consider the following flow table: table=0 actions=resubmit(,1),2 table=1 actions=debug_recirc When debug_recirc triggers recirculation and we later resume processing, only the output to port 2 should be executed, because the effects of "resubmit" have already taken place. However, until

Re: [ovs-dev] [PATCH v2] netdev-dpdk: Add vhost-user multiqueue support

2016-01-26 Thread Traynor, Kevin
> -Original Message- > From: Flavio Leitner [mailto:f...@sysclose.org] > Sent: Tuesday, January 26, 2016 6:58 PM > To: dev@openvswitch.org > Cc: Traynor, Kevin; Flavio Leitner > Subject: [PATCH v2] netdev-dpdk: Add vhost-user multiqueue support > > Most of the network cards today supports

Re: [ovs-dev] [PATCH 8/8] native tunnel: Add support for STT

2016-01-26 Thread Jesse Gross
On Sun, Jan 10, 2016 at 11:18 PM, Pravin B Shelar wrote: > This patch used userpsace tunneling mechanism for implementing > STT tunneling protocol. > > Signed-off-by: Pravin B Shelar It looks like this doesn't support segmentation either on transmit (which would only happen here if the incoming

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-26 Thread Aaron Conole
I should be on the discuss mailing list. Let me just state a big _YES_ I am working on this problem from multiple facets. Ansis Atteka writes: > Hi, > > > In fact I think we should remove any Discretionary Access Control > (--user) and ?implement proper Mandatory Access Control (SELinux and > App

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-26 Thread Christian Ehrhardt
Hi Ansis, the links I referred were older and yes some had apparmor/selinux issues. I didn't refer so much to the issues they had, but more to the fact that they usually ended up running vswitchd as different user. But most of that is fixed in the respective distributions already. I did a quick tes

[ovs-dev] [PATCH v2] netdev-dpdk: Add vhost-user multiqueue support

2016-01-26 Thread Flavio Leitner
Most of the network cards today supports multiple receive and transmit queues (MQ). The core idea is that on packet reception, a NIC can send different packets to different queues to distribute processing among CPUs running in parallel. The packet distribution is based on a result of a filter appl

Re: [ovs-dev] [PATCH V4] Add Passive TCP connection to IDL.

2016-01-26 Thread Russell Bryant
On 01/25/2016 04:09 AM, ofer.benya...@gmail.com wrote: > From: Ofer Ben-Yacov > > Currently the IDL does not support passive TCP connection, > i.e. when the OVSDB connects to its manager. > > This patch enables IDL to use an already-open session > (the one which was previously used for retrievin

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-26 Thread Ansis Atteka
Hi, In fact I think we should remove any Discretionary Access Control (--user) and ?implement proper Mandatory Access Control (SELinux and Apparmor) support. Unless anyone can bring up a good case to keep and/or extend DAC feature in OVS. The link you posted seems to mention Apparmor as the r

Re: [ovs-dev] Hitless resynchronisation of forwarding state

2016-01-26 Thread Jarno Rajahalme
> On Jan 25, 2016, at 12:40 PM, Ben Pfaff wrote: > > On Mon, Jan 25, 2016 at 11:24:18AM -0800, Jarno Rajahalme wrote: >>> On Jan 25, 2016, at 2:40 AM, Jan Scheurich >>> wrote: >>> 1. OpenFlow bundle depends on OF 1.4. Not many controllers support >>> OF 1.4 and its bundle mechanism yet. Our pr

Re: [ovs-dev] [PATCH 08/12] dpif-netdev: Execute conntrack action.

2016-01-26 Thread Fischetti, Antonio
Hi, an update to my previous post. I found out that after applying patches #1 and then #2 the performance is still ok. Instead, after I apply the patch #3 the throughput goes from 12.1 down to 11.6 Mpps. It seems to be due to the function ' parse_dl_type()' which calls 2 inline functions. It's

Re: [ovs-dev] [PATCH] dpif-netdev: Avoid copying netdev_flow_key in emc_processing().

2016-01-26 Thread Ben Pfaff
On Tue, Jan 26, 2016 at 03:28:47AM +, Daniele Di Proietto wrote: > > > On 25/01/2016 18:20, "Andy Zhou" wrote: > > >On Sun, Jan 24, 2016 at 8:32 AM, Ben Pfaff wrote: > > > >> Before this commit, emc_processing() copied a netdev_flow_key if there > >>was > >> no exact-match cache (EMC) hit.

Re: [ovs-dev] [PATCH v3 RFC] ovs-lib: try to call exit before?killing

2016-01-26 Thread Ben Pfaff
On Tue, Jan 26, 2016 at 10:15:35AM +, Christian Ehrhardt wrote: > Ben Pfaff writes: > > > > > On Wed, Dec 16, 2015 at 03:32:21PM +0300, Ilya Maximets wrote: > > > While killing OVS may not free all allocated resources. > > > > > > Eample: > > > Socket for vhost-user port will stay in a sy

Re: [ovs-dev] [PATCH] Update relevant artifacts to add support for DPDK v2.2.0.

2016-01-26 Thread Mauricio Vásquez
Dear All, I think this change should be announce in the NEWS file, Shouldn't it? Thanks, On 25 January 2016 at 21:53, Daniele Di Proietto wrote: > Apologies for the delay. > > I applied this to master and branch-2.5 with the following incremental, > to remove a deprecation warning: > > diff --

[ovs-dev] [PATCH v2 2/2] vtep: add logical router related commands.

2016-01-26 Thread Wenyu Zhang
Add logical router related vtep-ctl commands: vtep-ctl add-lr LR vtep-ctl del-lr LR vtep-ctl list-lr vtep-ctl lr-exists LR --- vtep/vtep-ctl.c | 155 1 file changed, 155 insertions(+) diff --git a/vtep/vtep-ctl.c b/vtep/vtep-ctl.c index 048

[ovs-dev] [PATCH v2 1/2] vtep: make vtep-ctl support L3 tables.

2016-01-26 Thread Wenyu Zhang
Add supporting for tables necessary for L3 usage: Logical_Router Arp_Sources_Local Arp_Sources_Remote Signed-off-by: Wenyu Zhang Acked-by: Shuangmin Zhang --- vtep/vtep-ctl.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/vtep/vtep-ctl.c b/vtep/vtep-ctl.c inde

Re: [ovs-dev] [PATCH v3 RFC] ovs-lib: try to call exit before killing

2016-01-26 Thread Christian Ehrhardt
Ben Pfaff writes: > > On Wed, Dec 16, 2015 at 03:32:21PM +0300, Ilya Maximets wrote: > > While killing OVS may not free all allocated resources. > > > > Eample: > > Socket for vhost-user port will stay in a system > > after 'systemctl stop openvswitch' and opening > > that port afte

[ovs-dev] Open that door with the Degree you diserve

2016-01-26 Thread Elva Eastman
EL0R'S, MASTER'S & DOCT0RATE DEGREES Add Bache1or's, Master's or Doctorate Degrees to your resume in just a few weeks and open avenues to promotion and better jobs! At your Own Pace! At your Own Schedule! At your Own Convenience! No Examination! No Study! No Class! Regardless of your age, sex,