[ovs-dev] [ovn-dev] ovn-nbctl: Improvement to the ovn-nbctl manpage

2016-07-01 Thread nickcooper-zhangtonghao
Hi Ben, Thank you for your advice. ovn-nbctl: the improvement to the ovn-nbctl manpage. If there are multiple logical switches or routers with a duplicate name, the configuration is slightly different. You should configure the logical switches or routers using the UUID instead of the n

Re: [ovs-dev] [RFC 3/3] Red Hat Systemd Integration

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote: > This commit builds upon some of the recent ovs-ctl changes to build a > more integrated systemd setup. A new service (openvswitch-network) is > added to track the ovs-vswitchd, and openvswitch-nonetwork is reserved > for the ovsdb-ser

Re: [ovs-dev] [RFC 2/3] rhel/ovsdb-server.service: Rename the nonetwork service

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 05:26:22PM -0400, Aaron Conole wrote: > Currently, openvswitch.service calls out to start > openvswitch-nonetwork.service. However, openvswitch-nonetwork.service > is better called ovsdb-server, since that is truly nonetwork. This > commit does make the file a bit of a mis

Re: [ovs-dev] [RFC 1/3] utilities/ovs-ctl.in: Allow non-monitoring daemons

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 05:26:21PM -0400, Aaron Conole wrote: > This commit allows the ovs-ctl command to spawn daemons without the > internal process monitor. This is useful when integrating with, > ex. systemd, which provides its own monitoring facilities. > > Signed-off-by: Aaron Conole Woul

Re: [ovs-dev] [PATCH 0/2] Misc. fixes to ovs-tcpdump

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 04:59:39PM -0400, Aaron Conole wrote: > From: Aaron Conole > > The ovs-tcpdump framework has a few minor issues related to edge cases, > such as dpdk bonds, and some tcpdump / tshark command line options. These > changes are some minor fixups for these cases. This was te

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

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 12:43:02PM -0700, William Tu wrote: > 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

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

2016-07-01 Thread Ben Pfaff
Joe, are you the right person to review this? On Fri, Jul 01, 2016 at 09:45:52AM -0700, William Tu wrote: > We generally try to keep the testsuite independent of the underlying > datapath. This patch removes the datapath-specific tests and macros. > > Tested-at: https://travis-ci.org/williamtu/ov

Re: [ovs-dev] [PATCH] pvector: Expose non-concurrent priority vector.

2016-07-01 Thread Ben Pfaff
On Tue, Jun 21, 2016 at 05:18:45PM -0700, Jarno Rajahalme wrote: > PMD threads use pvectors but do not need the overhead of the > concurrent version. Expose the internal non-concurrent version for > that use. > > Signed-off-by: Jarno Rajahalme Seems like a good idea, if we have a potential user

Re: [ovs-dev] [PATCH v5 00/11] Windows: Add support for debugging conntrack from userspace

2016-07-01 Thread Guru Shetty
On 1 July 2016 at 13:54, Nithin Raju wrote: > Good job with all the patches! I believe I have acked all of them. Thanks > for addressing all the comments. > > I applied the series, thank you! > -Original Message- > From: dev on behalf of Sairam Venugopal > > Date: Friday, July 1, 201

Re: [ovs-dev] [PATCH 2/2] ovs-ofctl: Clarify CT action documentation.

2016-07-01 Thread Ben Pfaff
On Tue, Jun 21, 2016 at 09:57:16AM -0700, Jarno Rajahalme wrote: > Since the 'commit' flag is required to set the mark and/or labels, the > set values are always available for following lookups. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff _

Re: [ovs-dev] [PATCH 1/2] tests: Ignore proxy configuration in kernel testsuite.

2016-07-01 Thread Ben Pfaff
On Tue, Jun 21, 2016 at 09:57:15AM -0700, Jarno Rajahalme wrote: > Add '--noproxy' parameter to 'wget' so that the test cases do not try > to use a proxy that may be configured in the test environment. > > Signed-off-by: Jarno Rajahalme We could do something like this to turn it off without mess

Re: [ovs-dev] [PATCH] route-table: Fix memory leak reported by valgrind.

2016-07-01 Thread Ben Pfaff
Hi William, please try this patch as a substitute for yours. It should ensure that pointers to nln_notifiers are to the beginning of the structs instead of to the middle, meaning that valgrind does not consider them "possible" leaks. diff --git a/lib/netlink-notifier.c b/lib/netlink-notifier.c in

Re: [ovs-dev] [PATCH] lib: Remove extra API dependency for ovs_thread_create()

2016-07-01 Thread Ben Pfaff
On Fri, Jun 17, 2016 at 04:25:32PM -0700, Andy Zhou wrote: > When calling ovs_thread_create() without calling fatal_signal_init() > first, ovs_thread_create() some times asserts. This dependency is > subtle and not very obvious. > > The root cause seems to be that, within ovs_thread_create(), the

[ovs-dev] hi!

2016-07-01 Thread dev
Hello! We are looking for employees working remotely. My name is Carey, I 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] Gift Suppler- China

2016-07-01 Thread shang kun
Dear Sir, Glad to get your info from China. We supply GIFT with good quality & competitive price meeting your purchase requirements.the best price will be provided after receipt of your requirements. 10 years experience in professional GIFT, capable production, promised delivery time and high l

Re: [ovs-dev] [PATCH 1/2] utilities: ovs-ctl: Drop duplicate line

2016-07-01 Thread Aaron Conole
Markos Chandras writes: > The --ovs-vswitchd-wrapper=WRAPPER line is referenced twice so drop it. > > Signed-off-by: Markos Chandras > --- > utilities/ovs-ctl.in | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/utilities/ovs-ctl.in b/utilities/ovs-ctl.in > index b2a97fc..8ec825b 100755 >

Re: [ovs-dev] [PATCH 2/2] utilities: ovs-ctl.in: Allow passing DPDK options to ovs-vswitchd

2016-07-01 Thread Aaron Conole
Markos Chandras writes: > The ovs-ctl script is used to launch ovs-vswitchd among other things. > However it does not make it possible to pass DPDK options to the > daemon. We fix this by explicitly looking and extracting the DPDK > options from the command line which is then reconstructed so it

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

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 03:11:04AM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > 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 7b6

[ovs-dev] [PATCH v2] ovs-pki: Use SHA-512 instead of SHA-1 as message digest.

2016-07-01 Thread Ben Pfaff
The upcoming OpenSSL 1.1.0 release disables use of SHA-1, which breaks the OVS unit tests, which use SHA-1. We last tried to switch to SHA-512 in 2014 with commit 9ff33ca75e9fcc ("ovs-pki: Use SHA-512 instead of MD5 as message digest."), but we had to downgrade to SHA-1 in commit 4a1f9610682d ("ov

[ovs-dev] [PATCH v3 24/28] datapath: backport: geneve: fix tx_errors statistics

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit efeb2267bba8aa893afdadfc9bae4790777c600c Author: Haishuang Yan geneve: fix tx_errors statistics Tx errors present summation of errors encountered while transmitting packets. Signed-off-by: Haishuang Yan Signed-off-by: David S. Miller Signed

[ovs-dev] [PATCH v3 26/28] datapath: backport: ovs/geneve: fix rtnl notifications on iface deletion

2016-07-01 Thread Pravin B Shelar
Upstream Commit: commit 41009481b690493c169ce85f591b9d32c6fd9422 Author: Nicolas Dichtel ovs/geneve: fix rtnl notifications on iface deletion The function geneve_dev_create_fb() (only used by ovs) never calls rtnl_configure_link(). The consequence is that dev->rtnl_link_state

[ovs-dev] [PATCH v3 25/28] datapath: backport: ovs/gre, geneve: fix error path when creating an iface

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit 106da663ff495e0aea3ac15b8317aa410754fcac Author: Nicolas Dichtel ovs/gre,geneve: fix error path when creating an iface After ipgre_newlink()/geneve_configure() call, the netdev is registered. Fixes: 7e059158d57b ("vxlan, gre, geneve: Set a large MTU o

Re: [ovs-dev] Bug#828478: [PATCH] ovs-pki: Use SHA-512 message digest when available.

2016-07-01 Thread Ben Pfaff
On Sun, Jun 26, 2016 at 08:55:04PM +0200, Kurt Roeckx wrote: > On Sun, Jun 26, 2016 at 11:05:35AM -0700, Ben Pfaff wrote: > > The upcoming OpenSSL 1.1.0 release disables use of SHA-1, which breaks the > > OVS unit tests, which use SHA-1. We last tried to switch to SHA-512 in > > 2014 with commit 9

[ovs-dev] [PATCH v3 27/28] datapath: backport: ovs/vxlan: fix rtnl notifications on iface deletion

2016-07-01 Thread Pravin B Shelar
At this point OVS tunneling is insync with upstream net branch (commit 9a0fee2b552b1). upstream commit: commit cf5da330bbdd0c06b05c525a3d1d58ccd82c87a6 Author: Nicolas Dichtel ovs/vxlan: fix rtnl notifications on iface deletion The function vxlan_dev_create() (only used by ovs)

[ovs-dev] [PATCH v3 23/28] datapath: backport: udp: prevent skbs lingering in tunnel socket queues

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit e5aed006be918af163eb397e45aa5ea6cefd5e01 Author: Hannes Frederic Sowa udp: prevent skbs lingering in tunnel socket queues In case we find a socket with encapsulation enabled we should call the encap_recv function even if just a udp header without paylo

[ovs-dev] [PATCH v3 21/28] datapath: backport: geneve: add IPv6 bits to geneve_fill_metadata_dst

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit b8812fa88371ae567c907448d9a7ba62d09b90c9 Author: John W. Linville geneve: add IPv6 bits to geneve_fill_metadata_dst Signed-off-by: John W. Linville Reviewed-by: Jesse Gross Signed-off-by: David S. Miller Signed-off-by: Pravin B Shelar Acked-by

[ovs-dev] [PATCH v3 00/28] datapath: compat: Add support for IPv6 Geneve and VxLAN tunnels.

2016-07-01 Thread Pravin B Shelar
This patch series update OVS compat layer to handle IPv6 UDP based tunnels. While doing this various fixes and updates to tunnel code are also brought in. This series sync OVS tunnel compat codebase upto latest net branch (commit 9a0fee2b552b1235fb1706ae1fc664ae74573be8). v2-v3: Addressed comments

[ovs-dev] [PATCH v3 22/28] datapath: backport: udp_offload: Set encapsulation before inner completes.

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit 229740c63169462a838a8b8e16391ed000934631 Author: Jarno Rajahalme udp_offload: Set encapsulation before inner completes. UDP tunnel segmentation code relies on the inner offsets being set for an UDP tunnel GSO packet, but the inner *_complete() function

[ovs-dev] [PATCH v3 28/28] datapath: backport: iptunnel: make rx/tx bytes counters consistent

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit bc22a0e2ea03b75b51a1f722f93821744b5b5ff1 Author: Nicolas Dichtel iptunnel: make rx/tx bytes counters consistent This was already done a long time ago in commit 64194c31a0b6 ("inet: Make tunnel RX/TX byte counters more consistent") but tx path was broken (

[ovs-dev] [PATCH v3 20/28] datapath: compat: get rid of OVS_CB inner header offsets.

2016-07-01 Thread Pravin B Shelar
OVS has GSO compat functionality which needs inner offset of the packet to segment a packet. older kernel did not include these offsets in skb, therefore these were stored in OVS_GSO_CB. Now OVS has dropped support for these old kernel, So none of the supported kernel needs this comapt code. Follow

[ovs-dev] [PATCH v3 18/28] datapath: compat: Use dst-cache for Geneve and VxLAN tunnels.

2016-07-01 Thread Pravin B Shelar
It partialy backport commit: commit d71785ffc7e7cae3fbdc4ea8a9d05b7a1c59f7b8 Author: Paolo Abeni net: add dst_cache to ovs vxlan lwtunnel In case of UDP traffic with datagram length below MTU this give about 2% performance increase when tunneling over ipv4 and about 60% w

[ovs-dev] [PATCH v3 17/28] datapath: Add support for IPv6 tunnels.

2016-07-01 Thread Pravin B Shelar
Mostly backports upstream commit along with other pieces to make IPv6 tunneling work. commit 6b26ba3a7d952e611dcde1f3f77ce63bcc70540a Author: Jiri Benc openvswitch: netlink attributes for IPv6 tunneling Add netlink attributes for IPv6 tunnel addresses. This enables IPv6 support

[ovs-dev] [PATCH v3 19/28] datapath: backport: openvswitch: correct encoding of set tunnel action attributes

2016-07-01 Thread Pravin B Shelar
upstream commit: commit e905eabc90a5b787d8708df164543ee295bea5f2 Author: Simon Horman openvswitch: correct encoding of set tunnel action attributes In a set action tunnel attributes should be encoded in a nested action. I noticed this because ovs-dpctl was reporting an e

[ovs-dev] [PATCH v3 15/28] datapath: compat: Add support for IPv6 UDP tunnel segmentation.

2016-07-01 Thread Pravin B Shelar
Next patch adds support for IPV6 Geneve and VXLAN, But support for UDP segmentation is available on all supported kernel. Following patch adds support for UDP tunnels over IPv6 for such kernels. Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross --- datapath/linux/compat/gso.c

[ovs-dev] [PATCH v3 14/28] datapath: compat: Prepare tnl-segmentation for ipv6.

2016-07-01 Thread Pravin B Shelar
Current tnl_skb_gso_segment() is written for ipv4 tunnels. Following patch make it generic so that it can also handle ipv6 tunnels. Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross --- datapath/linux/compat/gso.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a

[ovs-dev] [PATCH v3 11/28] datapath: compat: Update IPv6 in setup_udp_tunnel_sock()

2016-07-01 Thread Pravin B Shelar
Update setup_udp_tunnel_sock() to handle ipv6 sockets. Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross --- datapath/linux/compat/include/net/udp_tunnel.h | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/datapath/linux/compat/include/net/udp_tunnel.h b/d

[ovs-dev] [PATCH v3 13/28] datapath: backport: ip_tunnel: Move stats update to iptunnel_xmit()

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit 039f50629b7f860f36644ed1f34b27da9aa62f43 Author: Pravin B Shelar ip_tunnel: Move stats update to iptunnel_xmit() By moving stats update into iptunnel_xmit(), we can simplify iptunnel_xmit() usage. With this change there is no need to call another f

[ovs-dev] [PATCH v3 10/28] datapath: compat: Update udp_sock_create

2016-07-01 Thread Pravin B Shelar
Update udp-socket-create to create ipv6 socket currectly. Partially backports commit fd384412e199b ("udp_tunnel: Seperate ipv6 functions into its own file.") Signed-off-by: Pravin B Shelar --- acinclude.m4 | 2 + datapath/linux/Modules.mk

[ovs-dev] [PATCH v3 09/28] datapath: compat: rename HAVE_METADATA_DST to USE_UPSTREAM_TUNNEL

2016-07-01 Thread Pravin B Shelar
To better represent the meaning of symbol. Signed-off-by: Pravin B Shelar --- acinclude.m4 | 2 +- datapath/linux/compat/geneve.c | 2 +- datapath/linux/compat/gre.c | 4 ++-- datapath/linux/compat/gso.h

[ovs-dev] [PATCH v3 12/28] datapath: backport: udp: Add socket based GRO and config

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit 38fd2af24fcfda93f9fea3e53f26e48775ae9e09 Author: Tom Herbert udp: Add socket based GRO and config Add gro_receive and gro_complete to struct udp_tunnel_sock_cfg. Signed-off-by: Tom Herbert Signed-off-by: David S. Miller Signed-off-by: Pravin B

[ovs-dev] [PATCH v3 08/28] datapath: backport: ip_tunnel: add support for setting flow label via collect metadata

2016-07-01 Thread Pravin B Shelar
Update udp_tunnel6_xmit_skb(). Specificaly changes are related to setting ipv6 label. Upstream commit: commit 134611446dc657e1bbc73ca0e4e6b599df687db0 Author: Daniel Borkmann ip_tunnel: add support for setting flow label via collect metadata This patch extends udp_tunnel6_xmit_s

[ovs-dev] [PATCH v3 07/28] datapath: compat: Remove unnecessary iptunnel_xmit() declaration.

2016-07-01 Thread Pravin B Shelar
Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross --- datapath/linux/compat/include/net/ip_tunnels.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/datapath/linux/compat/include/net/ip_tunnels.h b/datapath/linux/compat/include/net/ip_tunnels.h index cea75a1..b9fa76f 100644 --

[ovs-dev] [PATCH v3 05/28] datapath: backport: ip_tunnel_core: iptunnel_handle_offloads returns int and doesn't free skb

2016-07-01 Thread Pravin B Shelar
There is return type change in upstream handle-offload functions. Following patch brings these changes in. This is backport of aed069df ("ip_tunnel_core: iptunnel_handle_offloads returns int and doesn't free skb") I have also removed duplicate definitions of tunnel_handle_offloads() from ip-tunnel

[ovs-dev] [PATCH v3 06/28] datapath: backport: tunnel: introduce ipv6_tun_rx_dst()

2016-07-01 Thread Pravin B Shelar
Update ovs_udp_tun_rx_dst() to handle ipv6 tunnels. This commit partially backports c29a70d2c ("tunnel: introduce udp_tun_rx_dst()") Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross --- datapath/linux/compat/include/net/dst_metadata.h | 46 ++-- datapath/linux/compat/in

[ovs-dev] [PATCH v3 04/28] datapath: backport: net: add dst_cache support

2016-07-01 Thread Pravin B Shelar
This backports dst-cache implementation from upstream implementation. commit 911362c70df5b766c243dc297fadeaced786ffd8 Author: Paolo Abeni net: add dst_cache support This patch add a generic, lockless dst cache implementation. The need for lock is avoided updating the dst cach

[ovs-dev] [PATCH v3 03/28] datapath: backport: tunnels: Remove encapsulation offloads on decap.

2016-07-01 Thread Pravin B Shelar
Following patch backports updated iptunnel pull function. Also brings in following upstream fix: commit a09a4c8dd1ec7f830e1fb9e59eb72bddc965d168 Author: Jesse Gross tunnels: Remove encapsulation offloads on decap. If a packet is either locally encapsulated or processed through G

[ovs-dev] [PATCH v3 01/28] datapath: compat: Refactor egress tunnel info

2016-07-01 Thread Pravin B Shelar
upstream tunnel egress info is retrieved using ndo_fill_metadata_dst. Since we do not have it on older kernel we need to keep vport operation to do same on these kernels. Following patch try to merge these to operations into one to avoid code duplication. This commit backports fc4099f1 ("openvswitc

[ovs-dev] [PATCH v3 02/28] datapath: compat: update iptunnel_pull_header()

2016-07-01 Thread Pravin B Shelar
Introduce xnet parameter to iptunnel_pull_header(). Signed-off-by: Pravin B Shelar --- acinclude.m4 | 1 + datapath/linux/compat/geneve.c | 3 +-- datapath/linux/compat/gre.c| 2 +- datapath/linux/compat/include/linux/skbuf

Re: [ovs-dev] [ovs-discuss] OVS fdb - MAC learning Subscription

2016-07-01 Thread Ben Pfaff
On Mon, Jun 27, 2016 at 07:21:53PM +0530, Kapil Adhikesavalu wrote: > is there a way to get notification on MACs learnt on a OVS bridge ? OVS doesn't provide a way to do that. If you implement MAC learning using the "learn" action, e.g. as described in the OVS tutorial under the tutorial/ directo

Re: [ovs-dev] [PATCH v2] ovsdb: Expose vhost-user socket directory in ovsdb

2016-07-01 Thread Ben Pfaff
On Mon, Jun 20, 2016 at 10:16:51AM +, Wojciechowicz, RobertX wrote: > Hi, > > > -Original Message- > > From: Ben Pfaff [mailto:b...@ovn.org] > > Sent: Wednesday, June 8, 2016 10:41 PM > > To: Wojciechowicz, RobertX > > Cc: dev@openvswitch.org > > Subject: Re: [ovs-dev] [PATCH v2] ovsd

Re: [ovs-dev] [PATCH v2 14/24] datapath: compat: Update Geneve and VxLAN modules.

2016-07-01 Thread pravin shelar
On Thu, Jun 30, 2016 at 7:38 PM, Jesse Gross wrote: > On Thu, Jun 30, 2016 at 1:06 AM, Pravin B Shelar wrote: >> This patch brings in various updates to upstream Geneve and VxLAN >> modules. For geneve this patch adds IPv6 support, for vxlan it adds >> VXLAN GPE feature. >> This should make OVS c

Re: [ovs-dev] [ovs-discuss] Sampling in datapath

2016-07-01 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 05:45:43PM +0300, Marcelo Caggiani Luizelli wrote: > I've seen that there is an implementation of sampling function in the ovs > linux datapath (in file datapath/actions.c). The issue is that this > functions is sending the packet to userspace (usually used by ipix/sflow > s

Re: [ovs-dev] Flow actions handling

2016-07-01 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 07:52:10PM +0530, Kapil Adhikesavalu wrote: > Hi, > > i have the following flow rule, > > *tun_id=0xfa0,in_port=10 actions=mod_vlan_vid:1,resubmit(0,2),resubmit(0,1)* > > and i need to know in which order actions will be performed. > > will the actions be performed in th

Re: [ovs-dev] [ovs-discuss] About the QoS of OpenvSwitch

2016-07-01 Thread Ben Pfaff
On Thu, Jun 30, 2016 at 06:15:05AM +, Xiao Ma (xima2) wrote: > Hi, All > > I want to use the QoS of OpenvSwitch to control the bandwidth based on the > vlan id. > So I deployed it as the topology attached,and configured the flow, the qos > rule, and used iperf tool to test it. > But the resu

Re: [ovs-dev] The name of logical switch and router is null

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 12:38:54AM +0800, nickcooper-zhangtonghao wrote: > When I create one or more logical router named ’null', using 'ovn-nbctl > lr-add’, there is no error information. > This is arguably an ovn-nbctl bug. It's not a bug. Any number of unnamed logical switches or logica

Re: [ovs-dev] The QoS feature minimum guaranteed bandwidth of OpenvSwitch not work

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 03:40:30AM +, Xiao Ma (xima2) wrote: > I want to use the QoS feature of OpenvSwitch to control the bandwidth based > on the vlan id(Scene 1) or port id(Scene 2). > So I deployed it as showed bellow,and configured the qos rules,the flows,and > used iperf tool to test it

Re: [ovs-dev] [PATCH 2/2] ovn: Add a case of policy based routing.

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 03:54:36PM -0700, Guru Shetty wrote: > On 1 July 2016 at 14:36, Ben Pfaff wrote: > > > On Fri, Jul 01, 2016 at 03:23:39AM -0700, Gurucharan Shetty wrote: > > > OVN currently supports multiple gateway routers (residing on > > > different chassis) connected to the same logic

Re: [ovs-dev] [PATCH V10] ovn-controller: reload configured SB probe timer

2016-07-01 Thread Ben Pfaff
On Wed, Jun 15, 2016 at 05:05:25PM -0700, ngh...@us.ibm.com wrote: > From: Nirapada Ghosh > > The probe timer between ovn-controller and OVN Southbound > can be configured using ovn-vsctl command, but that is not > effective on the fly. In other words, ovn-controller has > to be restarted to use

Re: [ovs-dev] [PATCH 2/2] ovn: Add a case of policy based routing.

2016-07-01 Thread Guru Shetty
On 1 July 2016 at 14:36, Ben Pfaff wrote: > On Fri, Jul 01, 2016 at 03:23:39AM -0700, Gurucharan Shetty wrote: > > OVN currently supports multiple gateway routers (residing on > > different chassis) connected to the same logical topology. > > > > When external traffic enters the logical topology,

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

2016-07-01 Thread Guru Shetty
On 1 July 2016 at 14:19, Ben Pfaff wrote: > On Fri, Jul 01, 2016 at 03:23:38AM -0700, Gurucharan Shetty wrote: > > The command 'lr-route-add' currently cannot update the output > > port. > > > > Signed-off-by: Gurucharan Shetty > > Our other "add" commands that take "--may-exist" insist that the

[ovs-dev] openvswitch_2.5.1~pre+git20160626-2_i386.changes ACCEPTED into unstable

2016-07-01 Thread Debian FTP Masters
Accepted: -BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Format: 1.8 Date: Fri, 01 Jul 2016 15:05:32 -0700 Source: openvswitch Binary: openvswitch-common openvswitch-switch openvswitch-ipsec openvswitch-pki openvswitch-testcontroller openvswitch-dbg python-openvswitch openvswitch-test openvs

[ovs-dev] Processing of openvswitch_2.5.1~pre+git20160626-2_i386.changes

2016-07-01 Thread Debian FTP Masters
openvswitch_2.5.1~pre+git20160626-2_i386.changes uploaded successfully to localhost along with the files: openvswitch_2.5.1~pre+git20160626-2.dsc openvswitch_2.5.1~pre+git20160626-2.debian.tar.xz openvswitch-common_2.5.1~pre+git20160626-2_i386.deb openvswitch-dbg_2.5.1~pre+git20160626-2_i3

Re: [ovs-dev] [PATCH v4] lib: Use a more accurate value for CPU count (sched_getaffinity).

2016-07-01 Thread Ben Pfaff
On Mon, Jun 27, 2016 at 04:51:49PM +0200, Quentin Monnet wrote: > 2016-06-23 (15:59 -0700) ~ Ben Pfaff > > On Thu, Jun 09, 2016 at 11:52:49AM +0200, Quentin Monnet wrote: > >> Signed-off-by: David Marchand > >> Signed-off-by: Liu Xiaofeng > >> Signed-off-by: Quentin Monnet > > > > Who is the a

Re: [ovs-dev] [PATCH 2/2] ovn: Add a case of policy based routing.

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 03:23:39AM -0700, Gurucharan Shetty wrote: > OVN currently supports multiple gateway routers (residing on > different chassis) connected to the same logical topology. > > When external traffic enters the logical topology, they can enter > from any gateway routers and reach

[ovs-dev] [RFC 3/3] Red Hat Systemd Integration

2016-07-01 Thread Aaron Conole
This commit builds upon some of the recent ovs-ctl changes to build a more integrated systemd setup. A new service (openvswitch-network) is added to track the ovs-vswitchd, and openvswitch-nonetwork is reserved for the ovsdb-server daemon. The systemd scripts still use ovs-ctl to actually initial

[ovs-dev] [RFC 2/3] rhel/ovsdb-server.service: Rename the nonetwork service

2016-07-01 Thread Aaron Conole
Currently, openvswitch.service calls out to start openvswitch-nonetwork.service. However, openvswitch-nonetwork.service is better called ovsdb-server, since that is truly nonetwork. This commit does make the file a bit of a misnomer - currently the ovsdb-server service will start the ovs-vswitchd

[ovs-dev] [RFC 1/3] utilities/ovs-ctl.in: Allow non-monitoring daemons

2016-07-01 Thread Aaron Conole
This commit allows the ovs-ctl command to spawn daemons without the internal process monitor. This is useful when integrating with, ex. systemd, which provides its own monitoring facilities. Signed-off-by: Aaron Conole --- utilities/ovs-ctl.8 | 5 + utilities/ovs-ctl.in | 1 + utilities/ov

[ovs-dev] [RFC 0/3] redhat: Improve the systemd integration

2016-07-01 Thread Aaron Conole
These patches make adjustments to the way systemd is done so that we have: * one daemon started per service * services which correspond to actual daemon names * a single 'dummy' service to start all at once * some convenient information to poll from systemd. These were tested quite a bit, but I'm

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

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 03:23:38AM -0700, Gurucharan Shetty wrote: > The command 'lr-route-add' currently cannot update the output > port. > > Signed-off-by: Gurucharan Shetty Our other "add" commands that take "--may-exist" insist that the "add" command is *not* changing anything. I don't know

Re: [ovs-dev] [PATCH] datapath-windows: Cleanup conntrack-tcp.c

2016-07-01 Thread Guru Shetty
On 30 June 2016 at 16:18, Nithin Raju wrote: > >>On 6/24/16, 6:14 PM, "Sairam Venugopal" wrote: > >> > >>>Update the code to use tcp->flags. This keeps the kernel conntrack-tcp.c > >>>file in sync with userspace version. > >>> > >>>This patch also addresses an warning - 'Comparison of a boolean

Re: [ovs-dev] [PATCH v5] ovn: Add software l2 gateway.

2016-07-01 Thread Russell Bryant
On Fri, Jun 24, 2016 at 9:29 PM, Justin Pettit wrote: > > > On Jun 23, 2016, at 12:34 PM, Russell Bryant wrote: > > > > This patch implements one approach to using ovn-controller to implement > > a software l2 gateway between logical and physical networks. > > > > A new logical port type called

[ovs-dev] [PATCH 2/2] utilities/ovs-tcpdump.in: Poll the process status

2016-07-01 Thread Aaron Conole
From: Aaron Conole Some options (such as -c X), when passed to tcpdump will cause it to halt. When this occurs, ovs-tcpdump will not recognize that such an event has happened, and will spew newlines across the screen running forever. To fix this, ovs-tcpdump can poll and then raise a KeyboardIn

[ovs-dev] [PATCH 1/2] utilities/ovs-tcpdump.in: Fix port lookups

2016-07-01 Thread Aaron Conole
From: Aaron Conole The original implementation of ovs-tcpdump conflated interfaces and ports needlessly. This commit changes ovs-tcpdump to only consider the port name when looking up the corresponding bridge. Reported-by: Jean-Tsung Hsiao Tested-by: Jean-Tsung Hsiao Signed-off-by: Aaron Cono

[ovs-dev] [PATCH 0/2] Misc. fixes to ovs-tcpdump

2016-07-01 Thread Aaron Conole
From: Aaron Conole The ovs-tcpdump framework has a few minor issues related to edge cases, such as dpdk bonds, and some tcpdump / tshark command line options. These changes are some minor fixups for these cases. This was tested with DPDK ports, non-dpdk ports, and by torturing the dump-cmd to t

Re: [ovs-dev] [PATCH v5 00/11] Windows: Add support for debugging conntrack from userspace

2016-07-01 Thread Nithin Raju
Good job with all the patches! I believe I have acked all of them. Thanks for addressing all the comments. -Original Message- From: dev on behalf of Sairam Venugopal Date: Friday, July 1, 2016 at 1:49 PM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v5 00/11] Windows: Add support

Re: [ovs-dev] [PATCH v5 10/11] Windows: Add conntrack dump and flush support in userspace

2016-07-01 Thread Nithin Raju
Acked-by: Nithin Raju -Original Message- From: dev on behalf of Sairam Venugopal Date: Friday, July 1, 2016 at 1:49 PM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v5 10/11] Windows: Add conntrack dump and flush support in userspace >Modify dpif-netlink.c and netlink-conntrac

Re: [ovs-dev] [PATCH v5 04/11] datapath-windows: Add support for flushing conntrack entries

2016-07-01 Thread Nithin Raju
Acked-by: Nithin Raju -Original Message- From: dev on behalf of Sairam Venugopal Date: Friday, July 1, 2016 at 1:49 PM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v5 04/11] datapath-windows: Add support for flushing conntrack entries >Flush out all conntrack entries or th

[ovs-dev] [PATCH v5 01/11] Windows: Add conntrack netfilter netlink definitions to kernel and userspace

2016-07-01 Thread Sairam Venugopal
Include netfilter-conntrack header definitions. This will be used by Windows userspace for adding debugging support in Conntrack. Few of these files are intentionally left blank to avoid removing #includes in userspace. New file - OvsDpInterfaceCtExt.h has been defined similar to OvsDpInterfaceExt.

[ovs-dev] [PATCH v5 05/11] datapath-windows: Add support for Conntrack IPCTNL_MSG_CT_DELETE cmd in Datapath.c

2016-07-01 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 v5 10/11] Windows: Add conntrack dump and flush support in userspace

2016-07-01 Thread Sairam Venugopal
Modify dpif-netlink.c and netlink-conntrack.c to send down dump and 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,

[ovs-dev] [PATCH v5 06/11] datapath-windows: Add support for dump-conntrack in datapath

2016-07-01 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 Acked-by: Nithin

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

2016-07-01 Thread Sairam Venugopal
This will be used by userspace for dumping conntrack entries - "ovs-dpctl dump-conntrack". Signed-off-by: Sairam Venugopal Acked-by: Paul-Daniel Boca Acked-by: Nithin Raju --- datapath-windows/ovsext/Datapath.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/dat

[ovs-dev] [PATCH v5 07/11] datapath-windows: Conntrack - Handle memory allocation failure

2016-07-01 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 Acked-by: Nithin Raju --- datapath-windows/ovsext/Conntrack-other.c | 5 +++-- datapath-windows/ovsext/Conntrack-tcp.c | 4 2 files changed, 7 insertions(+

[ovs-dev] [PATCH v5 04/11] datapath-windows: Add support for flushing conntrack entries

2016-07-01 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 | 93 ++

[ovs-dev] [PATCH v5 00/11] Windows: Add support for debugging conntrack from userspace

2016-07-01 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. v2

[ovs-dev] [PATCH v5 02/11] datapath-windows: Add support for Netfilter netlink message

2016-07-01 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 Acked-by: Nithin Raju --- datapath-win

[ovs-dev] [PATCH v5 03/11] datapath-windows: Add new NlFillOvsMsgForNfGenMsg method in Netlink.c

2016-07-01 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 Acked-by: Nithin Raju --- datapath-windows/ovsext/Netlink/Netlink.c | 39 +++

[ovs-dev] [PATCH v5 09/11] datapath-windows: Track the number of conntrack entries

2016-07-01 Thread Sairam Venugopal
Add a counter to track the number of connection tracking entries. Iterate over the conntrack entry table only if there are entries. Signed-off-by: Sairam Venugopal Acked-by: Nithin Raju --- datapath-windows/ovsext/Conntrack.c | 110 1 file changed, 62 insert

[ovs-dev] [PATCH v5 11/11] Windows: Ignore the dpif conversions for windows in netlink-conntrack.c

2016-07-01 Thread Sairam Venugopal
The flags and state sent by Windows datapath are currently in the userspace format. So prevent further translation. Signed-off-by: Sairam Venugopal Acked-by: Nithin Raju --- lib/netlink-conntrack.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lib/netlink-conntrack.c b/lib/net

Re: [ovs-dev] [PATCH] build: Skip datapath subdir on Windows

2016-07-01 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 06:36:12PM +, Alin Serdean wrote: > 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: > https://github.com/openvswitch/ovs/blob/master/datapath/Makefile.am#L34 > > and

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

2016-07-01 Thread Gurucharan Shetty
The command 'lr-route-add' currently cannot update the output port. Signed-off-by: Gurucharan Shetty --- ovn/utilities/ovn-nbctl.c | 4 tests/ovn-nbctl.at| 10 +- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/ovn/utilities/ovn-nbctl.c b/ovn/utilities/ovn-nb

[ovs-dev] [PATCH 2/2] ovn: Add a case of policy based routing.

2016-07-01 Thread Gurucharan Shetty
OVN currently supports multiple gateway routers (residing on different chassis) connected to the same logical topology. When external traffic enters the logical topology, they can enter from any gateway routers and reach its eventual destination. This is achieved with proper static routes configur

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-01 Thread Aaron Rosen
Yea sorry about that. I'll try and make sure I do that next time around. Aaron On Fri, Jul 1, 2016 at 7:39 AM, Ryan Moats wrote: > "dev" wrote on 06/30/2016 04:27:19 PM: > > > From: Aaron Rosen > > To: dev@openvswitch.org > > Cc: Aaron Rosen > > Date: 06/30/2016 04:29 PM > > Subject: [ovs-de

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

2016-07-01 Thread William Tu
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

Re: [ovs-dev] [PATCH v3 0/5] create tunnel devices using rtnetlink interface

2016-07-01 Thread Jesse Gross
On Fri, Jul 1, 2016 at 10:53 AM, Thadeu Lima de Souza Cascardo wrote: > On Thu, Jun 30, 2016 at 03:44:37PM -0700, Jesse Gross wrote: >> On Thu, Jun 30, 2016 at 12:25 PM, Thadeu Lima de Souza Cascardo >> wrote: >> > On Wed, Jun 29, 2016 at 09:38:00PM -0700, Jesse Gross wrote: >> >> On Wed, Jun 22,

Re: [ovs-dev] [PATCH V5 06/17] python tests: Skip IPsec test on Windows

2016-07-01 Thread Alin Serdean
Also paths in the test do not exist :) i.e. : " mkdir etc etc/init.d etc/racoon etc/racoon/certs mkdir usr usr/sbin" Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Friday, July 1, 2016 7:27 PM > Către: de

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

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit 1c1779fa54b2a9d4e1de990095d790d64b9e00a1 Author: Jarno Rajahalme 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 even

[ovs-dev] [PATCH branch-2.5 2/2] datapath: Only set mark and labels with a commit flag.

2016-07-01 Thread Pravin B Shelar
Upstream commit: commit 7d904c7bcd51f72579c0c3134a50896c5a3efb9f Author: Jarno Rajahalme Date: Tue Jun 21 14:59:38 2016 -0700 openvswitch: Only set mark and labels with a commit flag. Only set conntrack mark or labels when the commit flag is specified. This makes sure w

Re: [ovs-dev] [PATCH 2/2] datapath: Only set mark and labels with a commit flag.

2016-07-01 Thread pravin shelar
On Wed, Jun 29, 2016 at 7:14 AM, Joe Stringer wrote: > On 29 June 2016 at 15:35, Jarno Rajahalme wrote: >> Upstream commit: >> commit 7d904c7bcd51f72579c0c3134a50896c5a3efb9f >> Author: Jarno Rajahalme >> Date: Tue Jun 21 14:59:38 2016 -0700 >> >> openvswitch: Only set mark and

[ovs-dev] [PATCH] build: Skip datapath subdir on Windows

2016-07-01 Thread Alin Serdean
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: https://github.com/openvswitch/ovs/blob/master/datapath/Makefile.am#L34 and can be seen on: https://ci.appveyor.com/project/blp/ovs/build/1.0.1851#L322

  1   2   >