Re: [ovs-dev] [PATCH v2 1/3] ovn: Update ACL flow docs.

2016-03-09 Thread Han Zhou
On Wed, Mar 9, 2016 at 1:51 PM, Russell Bryant wrote: > > Apply some minor updates to the description of flows related to ACLs. > > Signed-off-by: Russell Bryant > --- > ovn/northd/ovn-northd.8.xml | 8 +--- > ovn/ovn-sb.xml | 11 +-- >

Re: [ovs-dev] [PATCH v2 2/3] ovn: Add ct_commit(ct_mark=INTEGER); action.

2016-03-09 Thread Han Zhou
On Wed, Mar 9, 2016 at 1:51 PM, Russell Bryant wrote: > > Update the "ct_commit;" logical flow action to optionally take a > parameter, setting the value of "ct_mark" to a 32-bit integer. > Supported ct_commit syntax now includes: > > ct_commit; > ct_commit(); >

Re: [ovs-dev] [PATCH v2 3/3] ovn: Apply ACL changes to existing connections.

2016-03-09 Thread Han Zhou
On Wed, Mar 9, 2016 at 1:51 PM, Russell Bryant wrote: > > Prior to this commit, once a connection had been committed to the > connection tracker, the connection would continue to be allowed, even > if the policy defined in the ACL table changed. This patch changes > the

[ovs-dev] RFC: OVN database options

2016-03-09 Thread Ben Pfaff
Requirements OVN uses two databases, the "northbound" and "southbound" databases, in a somewhat idiosyncratic manner. Each client of one of these databases maintains an in-memory replica of the database (or some subset of it), and the server sends it updates to this replica as they

Re: [ovs-dev] [RFC] OVN northbound address sets

2016-03-09 Thread Han Zhou
On Wed, Mar 9, 2016 at 1:32 PM, Ryan Moats wrote: > > > > "dev" wrote on 03/09/2016 03:12:07 PM: > > > From: Russell Bryant > > To: ovs dev > > Date: 03/09/2016 03:12 PM > > Subject: [ovs-dev] [RFC] OVN

[ovs-dev] [PATCH] rhel: Packaging for OVN is split as per functionality

2016-03-09 Thread bschanmu
rhel packaging for OVN is split to host(controller), central(northd), common(command line utilities) and docker(docker network plugin). This is similar to OVN packaging for Debian. Signed-off-by: Babu Shanmugam --- rhel/openvswitch-fedora.spec.in | 106

Re: [ovs-dev] [Fix branch-2.4 travis builds] travis: Use container infrastructure.

2016-03-09 Thread Joe Stringer
Hi Andy, thanks for fixing this. On 9 March 2016 at 18:05, Andy Zhou wrote: > From: Daniele Di Proietto As a general rule, it's nice to have the commit id direct in the message to show the upstream commit being backported. I've been doing it like this at

Re: [ovs-dev] [PATCH nf-next v9 8/8] openvswitch: Interface with NAT.

2016-03-09 Thread Joe Stringer
Hi Jarno, Thanks for working on this. Mostly just a few style things around #ifdefs below. On 9 March 2016 at 15:10, Jarno Rajahalme wrote: > Extend OVS conntrack interface to cover NAT. New nested > OVS_CT_ATTR_NAT attribute may be used to include NAT with a CT action. > A bare

Re: [ovs-dev] [PATCH nf-next v9 7/8] openvswitch: Delay conntrack helper call for new connections.

2016-03-09 Thread Joe Stringer
On 9 March 2016 at 15:10, Jarno Rajahalme wrote: > There is no need to help connections that are not confirmed, so we can > delay helping new connections to the time when they are confirmed. > This change is needed for NAT support, and having this as a separate > patch will make

Re: [ovs-dev] [PATCH nf-next v9 5/8] openvswitch: Find existing conntrack entry after upcall.

2016-03-09 Thread Joe Stringer
On 9 March 2016 at 15:10, Jarno Rajahalme wrote: > Add a new function ovs_ct_find_existing() to find an existing > conntrack entry for which this packet was already applied to. This is > only to be called when there is evidence that the packet was already > tracked and committed,

Re: [ovs-dev] [PATCH nf-next v9 4/8] openvswitch: Update the CT state key only after nf_conntrack_in().

2016-03-09 Thread Joe Stringer
On 9 March 2016 at 15:10, Jarno Rajahalme wrote: > Only a successful nf_conntrack_in() call can effect a connection state > change, so if suffices to update the key only after the > nf_conntrack_in() returns. "it" suffices to update... > This change is needed for the later NAT

[ovs-dev] [Fix branch-2.4 travis builds] travis: Use container infrastructure.

2016-03-09 Thread Andy Zhou
From: Daniele Di Proietto Recently some testcases have been failing in travis because of a warning related to the use of an L3 device (OpenVZ specific) inside the workers. To get travis tests working again we can move to the newer container infrastructure: this commit

[ovs-dev] [Fix branch-2.4 travis builds]Fix branch-2.4 travis builds

2016-03-09 Thread Andy Zhou
Branch 2.4 travis builds have been failing for a while, due to infrastructure changes at the travis ends. However, with constant build failures, it is hard to tell if any newer patches pushed into branch-2.4 contain genuine build breakers. Back port the following patches to restore the build

Re: [ovs-dev] [PATCH] netdev-dpdk: fix mbuf leaks

2016-03-09 Thread Yuanhan Liu
On Thu, Mar 10, 2016 at 01:33:03AM +, Daniele Di Proietto wrote: > Thanks for the patch! > > Were you able to hit this bug in your setup or did you just > find this by code inspection? Yes, it's a 100% reproducible bug if you have TSO enabled, which is a feature has been merged for DPDK v2.3

Re: [ovs-dev] [PATCH] netdev-dpdk: fix mbuf leaks

2016-03-09 Thread Daniele Di Proietto
Thanks for the patch! Were you able to hit this bug in your setup or did you just find this by code inspection? I'm asking because I'm wondering whether we should backport the fix. In any case I've applied this to master and added your name to the AUTHORS file, thanks! On 07/03/2016 17:50,

Re: [ovs-dev] [PATCH RFC 1/1] netdev-dpdk: NUMA Aware vHost

2016-03-09 Thread Daniele Di Proietto
Thanks for the patch, I'll put this in the use case list for my series if I need to resend it! It would be nice to get the numa socket information without linking OVS with libnuma, maybe using some DPDK api. From a quick look I didn't find any way, but maybe you know a better way. Some

[ovs-dev] [PATCH 4/6] tunneling: Handle multiple ip address for given device.

2016-03-09 Thread Pravin B Shelar
From: Pravin B Shelar Device can have multiple IP address but netdev_get_in4/6() returns only one configured IPv6 address. Following patch fixes it. OVS router is also updated to return source ip address for given destination, This is required when interface has multiple IP

[ovs-dev] [PATCH 2/6] netdev-linux: Fix netdev ipv6 notification

2016-03-09 Thread Pravin B Shelar
Listen to RTNLGRP_IPV6_IFINFO to get IPv6 address change notification. Signed-off-by: Pravin B Shelar --- lib/netdev-linux.c | 15 --- lib/rtnetlink.c| 6 -- 2 files changed, 16 insertions(+), 5 deletions(-) diff --git a/lib/netdev-linux.c

[ovs-dev] [PATCH 6/6] tunneling: Enable IPv6 tuneling.

2016-03-09 Thread Pravin B Shelar
From: Pravin B Shelar There is check to disable IPv6 tunneling. Following patch removes it and reintroduces the tunneling automake tests. This reverts commit 250bd94d1e500a89c76cac944e660bd9c07ac364. Signed-off-by: Pravin B Shelar ---

[ovs-dev] [PATCH 1/6] smap: bsd: Fix compilation error.

2016-03-09 Thread Pravin B Shelar
I saw following error while testing this series. ---8<--- In file included from lib/smap.c:16: ./lib/smap.h:75:56: warning: declaration of 'struct in6_addr' will not be visible outside of this function [-Wvisibility] void smap_add_ipv6(struct smap *, const char *, struct in6_addr *);

[ovs-dev] [PATCH 5/6] ovs-router: Do not allow routes over loopback device.

2016-03-09 Thread Pravin B Shelar
This just avoid unnecessary routes in the ovs-router. Signed-off-by: Pravin B Shelar --- lib/ovs-router.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/lib/ovs-router.c b/lib/ovs-router.c index 9b64ffb..3b11512 100644 --- a/lib/ovs-router.c +++ b/lib/ovs-router.c

[ovs-dev] [PATCH 3/6] lib: Fix compose nd

2016-03-09 Thread Pravin B Shelar
Following patch fixes number of issues with compose nd, like setting ip packet header, set ICMP opt-len, checksum. Signed-off-by: Pravin B Shelar --- lib/packets.c | 60 +-- 1 file changed, 42 insertions(+), 18

Re: [ovs-dev] [PATCH net-next] ovs: allow nl 'flow set' to use ufid without flow key

2016-03-09 Thread pravin shelar
On Wed, Mar 9, 2016 at 4:25 PM, Samuel Gauthier wrote: > Sorry, I missed that. Thank you for pointing it out. > > Although, set command is also used to reset the flow statistics, and the > action attribute seems optional. Would you find acceptable to make the key >

Re: [ovs-dev] [PATCH net-next] ovs: allow nl 'flow set' to use ufid without flow key

2016-03-09 Thread Samuel Gauthier
Sorry, I missed that. Thank you for pointing it out. Although, set command is also used to reset the flow statistics, and the action attribute seems optional. Would you find acceptable to make the key attribute mandatory only if the action attribute is provided? The ovs-dpctl command could then

Re: [ovs-dev] [PATCH v4 1/6] datapath-windows: Added recirculation support.

2016-03-09 Thread Sairam Venugopal
Hi Alin/Sorin, The OvsDoRecircFlowLookupOutput function currently assumes that the srcVport is always the Internal VPort. It also does not distinguish between Receive vs Transmit in the OvsCreateAndAddPackets call. Shouldn¹t the source VPORT point to the actual source VPORT instead of the

Re: [ovs-dev] [PATCH nf-next v8 3/8] openvswitch: Add commentary to conntrack.c

2016-03-09 Thread Jarno Rajahalme
Sergei, Just found this from my junk mail box, sorry. Will fix spelling, but I just sent v9 and will wait for other reviews before re-posting. Jarno > On Mar 9, 2016, at 5:50 AM, Sergei Shtylyov > wrote: > > Hello. > > On 3/9/2016 3:24 AM, Jarno

[ovs-dev] [PATCH nf-next v9 6/8] openvswitch: Handle NF_REPEAT in conntrack action.

2016-03-09 Thread Jarno Rajahalme
Repeat the nf_conntrack_in() call when it returns NF_REPEAT. This avoids dropping a SYN packet re-opening an existing TCP connection. Signed-off-by: Jarno Rajahalme Acked-by: Joe Stringer --- net/openvswitch/conntrack.c | 10 -- 1 file changed, 8

Re: [ovs-dev] [PATCH nf-next v8 8/8] openvswitch: Interface with NAT.

2016-03-09 Thread Jarno Rajahalme
rno-Rajahalme/netfilter-Remove-IP_CT_NEW_REPLY-definition/20160309-083126 > base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next master > config: i386-randconfig-x0-03091344 (attached as .config) > reproduce: ># save the attached .config to linux build tree >make

[ovs-dev] [PATCH nf-next v9 8/8] openvswitch: Interface with NAT.

2016-03-09 Thread Jarno Rajahalme
Extend OVS conntrack interface to cover NAT. New nested OVS_CT_ATTR_NAT attribute may be used to include NAT with a CT action. A bare OVS_CT_ATTR_NAT only mangles existing and expected connections. If OVS_NAT_ATTR_SRC or OVS_NAT_ATTR_DST is included within the nested attributes, new

[ovs-dev] [PATCH nf-next v9 7/8] openvswitch: Delay conntrack helper call for new connections.

2016-03-09 Thread Jarno Rajahalme
There is no need to help connections that are not confirmed, so we can delay helping new connections to the time when they are confirmed. This change is needed for NAT support, and having this as a separate patch will make the following NAT patch a bit easier to review. Signed-off-by: Jarno

[ovs-dev] [PATCH nf-next v9 1/8] netfilter: Remove IP_CT_NEW_REPLY definition.

2016-03-09 Thread Jarno Rajahalme
Remove the definition of IP_CT_NEW_REPLY from the kernel as it does not make sense. This allows the definition of IP_CT_NUMBER to be simplified as well. Signed-off-by: Jarno Rajahalme --- include/uapi/linux/netfilter/nf_conntrack_common.h | 12 +---

[ovs-dev] [PATCH nf-next v9 4/8] openvswitch: Update the CT state key only after nf_conntrack_in().

2016-03-09 Thread Jarno Rajahalme
Only a successful nf_conntrack_in() call can effect a connection state change, so if suffices to update the key only after the nf_conntrack_in() returns. This change is needed for the later NAT patches. Signed-off-by: Jarno Rajahalme --- net/openvswitch/conntrack.c | 7 ---

[ovs-dev] [PATCH nf-next v9 5/8] openvswitch: Find existing conntrack entry after upcall.

2016-03-09 Thread Jarno Rajahalme
Add a new function ovs_ct_find_existing() to find an existing conntrack entry for which this packet was already applied to. This is only to be called when there is evidence that the packet was already tracked and committed, but we lost the ct reference due to an userspace upcall.

[ovs-dev] [PATCH nf-next v9 3/8] openvswitch: Add commentary to conntrack.c

2016-03-09 Thread Jarno Rajahalme
This makes the code easier to understand and the following patches more focused. Signed-off-by: Jarno Rajahalme --- net/openvswitch/conntrack.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/net/openvswitch/conntrack.c

[ovs-dev] [PATCH nf-next v9 2/8] netfilter: Allow calling into nat helper without skb_dst.

2016-03-09 Thread Jarno Rajahalme
NAT checksum recalculation code assumes existence of skb_dst, which becomes a problem for a later patch in the series ("openvswitch: Interface with NAT."). Simplify this by removing the check on skb_dst, as the checksum will be dealt with later in the stack. Suggested-by: Pravin Shelar

[ovs-dev] [PATCH v2 1/3] ovn: Update ACL flow docs.

2016-03-09 Thread Russell Bryant
Apply some minor updates to the description of flows related to ACLs. Signed-off-by: Russell Bryant --- ovn/northd/ovn-northd.8.xml | 8 +--- ovn/ovn-sb.xml | 11 +-- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git

[ovs-dev] [PATCH v2 0/3] ovn: Apply ACL changes to existing connections.

2016-03-09 Thread Russell Bryant
Prior to this commit, once a connection had been committed to the connection tracker, the connection would continue to be allowed, even if the policy defined in the ACL table changed. This patch changes the implementation so that existing connections are affected by policy changes. The

[ovs-dev] [PATCH v2 2/3] ovn: Add ct_commit(ct_mark=INTEGER); action.

2016-03-09 Thread Russell Bryant
Update the "ct_commit;" logical flow action to optionally take a parameter, setting the value of "ct_mark" to a 32-bit integer. Supported ct_commit syntax now includes: ct_commit; ct_commit(); ct_commit(ct_mark=1); Setting ct_mark via this type of logical flow results in an OpenFlow

[ovs-dev] [PATCH v2 3/3] ovn: Apply ACL changes to existing connections.

2016-03-09 Thread Russell Bryant
Prior to this commit, once a connection had been committed to the connection tracker, the connection would continue to be allowed, even if the policy defined in the ACL table changed. This patch changes the implementation so that existing connections are affected by policy changes. The

Re: [ovs-dev] [RFC] OVN northbound address sets

2016-03-09 Thread Ryan Moats
"dev" wrote on 03/09/2016 03:12:07 PM: > From: Russell Bryant > To: ovs dev > Date: 03/09/2016 03:12 PM > Subject: [ovs-dev] [RFC] OVN northbound address sets > Sent by: "dev" > > I'd like to

Re: [ovs-dev] [PATCH net-next] ovs: allow nl 'flow set' to use ufid without flow key

2016-03-09 Thread pravin shelar
On Wed, Mar 9, 2016 at 9:05 AM, Samuel Gauthier wrote: > When we want to change a flow using netlink, we have to identify it to > be able to perform a lookup. Both the flow key and unique flow ID > (ufid) are valid identifiers, but we always have to specify the flow >

[ovs-dev] [CudaMailTagged] RSA Finance Marketing Leads of 2016

2016-03-09 Thread Emily White
Hi, Would you be interested in Finance Marketplace Lending + Investing user list or many Industry List of marketing and sales data we provides. And if you interested in other marketing industry let as know. We maintain 1.7Million contacts of CRM users with permission passed

[ovs-dev] [RFC] OVN northbound address sets

2016-03-09 Thread Russell Bryant
I'd like to propose a new feature for the OVN northbound database. If we reach some consensus, I will implement it. Overview: One use case for OVN ACLs includes matching on a set of IP addresses. A simple example: inport == "lport1" && ip.src == {10.0.0.1, 10.0.0.3, 10.0.7} This is only

[ovs-dev] [PATCH v8 10/10] Change physical_run to incremental processing

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Persist localvif_to_ofport and tunnels structures and change physical_run to incremental processing. Signed-off-by: RYAN D. MOATS --- ovn/controller/lflow.c|3 + ovn/controller/physical.c | 113

[ovs-dev] [PATCH v8 08/10] Convert binding_run to incremental processing.

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Persist all_lports structure and ensure that binding_run resets to process the entire port binding table when chassis are added/removed or when get_local_iface_ids finds new ports on the local vswitch. Signed-off-by: RYAN D. MOATS ---

[ovs-dev] [PATCH v8 3/3] Make flow table persistent in ovn controller

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS This is a prerequisite for incremental processing. Side effects: 1. Table rows are now tracked so that removed rows are correctly handled. 2. Hash by table id+priority+action added to help detect superseded flows. 3. Hash by insert seqno added to

[ovs-dev] [PATCH v8 09/10] Reset lflow processing when adding/removing patch ports

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS As lflow processing is incremental, reset it whenever a patch port is added or removed. Signed-off-by: RYAN D. MOATS --- ovn/controller/patch.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[ovs-dev] [PATCH v8 05/10] Persist local_datapaths

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Persist local_datapaths across runs so that a change can be used as a trigger to reset incremental flow processing. Signed-off-by: RYAN D. MOATS --- ovn/controller/binding.c| 41 --

[ovs-dev] [PATCH v8 07/10] Change encaps_run to work incrementally

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Side effects include tunnel context being persisted and no need to collect already defined OVS tunnels during each execution. Signed-off-by: RYAN D. MOATS --- ovn/controller/encaps.c | 123

[ovs-dev] [PATCH v8 04/10] Persist ports simap in logical_datapath

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Persist across runs so that a change to this simap can be used as a trigger for resetting incremental processing. Signed-off-by: RYAN D. MOATS --- ovn/controller/lflow.c | 125 1 files

[ovs-dev] [PATCH v8 06/10] Add incremental proessing to lflow_run

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS This code changes lflow_run to do incremental process of the logical flow table rather than processing the full table each run. Signed-off-by: RYAN D. MOATS --- ovn/controller/binding.c|3 ++ ovn/controller/lflow.c

[ovs-dev] [PATCH v8 01/10] Add useful information to ovn E2E tests

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Modify test 1739 to output the OF flows from all three hypervisors to help debug when something goes wrong. Signed-off-by: RYAN D. MOATS --- tests/ovn.at | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff

[ovs-dev] [PATCH v8 00/10] Implement incremental processing in ovn-controller

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Change ovn-controller to use incremental processing in encaps_run, binding_run, lflow_run and physical_run where easily feasible (patch_run and the mutlicast group processing in physical_run remain as complete rebuilds every cycle). This patch includes all

[ovs-dev] [PATCH v8 02/10] Present tracked changes in increasing change number order

2016-03-09 Thread Ryan Moats
From: RYAN D. MOATS Currently changes are added to the front of the track list, so they are looped through in LIFO order. Incremental processing is more efficient with a FIFO presentation, so (1) add new changes to the back of the track list, and (2) move updated changes to

[ovs-dev] [PATCH v10 6/6] NEWS: Announce the DPDK EAL configuration change

2016-03-09 Thread Aaron Conole
Previous commits have converted dpdk EAL initialization from requiring a ``--dpdk ... --`` command line arguments to using the Open vSwitch database. This change announces that as significant NEWS. Signed-off-by: Aaron Conole Signed-off-by: Kevin Traynor

[ovs-dev] [PATCH v10 4/6] netdev-dpdk: Allow arbitrary eal arguments

2016-03-09 Thread Aaron Conole
A previous change moved some commonly used arguments from commandline to the database, and with it the ability to pass arbitrary arguments to EAL. This change allows arbitrary eal arguments to be provided via a new db entry 'other_config:dpdk-extra' which will tokenize the string and add it to the

[ovs-dev] [PATCH v10 5/6] netdev-dpdk: Check dpdk-extra when reading db

2016-03-09 Thread Aaron Conole
A previous patch introduced the ability to pass arbitrary EAL command line options via the dpdk_extras database entry. This commit enhances that by warning the user when such a configuration is detected and prefering the value in the database. Suggested-by: Sean K Mooney

[ovs-dev] [PATCH v10 0/6] Convert DPDK configuration from command line to DB based

2016-03-09 Thread Aaron Conole
Currently, configuration of DPDK parameters is done via the command line through a --dpdk **OPTIONS** -- command line argument. This has a number of challenges, including: * It must be the first option passed to ovs-vswitchd * It is the only datapath feature in OVS to be configured on the command

[ovs-dev] [PATCH v10 1/6] netdev-dpdk: Restore thread affinity after DPDK init

2016-03-09 Thread Aaron Conole
When the DPDK init function is called, it changes the executing thread's CPU affinity to a single core specified in -c. This will result in the userspace bridge configuration thread being rebound, even if that is not the intent. This change fixes that behavior by rebinding to the original thread

[ovs-dev] [PATCH v10 3/6] netdev-dpdk: Autofill lcore coremask if absent

2016-03-09 Thread Aaron Conole
The user has control over the DPDK internal lcore coremask, but this parameter can be autofilled with a bit more intelligence. If the user does not fill this parameter in, we use the lowest set bit in the current task CPU affinity. Otherwise, we will reassign the current thread to the specified

[ovs-dev] [PATCH v10 2/6] netdev-dpdk: Convert initialization from cmdline to db

2016-03-09 Thread Aaron Conole
Existing DPDK integration is provided by use of command line options which must be split out and passed to librte in a special manner. However, this forces any configuration to be passed by way of a special DPDK flag, and interferes with ovs+dpdk packaging solutions. This commit delays dpdk

[ovs-dev] [PATCH net-next] ovs: allow nl 'flow set' to use ufid without flow key

2016-03-09 Thread Samuel Gauthier
When we want to change a flow using netlink, we have to identify it to be able to perform a lookup. Both the flow key and unique flow ID (ufid) are valid identifiers, but we always have to specify the flow key in the netlink message. When both attributes are there, the ufid is used. This commit

Re: [ovs-dev] Where to add GTP tunnel headers in datapath flow table of open vSwitch

2016-03-09 Thread Jesse Gross
On Tue, Mar 8, 2016 at 9:24 PM, Ajmer Singh wrote: > I have now question related to mapping of ofp_header->type (OFPT_) with > OFPRAW_contants > > struct ofp_header { > uint8_t version;/* An OpenFlow version number, e.g. OFP10_VERSION. > */ > uint8_t type;

[ovs-dev] [PATCH] Set reason before sending continuation packet-in to the controller

2016-03-09 Thread Numan Siddique
Signed-Off-by: Numan Siddique --- ofproto/ofproto-dpif-xlate.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index cd6eeab..e4692b3 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofproto-dpif-xlate.c

[ovs-dev] [PATCH] INSTALL.DPDK: remove issue with QEMU v2.4.0 and dpdkvhostuser

2016-03-09 Thread Mauricio Vasquez B
The patch mentioned in the commit e73b7508fb58 ("INSTALL.DPDK: Mention issue with QEMU v2.4.0 & dpdkvhostuser") is present in DPDK v2.2.0, then this issue is not valid anymore. Signed-off-by: Mauricio Vasquez B --- INSTALL.DPDK.md | 12 1

Re: [ovs-dev] [PATCH nf-next v8 3/8] openvswitch: Add commentary to conntrack.c

2016-03-09 Thread Sergei Shtylyov
Hello. On 3/9/2016 3:24 AM, Jarno Rajahalme wrote: This makes the code easier to understand and the following patches more focused. Signed-off-by: Jarno Rajahalme --- net/openvswitch/conntrack.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-)

Re: [ovs-dev] [PATCH v4 0/4] Support native dhcp in ovn controller

2016-03-09 Thread Numan Siddique
On 12/22/2015 01:31 PM, Ben Pfaff wrote: > Hi Babu and Numan. > > So far, we've been able to build OVN so that the logic of the system is > implemented in terms of OVN logical flows. That is, the logical flow > table determines what happens in every significant way in OVN. It would > be nice to

[ovs-dev] FW: Invoice 2016-M#289150

2016-03-09 Thread Sofia Cruz
Dear dev, Please find attached 2 invoices for processing. Yours sincerely, Sofia Cruz Account Manager __ This email has been scanned by the Symantec Email Security.cloud service.

[ovs-dev] FW: Invoice 2016-M#073206

2016-03-09 Thread Frances Lancaster
Dear dev, Please find attached 2 invoices for processing. Yours sincerely, Frances Lancaster Financial Manager __ This email has been scanned by the Symantec Email Security.cloud service.

Re: [ovs-dev] vhost-user invalid txqid cause discard of packets

2016-03-09 Thread Ilya Maximets
On 09.03.2016 12:13, Wang, Zhihong wrote: > > >> -Original Message- >> From: Ilya Maximets [mailto:i.maxim...@samsung.com] >> Sent: Wednesday, March 9, 2016 3:39 PM >> To: Wang, Zhihong ; dev@openvswitch.org >> Cc: Flavio Leitner ; Traynor, Kevin

Re: [ovs-dev] vhost-user invalid txqid cause discard of packets

2016-03-09 Thread Wang, Zhihong
> -Original Message- > From: Ilya Maximets [mailto:i.maxim...@samsung.com] > Sent: Wednesday, March 9, 2016 3:39 PM > To: Wang, Zhihong ; dev@openvswitch.org > Cc: Flavio Leitner ; Traynor, Kevin > ; > Dyasly Sergey

Re: [ovs-dev] Implementation of Packet-Out in Bundles

2016-03-09 Thread André Mantas
By the way, I'm still starting vswitchd with valgrind and detected some memory errors after using "sudo ovs-appctl exit". Is there other way to exit/stop ovs? To run I used: valgrind --tool=memcheck --leak-check=full ovs-vswitchd --pidfile --detach -v --log-file ==2344== HEAP SUMMARY: ==2344==

Re: [ovs-dev] [PATCH nf-next v8 8/8] openvswitch: Interface with NAT.

2016-03-09 Thread kbuild test robot
Hi Jarno, [auto build test ERROR on nf-next/master] url: https://github.com/0day-ci/linux/commits/Jarno-Rajahalme/netfilter-Remove-IP_CT_NEW_REPLY-definition/20160309-083126 base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next master config: i386-randconfig-x0-03091344