Re: [ovs-dev] [PATCH 4/4] ovn-controller: Use separate thread for packet-in processing.

2017-06-06 Thread Han Zhou
On Tue, Jun 6, 2017 at 3:56 PM, Ben Pfaff wrote: > > On Thu, May 25, 2017 at 05:26:47PM -0700, Han Zhou wrote: > > This patch introduces multi-threading for ovn-controller and use > > dedicated thread for packet-in processing as a start. It decouples > > packet-in processing and ovs

Re: [ovs-dev] [PATCH 3/3] netdev-linux: maintain original device's state

2017-06-06 Thread Ben Pfaff
On Mon, May 29, 2017 at 04:40:23PM -0300, Flavio Leitner wrote: > It is important to maintain the original state when > the device already exists in the system. > > Signed-off-by: Flavio Leitner Applied, thanks! ___ dev mailing list

Re: [ovs-dev] [PATCH 2/3] netdev-linux: make tap devices persistent.

2017-06-06 Thread Ben Pfaff
On Mon, May 29, 2017 at 04:40:22PM -0300, Flavio Leitner wrote: > When using data path type "netdev", bridge port is a tun device > and when OVS restarts, that device and its network configuration > is lost. > > This patch enables the tap device to persist instead. > > Signed-off-by: Flavio

Re: [ovs-dev] [PATCH 1/3] ovs-router: fix refcnt leak when program terminates.

2017-06-06 Thread Ben Pfaff
On Mon, May 29, 2017 at 04:40:21PM -0300, Flavio Leitner wrote: > Install a handler to flush routes and release devices when > the program is terminating. > > Signed-off-by: Flavio Leitner Thank you! I applied this to master. If you think that this is a bug fix that I should

Re: [ovs-dev] [PATCH 2/2] xlate: Use datapath clone action for patch port translation

2017-06-06 Thread Ben Pfaff
On Fri, May 26, 2017 at 08:47:45PM -0700, Andy Zhou wrote: > When available, use datapath 'clone' for patch port translation. > Clone provides a stronger guarantee that packet will be restored > after going through a patch port, Even in case the packet is > NAT'd by the bridge behind the patch

Re: [ovs-dev] [PATCH 1/4] ovn-controller: decouple localnet_port update from patch_run

2017-06-06 Thread Han Zhou
On Tue, Jun 6, 2017 at 12:31 PM, Ben Pfaff wrote: > > On Thu, May 25, 2017 at 05:26:44PM -0700, Han Zhou wrote: > > We figure out local datapaths in binding_run() but update the field > > localnet_port for each local datapath that has localnet port in > > patch_run(). This patch

Re: [ovs-dev] [PATCH 1/2] xlate: Add datapath clone generation API

2017-06-06 Thread Ben Pfaff
On Tue, Jun 06, 2017 at 04:55:19PM -0700, Ben Pfaff wrote: > I think that this implementation fails when it chooses > COMPOSE_CLONE_USING_ACTIONS, because in that case it does not save and > restore the base flow. I see now that I misunderstood. Never mind, on this point.

Re: [ovs-dev] [PATCH 1/2] xlate: Add datapath clone generation API

2017-06-06 Thread Ben Pfaff
On Fri, May 26, 2017 at 08:47:44PM -0700, Andy Zhou wrote: > There are three methods of translating openflow layer clone action. > Using datapath clone action, datapath sample action or using actions > to negating the any changes within a clone. Xlate logic selects > a specific method depends on

Re: [ovs-dev] [RFC PATCH 0/2] Generic encap/decap actions implementation

2017-06-06 Thread Yang, Yi Y
Ben, we'll send out new patch series based on the recently merged patches and new PTAP series, sending this in the very early phase was just to get some general comments about implementation, Jan has reviewed them and changed many things per his full ovs experience, so next post will have high

Re: [ovs-dev] [PATCH] test-hash: Fix unaligned pointer value error.

2017-06-06 Thread Ben Pfaff
On Fri, May 26, 2017 at 02:11:31PM -0700, Joe Stringer wrote: > Clang 4.0 complains: > > ../tests/test-hash.c:160:16: error: taking address of packed member 'b' of > class or structure 'offset_ovs_u128' may result in an unaligned pointer value > [-Werror,-Waddress-of-packed-member] >

Re: [ovs-dev] [PATCH 2/2] docs: Document dpdkr ports

2017-06-06 Thread Ben Pfaff
On Fri, May 26, 2017 at 03:12:38PM +0100, Stephen Finucane wrote: > I has an idea what these were but that idea was somewhat incorrect and > out-of-date. Add a minimal guide to fill in these gaps, along with a > warning about how useless these things generally are now (yay, > vhost-user). > >

[ovs-dev] Top 50 June 2017

2017-06-06 Thread Bonesca Mailing
Deze email nieuwsbrief werd in grafisch HTML formaat verzonden. Als u deze tekstversie ziet, verkiest uw email programma "gewone tekst" emails. U kan de originele nieuwsbrief online bekijken:

Re: [ovs-dev] [PATCH 1/2] docs: Clarify the superiority of dpdkvhostuserclient

2017-06-06 Thread Ben Pfaff
On Fri, May 26, 2017 at 03:12:37PM +0100, Stephen Finucane wrote: > Apparently dpdkvhostuser interfaces are inferior to dpdkvhostuserclient. > Explain why. > > Signed-off-by: Stephen Finucane > Cc: Ciara Loftus > Cc: Kevin Traynor

Re: [ovs-dev] [PATCH 4/4] ovn-controller: Use separate thread for packet-in processing.

2017-06-06 Thread Ben Pfaff
On Thu, May 25, 2017 at 05:26:47PM -0700, Han Zhou wrote: > This patch introduces multi-threading for ovn-controller and use > dedicated thread for packet-in processing as a start. It decouples > packet-in processing and ovs flow computing, so that packet-in inputs > won't trigger flow

Re: [ovs-dev] [PATCH 3/4] ovn-controller: refactor and abstract ovs_idl registering

2017-06-06 Thread Ben Pfaff
On Thu, May 25, 2017 at 05:26:46PM -0700, Han Zhou wrote: > Abstract as a function so that it can be used by other modules. > > Signed-off-by: Han Zhou This is reasonable on its own, so I applied it. Thank you! ___ dev mailing list

[ovs-dev] Las 4 perspectivas de calidad

2017-06-06 Thread Balanced Scorecard - En Línea
Lograr que la estrategía sea el objetivo de todos y de todos los días Balanced Scorecard: Aplicación e Implementación 30 de junio - Online en Vivo - 10:00 a 13:00 y de 15:00 a 18:00Hrs El BSC es un poderoso instrumento para medir el desempeño corporativo y se ha demostrado que es la

[ovs-dev] [PATCH] ovn-sb.xml: Update and improve documentation.

2017-06-06 Thread Ben Pfaff
Some of the abbreviations at the head of this document, like LN and PN, turn out to not be very useful, so expand them for clarity. Some of the statements in this document are more about planning the design than the current design. Remove these for clarity. Port_Binding rows used to all be

Re: [ovs-dev] [PATCH 2/2] ipfix: Update Timestamp when flow updated

2017-06-06 Thread Greg Rose
On 06/06/2017 08:22 AM, Ben Pfaff wrote: On Mon, Jun 05, 2017 at 08:34:32PM -0700, Greg Rose wrote: > On 06/05/2017 06:04 PM, Ben Pfaff wrote: >> From: Greg Rose >> >> Reported-by: Felix Konstantin Maurer >> Signed-off-by: Greg Rose

Re: [ovs-dev] [PATCH 1/4] ovn-controller: decouple localnet_port update from patch_run

2017-06-06 Thread Ben Pfaff
On Thu, May 25, 2017 at 05:26:44PM -0700, Han Zhou wrote: > We figure out local datapaths in binding_run() but update the field > localnet_port for each local datapath that has localnet port in > patch_run(). This patch updates the localnet_port field in binding_run > directly and removes the

Re: [ovs-dev] [RFC PATCH 0/2] Generic encap/decap actions implementation

2017-06-06 Thread Ben Pfaff
On Thu, Apr 20, 2017 at 11:26:39AM +0800, Yi Yang wrote: > This patchset implements "Generic encap/decap actions" (EXT-382), > it is based on top of the below patchsets: > > [PATCH v1 0/5] NSH userspace support > https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/331158.html > > [PATCH

Re: [ovs-dev] [PATCH v1 0/5] NSH userspace support

2017-06-06 Thread Ben Pfaff
On Thu, Apr 20, 2017 at 10:42:06AM +0800, Yi Yang wrote: > This patchset adds NSH userspace support, it just covers > parse, match and set_field NSH fields, actions encap_nsh > and decap_nsh will be included in the following patchset. > > This patchset is based on top of Zoltan's Now that the

Re: [ovs-dev] [PATCH] bfd: Detect Multiplier configuration

2017-06-06 Thread Ben Pfaff
On Tue, Jun 06, 2017 at 05:11:54PM +0200, Szucs Gabor wrote: > Mult value (bfd.DetectMult in RFC5880) is hard-coded and equal to 3 in > current openvswitch. As a consequence remote and local mult is the same. > > In this commit the mult (Detect Multiplier/bfd.DetectMult/Detect Mult) > can be set

Re: [ovs-dev] [PATCH V9 04/31] tc: Add tc flower functions

2017-06-06 Thread Joe Stringer
On 5 June 2017 at 23:01, Roi Dayan wrote: > > > On 05/06/2017 20:36, Joe Stringer wrote: >> >> On 3 June 2017 at 22:22, Roi Dayan wrote: >>> >>> >>> >>> On 01/06/2017 20:53, Joe Stringer wrote: On 1 June 2017 at 07:39, Roi Dayan

Re: [ovs-dev] [PATCH] conntrack : Use Rx checksum offload feature on DPDK ports for conntrack.

2017-06-06 Thread Darrell Ball
I did a first pass, but I will look thru. it more carefully and test it. The benefit is smaller than I had hoped, even in a simple case, but the code delta is also simple. Thanks Darrell On 6/6/17, 3:45 AM, "ovs-dev-boun...@openvswitch.org on behalf of Chandran, Sugesh"

[ovs-dev] [PATCH] bfd: Detect Multiplier configuration

2017-06-06 Thread Szucs Gabor
Mult value (bfd.DetectMult in RFC5880) is hard-coded and equal to 3 in current openvswitch. As a consequence remote and local mult is the same. In this commit the mult (Detect Multiplier/bfd.DetectMult/Detect Mult) can be set on each interface setting the mult= in bfd Column in Interface table of

Re: [ovs-dev] Try to insert new function

2017-06-06 Thread Ben Pfaff
On Tue, Jun 06, 2017 at 04:28:08PM +0200, Joaquin Alvarez Horcajo wrote: > we try to insert a new function to generate autonomus path, any have a flow > char about the process packet into the demond? Maybe you should be using an OpenFlow controller instead of modifying OVS.

Re: [ovs-dev] [PATCH v3] Update relevant artifacts to add support for DPDK 17.05.

2017-06-06 Thread Kavanagh, Mark B
>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] >On Behalf Of >mweglicx >Sent: Monday, June 5, 2017 1:31 PM >To: d...@openvswitch.org >Subject: [ovs-dev] [PATCH v3] Update relevant artifacts to add support for >DPDK 17.05. > >Following changes are applied: >-

[ovs-dev] Try to insert new function

2017-06-06 Thread Joaquin Alvarez Horcajo
Hi we try to insert a new function to generate autonomus path, any have a flow char about the process packet into the demond? thanks JAH --- El software de antivirus Avast ha analizado este correo electrónico en busca de virus. https://www.avast.com/antivirus

Re: [ovs-dev] [ovs-discuss] Enabling "Datapath does not support ct_zone" for LEDE tplink archer 2600 target board

2017-06-06 Thread Aaron Conole
Jimmy Carter writes: > Hi > > I am using ovs 2.7.0 in openwrt with kernel version 4.9.x for LEDE tplink > archer 2600 target board > But I see ovs logs like "Datapath does not support ct_zone" > > What packages are required to support ct zone ? I think you need a

Re: [ovs-dev] [PATCH V9 14/31] dpif-netlink: Use netdev flow put api to insert a flow

2017-06-06 Thread Roi Dayan
On 02/06/2017 23:08, Flavio Leitner wrote: On Sun, May 28, 2017 at 02:59:56PM +0300, Roi Dayan wrote: From: Paul Blakey Using the new netdev flow api operate will now try and offload flows to the relevant netdev of the input port. Other operate methods flows will come in

Re: [ovs-dev] [PATCH v1 1/4] ovn: l3ha, handling of multiple gateways

2017-06-06 Thread Liran Schour
Hi Miguel, I was thinking of using the add group table OVS action with type=select. Put each gateway into a dedicated bucket and then OVS will be able to load balance the traffic natively regardless if we will choose to do that matching L2 or L3. We should think about the cons and pros of L2

Re: [ovs-dev] [patch_v1] dpdk: Fix device cleanup.

2017-06-06 Thread Ilya Maximets
On 06.06.2017 11:30, Darrell Ball wrote: > On port deletion, device resources cannot be cleaned > up if the device was attached when vswitchd was started. > The issue is mainly due to introduction of the 'attached' > field for the netdev_dpdk device context. The logic > setting the 'attached'

Re: [ovs-dev] [PATCH] conntrack : Use Rx checksum offload feature on DPDK ports for conntrack.

2017-06-06 Thread Chandran, Sugesh
Ping!. Any other comments on this patch?? Regards _Sugesh > -Original Message- > From: Fischetti, Antonio > Sent: Friday, May 26, 2017 11:05 AM > To: Chandran, Sugesh ; ovs- > d...@openvswitch.org > Subject: RE: [ovs-dev] [PATCH] conntrack : Use Rx checksum

[ovs-dev] [PATCH v3] Copy external_ids from Logical_Switch_Port to SB database

2017-06-06 Thread Daniel Alvarez
This patch makes ovn-northd copy all string-string pairs in external_ids column of the Logical_Switch_Port table in Northbound database to the equivalent column of the Port_Binding table in Southbound database. OpenStack Neutron will add some useful data to NB database that can be later read by

Re: [ovs-dev] [PATCH v1 1/4] ovn: l3ha, handling of multiple gateways

2017-06-06 Thread Miguel Angel Ajo Pelayo
Hi Liran, For now we're focusing on Active/Backup implementation, A very brief introduction to the active/active problem can be found here [1]. I'm not an expert on this area and I haven't investigated yet. I'd guess we could have several options, like: * handling it at L3 level (ECMP), Or *