[ovs-dev] RETURNED MAIL: SEE TRANSCRIPT FOR DETAILS

2015-12-09 Thread Bounced mail
The message could not be delivered ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [ovs] ovn-nbctl:add db commands help (#93)

2015-12-09 Thread Wei Li
在 2015/12/8 22:57, Russell Bryant 写道: On 12/08/2015 01:06 AM, Wei Li wrote: 在 2015/12/7 23:30, Russell Bryant 写道: Can you update the man page, as well? I would like to do this,but have a question There is a reference to /db-ctl-base.man/ in /ovn-sbctl.8.in/ implement by "/.so lib/db-ctl-ba

[ovs-dev] [PATCH 4/9] odp-util: Commit ICMP set only for ICMP packets.

2015-12-09 Thread Daniele Di Proietto
commit_set_icmp_action() should do its job only if the packet is ICMP, otherwise there will be two problems: * A set ICMP action will be inserted in the ODP actions and the flow will be slow pathed. * The tp_src and tp_dst field will be unwildcarded. Normal TCP or UDP packets won't be impacted,

[ovs-dev] [PATCH 6/9] odp-util: Correctly [de]serialize mask for ND attributes.

2015-12-09 Thread Daniele Di Proietto
When converting between ODP attributes and struct flow_wildcards, we check that all the prerequisites are exact matched on the mask. For ND(ICMPv6) attributes, an exact match on tp_src and tp_dst (which in this context are the icmp type and code) shold look like htons(0xff), not htons(0x). Fi

[ovs-dev] [PATCH 2/9] ofproto-dpif-xlate: Fix revalidation in execute_controller_action().

2015-12-09 Thread Daniele Di Proietto
If there's no actual packet (e.g. during revalidation), execute_controller_action() exits right away, without calling xlate_commit_actions(). xlate_commit_actions() might have an influence on slow_path reason (which is included in the generated ODP actions), meaning that the revalidation will not

[ovs-dev] [PATCH 8/9] ofproto-dpif-xlate: Don't unwildcard tunnel attributes on set.

2015-12-09 Thread Daniele Di Proietto
When translating a set action we also unwildcard the field in question. This is done to correctly translate set actions with the value identical to the ingress flow, like in the following example: flow table: tcp,actions=set_field:80->tcp_dst,output:5 ingress packet: ...,tcp,tcp_dst=80 datapat

[ovs-dev] [PATCH 9/9] ofproto-dpif-xlate: Do not include non existing MPLS lse in wc.

2015-12-09 Thread Daniele Di Proietto
An action list like actions=push_mpls:0x8847,load:10->OXM_OF_MPLS_LABEL[] will generate an exact match on the newly pushed mpls label, because the load action needs to unwildcard its target to work properly. This doesn't make sense, because the original flow didn't have that label. An invalid ma

[ovs-dev] [PATCH 7/9] ofproto-dpif-xlate: Generate right mask when checking prereqs.

2015-12-09 Thread Daniele Di Proietto
During translation we need to unwildcard each member of the flow that we look at. When setting or moving a field, we need to look at (and consequently unwildcard) the field itself an all the prerequisites. The current code unwildcards the field and all the prerequisites and then reads them. This

[ovs-dev] [PATCH 3/9] tnl-ports: Generate mask with correct prerequisites.

2015-12-09 Thread Daniele Di Proietto
We should match on the transport ports only if the tunnel has a UDP header. It doesn't make sense to match on transport port for GRE tunnels. Also, to match on fragment bits we should use FLOW_NW_FRAG_MASK instead of 0xFF. FLOW_NW_FRAG_MASK is what we get if we convert to the ODP netlink format

[ovs-dev] [PATCH 5/9] odp-util: Return exact mask if netlink mask attribute is missing.

2015-12-09 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{,_udpif}() instead return 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 mas

[ovs-dev] [PATCH 1/9] dpif-netdev: Initialize match.tun_md in various places.

2015-12-09 Thread Daniele Di Proietto
This solves a crash in dp_netdev_flow_add(), when log level is debug. Signed-off-by: Daniele Di Proietto --- lib/dpif-netdev.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index a67ef05..3bf130d 100644 --- a/lib/dpif-netdev.c +++ b/lib/dpif-netdev.c

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

2015-12-09 Thread Daniele Di Proietto
Sometimes the ofproto layer creates a flow which is not liked by the revalidation for various reasons. This behavior, while not critical might impact the performance. This series aims to fix a lot of these bugs. The detection has been done by modifying OVS to revalidate a flow as soon as it is i

Re: [ovs-dev] [PATCH 1/3] datapath-windows: Add sequence tunneling information define

2015-12-09 Thread Nithin Raju
I suggested we get rid of ŒOVS_TNL_F_SEQ¹ in the 3/3 patch, since there¹s no way of using it. -- Nithin -Original Message- From: dev on behalf of Alin Serdean Date: Wednesday, December 2, 2015 at 9:50 AM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH 1/3] datapath-windows: Add seq

Re: [ovs-dev] [PATCH 3/3 v2] datapath-windows: Add GRE TEB support for windows datapath

2015-12-09 Thread Nithin Raju
Alin, Thanks for adding GRE support. The patch looks good for the most part. RFC 2890 calls for a much more elaborate handling of key and sequence numbers in the GRE header. Eg. Buffering by handling out of order sequence packets. In your patch, none of that is being done. Also, there¹s no way to

Re: [ovs-dev] [PATCH 05/14] Add OpenFlow support for new "arp" action.

2015-12-09 Thread Justin Pettit
> On Dec 8, 2015, at 5:08 PM, Ben Pfaff wrote: > > --- a/NEWS > +++ b/NEWS > @@ -1,5 +1,7 @@ > Post-v2.5.0 > - > + - OpenFlow: > + * New "arp" OpenFlow extension action for generating ARP packets. Once you have a design for packet injection from a controller, I think w

Re: [ovs-dev] [PATCH] datapath-windows: Added recirculation support.

2015-12-09 Thread Sairam Venugopal
Hey Sorin, Thanks for sending this out. I am currently working on a feature that relies on recirculation. I tried applying the patch and encountered the following errors in the log: NlAttrParse:1103 Required attr:1 missing _FlowNlGetCmdHandler:460 Attr Parsing failed for msg: E796E400 NlA

Re: [ovs-dev] [PATCH] datapath-windows: Fix compilation issue

2015-12-09 Thread Nithin Raju
That was my patch that broke it. I had fixed it in my local tree but forgot to send out the updated patch. Thanks for fixing this. Acked-by: Nithin Raju -Original Message- From: dev on behalf of Sairam Venugopal Date: Wednesday, December 9, 2015 at 5:31 PM To: "dev@openvswitch.org"

[ovs-dev] [PATCH] datapath-windows: Fix compilation issue

2015-12-09 Thread Sairam Venugopal
The previous patch was missing a ; Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Actions.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath-windows/ovsext/Actions.c b/datapath-windows/ovsext/Actions.c index 7d34458..f6c029b 100644 --- a/datapath-windows

Re: [ovs-dev] [PATCH 04/14] Use ip_parse() and ipv6_parse() and variants in more places.

2015-12-09 Thread Justin Pettit
> On Dec 8, 2015, at 5:08 PM, Ben Pfaff wrote: > > > -gw6 = in6addr_any; > +gw6 = in6addr_exact; Did you mean to change this from "any" to "exact"? This is a nice improvement. Other than the above comment: Acked-by: Justin Pettit --Justin

Re: [ovs-dev] [PATCH 03/14] packets: Add new functions for IPv4 and IPv6 address parsing.

2015-12-09 Thread Justin Pettit
Acked-by: Justin Pettit --Justin > On Dec 8, 2015, at 5:08 PM, Ben Pfaff wrote: > > These will be used in an upcoming patch to reduce duplicated code. > > Signed-off-by: Ben Pfaff > --- > lib/packets.c | 132 ++ > lib/packets.h | 7 ++

Re: [ovs-dev] [PATCH 02/14] packets: New macro ETH_ADDR_STRLEN.

2015-12-09 Thread Justin Pettit
Acked-by: Justin Pettit --Justin > On Dec 8, 2015, at 5:08 PM, Ben Pfaff wrote: > > An upcoming commit will introduce another user. > > Signed-off-by: Ben Pfaff > --- > lib/packets.h | 1 + > vswitchd/bridge.c | 5 +++-- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a

Re: [ovs-dev] [PATCH 01/14] sparse: Define INET_ADDRSTRLEN.

2015-12-09 Thread Justin Pettit
Acked-by: Justin Pettit --Justin > On Dec 8, 2015, at 5:08 PM, Ben Pfaff wrote: > > POSIX defines this but it was missing from the OVS header file definitions > for "sparse". > > Signed-off-by: Ben Pfaff > --- > include/sparse/netinet/in.h | 1 + > 1 file changed, 1 insertion(+) > > diff --

Re: [ovs-dev] [PATCH] ovn: Fix ct_state bit mappings in OVN symtab.

2015-12-09 Thread Joe Stringer
On 9 December 2015 at 08:19, Russell Bryant wrote: > On 12/08/2015 06:20 PM, Joe Stringer wrote: >> On 8 December 2015 at 14:34, Russell Bryant wrote: >>> The OVN symbol table contained outdated mappings between connection >>> states and the corresponding bit in the ct_state field. This patch >>

Re: [ovs-dev] [PATCH] CONTRIBUTING: Document the Fixes header.

2015-12-09 Thread Russell Bryant
On 12/09/2015 02:35 PM, Joe Stringer wrote: > On 9 December 2015 at 11:05, Russell Bryant wrote: >> Document the use of the Fixes header to refer to a commit that >> introduced a bug being fixed. >> >> Signed-off-by: Russell Bryant >> --- >> CONTRIBUTING.md | 8 >> 1 file changed, 8 ins

[ovs-dev] Invoice #46189846 from DataCorp Inc.

2015-12-09 Thread Ora Booker
Dear Customer, Reference nr. 46189846-1110 Our internal records show that you have an outstanding balance dating on youraccount. Previous invoice was for $618.73 and have yet to receive your payment. You can find the copy of the invoice enclosed to this letter. In case if you have already tran

Re: [ovs-dev] [PATCH] CONTRIBUTING: Document the Fixes header.

2015-12-09 Thread Joe Stringer
On 9 December 2015 at 11:05, Russell Bryant wrote: > Document the use of the Fixes header to refer to a commit that > introduced a bug being fixed. > > Signed-off-by: Russell Bryant > --- > CONTRIBUTING.md | 8 > 1 file changed, 8 insertions(+) > > diff --git a/CONTRIBUTING.md b/CONTRIB

[ovs-dev] [PATCH] CONTRIBUTING: Document the Fixes header.

2015-12-09 Thread Russell Bryant
Document the use of the Fixes header to refer to a commit that introduced a bug being fixed. Signed-off-by: Russell Bryant --- CONTRIBUTING.md | 8 1 file changed, 8 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 97ab9cb..247c1d9 100644 --- a/CONTRIBUTING.md +++ b/C

Re: [ovs-dev] [PATCH] tests: Fix typo in comment.

2015-12-09 Thread Justin Pettit
Acked-by: Justin Pettit --Justin > On Dec 9, 2015, at 9:31 AM, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > --- > tests/ofproto-dpif.at | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/tests/ofproto-dpif.at b/tests/ofproto-dpif.at > index 60734db..4c2a

Re: [ovs-dev] 802.1x authentication support in OVS

2015-12-09 Thread Justin Pettit
> On Dec 9, 2015, at 4:07 AM, > wrote: > > Hi Team, > > I would like to configure the 802.1x authentication in OVS. As of my > knowledge, current OVS does not support 802.1x authentication. If any support > already exist or any one provided the patch, please let me know the > details/confi

[ovs-dev] [PATCH] tests: Fix typo in comment.

2015-12-09 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- tests/ofproto-dpif.at | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/ofproto-dpif.at b/tests/ofproto-dpif.at index 60734db..4c2a995 100644 --- a/tests/ofproto-dpif.at +++ b/tests/ofproto-dpif.at @@ -3354,7 +3354,7 @@ mpls,dl_vl

[ovs-dev] Your order #83274151 - Corresponding Invoice #386D3034

2015-12-09 Thread Kenya Rodriquez
Dear Valued Customer, We are pleased to inform you that your order #83274151 has been processed andready to be dispatched. However, according to our records, above mentionedinvoice is still unpaid. We would highly appreciate if you sent your paymentpromptly. For your information, don't hesitate

Re: [ovs-dev] [PATCH] ovn: Fix ct_state bit mappings in OVN symtab.

2015-12-09 Thread Russell Bryant
On 12/08/2015 06:20 PM, Joe Stringer wrote: > On 8 December 2015 at 14:34, Russell Bryant wrote: >> The OVN symbol table contained outdated mappings between connection >> states and the corresponding bit in the ct_state field. This patch >> updates the symbol table with the proper values as defin

[ovs-dev] Your order #72307442 - Corresponding Invoice #0BEA7625

2015-12-09 Thread Mavis Caldwell
Dear Valued Customer, We are pleased to inform you that your order #72307442 has been processed andready to be dispatched. However, according to our records, above mentionedinvoice is still unpaid. We would highly appreciate if you sent your paymentpromptly. For your information, don't hesitate

Re: [ovs-dev] [PATCH net-next v4 4/8] openvswitch: Update the CT state key only after nf_conntrack_in().

2015-12-09 Thread Sergei Shtylyov
Hello. On 12/9/2015 4:01 AM, Jarno Rajahalme wrote: Only a successful nf_conntrack_in() call can effect a connection state change, so if suffices to update the key only after the nf_conntrack_in() returns. This change is needed for the later NAT patches. Signed-off-by: Jarno Rajahalme ---

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

2015-12-09 Thread Zoltan Kiss
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 Cc: Traynor, Kevin; dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: Clarify DPDK arguments. On Tue, Dec 08, 20

[ovs-dev] 802.1x authentication support in OVS

2015-12-09 Thread gayathri.manepalli
Hi Team, I would like to configure the 802.1x authentication in OVS. As of my knowledge, current OVS does not support 802.1x authentication. If any support already exist or any one provided the patch, please let me know the details/configuration. Thanks & Regards, Gayathri The information con

Re: [ovs-dev] [PATCH] FAQ: Add entry about different datapaths features.

2015-12-09 Thread Gray, Mark D
> From: Justin Pettit [mailto:jpet...@ovn.org] > Sent: Tuesday, December 8, 2015 5:03 PM > > > On Dec 8, 2015, at 1:01 AM, Gray, Mark D wrote: > > > >> +Feature | Linux upstream | Linux OVS tree | Userspace | > >> Hyper-V > | > >> +--|:--:|:-