Re: [ovs-dev] [PATCH] [PATCH v1] ovn: align lrp 'network' to lsp 'addresses'

2016-06-24 Thread Justin Pettit
> On Jun 23, 2016, at 10:14 PM, Zong Kai LI wrote: > > To be honest, I start to think about it just from IPv6 side, not IPv4. So I > may make some mistake, and afer I checked ovn-nb.xml for Logical_Switch_Port > table addresses column, I'm sure I've made a mistake. Maybe multiple IPv4 > addre

Re: [ovs-dev] [PATCH v3 2/2] ovn: Add address_set() support for ACLs.

2016-06-24 Thread Zong Kai LI
> > +/* Remember the names of all address sets currently in > > expr_address_sets > > + * so we can detect address sets that have been deleted. */ > > +struct sset cur_address_sets = SSET_INITIALIZER(&cur_address_sets); > > > This sset is not an address_set, but address_set names (or k

[ovs-dev] employees needed

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Tanya, 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!

[ovs-dev] Career growth

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Ada, 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!__

[ovs-dev] employees needed

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Christy, 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!__

[ovs-dev] [PATCH v4 0/2] QOS updates with DSCP support

2016-06-24 Thread bschanmu
v3 -> v4: - Rebased on top of master - Fixed a bug where the noop qos was set on tunnel ports instead of egress interfaces Babu Shanmugam (2): Check and allocate free qdisc queue id for ports with qos parameters DSCP marking on packets egressing VIF interface ovn/controller/binding.c |

[ovs-dev] [PATCH v4 1/2] Check and allocate free qdisc queue id for ports with qos parameters

2016-06-24 Thread bschanmu
ovn-northd processes the list of Port_Bindings and hashes the list of queues per chassis. When it finds a port with qos_parameters and without a queue_id, it allocates a free queue for the chassis that this port belongs. The queue_id information is stored in the options field of Port_binding table.

[ovs-dev] [PATCH v4 2/2] DSCP marking on packets egressing VIF interface

2016-06-24 Thread bschanmu
ovn-northd sets 'ip.dscp' to the DSCP value Signed-off-by: Babu Shanmugam --- ovn/controller/lflow.c | 2 +- ovn/northd/ovn-northd.c | 4 ovn/ovn-nb.xml | 6 ++ ovn/ovn-sb.xml | 5 + 4 files changed, 16 insertions(+), 1 deletion(-) diff --git a/ovn/controller/lfl

Re: [ovs-dev] [PATCH v3 2/3] Check and allocate free qdisc queue id for ports with qos parameters

2016-06-24 Thread Babu Shanmugam
I rebased this patch and pushed for review. Thank you, Babu On Friday 24 June 2016 09:50 AM, Ben Pfaff wrote: On Mon, Jun 13, 2016 at 02:00:20PM +0530, bscha...@redhat.com wrote: ovn-northd processes the list of Port_Bindings and hashes the list of queues per chassis. When it finds a port with

Re: [ovs-dev] [PATCH v3 3/3] DSCP marking on packets egressing VIF interface

2016-06-24 Thread Babu Shanmugam
On Friday 24 June 2016 09:52 AM, Ben Pfaff wrote: On Mon, Jun 13, 2016 at 02:00:21PM +0530, bscha...@redhat.com wrote: ovn-northd sets 'ip.dscp' to the DSCP value Signed-off-by: Babu Shanmugam The documentation says: Value should be in the range of 0 to 64. in a couple of places.

Re: [ovs-dev] [PATCH v3 1/2] Add address set support.

2016-06-24 Thread Babu Shanmugam
Hi Flavio, I agree with your suggestions and will include them in the next version. Thank you, Babu On Thursday 23 June 2016 10:46 PM, Flaviof wrote: On Thu, Jun 23, 2016 at 1:05 AM, > wrote: From: Russel Bryant mailto:russ...@ovn.org>> Update the OVN ex

Re: [ovs-dev] [PATCH v3 2/2] ovn: Add address_set() support for ACLs.

2016-06-24 Thread Babu Shanmugam
Hi Flavio, Thanks for reviewing. My comments are below. On Thursday 23 June 2016 10:33 PM, Flaviof wrote: On Thu, Jun 23, 2016 at 1:05 AM, > wrote: From: Russell Bryant mailto:russ...@ovn.org>> +/* Return true if the address sets match, false otherwise.

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

2016-06-24 Thread Ryan Moats
"dev" wrote on 06/23/2016 12:56:59 PM: > From: Hui Kang/Watson/IBM@IBMUS > To: dev@openvswitch.org > Date: 06/23/2016 12:57 PM > Subject: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on > creating and binding large number of lports > Sent by: "dev" > > > Hi, > In our scalability test

Re: [ovs-dev] [PATCH RFC 6/6] dpif-netdev: Add dpif-netdev/pmd-rxq-set appctl command.

2016-06-24 Thread Ilya Maximets
Hi, Ryan. Thanks for your attention to this series. I have the rebased and a little fixed version of this patch-set, but, actually, I'm waiting for comments about high level design in this thread: http://openvswitch.org/pipermail/dev/2016-June/073196.html . May be, I'll send a new version

[ovs-dev] [PATCH V2] ipfix: Export user specified virtual observation ID

2016-06-24 Thread Wenyu Zhang
In virtual network, users want more info about the virtual point to observe the traffic. It should be a string to provide clear info, not a simple interger ID. Introduce "other-config: virtual_obs_id" in IPFIX, which is a string configured by user. Introduce an enterprise IPFIX entity "virtualOb

Re: [ovs-dev] [PATCH 1/1] rpm: Update rhel/automake.mk to build via mock

2016-06-24 Thread Russell Bryant
Thanks for the patch! I appreciate the attention to detail in your first patch to OVS: including Signed-off-by, updating AUTHORS, and updating documentation for your changes. Thanks! I have one minor nit that could be addressed while you're updating this to account for Flavio's feedback. On Tue

[ovs-dev] Updated document

2016-06-24 Thread Linda Barber
Dear dev, The reference you requested is attached. Let me know if you have any questions. Best regards Linda Barber Chief Executive Officer - Food Packaging Company ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Updated document

2016-06-24 Thread Joseph Gross
Dear dev, The reference you requested is attached. Let me know if you have any questions. Best regards Joseph Gross Key Account Director Municipalities ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Career growth

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Claudette, 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!

[ovs-dev] Updated document

2016-06-24 Thread Robert Osborne
Dear dev, The reference you requested is attached. Let me know if you have any questions. Best regards Robert Osborne CEO, Cafedirect ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH] netdev-dpdk: Fix using uninitialized link_status.

2016-06-24 Thread Ilya Maximets
'rte_eth_link_get_nowait()' works only with physical ports. In case of vhost-user port, 'link' will stay uninitialized and there will be random messages in log about link status. Ex.: |dpdk(dpdk_watchdog2)|DBG|Port -1 Link Up - speed 1 Mbps - full-duplex Fix that by calling 'check_link_status

[ovs-dev] Cooperation with the great company

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Alta, 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!_

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

2016-06-24 Thread Russell Bryant
On Fri, Jun 24, 2016 at 8:12 AM, Ryan Moats wrote: > "dev" wrote on 06/23/2016 12:56:59 PM: > > > From: Hui Kang/Watson/IBM@IBMUS > > To: dev@openvswitch.org > > Date: 06/23/2016 12:57 PM > > Subject: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on > > creating and binding large numbe

[ovs-dev] Updated document

2016-06-24 Thread Hilton Reid
Dear dev, The reference you requested is attached. Let me know if you have any questions. Best regards Hilton Reid Product Director ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v1] Add new column compute_types to OVN_Southbound

2016-06-24 Thread Russell Bryant
On Wed, Jun 22, 2016 at 5:55 PM, Amitabha Biswas wrote: > > On Jun 22, 2016, at 2:30 PM, Guru Shetty wrote: > > > > > > > > On 21 June 2016 at 12:20, Amitabha Biswas azbis...@gmail.com>> wrote: > > This patch allows a OVN hypervisor administator to specify the > > type(s) of non-distributed log

Re: [ovs-dev] [PATCH] netdev-dpdk: Fix using uninitialized link_status.

2016-06-24 Thread Aaron Conole
Ilya Maximets writes: > 'rte_eth_link_get_nowait()' works only with physical ports. > In case of vhost-user port, 'link' will stay uninitialized and there > will be random messages in log about link status. > > Ex.: > |dpdk(dpdk_watchdog2)|DBG|Port -1 Link Up - speed 1 Mbps - full-duplex > >

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

2016-06-24 Thread Hui Kang
Hi, Ryan, Thanks for your comments; reply inline. - Hui Ryan Moats/Omaha/IBM wrote on 06/24/2016 08:12:38 AM: > From: Ryan Moats/Omaha/IBM > To: Hui Kang/Watson/IBM@IBMUS, "Ben Pfaff" , "Justin > Pettit" , "Yusheng Wang" > Cc: dev@openvswitch.org > Date: 06/24/2016 08:12 AM > Subject: Re: [ovs-

[ovs-dev] [PATCHv12 1/2] datapath:backport: openvswitch: Add packet truncation support.

2016-06-24 Thread William Tu
Upstream commit: commit f2a4d086ed4c588d32fe9b7aa67fead7280e7bf1 Author: William Tu Date: Fri Jun 10 11:49:33 2016 -0700 openvswitch: Add packet truncation support. The patch adds a new OVS action, OVS_ACTION_ATTR_TRUNC, in order to truncate packets. A 'max_len' is adde

[ovs-dev] [PATCHv12 0/2] datapath: Packet truncation backports.

2016-06-24 Thread William Tu
This series adds the packet truncate action to OVS kernel and userspace. The kernel datapath is upstreamed to net-next, so backports are provided for Linux kernel 3.10 - 4.6. Patch 1 provides the kernel code and compiles on each Linux version within this range. The backports are tested on Linux-

[ovs-dev] [PATCHv12 2/2] ofp-actions: Add truncate action.

2016-06-24 Thread William Tu
The patch adds a new action to support packet truncation. The new action is formatted as 'output(port=n,max_len=m)', as output to port n, with packet size being MIN(original_size, m). One use case is to enable port mirroring to send smaller packets to the destination port so that only useful pack

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

2016-06-24 Thread Hui Kang
> > My question is whether there is any particular reason to allocate that > > number > > of (struct ovn_port)? It seems to me there is room in this code to > optimize > > for performance. Thanks. > > > > - Hui > > > > > > [1] > > > https://github.com/openvswitch/ovs/blob/master/ovn/northd/ovn-no

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

2016-06-24 Thread manjugk
The original message was received at Fri, 24 Jun 2016 23:39:18 +0800 from ti.com [149.124.197.192] - The following addresses had permanent fatal errors - - Transcript of session follows - while talking to openvswitch.org.: >>> MAIL From:manj...@ti.com <<< 501 manj...@ti.com...

Re: [ovs-dev] [PATCH V2] ipfix: Export user specified virtual observation ID

2016-06-24 Thread Ben Pfaff
On Fri, Jun 24, 2016 at 05:25:57AM -0700, Wenyu Zhang wrote: > In virtual network, users want more info about the virtual point to observe > the traffic. > It should be a string to provide clear info, not a simple interger ID. > > Introduce "other-config: virtual_obs_id" in IPFIX, which is a stri

Re: [ovs-dev] [PATCHv12 2/2] ofp-actions: Add truncate action.

2016-06-24 Thread pravin shelar
On Fri, Jun 24, 2016 at 7:42 AM, William Tu wrote: > The patch adds a new action to support packet truncation. The new action > is formatted as 'output(port=n,max_len=m)', as output to port n, with > packet size being MIN(original_size, m). > > One use case is to enable port mirroring to send sma

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

2016-06-24 Thread Russell Bryant
On Fri, Jun 24, 2016 at 11:20 AM, Hui Kang wrote: > > > > My question is whether there is any particular reason to allocate that > > > number > > > of (struct ovn_port)? It seems to me there is room in this code to > > optimize > > > for performance. Thanks. > > > > > > - Hui > > > > > > > > > [1

[ovs-dev] [PATCH] [OVN-northd] Rename nbs/nbr to nbsp/nbrp

2016-06-24 Thread Hui Kang
These variables indicate ports in nb switches or routers. Signed-off-by: Hui Kang --- ovn/northd/ovn-northd.c | 196 1 file changed, 98 insertions(+), 98 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 1599e18..5

[ovs-dev] [PATCH] tests: Fixed ovsdb-monitor tests

2016-06-24 Thread Paul Boca
Redirect ovsdb-client stderr to /dev/null. This fixes the series of tests:1770 1771 1772 1773 1774 1775 1776 1777 1778 1779 1780 Signed-off-by: Paul-Daniel Boca --- tests/ovsdb-monitor.at | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/ovsdb-monitor.at b/tests/ovsd

Re: [ovs-dev] [PATCH v4] datapath-windows: Add GRE checksum

2016-06-24 Thread Nithin Raju
-Original Message- From: dev on behalf of Alin Serdean Date: Friday, June 17, 2016 at 1:00 PM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v4] datapath-windows: Add GRE checksum >This patch introduces GRE checksum computation if the userspace requires >it on Tx. On Rx we verify t

[ovs-dev] hi!

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Ann, 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!__

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

2016-06-24 Thread Ben Pfaff
On Fri, Jun 24, 2016 at 07:12:38AM -0500, Ryan Moats wrote: > Ben, Justin, Yusheng can one of you talk to an ETA for when the nlog > ovn-northd code base will start to land in the review queue? That will > provide input on whether doing an interim patch series is worth the effort > or not... My e

[ovs-dev] Flexible schedule

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Cheryl, 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!___

Re: [ovs-dev] [PATCH v2] ovn-northd, tests: Adding IPAM to ovn-northd.

2016-06-24 Thread Guru Shetty
On 23 June 2016 at 14:55, Nimay Desai wrote: > Added an IPv4 and MAC addresses management system to ovn-northd. When a > logical > switch's options:subnet field is set, logical ports attached to that switch > that do not have a MAC/IPv4 address will automatically be allocated a > globally > uniqu

Re: [ovs-dev] [PATCH v4] datapath-windows: Add GRE checksum

2016-06-24 Thread Guru Shetty
On 24 June 2016 at 10:55, Nithin Raju wrote: > -Original Message- > From: dev on behalf of Alin Serdean > > Date: Friday, June 17, 2016 at 1:00 PM > To: "dev@openvswitch.org" > Subject: [ovs-dev] [PATCH v4] datapath-windows: Add GRE checksum > > >This patch introduces GRE checksum comp

Re: [ovs-dev] [PATCH] tests: Fixed ovsdb-monitor tests

2016-06-24 Thread Lance Richardson
- Original Message - > From: "Paul Boca" > To: dev@openvswitch.org > Sent: Friday, June 24, 2016 12:51:49 PM > Subject: [ovs-dev] [PATCH] tests: Fixed ovsdb-monitor tests > > Redirect ovsdb-client stderr to /dev/null. > This fixes the series of tests:1770 1771 1772 1773 1774 1775 1776 177

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

2016-06-24 Thread Hui Kang
Ben Pfaff wrote on 06/24/2016 02:24:12 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: Hui Kang/Watson/IBM@IBMUS, Justin Pettit , > Yusheng Wang , dev@openvswitch.org > Date: 06/24/2016 02:24 PM > Subject: Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd > on creating a

[ovs-dev] [PATCH v2 6/9] datapath-windows: Add new NlFillOvsMsgForNfGenMsg method in Netlink.c

2016-06-24 Thread Sairam Venugopal
Create a new method to create and fill OvsMessage with NfGenMsg. This will be used for sending Netfilter based Netlink messages. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Netlink/Netlink.c | 39 +++ datapath-windows/ovsext/Netlink/Netlink.h | 4

[ovs-dev] [PATCH v2 4/9] datapath-windows: Add support for Conntrack IPCTNL_MSG_CT_DELETE cmd in Datapath.c

2016-06-24 Thread Sairam Venugopal
Create new NETLINK_CMD and NETLINK_FAMILY to assist in flushing conntrack entries. Modify Datapath.c to now support netfilter-netlink messages apart from the existing netfilter-generic messages. Also hookup the command handler to execute the OvsCtFlush in Conntrack.c Signed-off-by: Sairam Venugop

[ovs-dev] [PATCH v2 3/9] datapath-windows: Add support for flushing conntrack entries

2016-06-24 Thread Sairam Venugopal
Flush out all conntrack entries or those that match a given zone. Since the conntrack module is internal to OVS in Windows, this functionality needs to be added in. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Conntrack.c | 75 + 1 file changed,

[ovs-dev] [PATCH v2 2/9] datapath-windows: Add support for Netfilter netlink message

2016-06-24 Thread Sairam Venugopal
Introduce NF_GEN_MSG_HDR similar to GENL_MSG_HDR that will be used for communicating via netfilter-netlink channel. This will be used by userspace to retrieve and modify Conntrack data in Windows. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Netlink/Netlink.c | 11 +++

[ovs-dev] [PATCH v2 8/9] datapath-windows: Conntrack - Handle memory allocation failure

2016-06-24 Thread Sairam Venugopal
Return null if Windows fails to allocate memory for the conntrack entry. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Conntrack-other.c | 5 +++-- datapath-windows/ovsext/Conntrack-tcp.c | 4 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/datapath-windows/ov

[ovs-dev] [PATCH v2 0/9] Add support for debugging conntrack

2016-06-24 Thread Sairam Venugopal
Conntrack module is part of OVS datapath on Windows and lacks netlink support. The following patches adds support in Windows datapath for accepting netfilter-netlink messages from userspace and executing the command. The supported commands include flushing conntrack entries and dumping them. Ne

[ovs-dev] [PATCH v2 5/9] Windows: Add conntrack-flush support in userspace

2016-06-24 Thread Sairam Venugopal
Modify dpif-netlink.c and netlink-conntrack.c to send down flush command to Windows datapath. Include netlink-conntrack.c and netlink-conntrack.h in automake.mk for Windows binaries. Windows currently supports only NETLINK_GENERIC port. In order to support the NETLINK_NETFILTER messages, the port

[ovs-dev] [PATCH v2 7/9] datapath-windows: Add support for dump-conntrack in datapath

2016-06-24 Thread Sairam Venugopal
Create the methods used for dumping conntrack entries from the hyper-v datapath to userspace by means of netfilter netlink messages. Some of the attributes are not supported by the datapath and have been defaulted to 0. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Conntrack-tcp.c

[ovs-dev] [PATCH v2 1/9] Windows: Add netfilter-conntrack header files to Include folder

2016-06-24 Thread Sairam Venugopal
Include netfilter-conntrack header files in user-space. This will be used by Windows userspace for adding debugging support in Conntrack. Some of these files are intentionally left blank and will be updated once relevant support is added in Windows datapath. Signed-off-by: Sairam Venugopal ---

[ovs-dev] [PATCH v2 9/9] datapath-windows: Add support for Conntrack IPCTNL_MSG_CT_GET cmd in Datapath.c

2016-06-24 Thread Sairam Venugopal
This will be used by userspace for dumping conntrack entries - "ovs-dpctl dump-conntrack". Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Datapath.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/datapath-windows/ovsext/Datapath.c b/datapath-windo

[ovs-dev] [PATCH v8][PATCH 2/2] datapath-windows: Address minor alignment issues in Stt code.

2016-06-24 Thread Yin Lin
Signed-off-by: Yin Lin --- datapath-windows/ovsext/Stt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath-windows/ovsext/Stt.c b/datapath-windows/ovsext/Stt.c index 0bac5f2..5aaf6fe 100644 --- a/datapath-windows/ovsext/Stt.c +++ b/datapath-windows/ovsext/Stt.c @@ -875,

[ovs-dev] [PATCH v8][PATCH 1/2] datapath-windows: Add Geneve support

2016-06-24 Thread Yin Lin
Signed-off-by: Yin Lin --- datapath-windows/automake.mk | 2 + datapath-windows/ovsext/Actions.c | 72 ++- datapath-windows/ovsext/Debug.h| 1 + datapath-windows/ovsext/DpInternal.h | 29 ++- datapath-windows/ovsext/Flow.c | 179 +++-- datapa

Re: [ovs-dev] [PATCH] tests: Fixed ovsdb-monitor tests

2016-06-24 Thread Guru Shetty
On 24 June 2016 at 09:51, Paul Boca wrote: > Redirect ovsdb-client stderr to /dev/null. > This fixes the series of tests:1770 1771 1772 1773 1774 1775 1776 1777 > 1778 1779 1780 > > Signed-off-by: Paul-Daniel Boca > --- > tests/ovsdb-monitor.at | 5 +++-- > 1 file changed, 3 insertions(+), 2 de

Re: [ovs-dev] [PATCH v8][PATCH 1/2] datapath-windows: Add Geneve support

2016-06-24 Thread Guru Shetty
On 24 June 2016 at 11:49, Yin Lin wrote: > Signed-off-by: Yin Lin > is this just a rebase for the patch that Nithin Acked? > --- > datapath-windows/automake.mk | 2 + > datapath-windows/ovsext/Actions.c | 72 ++- > datapath-windows/ovsext/Debug.h| 1 + > datapa

Re: [ovs-dev] [PATCH] rhel: Fix RHEL package build breakage

2016-06-24 Thread Daniele Di Proietto
Works for me, thanks for fixing this! Acked-by: Daniele Di Proietto 2016-06-23 19:04 GMT-07:00 Ansis Atteka : > This patch fixes following error: > > error: Installed (but unpackaged) file(s) found: >/usr/bin/ovs-tcpdump >/usr/share/man/man8/ovs-tcpdump.8.gz > > Signed-off-by: Ansis Att

Re: [ovs-dev] [PATCH] tests: Fixed ovsdb-monitor tests

2016-06-24 Thread Lance Richardson
- Original Message - > From: "Guru Shetty" > To: "Paul Boca" > Cc: dev@openvswitch.org > Sent: Friday, June 24, 2016 2:50:35 PM > Subject: Re: [ovs-dev] [PATCH] tests: Fixed ovsdb-monitor tests > > On 24 June 2016 at 09:51, Paul Boca wrote: > > > Redirect ovsdb-client stderr to /dev/nu

[ovs-dev] [PATCH v3 2/9] datapath-windows: Add support for Netfilter netlink message

2016-06-24 Thread Sairam Venugopal
Introduce NF_GEN_MSG_HDR similar to GENL_MSG_HDR that will be used for communicating via netfilter-netlink channel. This will be used by userspace to retrieve and modify Conntrack data in Windows. Signed-off-by: Sairam Venugopal Acked-by: Paul-Daniel Boca --- datapath-windows/ovsext/Netlink/Net

[ovs-dev] [PATCH v3 5/9] Windows: Add conntrack-flush support in userspace

2016-06-24 Thread Sairam Venugopal
Modify dpif-netlink.c and netlink-conntrack.c to send down flush command to Windows datapath. Include netlink-conntrack.c and netlink-conntrack.h in automake.mk for Windows binaries. Windows currently supports only NETLINK_GENERIC port. In order to support the NETLINK_NETFILTER messages, the port

[ovs-dev] [PATCH v3 0/9] Add support for debugging conntrack

2016-06-24 Thread Sairam Venugopal
Conntrack module is part of OVS datapath on Windows and lacks netlink support. The following patches adds support in Windows datapath for accepting netfilter-netlink messages from userspace and executing the command. The supported commands include flushing conntrack entries and dumping them. Ne

[ovs-dev] [PATCH v3 1/9] Windows: Add netfilter-conntrack header files to Include folder

2016-06-24 Thread Sairam Venugopal
Include netfilter-conntrack header files in user-space. This will be used by Windows userspace for adding debugging support in Conntrack. Some of these files are intentionally left blank and will be updated once relevant support is added in Windows datapath. Signed-off-by: Sairam Venugopal Acked

[ovs-dev] [PATCH v3 8/9] datapath-windows: Conntrack - Handle memory allocation failure

2016-06-24 Thread Sairam Venugopal
Return null if Windows fails to allocate memory for the conntrack entry. Signed-off-by: Sairam Venugopal Acked-by: Paul-Daniel Boca --- datapath-windows/ovsext/Conntrack-other.c | 5 +++-- datapath-windows/ovsext/Conntrack-tcp.c | 4 2 files changed, 7 insertions(+), 2 deletions(-) diff

[ovs-dev] [PATCH v3 4/9] datapath-windows: Add support for Conntrack IPCTNL_MSG_CT_DELETE cmd in Datapath.c

2016-06-24 Thread Sairam Venugopal
Create new NETLINK_CMD and NETLINK_FAMILY to assist in flushing conntrack entries. Modify Datapath.c to now support netfilter-netlink messages apart from the existing netfilter-generic messages. Also hookup the command handler to execute the OvsCtFlush in Conntrack.c Signed-off-by: Sairam Venugop

[ovs-dev] [PATCH v3 3/9] datapath-windows: Add support for flushing conntrack entries

2016-06-24 Thread Sairam Venugopal
Flush out all conntrack entries or those that match a given zone. Since the conntrack module is internal to OVS in Windows, this functionality needs to be added in. Signed-off-by: Sairam Venugopal Acked-by: Paul-Daniel Boca --- datapath-windows/ovsext/Conntrack.c | 75 ++

[ovs-dev] [PATCH v3 6/9] datapath-windows: Add new NlFillOvsMsgForNfGenMsg method in Netlink.c

2016-06-24 Thread Sairam Venugopal
Create a new method to create and fill OvsMessage with NfGenMsg. This will be used for sending Netfilter based Netlink messages. Signed-off-by: Sairam Venugopal Acked-by: Paul-Daniel Boca --- datapath-windows/ovsext/Netlink/Netlink.c | 39 +++ datapath-windows/ovsext

[ovs-dev] [PATCH v3 7/9] datapath-windows: Add support for dump-conntrack in datapath

2016-06-24 Thread Sairam Venugopal
Create the methods used for dumping conntrack entries from the hyper-v datapath to userspace by means of netfilter netlink messages. Some of the attributes are not supported by the datapath and have been defaulted to 0. Signed-off-by: Sairam Venugopal Acked-by: Paul-Daniel Boca --- datapath-win

[ovs-dev] Returned mail: see transcript for details

2016-06-24 Thread Post Office
The original message was received at Sat, 25 Jun 2016 03:04:12 +0800 from openvswitch.org [184.220.219.91] - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo

[ovs-dev] beneficial proposition

2016-06-24 Thread dev
Hello! We are looking for employees working remotely. My name is Rosendo, 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!__

[ovs-dev] [PATCH v3 9/9] datapath-windows: Add support for Conntrack IPCTNL_MSG_CT_GET cmd in Datapath.c

2016-06-24 Thread Sairam Venugopal
This will be used by userspace for dumping conntrack entries - "ovs-dpctl dump-conntrack". Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Datapath.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/datapath-windows/ovsext/Datapath.c b/datapath-windo

[ovs-dev] [PATCH 0/4] Restore ovn-controller binding functionality.

2016-06-24 Thread Russell Bryant
"Convert binding_run to incremental processing" caused a regression in port binding handling. This was seen in OpenStack CI. The last commit here adds a test case that would have caught the issue in OVN. There are 3 reverts as two later patches conflicted with reverting patch 3/4. This is just

[ovs-dev] [PATCH 4/4] ovn: Test that port state goes up and down.

2016-06-24 Thread Russell Bryant
Some previous commits broke ovn-controller binding handling such that ovn-controller never cleared out the chassis column of the Port_Binding table. This broke OpenStack CI for OVN. This patch adds an OVN test case that would have caught this issue. Signed-off-by: Russell Bryant --- tests/ovn.

[ovs-dev] [PATCH 1/4] Revert "ovn-controller: Use new ovsdb-idl helpers to make logic more readable."

2016-06-24 Thread Russell Bryant
This reverts commit 9a33cd70f4f8bf5922ef05caf32e6da748c57f1a. Signed-off-by: Russell Bryant --- ovn/controller/binding.c | 11 +++ ovn/controller/encaps.c | 19 ++- ovn/controller/lport.c | 18 -- ovsdb/ovsdb-idlc.in | 12 4 files chan

[ovs-dev] [PATCH 3/4] Revert "Convert binding_run to incremental processing."

2016-06-24 Thread Russell Bryant
This reverts commit 263064aeaa31e758538773fac571dff0cb246cde. Signed-off-by: Russell Bryant --- ovn/controller/binding.c| 205 ++-- ovn/controller/binding.h| 4 +- ovn/controller/encaps.c | 3 - ovn/controller/ovn-controller.c | 29

[ovs-dev] [PATCH 2/4] Revert "lport: Persist lport_index and mcgroup_index structures."

2016-06-24 Thread Russell Bryant
This reverts commit 4470328bc4e0081479351e39a1964a78fe19be1b. Signed-off-by: Russell Bryant --- ovn/controller/lport.c | 221 +++- ovn/controller/lport.h | 22 +--- ovn/controller/ovn-controller.c | 16 ++- 3 files changed, 46 insertions(+)

Re: [ovs-dev] [PATCH 4/4] ovn: Test that port state goes up and down.

2016-06-24 Thread Russell Bryant
On Fri, Jun 24, 2016 at 3:52 PM, Russell Bryant wrote: > Some previous commits broke ovn-controller binding handling such that > ovn-controller never cleared out the chassis column of the Port_Binding > table. This broke OpenStack CI for OVN. This patch adds an OVN test > case that would have c

Re: [ovs-dev] [PATCH] rhel: Fix RHEL package build breakage

2016-06-24 Thread Aaron Conole
Ansis Atteka writes: > This patch fixes following error: > > error: Installed (but unpackaged) file(s) found: >/usr/bin/ovs-tcpdump >/usr/share/man/man8/ovs-tcpdump.8.gz > > Signed-off-by: Ansis Atteka > --- You have my ACK as well. Sorry. __

Re: [ovs-dev] [PATCH 0/4] Restore ovn-controller binding functionality.

2016-06-24 Thread Ben Pfaff
On Fri, Jun 24, 2016 at 03:52:07PM -0400, Russell Bryant wrote: > "Convert binding_run to incremental processing" caused a regression > in port binding handling. This was seen in OpenStack CI. The last > commit here adds a test case that would have caught the issue in OVN. > > There are 3 revert

Re: [ovs-dev] [PATCH] rhel: Fix RHEL package build breakage

2016-06-24 Thread Russell Bryant
On Thu, Jun 23, 2016 at 10:04 PM, Ansis Atteka wrote: > This patch fixes following error: > > error: Installed (but unpackaged) file(s) found: >/usr/bin/ovs-tcpdump >/usr/share/man/man8/ovs-tcpdump.8.gz > > Signed-off-by: Ansis Atteka > Acked-by: Russell Bryant -- Russell Bryant ___

Re: [ovs-dev] [PATCH 0/4] Restore ovn-controller binding functionality.

2016-06-24 Thread Russell Bryant
On Fri, Jun 24, 2016 at 3:59 PM, Ben Pfaff wrote: > On Fri, Jun 24, 2016 at 03:52:07PM -0400, Russell Bryant wrote: > > "Convert binding_run to incremental processing" caused a regression > > in port binding handling. This was seen in OpenStack CI. The last > > commit here adds a test case that

Re: [ovs-dev] [PATCH 0/4] Restore ovn-controller binding functionality.

2016-06-24 Thread Ryan Moats
Ben Pfaff wrote on 06/24/2016 02:59:30 PM: > From: Ben Pfaff > To: Russell Bryant > Cc: dev@openvswitch.org, Ryan Moats/Omaha/IBM@IBMUS > Date: 06/24/2016 03:00 PM > Subject: Re: [PATCH 0/4] Restore ovn-controller binding functionality. > > On Fri, Jun 24, 2016 at 03:52:07PM -0400, Russell Brya

Re: [ovs-dev] [PATCH v4 1/2] Check and allocate free qdisc queue id for ports with qos parameters

2016-06-24 Thread Ben Pfaff
On Fri, Jun 24, 2016 at 02:59:15PM +0530, bscha...@redhat.com wrote: > ovn-northd processes the list of Port_Bindings and hashes the list of > queues per chassis. When it finds a port with qos_parameters and without > a queue_id, it allocates a free queue for the chassis that this port belongs. > T

Re: [ovs-dev] [PATCH] rhel: Fix RHEL package build breakage

2016-06-24 Thread Ansis Atteka
On 24 June 2016 at 13:00, Russell Bryant wrote: > On Thu, Jun 23, 2016 at 10:04 PM, Ansis Atteka wrote: > > > This patch fixes following error: > > > > error: Installed (but unpackaged) file(s) found: > >/usr/bin/ovs-tcpdump > >/usr/share/man/man8/ovs-tcpdump.8.gz > > > > Signed-off-by:

Re: [ovs-dev] [PATCH v3 9/9] datapath-windows: Add support for Conntrack IPCTNL_MSG_CT_GET cmd in Datapath.c

2016-06-24 Thread Paul Boca
Acked-by: Paul-Daniel Boca > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Sairam > Venugopal > Sent: Friday, June 24, 2016 10:03 PM > To: dev@openvswitch.org > Subject: [ovs-dev] [PATCH v3 9/9] datapath-windows: Add support for > Conntrack IPCTNL_MSG_CT

[ovs-dev] [PATCH] ovn-sbctl: Change lport-(un)bind to lsp-(un)bind.

2016-06-24 Thread Russell Bryant
A previous commit changed the command names in ovn-nbctl from lport-* to lsp-*. Change lport-bind and lport-unbind in ovn-sbctl to match. Signed-off-by: Russell Bryant --- ovn/utilities/ovn-sbctl.c| 12 ++-- tests/ovn-controller-vtep.at | 10 +- tests/ovn-sbctl.at

[ovs-dev] [PATCH] Fix port binding update on OVS port delete events

2016-06-24 Thread Ryan Moats
Patch "Convert binding_run to incremental processing." introduced a bug where the port binding table is not correctly updated when an OVS port is deleted. Fix this by - persisting the lport shash used to record OVS ports - change get_local_iface_ids to return a bool indicating if the persisted l

Re: [ovs-dev] [PATCH V2] ipfix: Export user specified virtual observation ID

2016-06-24 Thread Ben Pfaff
On Fri, Jun 24, 2016 at 05:25:57AM -0700, Wenyu Zhang wrote: > In virtual network, users want more info about the virtual point to observe > the traffic. > It should be a string to provide clear info, not a simple interger ID. > > Introduce "other-config: virtual_obs_id" in IPFIX, which is a stri

[ovs-dev] [PATCH] bridge: Fix crash in bridge_configure_ipfix().

2016-06-24 Thread Daniele Di Proietto
'be_cfg' may be NULL at that point. Found by testsuite. 1051: ofproto-dpif - Flow IPFIX sanity check FAILED (ofproto-dpif.at:6002) 1052: ofproto-dpif - Flow IPFIX sanity check - tunnel set FAILED (ofproto-dpif.at:6041) 0 0x006ae830 in hmap_first_with_hash (hmap=0x110, hash=2537

Re: [ovs-dev] [PATCH v1] Add new column compute_types to OVN_Southbound

2016-06-24 Thread Amitabha Biswas
> On Jun 24, 2016, at 6:41 AM, Russell Bryant wrote: > > > On Wed, Jun 22, 2016 at 5:55 PM, Amitabha Biswas > wrote: > > On Jun 22, 2016, at 2:30 PM, Guru Shetty > > wrote: > > > > > > > > On 21 June 2016 at 12:20, Amitabha Biswas >

Re: [ovs-dev] [PATCH] Fix port binding update on OVS port delete events

2016-06-24 Thread Ryan Moats
Ryan Moats/Omaha/IBM@IBMUS wrote on 06/24/2016 03:39:28 PM: > From: Ryan Moats/Omaha/IBM@IBMUS > To: dev@openvswitch.org > Cc: b...@ovn.org, russ...@ovn.org, Ryan Moats/Omaha/IBM@IBMUS > Date: 06/24/2016 03:39 PM > Subject: [PATCH] Fix port binding update on OVS port delete events > > Patch "Conve

Re: [ovs-dev] [PATCH] bridge: Fix crash in bridge_configure_ipfix().

2016-06-24 Thread Daniele Di Proietto
Please discard this, Ben just reverted the change that introduced this. On 24/06/2016 13:46, "Daniele Di Proietto" wrote: >'be_cfg' may be NULL at that point. > >Found by testsuite. > >1051: ofproto-dpif - Flow IPFIX sanity check FAILED >(ofproto-dpif.at:6002) >1052: ofproto-dpif - Fl

Re: [ovs-dev] [PATCH v1] Add new column compute_types to OVN_Southbound

2016-06-24 Thread Russell Bryant
On Fri, Jun 24, 2016 at 4:37 PM, Amitabha Biswas wrote: > On Jun 24, 2016, at 6:41 AM, Russell Bryant wrote: > > > On Wed, Jun 22, 2016 at 5:55 PM, Amitabha Biswas > wrote: > >> > On Jun 22, 2016, at 2:30 PM, Guru Shetty wrote: >> > >> > >> > >> > On 21 June 2016 at 12:20, Amitabha Biswas > az

Re: [ovs-dev] [PATCH] acinclude: check for numa library

2016-06-24 Thread Daniele Di Proietto
Thanks, this certainly makes the error more explicit. Applied to master! 2016-06-20 2:53 GMT-07:00 Loftus, Ciara : > > > > Numa library is needed for NUMA aware vHost User functionality. > > Incase of missing numa package, the OVS DPDK configuration fails with > > "error: Could not find DPDK lib

Re: [ovs-dev] [PATCH V2] ipfix: Export user specified virtual observation ID

2016-06-24 Thread William Tu
Hi Wenyu, I was debugging a little bit and the issue is a NULL pointer deference of be_cfg at virtual_obs_id = smap_get(&be_cfg->other_config, "virtual_obs_id"); Maybe adding if (valid_be_cfg) check before the deference? I will leave you to fix it. Also I hope you can add a test case to this ca

Re: [ovs-dev] [PATCH 2/4] compat: ipv4: Pass struct net through ip_fragment.

2016-06-24 Thread Jesse Gross
On Wed, Jun 22, 2016 at 6:00 PM, Joe Stringer wrote: > diff --git a/acinclude.m4 b/acinclude.m4 > index 52d0209ab88a..8760948fcf9b 100644 > --- a/acinclude.m4 > +++ b/acinclude.m4 > @@ -408,7 +408,8 @@ AC_DEFUN([OVS_CHECK_LINUX_COMPAT], [ >[OVS_DEFINE([HAVE_INET_GET_LOCAL_PORT_

[ovs-dev] [PATCH 2/2] datapath-windows: Address minor alignment issues in Stt code.

2016-06-24 Thread Yin Lin
Signed-off-by: Yin Lin --- datapath-windows/ovsext/Stt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath-windows/ovsext/Stt.c b/datapath-windows/ovsext/Stt.c index 0bac5f2..5aaf6fe 100644 --- a/datapath-windows/ovsext/Stt.c +++ b/datapath-windows/ovsext/Stt.c @@ -875,

[ovs-dev] [Patch v9][PATCH 1/2] datapath-windows: Add Geneve support

2016-06-24 Thread Yin Lin
Signed-off-by: Yin Lin --- datapath-windows/automake.mk | 2 + datapath-windows/ovsext/Actions.c | 72 ++- datapath-windows/ovsext/Debug.h| 1 + datapath-windows/ovsext/DpInternal.h | 29 ++- datapath-windows/ovsext/Flow.c | 179 +++-- datapa

  1   2   >