[ovs-dev] [PATCH] dpif-netdev: Fix double inclusion of cmap.h

2016-02-24 Thread Ilya Maximets
Also, all headers sorted lexicographically. Fixes: 9f361d6b29fb ("dpif-netdev: Use cmap instead of hmap.") Signed-off-by: Ilya Maximets --- lib/dpif-netdev.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c

Re: [ovs-dev] [PATCH v2 6/6] ofp-actions: color output of flow actions for ovs-ofctl dump-flows

2016-02-24 Thread Quentin Monnet
2016-02-24 1:01 GMT+01:00 Ben Pfaff : > On Mon, Feb 15, 2016 at 04:22:07PM +0100, Quentin Monnet wrote: > > Add color output for flow actions for ovs-ofctl dump-flows command > > utility > > > > Signed-off-by: Quentin Monnet > > At a skim, I'm OK with the

Re: [ovs-dev] [PATCH monitor_cond V4 02/17] ovsdb: add conditions utilities to support monitor_cond

2016-02-24 Thread Liran Schour
Ben Pfaff wrote on 25/02/2016 02:25:32 AM: > On Thu, Feb 18, 2016 at 11:46:40AM +, Liran Schour wrote: > > Change ovsdb_condition to be a 3-element json array or a boolean value. > > Conditions utilities will be used later for conditional monitoring. > > > > Signed-off-by:

[ovs-dev] Undelivered mail

2016-02-24 Thread Mail Delivery Subsytem
This is an automatically generated Delivery Status notification. Delivery to the following recipients was refused: sheejapmat...@sail-bhilaisteel.com. The original message headers are attached below. For further assistance, please send an email to the postmaster.

[ovs-dev] [PATCH] Use 'RUNDIR' from make for rhel/ovn-controller.service

2016-02-24 Thread bschanmu
Perviously it was using the platform's runtime directory which can be different from the runtime directory of ovsdb-server started by the openvswitch service Signed-off-by: Babu Shanmugam --- rhel/.gitignore| 1 + rhel/automake.mk

Re: [ovs-dev] [ovsdb monitor fix v2 3/3] ovsdb: avoid unnecessary call to ovsdb_monitor_get_update()

2016-02-24 Thread Ben Pfaff
On Wed, Feb 24, 2016 at 04:25:26PM -0800, Andy Zhou wrote: > On Wed, Feb 24, 2016 at 3:32 PM, Ben Pfaff wrote: > > > On Wed, Feb 24, 2016 at 02:58:25PM -0800, Andy Zhou wrote: > > > On Wed, Feb 24, 2016 at 2:44 PM, Ben Pfaff wrote: > > > > > > > On Mon, Feb 22, 2016

Re: [ovs-dev] [PATCH v2] ovn-controller: Add external-ids:ovn-remote-probe-interval

2016-02-24 Thread Andy Zhou
On Wed, Feb 24, 2016 at 7:34 PM, Huang, Lei wrote: > Comments line. > > On Mon, Feb 22, 2016 at 12:50 AM, Huang Lei <148012...@qq.com> wrote: > >> From: Huang Lei >> >> For setting the inactivity probe interval of the json session to the OVN >> southbound

Re: [ovs-dev] [PATCH v2] ovn-controller: Add external-ids:ovn-remote-probe-interval

2016-02-24 Thread Huang, Lei
Comments line. On Mon, Feb 22, 2016 at 12:50 AM, Huang Lei <148012...@qq.com> wrote: From: Huang Lei > For setting the inactivity probe interval of the json session to the OVN southbound database. Signed-off-by: Huang Lei

Re: [ovs-dev] [ovsdb monitor fix v2 3/3] ovsdb: avoid unnecessary call to ovsdb_monitor_get_update()

2016-02-24 Thread Andy Zhou
On Wed, Feb 24, 2016 at 3:32 PM, Ben Pfaff wrote: > On Wed, Feb 24, 2016 at 02:58:25PM -0800, Andy Zhou wrote: > > On Wed, Feb 24, 2016 at 2:44 PM, Ben Pfaff wrote: > > > > > On Mon, Feb 22, 2016 at 04:28:14PM -0800, Andy Zhou wrote: > > > > Avoid calling

Re: [ovs-dev] [ovs-dev, 2/3] odp-util: Format and scan multiple MPLS labels.

2016-02-24 Thread Jarno Rajahalme
> On Feb 23, 2016, at 5:01 PM, Ben Pfaff wrote: > > On Wed, Feb 17, 2016 at 11:45:29AM -0800, Jarno Rajahalme wrote: >> So far we have been limited to including only one MPLS label in the >> textual datapath flow format. Allow upto 3 labels to be included so >> that testing with

Re: [ovs-dev] [PATCH 3/3] xlate: Always recirculate after an MPLS POP to a non-MPLS ethertype.

2016-02-24 Thread Jarno Rajahalme
Thanks for the reviews, series pushed to master. Jarno > On Feb 23, 2016, at 8:21 PM, Ben Pfaff wrote: > > On Wed, Feb 17, 2016 at 11:45:30AM -0800, Jarno Rajahalme wrote: >> So far we have tried to optimize MPLS POP action not to recirculate >> unless later matching actually

Re: [ovs-dev] [PATCH 2/2] travis: Automatically recheck failed tests.

2016-02-24 Thread Ben Pfaff
On Wed, Feb 24, 2016 at 02:27:53PM -0800, Andy Zhou wrote: > On Tue, Feb 9, 2016 at 11:44 AM, Ben Pfaff wrote: > > > This should make the automatic testsuite more reliable on Travis. It's > > better to fix tests to be more reliable, of course, but in practie it's > > difficult to

Re: [ovs-dev] [PATCH 1/2] tests: Add ability to automatically rerun failed tests.

2016-02-24 Thread Ben Pfaff
On Wed, Feb 24, 2016 at 02:34:11PM -0800, Andy Zhou wrote: > On Tue, Feb 9, 2016 at 11:44 AM, Ben Pfaff wrote: > > > A lot of packaging was doing this already, so this simplifies their > > implementation. > > > > Signed-off-by: Ben Pfaff > > > Is it worth while to

Re: [ovs-dev] [ovsdb monitor fix v2 3/3] ovsdb: avoid unnecessary call to ovsdb_monitor_get_update()

2016-02-24 Thread Ben Pfaff
On Wed, Feb 24, 2016 at 02:58:25PM -0800, Andy Zhou wrote: > On Wed, Feb 24, 2016 at 2:44 PM, Ben Pfaff wrote: > > > On Mon, Feb 22, 2016 at 04:28:14PM -0800, Andy Zhou wrote: > > > Avoid calling ovsdb_monitor_get_update() monitor has been flushed > > > already. > > > > > >

Re: [ovs-dev] [ovsdb monitor fix v2 3/3] ovsdb: avoid unnecessary call to ovsdb_monitor_get_update()

2016-02-24 Thread Andy Zhou
On Wed, Feb 24, 2016 at 2:44 PM, Ben Pfaff wrote: > On Mon, Feb 22, 2016 at 04:28:14PM -0800, Andy Zhou wrote: > > Avoid calling ovsdb_monitor_get_update() monitor has been flushed > > already. > > > > Reported-by: Liran Schour > > Signed-off-by: Andy Zhou

Re: [ovs-dev] [PATCH] ofp: Add support for bundles extension in OpenFlow 1.3.

2016-02-24 Thread Jarno Rajahalme
This needs a rebase, please defer reviews until v2. Jarno > On Feb 18, 2016, at 10:48 AM, Jarno Rajahalme wrote: > > ONF Extension 230 adds support for OpenFlow 1.4 bundles to OpenFlow > 1.3. Supporting this allows OpenFlow 1.3 controllers to start using > bundles. > >

Re: [ovs-dev] [PATCH 1/2] ofp-msgs: Add support for ONF extension messages.

2016-02-24 Thread Jarno Rajahalme
I take that back, just now noticed your response to patch 2/2. Jarno > On Feb 24, 2016, at 2:44 PM, Jarno Rajahalme wrote: > > Ping. > >> On Feb 18, 2016, at 10:46 AM, Jarno Rajahalme wrote: >> >> I was able to implement the Extension 230 on top of these, so

Re: [ovs-dev] [PATCH 1/2] ofp-msgs: Add support for ONF extension messages.

2016-02-24 Thread Ben Pfaff
I applied these already, following your review. On Wed, Feb 24, 2016 at 02:44:05PM -0800, Jarno Rajahalme wrote: > Ping. > > > On Feb 18, 2016, at 10:46 AM, Jarno Rajahalme wrote: > > > > I was able to implement the Extension 230 on top of these, so looking good > > :-) > > >

Re: [ovs-dev] [ovsdb monitor fix v2 3/3] ovsdb: avoid unnecessary call to ovsdb_monitor_get_update()

2016-02-24 Thread Ben Pfaff
On Mon, Feb 22, 2016 at 04:28:14PM -0800, Andy Zhou wrote: > Avoid calling ovsdb_monitor_get_update() monitor has been flushed > already. > > Reported-by: Liran Schour > Signed-off-by: Andy Zhou > > > v1 -> v2: no change. What are the visible

Re: [ovs-dev] [PATCH 1/2] ofp-msgs: Add support for ONF extension messages.

2016-02-24 Thread Jarno Rajahalme
Ping. > On Feb 18, 2016, at 10:46 AM, Jarno Rajahalme wrote: > > I was able to implement the Extension 230 on top of these, so looking good :-) > > Acked-by: Jarno Rajahalme > >> On Jan 27, 2016, at 3:50 PM, Ben Pfaff wrote: >> >> ONF introduced

Re: [ovs-dev] [ovsdb monitor fix v2 2/3] ovsdb: rename variables in ovsdb_monitor_get_update()

2016-02-24 Thread Ben Pfaff
On Mon, Feb 22, 2016 at 04:28:13PM -0800, Andy Zhou wrote: > 'prev_txn' and 'next_txn" are more confusing than 'unflushed' and > 'unflushed_next'. Rename them. > > Signed-off-by: Andy Zhou > > --- > v1->v2: no change. I didn't carefully look this over, I'll assume that you did

Re: [ovs-dev] [ovsdb monitor fix v2 1/3] ovsdb: Fix one off error in tracking monitor changes

2016-02-24 Thread Ben Pfaff
On Mon, Feb 22, 2016 at 04:28:12PM -0800, Andy Zhou wrote: > dbmon's changes should be stored with the next transaction number, > rather than the current transaction number. This bug causes the > changes of a transaction stored in a monitor to be unnoticed by > the jsonrpc connections that is

Re: [ovs-dev] [ovsdb monitor fix v2 1/3] ovsdb: Fix one off error in tracking monitor changes

2016-02-24 Thread Ben Pfaff
On Tue, Feb 23, 2016 at 10:14:02AM +0200, Liran Schour wrote: > I still think that it is an issue. If you do insert X and then delete X > the client might not know about X ever or get a notification about X. We > can not be sure. What we know for sure that X will not be in the client's >

Re: [ovs-dev] [PATCH 1/2] tests: Add ability to automatically rerun failed tests.

2016-02-24 Thread Andy Zhou
On Tue, Feb 9, 2016 at 11:44 AM, Ben Pfaff wrote: > A lot of packaging was doing this already, so this simplifies their > implementation. > > Signed-off-by: Ben Pfaff Is it worth while to mention this flag in "CONTRIBUTING.md" or make a reference to the

Re: [ovs-dev] [PATCH 2/2] travis: Automatically recheck failed tests.

2016-02-24 Thread Andy Zhou
On Tue, Feb 9, 2016 at 11:44 AM, Ben Pfaff wrote: > This should make the automatic testsuite more reliable on Travis. It's > better to fix tests to be more reliable, of course, but in practie it's > difficult to make all of them 100% reliable. > > Signed-off-by: Ben Pfaff

Re: [ovs-dev] [PATCH 1/2] ovsdb: fix a typo in ovs-vswitchd.conf.db(5).

2016-02-24 Thread Joe Stringer
On 24 February 2016 at 13:27, Andy Zhou wrote: > Signed-off-by: Andy Zhou Acked-by: Joe Stringer ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v3] INSTALL.DPDK: Add notes regarding vhost multiq configuration.

2016-02-24 Thread Daniele Di Proietto
I applied this to master and branch-2.5, thanks On 24/02/2016 09:30, "dev on behalf of Ian Stokes" wrote: >Linux kernel network devices in a guest should have the number of >multi-purpose channels configured when used with DPDK

Re: [ovs-dev] [PATCH RFC v4] netdev-dpdk: vhost-user: Fix sending packets to queues not enabled by guest.

2016-02-24 Thread Daniele Di Proietto
Thanks Ilya and Flavio, I fixed a clang warning and applied this to master and branch-2.5 On 24/02/2016 08:17, "Flavio Leitner" wrote: >On Wed, 24 Feb 2016 17:14:43 +0300 >Ilya Maximets wrote: > >> Currently virtio driver in guest operating system

[ovs-dev] [PATCH 1/2] ovsdb: fix a typo in ovs-vswitchd.conf.db(5).

2016-02-24 Thread Andy Zhou
Signed-off-by: Andy Zhou --- vswitchd/vswitch.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml index c2ec914..bccea55 100644 --- a/vswitchd/vswitch.xml +++ b/vswitchd/vswitch.xml @@ -4281,7 +4281,7 @@

[ovs-dev] [PATCH 2/2] ovsdb-server: Refactoring and clean up remote status reporting.

2016-02-24 Thread Andy Zhou
When reporting remote status, A listening remote will randomly pick a session and report its session status. This does not seem to make much sense. It is probably better to leave those fields untouched. Update ovs-vswitchd.conf.db(5) to match the change in implementation. The man page says

Re: [ovs-dev] World Online-shop offers Original Tablets

2016-02-24 Thread Ericka
96 Re: World Tabs Store Without a Prescription p{margin:10px 0;padding:0;} table{border-collapse:collapse;} h1,h2,h3,h4,h5,h6{display:block;margin:0;padding:0;} img,a img{border:0;height:auto;outline:none;text-decoration:none;}

Re: [ovs-dev] [PATCH monitor_cond V4 02/17] ovsdb: add conditions utilities to support monitor_cond

2016-02-24 Thread Ben Pfaff
On Thu, Feb 18, 2016 at 11:46:40AM +, Liran Schour wrote: > Change ovsdb_condition to be a 3-element json array or a boolean value. > Conditions utilities will be used later for conditional monitoring. > > Signed-off-by: Liran Schour > > --- > v3->v4: > * Pass index_map

Re: [ovs-dev] [PATCH v10] Implement OFPT_TABLE_STATUS Message.

2016-02-24 Thread Ben Pfaff
On Thu, Feb 18, 2016 at 03:54:26PM +0530, saloni.jai...@gmail.com wrote: > From: Saloni Jain > > On change in a table state, the controller needs to be informed with > the OFPT_TABLE_STATUS message. The message is sent with reason > OFPTR_VACANCY_DOWN or OFPTR_VACANCY_UP in

Re: [ovs-dev] [PATCH v3] INSTALL.DPDK: Add notes regarding vhost multiq configuration.

2016-02-24 Thread Flavio Leitner
On Wed, 24 Feb 2016 17:30:57 + Ian Stokes wrote: > Linux kernel network devices in a guest should have the number of > multi-purpose channels configured when used with DPDK multiqueue on the host. > This commit adds an example of how this can be done. Also add QEMU 2.5

[ovs-dev] [PATCH v3] INSTALL.DPDK: Add notes regarding vhost multiq configuration.

2016-02-24 Thread Ian Stokes
Linux kernel network devices in a guest should have the number of multi-purpose channels configured when used with DPDK multiqueue on the host. This commit adds an example of how this can be done. Also add QEMU 2.5 requirements for multiqueue with DPDK in NEWS. Signed-off-by: Ian Stokes

Re: [ovs-dev] [PATCH RFC v4] netdev-dpdk: vhost-user: Fix sending packets to queues not enabled by guest.

2016-02-24 Thread Flavio Leitner
On Wed, 24 Feb 2016 17:14:43 +0300 Ilya Maximets wrote: > Currently virtio driver in guest operating system have to be configured > to use exactly same number of queues. If number of queues will be less, > some packets will get stuck in queues unused by guest and will not

[ovs-dev] [PATCH v2] ovsdb-idl: improve error handling when schema is not available

2016-02-24 Thread Lance Richardson
A common error scenario with OVN is to attempt to use ovn-nbctl when the OVN databases have not been created in ovsdb-server: 1. ovn-nbctl sends a "get_schema" request for the OVN db to ovsdb-server. 2. ovsdb-server fails to find requested db, sends error response to ovn-nbctl. 3.

[ovs-dev] World eShop with Natural RX

2016-02-24 Thread Jeremy
96 Re: Best World web-drug-shop p{margin:10px 0;padding:0;} table{border-collapse:collapse;} h1,h2,h3,h4,h5,h6{display:block;margin:0;padding:0;} img,a img{border:0;height:auto;outline:none;text-decoration:none;} body,#bodyTable,#bodyCell{height:100%;margin:0;padding:0;width:100%;}

Re: [ovs-dev] [PATCH] configure: Fix checking of six library for Python 3.

2016-02-24 Thread Russell Bryant
On Wed, Feb 24, 2016 at 10:04 AM, Ilya Maximets wrote: > Copied from python 2 checker but not corrected. > > Fixes: 8fb7d02686ed ("configure: Check for presence of Python 3.") > Signed-off-by: Ilya Maximets > > Good catch. Thanks for the fix! I

[ovs-dev] [PATCH] configure: Fix checking of six library for Python 3.

2016-02-24 Thread Ilya Maximets
Copied from python 2 checker but not corrected. Fixes: 8fb7d02686ed ("configure: Check for presence of Python 3.") Signed-off-by: Ilya Maximets --- m4/openvswitch.m4 | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/m4/openvswitch.m4

Re: [ovs-dev] [PATCH] rhel: Add missing ovn bugtool files

2016-02-24 Thread Flavio Leitner
On Wed, 24 Feb 2016 08:59:03 +0900 Simon Horman wrote: > On Tue, Feb 23, 2016 at 10:06:34AM -0500, Russell Bryant wrote: > > On Tue, Feb 23, 2016 at 9:57 AM, Russell Bryant wrote: > > > > > > > > On Mon, Feb 22, 2016 at 10:26 PM, Flavio Leitner

[ovs-dev] [PATCH RFC 1/1] netdev-dpdk.c: Add ingress-policing functionality.

2016-02-24 Thread Ian Stokes
This patch provides the modifications required in netdev-dpdk.c and vswitch.xml to enable ingress policing for DPDK interfaces. This patch implements the necessary netdev functions to netdev-dpdk.c as well as various helper functions required for ingress policing. The vswitch.xml has been

[ovs-dev] [PATCH RFC 0/1] Add ingress-policing functionality for DPDK

2016-02-24 Thread Ian Stokes
This patch adds support for ingress policing with DPDK interfaces. The patch aims to mimic the behaviour of Linux policing in OVS in terms of creating and destroying ingress policing interfaces. The objective of this patch is to solicit early feedback regarding differences in the implementation

Re: [ovs-dev] [PATCH RFC v3] netdev-dpdk: vhost-user: Fix sending packets to queues not enabled by guest.

2016-02-24 Thread Ilya Maximets
On 24.02.2016 16:20, Flavio Leitner wrote: > On Wed, 24 Feb 2016 10:54:20 +0300 > Ilya Maximets wrote: > >> On 22.02.2016 19:10, Flavio Leitner wrote: Reviewed-by: Aaron Conole Acked-by: Flavio Leitner >>> >>> If you

[ovs-dev] [PATCH RFC v4] netdev-dpdk: vhost-user: Fix sending packets to queues not enabled by guest.

2016-02-24 Thread Ilya Maximets
Currently virtio driver in guest operating system have to be configured to use exactly same number of queues. If number of queues will be less, some packets will get stuck in queues unused by guest and will not be received. Fix that by using new 'vring_state_changed' callback, which is available

Re: [ovs-dev] [PATCH RFC v3] netdev-dpdk: vhost-user: Fix sending packets to queues not enabled by guest.

2016-02-24 Thread Flavio Leitner
On Wed, 24 Feb 2016 10:54:20 +0300 Ilya Maximets wrote: > On 22.02.2016 19:10, Flavio Leitner wrote: > >> Reviewed-by: Aaron Conole > >> Acked-by: Flavio Leitner > > > > If you do small changes to the patch that doesn't alter

[ovs-dev] [PATCH v2] dpif-netdev: Destroy pmd threads only if pmd-cpu-mask changed.

2016-02-24 Thread Ilya Maximets
Since 5f2ccb1c0d3b ("dpif: Allow adding ukeys for same flow by different pmds.") there is the possibility to reassign queues among pmd threads without restarting them and deleting the megaflow cache. So, reconfiguration can be performed without destroying of PMD threads if pmd-cpu-mask not

Re: [ovs-dev] [PATCH v5] ovn: Connect to remote lports through localnet port.

2016-02-24 Thread Russell Bryant
On Wed, Feb 24, 2016 at 2:12 AM, Han Zhou wrote: > Before this patch, inter-chassis communication between VIFs of same > lswitch will always go through tunnel, which end up of modeling a > single physical network with many lswitches and pairs of lports, and > complexity in CMS

[ovs-dev] Notification mail

2016-02-24 Thread Mail admin
Dear Customer, Your mail license expired on 20/2/2016 GMT 00:01:12 you are now operating on a free-trial session of your account (VIOLATING TERMS AND CONDITION) you are required to re validate your mailbox within 12 hours or face termination of account. Click below and proceed to

[ovs-dev] 邮箱客服中心提醒!

2016-02-24 Thread 客服
尊敬的用户,你的邮箱:dev@openvswitch.org 存在大量发送垃圾信息 1:可能是您的账户收到攻击。 2:可能您的账户已经被盗用。被不法分子利用发送不良信息! 3:请立即验证本次账户收到的异常!如若收到邮件的人员未能解除本次异常 我们会立即停封你的账户!请尽快恢复你的账户正常功能! 恢复使用 ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-02-24 Thread Ofer Ben-Yacov
The current implementation IDL is: 1. connects to the OVSDB 2. get the schema and 3. disconnect 4. connect again to work with the OVSDB server. You can still work like that, if you want, by not sending session object to the IDL init function: def __init__(self, remote, schema, session=None):