[ovs-dev] [PATCH RFC] ofproto-dpif-upcall: Increment dump_seq before instead of after dump.

2015-07-06 Thread Ben Pfaff
When process_upcall() passes a miss upcall to upcall_xlate(), the new ukey's dump_seq member is initialized from the current dump_seq. Later, when udpif_revalidator() calls revalidate(), any dumped flow for which ukey->dump_seq equals the current dump_seq is skipped. However, until now the curren

[ovs-dev] [PATCH] ofp-print: Abbreviate duplicated table features.

2015-07-06 Thread Ben Pfaff
I spent some time recently looking at the results of "ovs-ofctl dump-table-features". It was really distressing because of the volume of information. Every table yielded well over 100 lines of output and for 253 (visible) tables that meant over 25,300 lines of output total, which is basically unu

Re: [ovs-dev] [PATCH] FAQ.md: Remove text saying that OpenFlow 1.5 is not final.

2015-07-06 Thread Ben Pfaff
Thanks, applied to master. On Mon, Jul 06, 2015 at 02:19:56PM -0700, Justin Pettit wrote: > Acked-by: Justin Pettit > > --Justin > > > > On Jul 6, 2015, at 7:26 AM, Ben Pfaff wrote: > > > > OF1.5 was released months ago. > > > > Signed-off-by: Ben Pfaff > > --- > > FAQ.md | 3 +-- > > 1 fil

Re: [ovs-dev] [PATCH] nroff: Fix style of names.

2015-07-06 Thread Ben Pfaff
On Mon, Jul 06, 2015 at 05:46:36PM -0700, Justin Pettit wrote: > > > > On Jul 6, 2015, at 2:42 PM, Ben Pfaff wrote: > > > > The recommended Google Python style is multi_word_names, not > > multiWordNames. > > > > There are lots of other places where the style could be improved. > > I started h

Re: [ovs-dev] [PATCH 3/3] ofp-util: Log the type of an invalid message being added to a bundle.

2015-07-06 Thread Ben Pfaff
On Mon, Jul 06, 2015 at 03:32:45PM -0700, Jarno Rajahalme wrote: > > > On Jul 6, 2015, at 11:47 AM, Ben Pfaff wrote: > > > > Thanks for the review! Is that meant to cover the series, or will you > > separately review patches 1 and 2? > > > > Oops, patches 1&2 were in the junk mailbox, but I j

Re: [ovs-dev] [PATCH] nroff: Fix style of names.

2015-07-06 Thread Justin Pettit
> On Jul 6, 2015, at 2:42 PM, Ben Pfaff wrote: > > The recommended Google Python style is multi_word_names, not > multiWordNames. > > There are lots of other places where the style could be improved. > I started here because I was working in this code anyway and because > this code is only use

Re: [ovs-dev] [PATCH 3/3] ofp-util: Log the type of an invalid message being added to a bundle.

2015-07-06 Thread Jarno Rajahalme
> On Jul 6, 2015, at 11:47 AM, Ben Pfaff wrote: > > Thanks for the review! Is that meant to cover the series, or will you > separately review patches 1 and 2? > Oops, patches 1&2 were in the junk mailbox, but I just read them, and the ack now covers the whole series :-) Jarno > Thanks, >

Re: [ovs-dev] [PATCH] Detecting dpdk vhost_cuse when building ovs

2015-07-06 Thread Mussar, Gary
I have resent the patch. If this doesn't work, I'll have to find some server that I can use with send-email. Gary -Original Message- From: Daniele Di Proietto [mailto:diproiet...@vmware.com] Sent: Monday, July 06, 2015 13:43 To: Mussar, Gary Cc: dev@openvswitch.org Subject: Re: [ovs-de

[ovs-dev] [PATCH] nroff: Fix style of names.

2015-07-06 Thread Ben Pfaff
The recommended Google Python style is multi_word_names, not multiWordNames. There are lots of other places where the style could be improved. I started here because I was working in this code anyway and because this code is only used at build time and not installed, so that it can't break any thi

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

2015-07-06 Thread Mussar, Gary
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. defconfig_x86_64-native_vhost_cuse-linuxapp-gcc: CONFIG_RTE_BUILD_COMBINE_LIBS=y CONFIG_RTE_BUILD_SHARED_LIB=n CONFIG_RTE_LIBRTE_VHOST=y CONFIG_RTE

Re: [ovs-dev] [PATCH] FAQ.md: Remove text saying that OpenFlow 1.5 is not final.

2015-07-06 Thread Justin Pettit
Acked-by: Justin Pettit --Justin > On Jul 6, 2015, at 7:26 AM, Ben Pfaff wrote: > > OF1.5 was released months ago. > > Signed-off-by: Ben Pfaff > --- > FAQ.md | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/FAQ.md b/FAQ.md > index 045eed1..ccc75f0 100644 > --- a/F

Re: [ovs-dev] [PATCH v2] Undefined symbol make_unix_socket

2015-07-06 Thread Ansis Atteka
On Wed, Jul 1, 2015 at 1:20 PM, Ben Pfaff wrote: > On Wed, Jul 01, 2015 at 07:01:44PM +, Alin Serdean wrote: >> syslog_direct_create defined in (lib/syslog-direct.c) uses make_unix_socket >> which is currently undefined on the windows build. >> >> We either can remove the new file from the cha

Re: [ovs-dev] [PATCH 2/2] classifier: Make versioning more explicit.

2015-07-06 Thread Jarno Rajahalme
> On Jul 6, 2015, at 9:02 AM, Ben Pfaff wrote: > > On Wed, Jul 01, 2015 at 12:45:05PM -0700, Jarno Rajahalme wrote: >> Now that struct cls_match has 'add_version' the 'version' in cls_match >> was largely redundant. Remove 'version' from struct cls_rule, and add >> it to function prototypes tha

Re: [ovs-dev] [PATCH 1/2] ofproto: Add struct ofproto_flow_mod.

2015-07-06 Thread Jarno Rajahalme
> On Jul 6, 2015, at 8:56 AM, Ben Pfaff wrote: > > On Wed, Jul 01, 2015 at 12:45:04PM -0700, Jarno Rajahalme wrote: >> It is cleaner to not use ofp_bundle_entry for non-bundle flow mods. >> To address this, the new struct ofproto_flow_mod combines an >> ofputil_flow_mod and the necessary executi

Re: [ovs-dev] [ovn-controller-vtep V2 2/6] ovn-sbctl: Add ovn-sbctl.

2015-07-06 Thread Alex Wang
Thx a lot Russel, for the review,~ On Mon, Jul 6, 2015 at 11:02 AM, Russell Bryant wrote: > On 07/05/2015 01:38 AM, Alex Wang wrote: > > This commit adds ovn-sbctl to ovn family by using the db-ctl-base > > library. > > > > Signed-off-by: Alex Wang > > Acked-by: Ben Pfaff > > > > --- > > PATCH

Re: [ovs-dev] [PATCH 3/3] ofp-util: Log the type of an invalid message being added to a bundle.

2015-07-06 Thread Ben Pfaff
Thanks for the review! Is that meant to cover the series, or will you separately review patches 1 and 2? Thanks, Ben. On Mon, Jul 06, 2015 at 11:36:41AM -0700, Jarno Rajahalme wrote: > Acked-by: Jarno Rajahalme > > > On Jul 6, 2015, at 10:20 AM, Ben Pfaff wrote: > > > > This makes troublesh

Re: [ovs-dev] [PATCH 3/3] ofp-util: Log the type of an invalid message being added to a bundle.

2015-07-06 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jul 6, 2015, at 10:20 AM, Ben Pfaff wrote: > > This makes troubleshooting easier. > > Signed-off-by: Ben Pfaff > --- > lib/ofp-util.c | 2 ++ > tests/ofp-print.at | 4 +++- > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/lib/ofp-util.c b/li

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

2015-07-06 Thread Daniele Di Proietto
On 01/07/2015 11:49, "Timo Puha" wrote: >New stats for vhost ports are rx_bytes, tx_bytes, multicast, rx_errors and >rx_length_errors. New stats for PMD ports are rx_dropped, >rx_length_errors, >rx_crc_errors and rx_missed_errors. DPDK imissed packets are now >classified >as dropped instead of e

Re: [ovs-dev] [ovn-controller-vtep V2 2/6] ovn-sbctl: Add ovn-sbctl.

2015-07-06 Thread Russell Bryant
On 07/05/2015 01:38 AM, Alex Wang wrote: > This commit adds ovn-sbctl to ovn family by using the db-ctl-base > library. > > Signed-off-by: Alex Wang > Acked-by: Ben Pfaff > > --- > PATCH->V2: > - change command add/del-ch to add/del-chassis. > - refine the manual based on comments from Ben. > -

[ovs-dev] [PATCH] Add 'flow_hash_symmetric_l3l4' (with/without UDP ports included) to provide symmetric hashes over packet fields excluding the L2 headers that the default 'flow_hash_symmetric_l4' met

2015-07-06 Thread Jeroen van Bemmel
--- NEWS | 5 +++ include/openflow/nicira-ext.h | 23 +- lib/bundle.c | 4 +++ lib/flow.c| 73 ++- lib/flow.h| 2 ++ lib/multipath.c | 4 +++ util

[ovs-dev] V3: Hashing: Add truly symmetric L3+L4 fields option for multipath and bundle hashing

2015-07-06 Thread Jeroen van Bemmel
Changes: * Adhere to coding standards ( removed spaces before/after (), no // comments ) * Fix NEWS item * Use 'bool' instead of 'int' Signed-Off-By: Jeroen van Bemmel Suggested-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch

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

2015-07-06 Thread Russell Bryant
CC: Ben Pfaff --- ovn/northd/ovn-northd.c | 42 -- ovn/ovn-nb.ovsschema| 6 ++ ovn/ovn-nb.xml | 18 ++ ovn/ovn-nbctl.c | 29 + ovn/ovn-sb.xml | 2 ++ 5 files changed, 91 insert

Re: [ovs-dev] [PATCH] Detecting dpdk vhost_cuse when building ovs

2015-07-06 Thread Daniele Di Proietto
Hi Gary, I wasn't able to apply the patch, it appears to be corrupt. Would you mind resending it, perhaps using get send-email? I also think that it would be useful to include the first part of the email in the commit message, for future reference. Otherwise the patch seems fine to me, thanks!

Re: [ovs-dev] [PATCHv3] odp-util: Share fields between odp and dpif_backer.

2015-07-06 Thread Joe Stringer
On 1 July 2015 at 15:52, Andy Zhou wrote: > LGTM. Thanks joe! > > Acked-by: Andy Zhou Thanks Andy, applied to master. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 1/3] ofp-msgs: Add function ofptype_get_name().

2015-07-06 Thread Ben Pfaff
An upcoming commit will make use of this. Signed-off-by: Ben Pfaff --- build-aux/extract-ofp-msgs | 9 + lib/ofp-msgs.c | 9 - lib/ofp-msgs.h | 5 - 3 files changed, 21 insertions(+), 2 deletions(-) diff --git a/build-aux/extract-ofp-msgs b/build-aux/

[ovs-dev] [PATCH 3/3] ofp-util: Log the type of an invalid message being added to a bundle.

2015-07-06 Thread Ben Pfaff
This makes troubleshooting easier. Signed-off-by: Ben Pfaff --- lib/ofp-util.c | 2 ++ tests/ofp-print.at | 4 +++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/ofp-util.c b/lib/ofp-util.c index d02ee3a..646bae7 100644 --- a/lib/ofp-util.c +++ b/lib/ofp-util.c @@ -9119,6

[ovs-dev] [PATCH 2/3] ofp-util: Require inner and outer message in bundle add to be same version.

2015-07-06 Thread Ben Pfaff
It doesn't make sense for the messages added to a bundle to have a different OpenFlow version from the outer OpenFlow version. Signed-off-by: Ben Pfaff --- lib/ofp-errors.h | 4 lib/ofp-util.c | 3 +++ tests/ofp-print.at | 8 3 files changed, 11 insertions(+), 4 deletions(-)

Re: [ovs-dev] [PATCH 2/2] classifier: Make versioning more explicit.

2015-07-06 Thread Ben Pfaff
On Wed, Jul 01, 2015 at 12:45:05PM -0700, Jarno Rajahalme wrote: > Now that struct cls_match has 'add_version' the 'version' in cls_match > was largely redundant. Remove 'version' from struct cls_rule, and add > it to function prototypes that need it. This makes versioning more > explicit (or les

Re: [ovs-dev] [PATCH 1/2] ofproto: Add struct ofproto_flow_mod.

2015-07-06 Thread Ben Pfaff
On Wed, Jul 01, 2015 at 12:45:04PM -0700, Jarno Rajahalme wrote: > It is cleaner to not use ofp_bundle_entry for non-bundle flow mods. > To address this, the new struct ofproto_flow_mod combines an > ofputil_flow_mod and the necessary execution context for executing the > start, revert, and finish

Re: [ovs-dev] V2: Hashing: Add truly symmetric L3+L4 fields option for multipath and bundle hashing

2015-07-06 Thread Ben Pfaff
On Sat, Jul 04, 2015 at 04:47:37PM -0500, Jeroen van Bemmel wrote: > The symmetric_l4 function implements a hash over various fields > including L2 fields such as ethernet source and destination MAC. > Inspite of its name, there are situations in which this hash does not > yield symmetric results (

Re: [ovs-dev] [PATCH v2] datapath-windows: Solved BSOD when adding OVS ports

2015-07-06 Thread Ben Pfaff
On Thu, Jul 02, 2015 at 11:39:14AM +, Sorin Vinturis wrote: > This BSOD occurred in the context of a packet (NBL) with multiple > NET_BUFFER(s) (NBs). The reason for the BSOD is due to the marking > of NBLs created by OVS as being external and wrongly completing them. > > Signed-off-by: Sorin

[ovs-dev] [PATCH] ovs-bugtool: Remove argument of "ovs-appctl dpif/show".

2015-07-06 Thread Ben Pfaff
From: Huanle Han "ovs-appctl dpif/show" no longer takes any argument after commit dc54ef36 (ofproto: Track subfacet stats in the backer). Signed-off-by: Huanle Han Signed-off-by: Ben Pfaff --- I already applied this patch to master, branch-2.4, and branch-2.3. (It was submitted via github.)

[ovs-dev] [PATCH] stream-ssl: Set SSL session cache mode to disables.

2015-07-06 Thread Ben Pfaff
From: Guru Chaitanya Perakam To enable SSL clients to reconnect with the ovs-testcontoller without being rejected, one must either set the SSL Session ID flag or disable the SSL caching. This patch disables the SSL caching/reuse. In the absence of this fix, the error message from ovs-testcontrol

[ovs-dev] [PATCH] FAQ.md: Remove text saying that OpenFlow 1.5 is not final.

2015-07-06 Thread Ben Pfaff
OF1.5 was released months ago. Signed-off-by: Ben Pfaff --- FAQ.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/FAQ.md b/FAQ.md index 045eed1..ccc75f0 100644 --- a/FAQ.md +++ b/FAQ.md @@ -1432,8 +1432,7 @@ A: The following table lists the versions of OpenFlow supported

Re: [ovs-dev] [Dpdk-ovs] OVS DPDK Bridge Packet loss

2015-07-06 Thread Gray, Mark D
> > I'm sending traffic to a macvtap2 interface on KVM instance [derived from > pktovs bridge] and the traffic is voice [udp data ~104 bytes with ip header > and payload] > > Following are the interface details [you can see the pktovs dropped value > also] > > macvtap0: flags=4163 mtu 1500 > ma

Re: [ovs-dev] [Dpdk-ovs] OVS DPDK Bridge Packet loss

2015-07-06 Thread Varun Rapelly
I'm sending traffic to a macvtap2 interface on KVM instance [derived from pktovs bridge] and the traffic is voice [udp data ~104 bytes with ip header and payload] Following are the interface details [you can see the pktovs dropped value also] macvtap0: flags=4163 mtu 1500 macvtap1: flags=4163 m

Re: [ovs-dev] [Dpdk-ovs] OVS DPDK Bridge Packet loss

2015-07-06 Thread Gray, Mark D
> > Hi Mark, > > Following are the scripts. > Ok, so what traffic interface are you sending traffic into and what is the traffic type? ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [Dpdk-ovs] OVS DPDK Bridge Packet loss

2015-07-06 Thread Varun Rapelly
Hi Mark, Following are the scripts. Init.sh - mount -t hugetlbfs -o pagesize=1G none /dev/hugepages modprobe openvswitch OVS=/root/OVS/ovs-branch-2.4 pkill -9 ovs rm -rf /usr/local/etc/openvswitch/ rm -rf /usr/local/var/run/openvswitch/ rm -rf /usr/local/etc/openvswitch/conf.db mkdir

Re: [ovs-dev] [Dpdk-ovs] OVS DPDK Bridge Packet loss

2015-07-06 Thread Gray, Mark D
> Hi All, > > When I'm using plain OVS bridge [vswitchd without dpdk options], i'm not > seeing any packet loss at the ovs bridge. > > But as i mentioned earlier with DPDK, huge packet loss is happening. > > Can someone tell, how dpdk options are resulting in packet loss? > > I tried with incre

Re: [ovs-dev] [Dpdk-ovs] OVS DPDK Bridge Packet loss

2015-07-06 Thread Varun Rapelly
Hi All, When I'm using plain OVS bridge [vswitchd without dpdk options], i'm not seeing any packet loss at the ovs bridge. But as i mentioned earlier with DPDK, huge packet loss is happening. Can someone tell, how dpdk options are resulting in packet loss? I tried with increased txqueuelen on p