Re: [ovs-dev] [PATCH 1/3 v7] Windows: Add internal switch port per OVS bridge

Thanks for the patch! Acked-by: Sairam Venugopal On 12/15/16, 11:37 AM, "Alin Serdean" wrote: >This patch updates the following commands in the vswitch: > ovs-vsctl add-br br-test > ovs-vsctl del-br br-test > >ovs-vsctl add-br br-test: >

Re: [ovs-dev] [PATCH 3/3 v7] Windows: document multiple NIC support setup

Acked-by: Sairam Venugopal On 12/15/16, 11:37 AM, "Alin Serdean" wrote: >This patch updates the documentation on how to set up OVS with multiple >NICs. > >Also update the documentation to show users how new internal ports are >created >

Re: [ovs-dev] [PATCH] ovn-controller: Fix conntrack zone in gateway routers.

On Mon, Dec 19, 2016 at 04:12:17AM -0800, Gurucharan Shetty wrote: > The gateway router was using the ct_next action to > reassemble packets. But ct_next action by default would > use the zone allocated for a logical port and in case of > gateway routers that value was zero. This would make >

Re: [ovs-dev] [PATCH v3 13/16] ovn-controller: Drop most uses of OVS patch ports.

On Sun, Dec 18, 2016 at 10:59:18PM -0800, Mickey Spiegel wrote: > On Sun, Dec 18, 2016 at 12:18 AM, Ben Pfaff wrote: > > > Until now, ovn-controller has implemented OVN logical patch ports and > > l3gateway ports in terms of OVS patch ports. It is a hassle to create and > >

[ovs-dev] [PATCH RFC] dpcls: Avoid one 8-byte chunk in subtable mask.

This patch skips the chunk comprising of dp_hash and in_port in the subtable mask when the dpcls refers to a physical port. This will slightly speed up the hash computation as one expensive function call to hash_add64() can be skipped. The idea behind is that the packets coming from a physical

Re: [ovs-dev] OVS2.6+dpdk IPv6 Conntrack support

> On Dec 19, 2016, at 1:29 PM, Darrell Ball wrote: > > > > On 12/16/16, 1:25 PM, "ovs-dev-boun...@openvswitch.org > on behalf of Jarno Rajahalme" > on >

Re: [ovs-dev] [RFC: fedora ovn packaging 0/5] RFC fedora ovn packaging

On Fri, Dec 16, 2016 at 6:19 PM, Russell Bryant wrote: > > > On Thu, Dec 15, 2016 at 8:35 AM, Andy Zhou wrote: > >> The goal of this series is to make it easier to set up OVN in >> a larger setup, i.e. not having all services running on a single host. >> >>

Re: [ovs-dev] [PATCH] netdev-dpdk: Use intermediate queue during packet transmission.

Thanks Ilya and Aaron for reviewing this patch and providing your comments, my reply inline. >-Original Message- >From: Ilya Maximets [mailto:i.maxim...@samsung.com] >Sent: Monday, December 19, 2016 8:41 AM >To: Aaron Conole ; Bodireddy, Bhanuprakash

Re: [ovs-dev] [PATCH v3 15/16] ovsdb-idl: Change interface to conditional monitoring.

On Mon, Dec 19, 2016 at 10:34:20AM +0200, Liran Schour wrote: > ovs-dev-boun...@openvswitch.org wrote on 18/12/2016 10:18:33 AM: > > > Most users of OVSDB react to whatever is currently in their view of the > > database, as opposed to keeping track of changes and reacting to those > > changes

Re: [ovs-dev] strong empheral references?

On Thu, Dec 15, 2016 at 03:14:47PM -0800, Ben Pfaff wrote: > This was implemented because it led to databases that couldn't be read > from disk upon restart. The code comment gives more of the story: > > +if (ovsdb_base_type_is_strong_ref(base) && !refTable->is_root) { > +/* We

[ovs-dev] [PATCH] openvswitch: FTBFS with dpkg-buildpackage.

The debian packages are ready. This patch fixes the bug #831924 reported at debian bug tracking system. With this patch, openvswitch-2.6.1 will be upload to the Debian archive. If we build the packages with "dpkg-buildpackage --target binary-indep", an error state arises. debian/rules should be

Re: [ovs-dev] [PATCH v3 1/3] netdev-dpdk: add hotplug support

On 12/15/2016 11:54 AM, Ciara Loftus wrote: > In order to use dpdk ports in ovs they have to be bound to a DPDK > compatible driver before ovs is started. > > This patch adds the possibility to hotplug (or hot-unplug) a device > after ovs has been started. The implementation adds two appctl

Re: [ovs-dev] [PATCH v3 16/16] ovn-controller: Monitor only necessary southbound rows.

Ben Pfaff wrote on 18/12/2016 10:18:34 AM: > Until now, ovn-controller has replicated all of the southbound database > (through the IDL). This is inefficient, especially in a large OVN setup > where many logical networks are not present on an individual hypervisor. > This commit

Re: [ovs-dev] Sync on PTAP, EXT-382 and NSH

Hi Jan, On Sun, 18 Dec 2016 14:44:03 +, Jan Scheurich wrote: > I would like to call for a final sync meeting before the Christmas break. you managed to hit almost the only slot in this week when I have a conflict with another meeting. I'm not available on Tuesday starting at the time you

Re: [ovs-dev] [PATCH] netdev-dpdk: Use intermediate queue during packet transmission.

Hi, I didn't test this patch yet. Bhanu, could you please describe your test scenario and performance results in more details. It'll be nice if you provide throughput and latency measurement results for different scenarios and packet sizes. Latency is important here. About the patch itself:

Re: [ovs-dev] [PATCH v3 15/16] ovsdb-idl: Change interface to conditional monitoring.

ovs-dev-boun...@openvswitch.org wrote on 18/12/2016 10:18:33 AM: > Most users of OVSDB react to whatever is currently in their view of the > database, as opposed to keeping track of changes and reacting to those > changes individually. The interface to conditional monitoring was > different, in

Re: [ovs-dev] OVS2.6+dpdk IPv6 Conntrack support

Thanks. We will avoid sending the ND packets to conntrack. In the long run, its good to fix this inconsistency as Jarno suggested. On Sat, Dec 17, 2016 at 2:55 AM, Jarno Rajahalme wrote: > > > On Dec 16, 2016, at 9:12 AM, Sridhar Gaddam wrote: > > > > ​​ > >

Re: [ovs-dev] [PATCH v3 00/16] ovn: get rid of most uses of patch ports

On Sun, Dec 18, 2016 at 12:18 AM, Ben Pfaff wrote: > v1->v2: >- Fixed some bugs reported by Mickey throughout the series. >- Patches 1 and 2 are new. >- Patch 3 is modified in various ways, most notably to avoid using the >datapath "sample" action, to add a test,