Re: [ovs-dev] [PATCH] Patch v3: ovn-controller-vtep: Support BUM traffic for the VTEP Schema

2016-04-25 Thread Justin Pettit
> On Apr 25, 2016, at 9:39 PM, Darrell Ball wrote: > > Thanks for the review; > The suggestions are fine; > I added a couple comments inline. Great! Thanks for the patch. I pushed the change. --Justin ___ dev mailing list dev@openvswitch.org http

Re: [ovs-dev] [PATCH v1] Support port level IPFIX

2016-04-25 Thread Daniel Ye
Hi Ben, Thanks for your review. I have sent a second patch with signed-off tag. Please review it again. For your comments, I listed the answers of mine below: 1. "I worry that we'll end up with a fourth, and a fifth, ..." With flow IPFIX, Bridge IPFIX and Port level IPFIX, we can handle almo

Re: [ovs-dev] [PATCH] Patch v3: ovn-controller-vtep: Support BUM traffic for the VTEP Schema

2016-04-25 Thread Darrell Ball
Thanks for the review; The suggestions are fine; I added a couple comments inline. Darrell On Mon, Apr 25, 2016 at 3:44 PM, Justin Pettit wrote: > > > On Apr 5, 2016, at 1:13 PM, Darrell Ball wrote: > > > > This patch implements BUM support in the VTEP schema. This relates to > BUM > > traffi

Re: [ovs-dev] [patch_v2] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
This patch is superseded I sent an updated patch with a couple bugs fixed in ovs-vtep Darrell On Mon, Apr 25, 2016 at 1:43 PM, Darrell Ball wrote: > This patch series updates the vtep schema, vtep-ctl commands and vtep > simulator to support source node replication in addition to service node >

[ovs-dev] [patch_v3] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
Offline discussion is documented in v1->v2 below. There are various considerations to what kind of replication configuration is needed and what is "better" varies with the specific requirements. Feel free to suggest otherwise. This patch series updates the vtep schema to support source node replic

[ovs-dev] [patch_v3] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
This patch series updates the vtep schema, vtep-ctl commands and vtep simulator to support source node replication in addition to service node replication per logical switch. The default replication mode is service node as that was the only mode previously supported. Source node replication mode

[ovs-dev] [PATCH] tnl-neigh-cache: check for arp expiration.

2016-04-25 Thread Pravin B Shelar
The neighbor entry expiry is only checked in dpif-poll event handler, But in absence of any event we could keep using arp entry forever. This patch changes it to check expiration on each lookup. Signed-off-by: Pravin B Shelar --- lib/tnl-neigh-cache.c | 10 +- 1 file changed, 9 insertion

Re: [ovs-dev] [PATCH] Patch v3: ovn-controller-vtep: Support BUM traffic for the VTEP Schema

2016-04-25 Thread Justin Pettit
> On Apr 5, 2016, at 1:13 PM, Darrell Ball wrote: > > This patch implements BUM support in the VTEP schema. This relates to BUM > traffic flowing from a gateway towards HVs. This code would be relevant to HW > gateways and the ovs-vtep simulator. In order to do this, the mcast macs > remote >

Re: [ovs-dev] [PATCH v2] datapath-windows: Pause switch state on PnP event

2016-04-25 Thread Sairam Venugopal
Thanks for incorporating the review comments. I noticed that V2 hasn¹t been ACKED yet. Acked-by: Sairam Venugopal On 3/10/16, 5:33 AM, "Alin Serdean" wrote: >A PnP(plug and play) event will be triggered before trying to disable >the extension. We could use this PnP event to prepare for deta

[ovs-dev] [PATCH v2] stt: linearize for CONFIG_SLUB case

2016-04-25 Thread Pravin B Shelar
STT implementation we saw performance improvements with linearizing skb for SLUB case. So following patch skips zero copy operation for such a case. Performance number for large packet TCP test using netperf. OVS branch TCP Host0 Host1 versionGbps CPU% CPU%

Re: [ovs-dev] [PATCH v2] datapath-windows: Improved offloading on STT tunnel

2016-04-25 Thread Sairam Venugopal
Hi Paul, Please see the comments inlined. Thanks, Sairam On 4/18/16, 1:31 PM, "Paul Boca" wrote: >Added OvsExtractLayers - populates only the layers field without >unnecessary >memory operations for flow part >If in STT header the flags are 0 then force packets checksums calculation >Ensure co

[ovs-dev] [patch_v2] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
Offline discussion is documented in v1->v2 below. There are various considerations to what kind of replication configuration is needed and what is "better" varies with the specific requirements. Feel free to suggest otherwise. This patch series updates the vtep schema to support source node replic

[ovs-dev] [patch_v2] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
This patch series updates the vtep schema, vtep-ctl commands and vtep simulator to support source node replication in addition to service node replication per logical switch. The default replication mode is service node as that was the only mode previously supported. Source node replication mode

Re: [ovs-dev] [PATCH] netdev-dpdk: Set pmd thread priority

2016-04-25 Thread Bodireddy, Bhanuprakash
Thanks for looking in to the patch Kevin, please see my reply inline. > -Original Message- > From: Traynor, Kevin > Sent: Monday, April 25, 2016 6:08 PM > To: Bodireddy, Bhanuprakash ; > dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] netdev-dpdk: Set pmd thread priority > > On 21/04

[ovs-dev] [PATCH] Remove "VLAN splinters" feature.

2016-04-25 Thread Pravin B Shelar
The "VLAN splinters" feature works around buggy device drivers in old Linux versions. But support for the old kernel is dropped, So now all supported kernel vlan drivers should be working fine with OVS kernel datapath. Following patch removes this deprecated feature. Signed-off-by: Pravin B Shelar

Re: [ovs-dev] [PATCH] netdev-dpdk: Set pmd thread priority

2016-04-25 Thread Traynor, Kevin
On 21/04/2016 16:16, Bhanuprakash Bodireddy wrote: Set the DPDK pmd thread scheduling policy to SCHED_RR and static priority to highest priority value of the policy. This is to deal with pmd thread starvation case where another cpu hogging process can get scheduled/affinitized to the same core wh

Re: [ovs-dev] [PATCH v2 04/15] conntrack: New userspace connection tracker.

2016-04-25 Thread Fischetti, Antonio
Hi Daniele, some comments inline prefixed with [Antonio F]. Regards, Antonio > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Daniele Di > Proietto > Sent: Saturday, April 16, 2016 1:03 AM > To: dev@openvswitch.org > Subject: [ovs-dev] [PATCH v2 04/15] c

Re: [ovs-dev] SAI ofproto provider for OvS

2016-04-25 Thread Petr Machata
Ben Pfaff writes: >> > want to make sure that the additional providers can be built without >> > installing code that is available only under NDA. >> >> That is understood. The SAI backend will be optional. > > That's not exactly what I mean. I mean, there must be a way for users > to compile

Re: [ovs-dev] [PATCH v2 00/15] Userspace (DPDK) connection tracker

2016-04-25 Thread Fischetti, Antonio
Hi Daniele, below are - Regression Tests results - ALWAYS_INLINE check Regression Tests on Performance === I did some regression tests on performance, it looks like there's no performance drop. I've sent mono-directional data with 64-Byte packets at line rate o

[ovs-dev] [PATCH] datapath-windows: Validate netlink packets integrity

2016-04-25 Thread Paul Boca
Solved access violation when trying to acces netling message - obtained with forged IOCTLs Signed-off-by: Paul-Daniel Boca --- datapath-windows/ovsext/Datapath.c| 45 +--- datapath-windows/ovsext/Flow.c| 42 ++- datapath-windows/ovsext/Netlink

Re: [ovs-dev] [PATCH 0/2] ovn: QOS updates with DSCP support

2016-04-25 Thread Babu Shanmugam
On Friday 22 April 2016 10:51 PM, Ben Pfaff wrote: On Fri, Apr 22, 2016 at 12:44:12PM +0530, bscha...@redhat.com wrote: From: Babu Shanmugam Following are done through this series 1. Changed the old approach of policing the packets. It is now shaped with queues. Changed the Logical_Port op

[ovs-dev] Status

2016-04-25 Thread Mail Delivery Subsystem
Dear user of openvswitch.org, We have received reports that your account has been used to send a huge amount of unsolicited email during the last week. Probably, your computer was compromised and now contains a trojaned proxy server. We recommend that you follow our instructions in order to kee

[ovs-dev] Mail System Error - Returned Mail

2016-04-25 Thread Post Office
The original message was received at Mon, 25 Apr 2016 14:50:28 +0530 from [205.105.134.93] - The following addresses had permanent fatal errors - dev@openvswitch.org ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/list

[ovs-dev] [CudaMailTagged] Comparing the flow with the packet

2016-04-25 Thread Amrane Ait Zeouay
Hi everyone, I'm new to OVS and i wanted to know which function that compare the flow of a table with the flow of extracted from a packet. Thank you and have a nice day. Best regards. -- Amrane Ait Zeouay Engineer Student in The Development of Software and Systems University of Western Brittan

Re: [ovs-dev] [PATCH net-next 2/9] libnl: nla_put_le64(): align on a 64-bit area

2016-04-25 Thread Nicolas Dichtel
Le 23/04/2016 19:28, Alexander Aring a écrit : Hi, On 04/23/2016 07:05 PM, Alexander Aring wrote: ... if this is really needed, then nla_put_u64/be64/etc need to be changed also, Okay, I found PATCH 3/9 do it for be64, but what's about u64? It's also planned. I will send several "small" se

[ovs-dev] Consideration of ce179f11 for 2.5 branch?

2016-04-25 Thread Christian Ehrhardt
Hi, I've recently run a lot into extremely huge logfiles. I found it related to messages like: 2016-04-25T06:25:29.464Z|3|dpif_netdev(pmd18)|INFO|Core 9 processing port 'vhost-user-1' With many ports and queues set up it really gets bad with almost a Gigabyte for 512 ports and even up to stall

[ovs-dev] [PATCH v2] lib: protect daemon_set_new_user against non existing user:group specs

2016-04-25 Thread Christian Ehrhardt
From the manpages of getgrnam_r (getpwnam_r is similar): "If no matching group record was found, these functions return 0 and store NULL in *result." The code checked only against errors, but non existing users didn't set e != 0 therefore the code could try to set arbitrary uid/gid values. *Updat