Gurucharan Shetty | 1 Jul 12:23 2016

[PATCH 1/2] ovn-nbctl: Ability to update a route's output port.

The command 'lr-route-add' currently cannot update the output

Signed-off-by: Gurucharan Shetty <guru <at>>
 ovn/utilities/ovn-nbctl.c |  4 ++++
 tests/        | 10 +++++++++-
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/ovn/utilities/ovn-nbctl.c b/ovn/utilities/ovn-nbctl.c
index 345647a..abeba0b 100644
--- a/ovn/utilities/ovn-nbctl.c
+++ b/ovn/utilities/ovn-nbctl.c
 <at>  <at>  -1383,6 +1383,10  <at>  <at>  nbctl_lr_route_add(struct ctl_context *ctx)
         nbrec_logical_router_static_route_set_ip_prefix(route, prefix);
         nbrec_logical_router_static_route_set_nexthop(route, next_hop);
+        if (ctx->argc == 5) {
+            nbrec_logical_router_static_route_set_output_port(route,
+                                                              ctx->argv[4]);
+        }
diff --git a/tests/ b/tests/
index d2d74b6..0c756ed 100644
--- a/tests/
+++ b/tests/
 <at>  <at>  -388,6 +388,14  <at>  <at>  IPv4 Routes
(Continue reading)

William Tu | 1 Jul 21:43 2016

[PATCHv2] ofproto-dpif-sflow: Add snaplen for sample action and sFlow.

This patch adds a 'snaplen' field in sample action (nx_action_sample2).
Currently, sample action is used by sFlow and IPFIX. For IPFIX, nothing
is changed.  For sFlow configuration, the patch translates header=N to
a sample action with snaplen=N, then the snaplen=N translates to trunc(N)
in kernel datapath.  Thus, only N bytes instead of full-packet size will
be copied from kernel to userspace, saving the copying overhead.

Also, the patch parses OVS_PACKET_ATTR_LEN from nlattr to upcall related
structures so the sFlow receiver knows the original packet size before it
is truncated.

Signed-off-by: William Tu <u9012063 <at>>
- Add snaplen to nx_action_sample2
- Since truncate then userspace action is supported in userspace datapath,
  ,commit aaca4fe0ce9e (ofp-actions: Add truncate action.), remove
  datapath-specific implementation and testcase.
 include/openvswitch/ofp-actions.h |  1 +
 lib/dpif-netlink.c                |  4 +++-
 lib/dpif.h                        |  2 +-
 lib/ofp-actions.c                 | 12 +++++++++++-
 ofproto/ofproto-dpif-sflow.c      | 16 +++++++++++++---
 ofproto/ofproto-dpif-sflow.h      |  2 ++
 ofproto/ofproto-dpif-upcall.c     | 21 ++++++++++++++++-----
 ofproto/ofproto-dpif-xlate.c      | 22 ++++++++++++++++++++--
 tests/              |  4 ++--
(Continue reading)

Pravin B Shelar | 1 Jul 20:37 2016

[PATCH branch-2.5 1/2] datapath: backport: Set mark and labels before confirming.

Upstream commit:
    commit 1c1779fa54b2a9d4e1de990095d790d64b9e00a1
    Author: Jarno Rajahalme <jarno <at>>
    Date:   Tue Jun 21 14:59:37 2016 -0700

    openvswitch: Set mark and labels before confirming.

    Set conntrack mark and labels right before committing so that
    the initial conntrack NEW event has the mark and labels.

    Signed-off-by: Jarno Rajahalme <jarno <at>>
    Acked-by: Joe Stringer <joe <at>>
    Signed-off-by: David S. Miller <davem <at>>

Signed-off-by: Pravin B Shelar <pshelar <at>>
 datapath/conntrack.c | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/datapath/conntrack.c b/datapath/conntrack.c
index 795ed91..1ef6828 100644
--- a/datapath/conntrack.c
+++ b/datapath/conntrack.c
 <at>  <at>  -484,9 +484,6  <at>  <at>  static int ovs_ct_commit(struct net *net, struct sw_flow_key *key,
 	err = __ovs_ct_lookup(net, key, info, skb);
 	if (err)
 		return err;
-	if (nf_conntrack_confirm(skb) != NF_ACCEPT)
-		return -EINVAL;
(Continue reading)

Alin Serdean | 1 Jul 20:36 2016

[PATCH] build: Skip datapath subdir on Windows

This patch makes automake skip the 'datapath' folder under Windows.

The main reason is that msys does not have an implementation of rev
used by:

and can be seen on:

Another solution would be to skip COMPAT_GET_FUNCTIONS on windows or
to modify the use of 'rev'.

Signed-off-by: Alin Gabriel Serdean <aserdean <at>>
--- | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ b/
index 8cb8523..c251ef5 100644
--- a/
+++ b/
 <at>  <at>  -7,7 +7,9  <at>  <at> 

 AUTOMAKE_OPTIONS = foreign subdir-objects
+if !WIN32
 SUBDIRS = datapath

(Continue reading)

William Tu | 1 Jul 18:45 2016

[PATCH] system-traffic: Remove datapath specific tests and macro.

We generally try to keep the testsuite independent of the underlying
datapath. This patch removes the datapath-specific tests and macros.

Signed-off-by: William Tu <u9012063 <at>>
 tests/      |  7 -------
 tests/          | 13 ++-----------
 tests/ |  7 -------
 3 files changed, 2 insertions(+), 25 deletions(-)

diff --git a/tests/ b/tests/
index a3e4dd7..cee0510 100644
--- a/tests/
+++ b/tests/
 <at>  <at>  -66,10 +66,3  <at>  <at>  m4_define([CHECK_CONNTRACK],
      on_exit 'ovstest test-netlink-conntrack flush'
-# Ignore the CHECK_USER_DP and execute the CHECK_KERNEL_DP
-m4_define([CHECK_KERNEL_DP], [$1])
-m4_define([CHECK_USER_DP], [])
diff --git a/tests/ b/tests/
index 252ed20..f8e7279 100644
--- a/tests/
+++ b/tests/
(Continue reading)

Paul Boca | 1 Jul 18:27 2016

[PATCH V5] windows: Added lockf function and lock PID file

If the PID file isn't locked then detects it as stale and
bails out without doing anything. Because of this lots of Python tests fail.
Also this protects the PID file from being overwritten.

I used only shared lock, in order to be compatible with Python tests,
which try to acquire the lock exclusively. On Windows if the exclusive lock
is used, than the read access is denied too for other instances of this file.

Signed-off-by: Paul-Daniel Boca <pboca <at>>
V2: No changes
V3: No changes
V4: No changes
V5: Removed this patch from Python series.
    Removed unused constants and defined 0xFFFF0000 as constant.
    Updated commit text.
 lib/daemon-windows.c | 43 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/lib/daemon-windows.c b/lib/daemon-windows.c
index 8cf0fea..4c4724b 100644
--- a/lib/daemon-windows.c
+++ b/lib/daemon-windows.c
 <at>  <at>  -18,6 +18,7  <at>  <at> 
 #include "daemon.h"
 #include "daemon-private.h"
 #include <stdio.h>
+#include <io.h>
 #include <stdlib.h>
(Continue reading)

Paul Boca | 1 Jul 18:26 2016

[PATCH V5 00/17] Fix and enable Python tests on Windows

This series of patches ports the python damons to Windows and
fixes the tests to work on Windows.
There are still some python tests that fail on Windows, from which
some are failing on Linux too.
The patches in this series are dependent and are related to
python tests on Windows.

V2: Fixes more porting issues for Windows
V3: Fixes all python tests
    This series was extended from 16 to 21 patches in order to solve
    all python related tests in one series.
    To run the tests on windows 'pypiwin32' is required to be
    installed for both Python2 and Python3 (pip install pypiwin32).
V4: Rebased the entire series, if didn't apply anymore.
    Fixed delete of PID file on exit of python daemon, this leaded to
    removal of patch "python tests: Don't check if the PID file
    gets deleted".
V5: Removed "windows: Added lockf function and lock PID file to be sent
    individually and removed already merged patches from this series.
Paul-Daniel Boca (17):
  python tests: Implemented signal.alarm for Windows
  python tests: Register signal handlers only on supported types on
  python tests: Fixed ctl file name for Windows
  python tests: Fixed unixctl python tests for Windows
  python tests: Added fcntl module for Windows
  python tests: Skip IPsec test on Windows
  python tests: Skip TCP6 idl tests on Windows
  python tests: Skip python tests on Windows where POSIX pid is used
  python tests: Fixed OSError not iterable on Windows
(Continue reading)

Mrs Michelle Richard | 1 Jul 16:50 2016

Re: Информация

Привет мои дорогие,

Я знаю, что это письмо может быть сюрпризом для вас, но я
считаю, что
вы будете искренни и честны, чтобы исполнить мое
последнее желание. Я
принести мир и любовь к вам. Именно по благодати
Божией, у меня не
было выбора, кроме как делать то, что является
законным и правильным в
глазах Бога для вечной жизни, и в глазах человека за свидетельство
милости и славы Бога на мою жизнь. Моя дорогая, я послал
эту почту
молилась она нашла вас в хорошем состоянии здоровья,
так как я сам в
очень критическом состоянии здоровья, в котором я
сплю каждую ночь, не
зная, если я могу быть живым, чтобы увидеть на
следующий день. Я г-жа
Мишель Ричардс вдова, гражданин Соединенного Королевства
(Великобритании) Im госпел христианские книги
писателя. Я страдал от
долгого рака легких я не могу общаться с вами по
телефону время,
потому что рак повлиял на мою способность слышать и
мой мозг, от всех
показаний мои условия действительно ухудшается, и
это совершенно
очевидно, что, по моим врачам они посоветовали мне,
(Continue reading)

dev | 1 Jul 12:55 2016



We are looking for employees working remotely.

My name is Dollie, am the personnel manager of a large International company.
Most of the work you can do from home, that is, at a distance.

Salary is $2500-$5000.

If you are interested in this offer, please visit 
Our Site

Best regards!
dev mailing list
dev <at>
Jan Scheurich | 1 Jul 13:44 2016

[RFC Patch] dpif-netdev: dpcls per in_port with sorted subtables

This is a redesigned version of the earlier RFC patch “dpif-netdev: Sorted
subtable vectors per in_port in dpcls”. 

The user-space datapath (dpif-netdev) consists of a first level "exact match
cache" (EMC) matching on 5-tuples and the normal megaflow classifier. With
many parallel packet flows (e.g. TCP connections) the EMC becomes inefficient
and the OVS forwarding performance is determined by the megaflow classifier.

The megaflow classifier (dpcls) consists of a variable number of hash tables
(aka subtables), each containing megaflow entries with the same mask of
packet header and metadata fields to match upon. A dpcls lookup matches a
given packet against all subtables in sequence until it hits a match. As
megaflow cache entries are by construction non-overlapping, the first match
is the only match.

Today the order of the subtables in the dpcls is essentially random so that
on average a dpcsl lookup has to visit N/2 subtables for a hit, when N is the
total number of subtables. Even though every single hash-table lookup is
fast, the performance of the current dpcls degrades when there are many

How does the patch address this issue:

In reality there is often a strong correlation between the ingress port and a
small subset of subtables that have hits. The entire megaflow cache typically
decomposes nicely into partitions that are hit only by packets entering from
a range of similar ports (e.g. traffic from Phy  -> VM vs. traffic from VM ->

Therefore, maintaining a separate dpcls instance per ingress port with its
(Continue reading)

Justin Pettit | 1 Jul 12:11 2016

[PATCH] OpenFlow: Update comment about non-experimental versions.

Signed-off-by: Justin Pettit <jpettit <at>>
 include/openflow/openflow-common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/openflow/openflow-common.h b/include/openflow/openflow-common.h
index 7b619a9..80a0055 100644
--- a/include/openflow/openflow-common.h
+++ b/include/openflow/openflow-common.h
 <at>  <at>  -65,7 +65,7  <at>  <at> 
 #endif /* __cplusplus */

 /* Version number:
- * Non-experimental versions released: 0x01 0x02
+ * Non-experimental versions released: 0x01 -- 0x04
  * Experimental versions released: 0x81 -- 0x99
 /* The most significant bit being set in the version field indicates an


dev mailing list
dev <at>