[ovs-dev] OVN - QoS (Neutron API)

2015-07-09 Thread Gal Sagie
Hello all, As of Liberty, neutron will support a QoS API which can be configured on the logical ports. (Currently only bandwidth limiting will be supported) I would like to offer to implement this back-to-back in Neutron OVN plugin and in OVN. For this feature we would need to extend the OVN NB

Re: [ovs-dev] [PATCH] netdev-dpdk: Add some missing statistics.

2015-07-09 Thread Weglicki, MichalX
Hello, >Do you know how much it is going to impact the vhost throughput? Other than >this the change looks good to me. I compared the results based on (OVS head) vs (OVS head + patch) and seems there is very small performance drop: 4,763,016 (HEAD) vs 4,699,571(Patch). -Original Message

Re: [ovs-dev] [RFC] ovn: physical network integraiton (WIP)

2015-07-09 Thread Russell Bryant
(re-adding ovs dev list) On 07/09/2015 10:52 AM, Salvatore Orlando wrote: > > > On 8 July 2015 at 16:12, Russell Bryant > wrote: > > On 07/07/2015 06:06 PM, Salvatore Orlando wrote: > > Reading your post and the associated patch it seems that you're treating

[ovs-dev] [PATCH] tests: Skip IPv6 tests if the system does not support IPv6.

2015-07-09 Thread Ben Pfaff
This is only for the tests that actually create IPv6 sockets. The tests that merely use IPv6 addresses in flow entries, etc., do not depend on kernel support. Reported-by: 俊 赵 Reported-at: http://openvswitch.org/pipermail/discuss/2015-July/018173.html Signed-off-by: Ben Pfaff --- AUTHORS

[ovs-dev] [RFC PATCH 1/6] tests: Check for core files before exiting.

2015-07-09 Thread Jarno Rajahalme
I've seen core files appear and then be automatically removed as the test case was successful. Such success is highly doubtful, so fail the test cases if any core files exist at the end of the test. Signed-off-by: Jarno Rajahalme --- tests/ofproto-macros.at |6 +- 1 file changed, 5 inse

[ovs-dev] [RFC PATCH 0/6] Increase miniflow's capacity.

2015-07-09 Thread Jarno Rajahalme
Upto now struct miniflow has been limited to 63 64-bit units. This series increases this capacity to 128 64-bit units. For presimed performance reasons the new miniflow uses one 64-bit map for tunnel metadata and another for the rest of the metadata and the fields extracted from packet headers.

[ovs-dev] [RFC PATCH 6/6] flow: Split miniflow's map.

2015-07-09 Thread Jarno Rajahalme
Use two maps in miniflow to allow for expansion of struct flow past 512 bytes. We now have one map for tunnel related fields, and another for the rest of the packet metadata and actual packet header fields. This split has the benefit that for non-tunneled packets the overhead should be minimal. S

[ovs-dev] [RFC PATCH 2/6] meta-flow: Add a missing break statement.

2015-07-09 Thread Jarno Rajahalme
Signed-off-by: Jarno Rajahalme --- lib/meta-flow.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/meta-flow.c b/lib/meta-flow.c index 21a13b4..541143c 100644 --- a/lib/meta-flow.c +++ b/lib/meta-flow.c @@ -1114,7 +1114,7 @@ mf_set_flow_value(const struct mf_field *mf,

[ovs-dev] [RFC PATCH 5/6] flow: Eliminate miniflow_clone() and minimask_clone().

2015-07-09 Thread Jarno Rajahalme
miniflow_clone() and minimask_clone() are no longer used, remove them from the API. Now that miniflow data is always inlined, it makes sense to rename miniflow_clone_inline() miniflow_clone(). Signed-off-by: Jarno Rajahalme --- lib/classifier.c|8 lib/flow.c |

[ovs-dev] [RFC PATCH 4/6] match: Single malloc minimatch.

2015-07-09 Thread Jarno Rajahalme
Allocate the miniflow and minimask in struct minimatch at once, so that they are consecutive in memory. This halves the number of allocations, and allows smaller minimatches to share the same cache line. After this a minimatch has one heap allocation for all it's data. Previously it had either no

[ovs-dev] [RFC PATCH 3/6] flow: Always inline miniflows.

2015-07-09 Thread Jarno Rajahalme
Now that performance critical code already inlines miniflows and minimasks, we can simplify struct miniflow by always dynamically allocating miniflows and minimasks to the correct size. This changes the struct minimatch to always contain pointers to its miniflow and minimask. Signed-off-by: Jarno

Re: [ovs-dev] [PATCH] Fix detection of vhost_cuse in dpdk rte_config.h

2015-07-09 Thread Daniele Di Proietto
The patch makes sense for branch-2.4 and master. Thanks! Acked-by: Daniele Di Proietto On 06/07/2015 22:41, "Mussar, Gary" wrote: >Fix detection of vhost_cuse in dpdk rte_config.h > >Dpdk allows users to create a config that includes other config files and >then override values. > >Eg. >defcon

Re: [ovs-dev] [PATCH 1/3] db-ctl-base: fix a few typos

2015-07-09 Thread Andy Zhou
On Wed, Jul 8, 2015 at 11:37 PM, Alex Wang wrote: > Acked-by: Alex Wang Thanks. pushed to master. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/3] db-ctl-base: Do not expose get_table() API

2015-07-09 Thread Andy Zhou
On Wed, Jul 8, 2015 at 11:44 PM, Alex Wang wrote: > > > On Tue, Jul 7, 2015 at 8:08 PM, Andy Zhou wrote: >> >> Both get_table() and set_cloum() APIs are mostly used within db-ctl-base >> library. This patch makes both private to the library. >> >> Add a new ctl_set_colum() API for library client.

[ovs-dev] [db-ctl-base v2 1/3] db-ctl-base: do not expose get_table() API

2015-07-09 Thread Andy Zhou
Both get_table() and set_cloum() APIs are mostly used within db-ctl-base library. This patch makes both private to the library. Add a new ctl_set_colum() API for library client. The changes are cleanups. No functional changes. Signed-off-by: Andy Zhou --- lib/db-ctl-base.c | 15 +++

[ovs-dev] [db-ctl-base v2 0/3] db-ctl-base cleanup

2015-07-09 Thread Andy Zhou
Minor cleanups in db-ctl-base library, and in prepare for ovsdb join related changes in the library. v1->v2: Drop the first patch that has been committed. group static functions together Fix comments Andy Zhou (3): db-ctl-base: do not expose get_table() API db-ctl-base: do not r

[ovs-dev] [db-ctl-base v2 3/3] db-ctl-base: group static functions together

2015-07-09 Thread Andy Zhou
This file follows a convention that all static functions are grouped towards the beginning, ahead of public functions. Re-arrange the code to confirm to this convention. No functional changes. Signed-off-by: Andy Zhou --- lib/db-ctl-base.c | 166 +++--

[ovs-dev] [db-ctl-base v2 2/3] db-ctl-base: do not require client to expose the "tables" variable

2015-07-09 Thread Andy Zhou
Instead, client now pass it via the modified ctl_init() API. Siigned-off-by: Andy Zhou --- lib/db-ctl-base.c | 8 +++- lib/db-ctl-base.h | 8 ++-- utilities/ovs-vsctl.c | 4 ++-- vtep/vtep-ctl.c | 4 ++-- 4 files changed, 13 insertions(+), 11 deletions(-) diff --git a/lib/

[ovs-dev] RE

2015-07-09 Thread Flynn, Anne
I have a proposal for you kindly contact me via:: idu...@gmail.com -- This email remains the property of the ACT Education & Training Directorate. This transmission and

[ovs-dev] [PATCH] lib/automake.mk: Fix dependency typos

2015-07-09 Thread Kyle Upton
Fix dependency issues with lib/ofp-errors.lo and lib/ofp-msgs.lo that caused problems when building debian packages. Signed-off-by: Kyle Upton --- lib/automake.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/automake.mk b/lib/automake.mk index f72eb32..fe40e4a 100

[ovs-dev] [PATCH] doc: Document proposed OVN Gateway HA design.

2015-07-09 Thread Ethan Jackson
High availability for gateways in network virtualization deployments is fairly difficult to get right. There are a ton of options, most of which are too complicated or perform badly. To help solve this problem, this patch proposes an HA design based on some of the lessons learned building similar

[ovs-dev] About one abort in vswitchd

2015-07-09 Thread 马啸
Hi,all I am one engineer from UnitedStack, one OpenStack Provider. We are using OpenvSwitch as the software-switch in OpenStack Compute and Network Node, and we enabled sflow to monitor the traffic. And one crash happened. The core-dump information is attached, could anybody help us to sol

Re: [ovs-dev] About one abort in vswitchd

2015-07-09 Thread Alex Wang
Hey, Could you send the core dump info (did not see any attachment)? I assume you mean the gdb printout showing what causes the crash. Also, could you provide the ovs version you are using? I'm trying to debug an ipfix related crash, could we related, Thanks, 谢谢, Alex Wang, On Thu, Jul 9, 201

Re: [ovs-dev] About one abort in vswitchd

2015-07-09 Thread Jesse Gross
This could be related to b953042214201e2693a485a8ba8b19f69e5bdf34 ("datapath: simplify sample action implementation"). I would check that you are using OVS 2.3.2 for anything related to sampling. On Thu, Jul 9, 2015 at 8:43 PM, Alex Wang wrote: > Hey, > > Could you send the core dump info (did no