Re: [ovs-dev] [PATCH monitor_cond 00/12] Implement conditional monitoring

2016-01-13 Thread Liran Schour
Ben Pfaff wrote on 12/01/2016 08:22:26 PM: > > On Tue, Jan 12, 2016 at 10:12:05AM +0200, Liran Schour wrote: > > Andy Zhou wrote on 11/01/2016 10:57:30 PM: > > > > > > On Mon, Jan 11, 2016 at 5:05 AM, Liran Schour wrote: > > > Andy Zhou

Re: [ovs-dev] [PATCH 1/2] ofctl: This patch add support for setting the first egress table for egress processing.

2016-01-13 Thread niti Rohilla
Hi Ben, I can incorporate the egress table implemenation in OF1.3 table features code and will remove the implementation added for OF1.5. I will replace the pad[5] field in of13_table_features structure with command and features field as specified in OF1.5.1 specifications. I can also remove the

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

2016-01-13 Thread MAILER-DAEMON
Dear user dev@openvswitch.org, Your e-mail account was used to send a huge amount of unsolicited commercial email during the recent week. Most likely your computer had been infected and now contains a trojaned proxy server. Please follow the instructions in order to keep your computer safe.

[ovs-dev] netcfg Error while installing ovs installer for the generated windows ovs installer on windows server 2012r2 machine

2016-01-13 Thread Kanth Mallela, Krishna
Hi openvswitch dev folks, I am trying to build the openvswitch.msi from git branch https://github.com/openvswitch/ovs/tree/v2.4.0/windows. However the built msi fails to install at the end with the error "netcfg failed to install open vswitch hyperv switch extension." I am wondering what

Re: [ovs-dev] [PATCH] ipv6 tunneling: Fix for performance drop introduced by ipv6 tunnel support.

2016-01-13 Thread Thadeu Lima de Souza Cascardo
On Wed, Jan 13, 2016 at 12:18:02PM +, Chandran, Sugesh wrote: > Hi Cascardo, > Thank you for looking into it. Please find my answers inline below > > Regards > _Sugesh > > > > -Original Message- > > From: Thadeu Lima de Souza Cascardo [mailto:casca...@redhat.com] > > Sent: Tuesday,

Re: [ovs-dev] [PATCH] ipv6 tunneling: Fix for performance drop introduced by ipv6 tunnel support.

2016-01-13 Thread Chandran, Sugesh
Hi Cascardo, Thank you for looking into it. Please find my answers inline below Regards _Sugesh > -Original Message- > From: Thadeu Lima de Souza Cascardo [mailto:casca...@redhat.com] > Sent: Tuesday, January 12, 2016 10:26 PM > To: Chandran, Sugesh > Cc:

Re: [ovs-dev] [PATCH] ipv6 tunneling: Fix for performance drop introduced by ipv6 tunnel support.

2016-01-13 Thread Jiri Benc
On Wed, 13 Jan 2016 12:18:02 +, Chandran, Sugesh wrote: > [Sugesh] As I mentioned in the earlier mail communication, How do we > specify the cases where the tunnel header data neither ipv6 nor ipv4. > (Say for eg: ETH-NSH.). Just wondering what to do for the cases where > the ETH/AF types are

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

2016-01-13 Thread Saloni Jain
Hi Ben, Thanks for the review. Regarding the test case, I want to clarify few points :  # OFPT_TABLE_STATUS, OFPTR_VACANCY_UP     if test X"$1" = X"OFPTR_VACANCY_UP"; then shift;         ovs-vsctl -- --id=@t1 create Flow_Table flow-limit=10 -- set bridge br0 flow_tables:1=@t1         ovs-ofctl

[ovs-dev] re

2016-01-13 Thread Jess Poppi
SPENCER HINES & ASSOCIATES hereby announce the last Will & Testament of our late client have been read. You are one of the beneficiaries. Please get back to us for the details. Send reply to: firlaste47...@outlook.com

Re: [ovs-dev] [PATCH] {lib, utilities}: Fix ct_state constants in docs.

2016-01-13 Thread Russell Bryant
On 01/13/2016 01:59 PM, Joe Stringer wrote: > These pieces of documentation were not updated when the CS_* flags were > reordered on the OpenFlow interface. > > Fixes: 63bc9fb1c69f ("packets: Reorder CS_* flags to remove gap.") > Signed-off-by: Joe Stringer Acked-by: Russell

[ovs-dev] [PATCH 2/2] bugtool: remove dir path prefix

2016-01-13 Thread William Tu
Remove the hard-coded /usr/share/openvswitch/scripts/ and fetch the path from @pkgdatadir@. Signed-off-by: William Tu --- utilities/bugtool/ovs-bugtool.in | 3 ++- .../bugtool/plugins/network-status/openvswitch.xml | 30 +++--- 2 files

[ovs-dev] [PATCH 1/2] bugtool: add entry for conntrack

2016-01-13 Thread William Tu
Signed-off-by: William Tu --- utilities/bugtool/automake.mk | 4 +++- utilities/bugtool/ovs-bugtool-conntrack-dump | 19 +++ .../bugtool/plugins/network-status/openvswitch.xml| 1 + 3 files changed, 23 insertions(+), 1

Re: [ovs-dev] [PATCH 2/2] openvswitch: make auto-attach logic disable-able

2016-01-13 Thread Ben Pfaff
On Tue, Jan 12, 2016 at 09:25:16PM +0200, Alexandru Ardelean wrote: > On Tue, Jan 12, 2016 at 6:23 PM, Ben Pfaff wrote: > > > On Tue, Jan 12, 2016 at 09:26:35AM +0200, Alexandru Ardelean wrote: > > > But as it turns out, OVS is getting bigger and bigger with each release, > > >

Re: [ovs-dev] [PATCH 2/2] bugtool: remove dir path prefix

2016-01-13 Thread Guru Shetty
On 13 January 2016 at 11:36, William Tu wrote: > Remove the hard-coded /usr/share/openvswitch/scripts/ and fetch the path > from @pkgdatadir@. > > Signed-off-by: William Tu > --- > utilities/bugtool/ovs-bugtool.in | 3 ++- >

Re: [ovs-dev] [PATCH v2] net: preserve IP control block during GSO segmentation

2016-01-13 Thread David Miller
From: Konstantin Khlebnikov Date: Fri, 08 Jan 2016 15:21:46 +0300 > Skb_gso_segment() uses skb control block during segmentation. > This patch adds 32-bytes room for previous control block which > will be copied into all resulting segments. > > This patch fixes kernel crash

Re: [ovs-dev] [PATCH 2/2] openvswitch: make auto-attach logic disable-able

2016-01-13 Thread Alexandru Ardelean
On Wed, Jan 13, 2016 at 9:50 PM, Ben Pfaff wrote: > On Tue, Jan 12, 2016 at 09:25:16PM +0200, Alexandru Ardelean wrote: > > On Tue, Jan 12, 2016 at 6:23 PM, Ben Pfaff wrote: > > > > > On Tue, Jan 12, 2016 at 09:26:35AM +0200, Alexandru Ardelean wrote: > > > > But as

[ovs-dev] [PATCH] {lib, utilities}: Fix ct_state constants in docs.

2016-01-13 Thread Joe Stringer
These pieces of documentation were not updated when the CS_* flags were reordered on the OpenFlow interface. Fixes: 63bc9fb1c69f ("packets: Reorder CS_* flags to remove gap.") Signed-off-by: Joe Stringer --- lib/meta-flow.h | 8 utilities/ovs-ofctl.8.in | 32

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

2016-01-13 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 breaks from the way most other things are configured in OVS * It

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

2016-01-13 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 v3 1/3] netdev-dpdk: Restore thread affinity after DPDK init

2016-01-13 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

Re: [ovs-dev] [PATCH v2 2/3] netdev-dpdk: Convert initialization from cmdline to db

2016-01-13 Thread Aaron Conole
Aaron Conole writes: > "Traynor, Kevin" writes: >>> -Original Message- >>> From: Aaron Conole [mailto:acon...@redhat.com] >>> Sent: Monday, January 4, 2016 9:47 PM >>> To: dev@openvswitch.org; Flavio Leitner; Traynor, Kevin >>> Subject: [PATCH

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

2016-01-13 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

Re: [ovs-dev] I don't know if this bug still exist ?

2016-01-13 Thread Ben Pfaff
On Thu, Jan 14, 2016 at 11:07:02AM +0800, yewgang wrote: > Hi all, > > In OVS-2.3.0, after I add 100 meters in br0, I found "***decode error: > OFPBRC_BAD_LEN***" sometimes after I type "ovs-ofctl dump-meters br0". > > After I check code, I found code should change in >

[ovs-dev] OVS PatchWork Update

2016-01-13 Thread Ubuntu
30+ Day Old Patches === ID State Date Name -- - 548008 New2015-11-24 11:19:01 [ovs-dev,v4,2/4] ovn: New flows for DHCP tranffic 548010 New2015-11-24 11:19:02 [ovs-dev,v4,3/4] ovn: Process dhcp packet-ins

Re: [ovs-dev] Wildcard Matching optimization idea

2016-01-13 Thread Wei Li
insert a rule to "mirror" and swap "mirror" and "real" then real does not contain the new inserted rule how to sync? take the "long" time as inserting to "mirror"? 在 2016/1/12 18:37, Fischetti, Antonio 写道: Thanks Zoltan for your questions. Anyone else has any new feedback? -Original

[ovs-dev] [PATCH] ofproto: Fix memory leak and memory exhaustion bugs in group_mod.

2016-01-13 Thread Ben Pfaff
In handle_group_mod() cases where adding a group failed, nothing freed the list of buckets, causing a leak. The same was true in every case of modifying a group. This commit fixes the problem by changing add_group() to never steal or free the buckets (modify_group() already acted this way) and

Re: [ovs-dev] [PATCH] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread pravin shelar
On Wed, Jan 13, 2016 at 4:24 PM, Joe Stringer wrote: > On 13 January 2016 at 16:17, Pravin Shelar wrote: >> On Wed, Jan 13, 2016 at 4:07 PM, Pravin B Shelar wrote: >>> nf-hook is not unregistered on STT device delete, But when >>> second

Re: [ovs-dev] [PATCH 2/2] openvswitch: make auto-attach logic disable-able

2016-01-13 Thread Ben Pfaff
Ben Pfaff wrote: > How much does this commit trim from the binary size? Alexandru Ardelean wrote: > Roughly a 34k decrease in binary size, decrease being in libofproto and > libopenvswitch libs > Quite a small decrease, so more --disable- flags would be needed. > My next thought would have been

Re: [ovs-dev] [PATCH v2] net: preserve IP control block during GSO segmentation

2016-01-13 Thread Florian Westphal
David Miller wrote: > From: Konstantin Khlebnikov > Date: Fri, 08 Jan 2016 15:21:46 +0300 > > > Skb_gso_segment() uses skb control block during segmentation. > > This patch adds 32-bytes room for previous control block which > > will be copied into all

Re: [ovs-dev] [PATCH] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread Pravin Shelar
On Wed, Jan 13, 2016 at 4:07 PM, Pravin B Shelar wrote: > nf-hook is not unregistered on STT device delete, But when > second time it was created it nf-hook is again registered. > which causes following softlockup. > Following patch fixes it by registering nf-hook only on very

[ovs-dev] [PATCH] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread Pravin B Shelar
nf-hook is not unregistered on STT device delete, But when second time it was created it nf-hook is again registered. which causes following softlockup. Following patch fixes it by registering nf-hook only on very first stt device. ---8<--- BUG: soft lockup - CPU#1 stuck for 22s!

[ovs-dev] ovs segments frames when transmitting to veth

2016-01-13 Thread Ramu Ramamurthy
Problem: - We are running centos 7.2 (ovs-kernel module from distribution) with OVS 2.3.1. Large tcp frames (> 30Kbytes gro'd) are coming into a vxlan-tunnel port and switched by an ovs-bridge onto a veth port vxlan-tunnel--(gro'd tcp frames sized > 30K)-->OVS-bridge--(frames

[ovs-dev] [PATCH v2] ovs-bugtool: Add conntrack output

2016-01-13 Thread William Tu
Add a script to show all the connection entries in the tracker. Signed-off-by: William Tu --- utilities/bugtool/automake.mk | 4 +++- utilities/bugtool/ovs-bugtool-conntrack-dump | 19 +++

Re: [ovs-dev] [PATCH] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread Joe Stringer
On 13 January 2016 at 16:17, Pravin Shelar wrote: > On Wed, Jan 13, 2016 at 4:07 PM, Pravin B Shelar wrote: >> nf-hook is not unregistered on STT device delete, But when >> second time it was created it nf-hook is again registered. >> which causes

Re: [ovs-dev] ovs segments frames when transmitting to veth

2016-01-13 Thread Jesse Gross
On Wed, Jan 13, 2016 at 3:51 PM, Ramu Ramamurthy wrote: > > Problem: > - > > We are running centos 7.2 (ovs-kernel module from distribution) with OVS > 2.3.1. > > Large tcp frames (> 30Kbytes gro'd) are coming into a vxlan-tunnel port and > switched by an

Re: [ovs-dev] [PATCH] {lib, utilities}: Fix ct_state constants in docs.

2016-01-13 Thread Joe Stringer
On 13 January 2016 at 11:03, Russell Bryant wrote: > On 01/13/2016 01:59 PM, Joe Stringer wrote: >> These pieces of documentation were not updated when the CS_* flags were >> reordered on the OpenFlow interface. >> >> Fixes: 63bc9fb1c69f ("packets: Reorder CS_* flags to remove

[ovs-dev] [PATCH v2] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread Pravin B Shelar
nf-hook is not unregistered on STT device delete, But when second time it was created it nf-hook is again registered. which causes following softlockup. Following patch fixes it by registering nf-hook only on very first stt device. ---8<--- BUG: soft lockup - CPU#1 stuck for 22s!

Re: [ovs-dev] [PATCH v2] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread Joe Stringer
On 13 January 2016 at 16:42, Pravin B Shelar wrote: > nf-hook is not unregistered on STT device delete, But when > second time it was created it nf-hook is again registered. > which causes following softlockup. > Following patch fixes it by registering nf-hook only on very >

Re: [ovs-dev] ovs segments frames when transmitting to veth

2016-01-13 Thread Ramu Ramamurthy
On 2016-01-13 16:22, Jesse Gross wrote: On Wed, Jan 13, 2016 at 3:51 PM, Ramu Ramamurthy wrote: Problem: - We are running centos 7.2 (ovs-kernel module from distribution) with OVS 2.3.1. Large tcp frames (> 30Kbytes gro'd) are coming into a

Re: [ovs-dev] [PATCH 2/2] bugtool: remove dir path prefix

2016-01-13 Thread William Tu
Hi Guru, Thank you. I've removed this change and submitted v2 patch. Regards, William On Wed, Jan 13, 2016 at 1:04 PM, Guru Shetty wrote: > > > On 13 January 2016 at 11:36, William Tu wrote: > >> Remove the hard-coded /usr/share/openvswitch/scripts/ and

Re: [ovs-dev] [PATCH v2] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread Joe Stringer
On 13 January 2016 at 16:42, Pravin B Shelar wrote: > nf-hook is not unregistered on STT device delete, But when > second time it was created it nf-hook is again registered. > which causes following softlockup. > Following patch fixes it by registering nf-hook only on very >

Re: [ovs-dev] [PATCH v2] datapath: STT: Fix nf-hook softlockup.

2016-01-13 Thread Joe Stringer
On 13 January 2016 at 18:11, Joe Stringer wrote: > On 13 January 2016 at 16:42, Pravin B Shelar wrote: >> nf-hook is not unregistered on STT device delete, But when >> second time it was created it nf-hook is again registered. >> which causes following

[ovs-dev] I don't know if this bug still exist ?

2016-01-13 Thread yewgang
Hi all, In OVS-2.3.0, after I add 100 meters in br0, I found "***decode error: OFPBRC_BAD_LEN***" sometimes after I type "ovs-ofctl dump-meters br0". After I check code, I found code should change in "ofputil_append_meter_config" function like this : +reply = (void *)((uint8_t

Re: [ovs-dev] OVN physical network support

2016-01-13 Thread Russell Bryant
On 01/13/2016 02:50 AM, Han Zhou wrote: > Currently we have "localnet" in OVN to connect VIF to physical network, > but there seems to be no way to support intra-hypervisor communication > with exiting model mentioned in ovn-sb.xml: > > localnet > > A connection

Re: [ovs-dev] [PATCH 0/3] QOS support in OVN

2016-01-13 Thread Russell Bryant
On 01/11/2016 08:19 PM, Ben Pfaff wrote: > On Tue, Jan 05, 2016 at 07:33:16PM +0530, bscha...@redhat.com wrote: >> This patch series enables QOS support in OVN. Only two parameters >> (policing_rate and policing_burst) are enabled through this patch series. >> >> Babu Shanmugam (3): >> ovn:

Re: [ovs-dev] [PATCH 1/2] ofctl: This patch add support for setting the first egress table for egress processing.

2016-01-13 Thread Ben Pfaff
On Wed, Jan 13, 2016 at 03:18:02PM +0530, niti Rohilla wrote: > I can incorporate the egress table implemenation in OF1.3 table features > code and will remove the implementation added for OF1.5. Good. > I will replace the pad[5] field in of13_table_features structure with > command and features

Re: [ovs-dev] [PATCH 2/2] openvswitch: make auto-attach logic disable-able

2016-01-13 Thread Jesse Gross
On Tue, Jan 12, 2016 at 11:12 PM, Alexandru Ardelean wrote: > On Tue, Jan 12, 2016 at 9:25 PM, Alexandru Ardelean > wrote: > >> On Tue, Jan 12, 2016 at 6:23 PM, Ben Pfaff wrote: >> >>> On Tue, Jan 12, 2016 at 09:26:35AM +0200,

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

2016-01-13 Thread Ben Pfaff
On Wed, Jan 13, 2016 at 05:44:16PM +0530, Saloni Jain wrote: > Hi Ben, > > Thanks for the review. > Regarding the test case, I want to clarify few points : > >  # OFPT_TABLE_STATUS, OFPTR_VACANCY_UP >     if test X"$1" = X"OFPTR_VACANCY_UP"; then shift; >         ovs-vsctl -- --id=@t1 create

Re: [ovs-dev] [PATCH 0/3] QOS support in OVN

2016-01-13 Thread Russell Bryant
On 01/13/2016 11:30 AM, Russell Bryant wrote: > On 01/11/2016 08:19 PM, Ben Pfaff wrote: >> On Tue, Jan 05, 2016 at 07:33:16PM +0530, bscha...@redhat.com wrote: >>> This patch series enables QOS support in OVN. Only two parameters >>> (policing_rate and policing_burst) are enabled through this

Re: [ovs-dev] [PATCH 2/2] openvswitch: make auto-attach logic disable-able

2016-01-13 Thread Alexandru Ardelean
On Wed, Jan 13, 2016 at 6:47 PM, Jesse Gross wrote: > On Tue, Jan 12, 2016 at 11:12 PM, Alexandru Ardelean > wrote: > > On Tue, Jan 12, 2016 at 9:25 PM, Alexandru Ardelean < > ardeleana...@gmail.com> > > wrote: > > > >> On Tue, Jan 12, 2016 at 6:23 PM,