[ovs-dev] [CudaMailTagged] [PATCH] geneve-map-rename: rename geneve-map to tlv-map.

2015-12-14 Thread Mengke Liu
This patch renames the command name related with geneve-map to a more generic name as following: add-geneve-map -> add-tlv-map del-geneve-map -> del-tlv-map dump-geneve-map -> dump-tlv-map It also renames the Geneve_table to tlv_table. By doing this renaming, the NSH variable context header(the s

Re: [ovs-dev] [PATCH 0/7] Enable NSH based Service Function Chaining support in OVS

2015-12-14 Thread Liu, Mengke
> -Original Message- > From: Jesse Gross [mailto:je...@kernel.org] > Sent: Saturday, December 12, 2015 1:41 AM > To: Liu, Mengke > Cc: dev@openvswitch.org; Pritesh Kothari (pritkoth) ; > Zhou, Danny ; Li, Ricky ; > pa...@cisco.com > Subject: Re: [ovs-dev] [PATCH 0/7] Enable NSH based Ser

[ovs-dev] Returned mail: see transcript for details

2015-12-14 Thread Bounced mail
The original message was received at Mon, 14 Dec 2015 11:47:57 +0300 from openvswitch.org [86.233.57.199] - The following addresses had permanent fatal errors - - Transcript of the session follows - ... while talking to 57.122.35.213: >>> RCPT To: <<< 550 MAILBOX NOT FOUND

[ovs-dev] [PATCH] jsonrpc-server: Use prototype style for ovsdb_jsonrpc_disable_monitor2().

2015-12-14 Thread Ben Pfaff
Without "void", this is a pre-ANSI style function definition that has subtly different semantics. Found by sparse. Signed-off-by: Ben Pfaff --- ovsdb/jsonrpc-server.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ovsdb/jsonrpc-server.c b/ovsdb/jsonrpc-server.c index 1dcd39

Re: [ovs-dev] [PATCH] match: Add support for matching IGMP fields.

2015-12-14 Thread Ben Pfaff
On Thu, Dec 10, 2015 at 01:42:41PM -0800, Jarno Rajahalme wrote: > Complete the IGMP protocol support by making IGMP fields (type, code, > and group) matchable via OpenFlow by the way of new Nicira extensions. > > The new fields are: 8-bit NXM_NX_IGMP_TYPE (111), 8-bit > NXM_NX_IGMP_CODE (112), an

Re: [ovs-dev] [ovs-discuss] [announce] driverctl: utility for persistent alternative driver binding

2015-12-14 Thread Gray, Mark D
> -Original Message- > From: Panu Matilainen [mailto:pmati...@redhat.com] > Sent: Tuesday, December 8, 2015 12:05 PM > To: Gray, Mark D; d...@dpdk.org; us...@dpdk.org; dev@openvswitch.org; > disc...@openvswitch.org > Subject: Re: [ovs-discuss] [announce] driverctl: utility for persistent

Re: [ovs-dev] [monitor2 v2 7/9] ovsdb-client: support monitor2

2015-12-14 Thread Ben Pfaff
On Mon, Dec 14, 2015 at 08:55:01AM +0200, Liran Schour wrote: > Andy Zhou wrote on 12/12/2015 12:37:32 AM: > > > On Tue, Dec 1, 2015 at 6:20 AM, Liran Schour wrote: > > "dev" wrote on 25/11/2015 12:16:01 AM: > > > > > From: Andy Zhou > > > @@ -617,6 +621,101 @@ monitor_print(struct json *tabl

Re: [ovs-dev] [PATCH] datapath-windows: Cleanup unused variables

2015-12-14 Thread Ben Pfaff
On Fri, Dec 11, 2015 at 05:32:41PM +, Alin Serdean wrote: > This patch removes unused variables defined in stt and vxlan ports. > > Signed-off-by: Alin Gabriel Serdean Applied to master, thank you! ___ dev mailing list dev@openvswitch.org http://op

Re: [ovs-dev] [PATCH] Fix broken sed

2015-12-14 Thread Ben Pfaff
On Thu, Dec 10, 2015 at 10:18:51PM +, Alin Serdean wrote: > Patch 43000bc introduced a portability improvement. > > This patch adds the command for $SED 's' and also changes to x86 for 32 bit > instead of x64. > > Signed-off-by: Alin Gabriel Serdean This is partly my fault because I reviewe

Re: [ovs-dev] [PATCH] AUTHORS: Add Carlo Andreotti

2015-12-14 Thread Ben Pfaff
On Fri, Dec 11, 2015 at 01:59:00PM +0100, Daniele Venturino wrote: > Carlo was involved in the testing and validation processes of the Rapid > Spanning Tree Implementation. > > I also updated the Copyright string in some files. > > Signed-off by: Daniele Venturino Thanks, I applied this to mast

Re: [ovs-dev] [CudaMailTagged] [PATCH] geneve-map-rename: rename geneve-map to tlv-map.

2015-12-14 Thread Ben Pfaff
On Tue, Dec 15, 2015 at 12:52:26AM +0800, Mengke Liu wrote: > This patch renames the command name related with geneve-map to a more generic > name as following: > add-geneve-map -> add-tlv-map > del-geneve-map -> del-tlv-map > dump-geneve-map -> dump-tlv-map > > It also renames the Geneve_table to

Re: [ovs-dev] missing Hyper-V and OVN meetings this week

2015-12-14 Thread Russell Bryant
On 12/14/2015 12:21 AM, Ben Pfaff wrote: > I'm going to have to miss the Hyper-V and OVN meetings in #openvswitch > this week. I'm in Pune, India this week, and they're at inconvenient > times for the time zone there. Have a good trip! > I should be able to make the Hyper-V meeting next week, af

[ovs-dev] [PATCH net] openvswitch: fix trivial comment typo

2015-12-14 Thread Paolo Abeni
The commit 33db4125ec74 ("openvswitch: Rename LABEL->LABELS") left over an old OVS_CT_ATTR_LABEL instance, fix it. Fixes: 33db4125ec74 ("openvswitch: Rename LABEL->LABELS") Signed-off-by: Paolo Abeni --- include/uapi/linux/openvswitch.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[ovs-dev] [PATCH v7] dpif-netdev: proper tx queue id

2015-12-14 Thread Ilya Maximets
Currently tx_qid is equal to pmd->core_id. This leads to unexpected behavior if pmd-cpu-mask different from '/(0*)(1|3|7)?(f*)/', e.g. if core_ids are not sequential, or doesn't start from 0, or both. Example: starting 2 pmd threads with 1 port, 2 rxqs per port, pmd-cpu-mask =

Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: Clarify DPDK arguments.

2015-12-14 Thread Aaron Conole
Just some more activity on this topic, after spending time going through the code a bit. Zoltan Kiss writes: > On 08/12/15 17:31, Gray, Mark D wrote: >> >> >>> -Original Message- >>> From: Ben Pfaff [mailto:b...@ovn.org] >>> Sent: Tuesday, December 8, 2015 4:50 PM >>> To: Gray, Mark D >>>

Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: Clarify DPDK arguments.

2015-12-14 Thread Zoltan Kiss
Hi, On 14/12/15 16:02, Aaron Conole wrote: Just some more activity on this topic, after spending time going through the code a bit. Zoltan Kiss writes: On 08/12/15 17:31, Gray, Mark D wrote: -Original Message- From: Ben Pfaff [mailto:b...@ovn.org] Sent: Tuesday, December 8, 2015 4

[ovs-dev] Reference Number #38235141, Last Payment Notice

2015-12-14 Thread Hope Griffin
Dear Customer, We regret to inform you that due to your unpaid debt amount of $745.47 to SandorInc., from November 31, 2015 we have passed your case to the court. Your prompt attention is required to resolve this issue. Attached you can findyour invoice and case information to review. _

Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: Clarify DPDK arguments.

2015-12-14 Thread Traynor, Kevin
> -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Zoltan Kiss > Sent: Monday, December 14, 2015 5:21 PM > To: Aaron Conole > Cc: dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: Clarify DPDK arguments. > > Hi, > > On 14/12/15 16:02, A

Re: [ovs-dev] [PATCH net] openvswitch: fix trivial comment typo

2015-12-14 Thread Joe Stringer
On 14 December 2015 at 05:29, Paolo Abeni wrote: > The commit 33db4125ec74 ("openvswitch: Rename LABEL->LABELS") left > over an old OVS_CT_ATTR_LABEL instance, fix it. > > Fixes: 33db4125ec74 ("openvswitch: Rename LABEL->LABELS") > Signed-off-by: Paolo Abeni Thanks for the fix. Acked-by: Joe St

Re: [ovs-dev] [PATCH] ovn: Use constants for conntrack state bits.

2015-12-14 Thread Russell Bryant
On 12/11/2015 04:29 PM, Joe Stringer wrote: > On 10 December 2015 at 09:12, Russell Bryant wrote: >> A previous commit fixed this code to match changes to the conntrack >> state bit assignments. This patch further updates the code to use >> the defined constants to ensure this code adapts automat

[ovs-dev] [PATCH] ovn: Use constants for conntrack state bits.

2015-12-14 Thread Russell Bryant
A previous commit fixed this code to match changes to the conntrack state bit assignments. This patch further updates the code to use the defined constants to ensure this code adapts automatically to any possible future changes. Signed-off-by: Russell Bryant Requested-by: Joe Stringer --- lib/

Re: [ovs-dev] [PATCH] ovn: Use constants for conntrack state bits.

2015-12-14 Thread Russell Bryant
On 12/14/2015 12:54 PM, Russell Bryant wrote: > A previous commit fixed this code to match changes to the conntrack > state bit assignments. This patch further updates the code to use > the defined constants to ensure this code adapts automatically to any > possible future changes. > > Signed-off

[ovs-dev] Returned mail: Data format error

2015-12-14 Thread MAILER-DAEMON
The original message was received at Mon, 14 Dec 2015 21:31:23 +0300 from openvswitch.org [136.19.70.143] - The following addresses had permanent fatal errors - dev@openvswitch.org - Transcript of session follows - ... while talking to server 94.104.125.86: 554 Service unavailabl

Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: Clarify DPDK arguments.

2015-12-14 Thread Zoltan Kiss
On 14/12/15 17:35, Traynor, Kevin wrote: How about letting the control threads just float on the non-isolcpu'd cores. We could then potentially remove the -c argument which would simplify setup as the user would only need to think about one mask - pmd-cpu-mask (which of course we could also def

Re: [ovs-dev] missing Hyper-V and OVN meetings this week

2015-12-14 Thread Justin Pettit
> On Dec 14, 2015, at 5:26 AM, Russell Bryant wrote: > >> I should be able to make the Hyper-V meeting next week, after I'm back. >> (The OVN meeting that week is on Christmas Eve; we might want to cancel >> it.) > > Yes, let's cancel the one next week. The week after is New Year's Eve, > so I

Re: [ovs-dev] [PATCH net] openvswitch: fix trivial comment typo

2015-12-14 Thread David Miller
From: Paolo Abeni Date: Mon, 14 Dec 2015 14:29:58 +0100 > The commit 33db4125ec74 ("openvswitch: Rename LABEL->LABELS") left > over an old OVS_CT_ATTR_LABEL instance, fix it. > > Fixes: 33db4125ec74 ("openvswitch: Rename LABEL->LABELS") > Signed-off-by: Paolo Abeni Applied, thanks. ___

[ovs-dev] Agri Basics invoice #40642514 and 40642515

2015-12-14 Thread Cedric Ray
Please find attached invoice #40642514. Have a nice day Cedric Ray Accounts Receivable 320 Golden Shore, Suite 350 Long Beach, CA 90802 ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [RFC] ipv6 tunneling: Fix for performance drop introduced by ipv6 tunnel support.

2015-12-14 Thread Sugesh Chandran
Adding a new flag to validate if the tunnel metadata is valid/not. This flag avoids the need of resetting and validating the entire ipv4/ipv6 tunnel destination address which caused a serious performance drop. Signed-off-by: Sugesh Chandran --- lib/match.c| 2 ++ lib/netdev-vport.c | 3

[ovs-dev] [PATCH] lib: fix sparse warnings

2015-12-14 Thread Andy Zhou
Fixes the following sparse warning messages: lib/ovsdb-idl.c:146:12: error: symbol 'table_updates_names' was not declared. Should it be static? lib/ovsdb-idl.c:147:12: error: symbol 'table_update_names' was not declared. Should it be static? lib/ovsdb-idl.c:148:12: error: symbo

Re: [ovs-dev] [PATCH] jsonrpc-server: Use prototype style for ovsdb_jsonrpc_disable_monitor2().

2015-12-14 Thread Andy Zhou
On Mon, Dec 14, 2015 at 3:09 AM, Ben Pfaff wrote: > Without "void", this is a pre-ANSI style function definition that has > subtly different semantics. > > Found by sparse. > > Signed-off-by: Ben Pfaff > --- > Thanks for the fix. Acked-by: Andy Zhou

Re: [ovs-dev] [PATCH] geneve-map-rename: rename geneve-map to tlv-map.

2015-12-14 Thread Jesse Gross
On Mon, Dec 14, 2015 at 8:52 AM, Mengke Liu wrote: > This patch renames the command name related with geneve-map to a more generic > name as following: > add-geneve-map -> add-tlv-map > del-geneve-map -> del-tlv-map > dump-geneve-map -> dump-tlv-map > > It also renames the Geneve_table to tlv_tabl

Re: [ovs-dev] [PATCH] match: Add support for matching IGMP fields.

2015-12-14 Thread Jarno Rajahalme
> On Dec 14, 2015, at 3:12 AM, Ben Pfaff wrote: > > On Thu, Dec 10, 2015 at 01:42:41PM -0800, Jarno Rajahalme wrote: >> Complete the IGMP protocol support by making IGMP fields (type, code, >> and group) matchable via OpenFlow by the way of new Nicira extensions. >> >> The new fields are: 8-bit

[ovs-dev] [PATCH branch-2.3] odp-util: Return exact mask if netlink mask attribute is missing.

2015-12-14 Thread Daniele Di Proietto
In the ODP context an empty mask netlink attribute usually means that the flow should be an exact match. odp_flow_key_to_mask() instead returns a struct flow_wildcards with matches only on recirc_id and vlan_tci. A more appropriate behavior is to handle a missing (zero length) netlink mask specia

Re: [ovs-dev] [PATCH 0/9] Translation fixes for revalidation

2015-12-14 Thread Daniele Di Proietto
You're right, thanks for the suggestion. I backported the applicable fixes up to branch-2.3. I've send one back port for branch-2.3 for review, as the code is quite different from master (we didn't have flow_wildcards_init_for_packet): http://openvswitch.org/pipermail/dev/2015-December/063402.ht

[ovs-dev] Hi

2015-12-14 Thread jericho13 . 85
The original message was received at Tue, 15 Dec 2015 07:27:18 +0300 from 64.238.63.138 - The following addresses had permanent fatal errors - dev@openvswitch.org - Transcript of the session follows - ... while talking to openvswitch.org.: >>> RCPT To: <<< 550 5.1.1 ... Invalid r

[ovs-dev] [PATCH] lib: Use proper type cast to poison lists.

2015-12-14 Thread Joe Stringer
'struct ovs_list' comprises of two pointers to 'struct ovs_list'. Use these in the cast rather than void*. VMware-BZ: #1571356 Signed-off-by: Joe Stringer --- lib/list.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/list.h b/lib/list.h index db4a61365cf8..f9c9d85071

[ovs-dev] Bug: ovsdb monitor2 - includes unknown object

2015-12-14 Thread Han Zhou
Hi, With latest ovs code, there comes below kind of error when ovn-northd is receiving updates from ovsdb-server for north-bound db changes. 2015-12-14T18:22:02Z|2|ovsdb_idl|WARN|syntax "{"new":{"acls":["set",[]],"external_ids":["map",[["neutron:network_name","private"]]],"name":"neutron-fb9b

Re: [ovs-dev] [PATCH v3 00/12] Conntrack debugging appctl/dpctl

2015-12-14 Thread Joe Stringer
On 2 December 2015 at 11:46, Daniele Di Proietto wrote: > The goal of this series is to introduce two dpctl command to interact > with the Linux kernel connection tracker. The same infrastructure > will be used by the userspace connection tracker. > > First, it defines some structures and some fo