Re: [ovs-dev] [PATCH v2 2/2] ovn-northd: Add logical flows to support native DNS

2017-03-22 Thread Numan Siddique
On Wed, Mar 22, 2017 at 10:43 PM, Guru Shetty wrote: > > > On 22 March 2017 at 09:51, Guru Shetty wrote: > >> >> >> On 13 March 2017 at 00:32, wrote: >> >>> From: Numan Siddique >>> >>> OVN implements native DNS resolution

Re: [ovs-dev] [PATCH] meta-flow: Remove metadata prerequisite on ether type.

2017-03-22 Thread Numan Siddique
On Thu, Mar 23, 2017 at 5:18 AM, Jarno Rajahalme wrote: > Conntrack original direction tuple fields depend on the conntrack > state and the type of the packet that was tracked. These dependencies > were encoded as OpenFlow prerequisites in commit daf4d3c18da4 ("odp: > Support

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

2017-03-22 Thread Numan Siddique
On Thu, Mar 23, 2017 at 5:19 AM, Jarno Rajahalme wrote: > On Mar 22, 2017, at 1:49 PM, Jarno Rajahalme wrote: > > > > > >> On Mar 22, 2017, at 1:43 PM, Ben Pfaff wrote: > >> > >> On Wed, Mar 22, 2017 at 01:40:10PM -0700, Jarno Rajahalme wrote: > >>>

[ovs-dev] OVN meeting report

2017-03-22 Thread Ben Pfaff
Hello everyone. I am not sure whether I am going to be able to attend the OVN meeting tomorrow, because I will be in another possibly distracting meeting, so I'm going to give my report here. Toward the end of last week I did a full pass of reviews through patchwork. The most notable result, I

Re: [ovs-dev] [PATCH] mirror: Allow concurrent lookups.

2017-03-22 Thread Jarno Rajahalme
> On Mar 8, 2017, at 4:15 PM, Ben Pfaff wrote: > > On Tue, Feb 21, 2017 at 07:44:02PM -0800, Jarno Rajahalme wrote: >> Handler threads use a selection of mirror functions with the >> assumption that the data referred to is RCU protected, while the >> implementation has not

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

2017-03-22 Thread Jarno Rajahalme
On Mar 22, 2017, at 1:49 PM, Jarno Rajahalme wrote: > > >> On Mar 22, 2017, at 1:43 PM, Ben Pfaff wrote: >> >> On Wed, Mar 22, 2017 at 01:40:10PM -0700, Jarno Rajahalme wrote: >>> On Mar 22, 2017, at 1:21 PM, Ben Pfaff wrote: On

[ovs-dev] [PATCH] meta-flow: Remove metadata prerequisite on ether type.

2017-03-22 Thread Jarno Rajahalme
Conntrack original direction tuple fields depend on the conntrack state and the type of the packet that was tracked. These dependencies were encoded as OpenFlow prerequisites in commit daf4d3c18da4 ("odp: Support conntrack orig tuple key."). However, having a prerequisite from a metadata field

[ovs-dev] New Arrivals

2017-03-22 Thread Bonesca - Jona
  [ View in browser ]( http://r.newsletter.bonescamail.nl/nru6rae6oatrf.html )         Some new arrivals of this week for your information:   Hilsa 800-1000 (19-20 pcs/box so very larga grading!!) Rohu Gutted 2-3 & 3 Kg up  Nile Perch Headed and Gutted 1-3 & 3-7 Kg Nile Perch Whole Gutted 1-3

[ovs-dev] [PATCH 3/3 v2] datapath-windows: NAT integration with conntrack

2017-03-22 Thread Yin Lin
Signed-off-by: Yin Lin --- datapath-windows/ovsext/Actions.c | 179 --- datapath-windows/ovsext/Actions.h | 77 + datapath-windows/ovsext/Conntrack-nat.c | 3 +- datapath-windows/ovsext/Conntrack.c | 184

[ovs-dev] [PATCH 2/3 v2] datapath-windows: Add NAT module in conntrack

2017-03-22 Thread Yin Lin
Signed-off-by: Yin Lin Issue: # Change-Id: I6f37360c36525548b343f0016304015fec8aba7d --- datapath-windows/automake.mk| 2 + datapath-windows/ovsext/Conntrack-nat.c | 437 datapath-windows/ovsext/Conntrack-nat.h | 39 +++ 3 files

[ovs-dev] [PATCH 1/3 v2] datapath-windows: Add support for NAT in conntrack

2017-03-22 Thread Yin Lin
Add support for parsing netlink attributes related to NAT in conntrack. Co-Authored-by: Anand Kumar Co-Authored-by: Darrell Ball Signed-off-by: Yin Lin --- datapath-windows/ovsext/Conntrack.c | 73 +++--

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

2017-03-22 Thread Ben Pfaff
On Wed, Mar 22, 2017 at 01:40:10PM -0700, Jarno Rajahalme wrote: > > > On Mar 22, 2017, at 1:21 PM, Ben Pfaff wrote: > > > > On Wed, Mar 22, 2017 at 09:30:36PM +0530, nusid...@redhat.com wrote: > >> From: Numan Siddique > >> > >> When ovs-vswitchd sends the

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

2017-03-22 Thread Jarno Rajahalme
> On Mar 22, 2017, at 1:21 PM, Ben Pfaff wrote: > > On Wed, Mar 22, 2017 at 09:30:36PM +0530, nusid...@redhat.com wrote: >> From: Numan Siddique >> >> When ovs-vswitchd sends the NX_PACKET_IN2 message, it may not >> encode ETH_TYPE of the packet. And with

Re: [ovs-dev] [RFC] sandbox: use ssl for ovn-controller to sb db connection

2017-03-22 Thread Russell Bryant
On Wed, Mar 22, 2017 at 1:19 PM, Lance Richardson wrote: > When SSL support is available, use SSL for the ovn-controller > to southbound database connection. When configured without > SSL, unix socket connections are used. > > Signed-off-by: Lance Richardson

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

2017-03-22 Thread Numan Siddique
On Mar 22, 2017 11:56 PM, "Joe Stringer" wrote: On 22 March 2017 at 09:00, wrote: > From: Numan Siddique > > When ovs-vswitchd sends the NX_PACKET_IN2 message, it may not > encode ETH_TYPE of the packet. Hmm, I'm not sure why this would

[ovs-dev] Superlinha - Central de Atendimento - Santander

2017-03-22 Thread Superlinha - Central de Atendimento - Santander
Caso n�o esteja visualizando, Abra Aqui APERTE PARA VISUALIZAR ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

2017-03-22 Thread Joe Stringer
On 22 March 2017 at 09:00, wrote: > From: Numan Siddique > > When ovs-vswitchd sends the NX_PACKET_IN2 message, it may not > encode ETH_TYPE of the packet. Hmm, I'm not sure why this would be? > And with the commit daf4d3c18da > ("odp: Support

[ovs-dev] [RFC] sandbox: use ssl for ovn-controller to sb db connection

2017-03-22 Thread Lance Richardson
When SSL support is available, use SSL for the ovn-controller to southbound database connection. When configured without SSL, unix socket connections are used. Signed-off-by: Lance Richardson --- tutorial/automake.mk | 3 ++- tutorial/ovs-sandbox | 42

Re: [ovs-dev] [PATCH v2 2/2] ovn-northd: Add logical flows to support native DNS

2017-03-22 Thread Guru Shetty
On 22 March 2017 at 09:51, Guru Shetty wrote: > > > On 13 March 2017 at 00:32, wrote: > >> From: Numan Siddique >> >> OVN implements native DNS resolution which can be used to resolve the >> internal DNS names belonging to a logical

Re: [ovs-dev] [PATCH v2 2/2] ovn-northd: Add logical flows to support native DNS

2017-03-22 Thread Guru Shetty
On 13 March 2017 at 00:32, wrote: > From: Numan Siddique > > OVN implements native DNS resolution which can be used to resolve the > internal DNS names belonging to a logical datapath. > > To support this, the logical ports should be configured with the

Re: [ovs-dev] [RFC] lib/automake.mk: remove runtime directories

2017-03-22 Thread Eric Garver
On Tue, Mar 21, 2017 at 02:20:30PM -0400, Aaron Conole wrote: > Aaron Conole writes: > > > The Open vSwitch run, log, and DB directories are installed as part of the > > normal `make install` process. However, this means they are created with > > user and group ownership

Re: [ovs-dev] [PATCH] datapath-windows: Add support for OVS_CT_ATTR_FORCE_COMMIT

2017-03-22 Thread Guru Shetty
On 21 March 2017 at 00:02, Sairam Venugopal wrote: > Add support for handling OVS_CT_ATTR_FORCE_COMMIT in Conntrack action. > When this flag is specified, it implicitly means commit and deletes > entries in the reverse direction. > > Signed-off-by: Sairam Venugopal

Re: [ovs-dev] [PATCH v4 1/2] ovn: Gratuitous ARP for centralized NAT rules on a distributed router

2017-03-22 Thread Guru Shetty
> > > >> >>> This is achieved by extending >>> the syntax for "options:nat-addresses" in the southbound database, >>> allowing the condition 'is_chassis_resident("LPORT_NAME")' to be >>> appended >>> after the MAC and IP addresses. This condition is automatically inserted >>> by ovn-northd when

[ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

2017-03-22 Thread nusiddiq
From: Numan Siddique When ovs-vswitchd sends the NX_PACKET_IN2 message, it may not encode ETH_TYPE of the packet. And with the commit daf4d3c18da ("odp: Support conntrack orig tuple key."), the conntrack fields are encoded, if set. After the commit 7befb20d0f70 ("nx-match:

Re: [ovs-dev] [PATCH] ofproto-dpif : propagate may_enable flag as link aliveness

2017-03-22 Thread László Sürü
Thanks for the comments! I've applied the recommended style. Indeed, ofport_modified() in the first patch may reported wrong status - even if only for a short time. To avoid this, liveness handling at port state change has been moved out to port_modified() instead and port status update

[ovs-dev] [PATCH ovs V5 15/24] netdev-tc-offloads: Implement netdev flow del using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 33 ++--- 1 file changed, 30 insertions(+), 3 deletions(-) diff --git a/lib/netdev-tc-offloads.c

[ovs-dev] [PATCH ovs V5 14/24] dpif-netlink: Use netdev flow del api to delete a flow

2017-03-22 Thread Roi Dayan
From: Paul Blakey If a flow was offloaded to a netdev we delete it using netdev flow api. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 12 +++- lib/netdev.c | 15 +++

[ovs-dev] [PATCH ovs V5 16/24] dpif-netlink: Use netdev flow get api to query a flow

2017-03-22 Thread Roi Dayan
From: Paul Blakey Search all datapath added netdevs for a given flow using netdev flow api and parse it back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 50

[ovs-dev] [PATCH ovs V5 18/24] netdev-linux: Disallow setting policing when configured with hw offload

2017-03-22 Thread Roi Dayan
From: Paul Blakey Notify as not supported. Otherwise the ingress qdisc is being removed and offload rules will be removed. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-linux.c | 8 1 file changed, 8

[ovs-dev] [PATCH ovs V5 20/24] netdev-tc-offloads: Add ingress on netdev flow api init

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/lib/netdev-tc-offloads.c

[ovs-dev] [PATCH ovs V5 19/24] netdev-vport: Use common offloads interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey netdev vports are backed by actualy netdev at the kernel level, so they can use the common netdev-tc offloads interface for flow offloading (if enabled). Signed-off-by: Paul Blakey Signed-off-by: Simon Horman

[ovs-dev] [PATCH ovs V5 17/24] netdev-tc-offloads: Implement flow get using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Search the requested ufid for a offloaded flow, and if found, dump and parse it back to required format. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 49

[ovs-dev] [PATCH ovs V5 23/24] compat: Add tc compatibility headers for old kernels

2017-03-22 Thread Roi Dayan
From: Paul Blakey Added compatibility headers for actions vlan and tunnel key. Do not use compat code when compiling kernel datapath there is no need for it as TC compatibility is not provided there. In other words, the compat code is only used when compiling user-space code

[ovs-dev] [PATCH ovs V5 22/24] tests: Add system-offloads-testsuite

2017-03-22 Thread Roi Dayan
From: Paul Blakey The new system-offloads-testsuite, which can be launched via `make check-offloads`, tests offloading capabilities to makes sure that certian flows are actually offloaded. The tests run on virtual netdevices (VETH). Signed-off-by: Paul Blakey

[ovs-dev] [PATCH ovs V5 24/24] netdev: Init flow api on already added ports on offload enable

2017-03-22 Thread Roi Dayan
From: Paul Blakey Ports already added to a switch are not being initialized for offloading so when enabling offload we need to go over those ports. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev.c | 14

[ovs-dev] [PATCH ovs V5 13/24] netdev-tc-offloads: Implement netdev flow put using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Currently only tunnel offload is supported. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 324 ++- 1 file changed, 317 insertions(+), 7

[ovs-dev] [PATCH ovs V5 12/24] netdev-tc-offloads: Add flower mask to priority map

2017-03-22 Thread Roi Dayan
From: Paul Blakey Flower classifer requires a different priority per mask, so we hash the mask and generate a new priority for each new mask used. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 38

[ovs-dev] [PATCH ovs V5 01/24] tc: Add tc flower interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Add tc flower interface that will be used to offload flows via tc flower classifier. Depending on the flag used (skip_sw/hw) flower will pass those to HW or handle them itself. Move some tc related functions from netdev-linux.c to tc.c Co-authored-by:

[ovs-dev] [PATCH ovs V5 08/24] dpif-netlink: Dump netdevs flows on flow dump

2017-03-22 Thread Roi Dayan
From: Paul Blakey While dumping flows, dump flows that were offloaded to netdev and parse them back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 179

[ovs-dev] [PATCH ovs V5 10/24] netdev-tc-offloads: Implement netdev flow dump api using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 182 --- 1 file changed, 174 insertions(+), 8 deletions(-) diff --git

[ovs-dev] [PATCH ovs V5 00/24] Introducing HW offload support for openvswitch

2017-03-22 Thread Roi Dayan
This patch series introduces rule offload functionality to dpif-netlink via netdev ports new flow offloading API. The user can specify whether to enable rule offloading or not via OVS configuration. Netdev providers are able to implement netdev flow offload API in order to offload rules. This

[ovs-dev] [PATCH ovs V5 09/24] netdev-tc-offloads: Add ufid to tc/netdev map

2017-03-22 Thread Roi Dayan
From: Paul Blakey Flows offloaded to tc are identified by priority and handle pair while OVS flows are identified by ufid. Added a hash map to convert between the two for later retrieval and deleting of offloaded flows. Signed-off-by: Paul Blakey

[ovs-dev] [PATCH ovs V5 11/24] dpif-netlink: Use netdev flow put api to insert a flow

2017-03-22 Thread Roi Dayan
From: Paul Blakey Using the new netdev flow api operate will now try and offload flows to the relevant netdev of the input port. Other operate methods flows will come in later patches. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan

[ovs-dev] [PATCH ovs V5 02/24] netdev: Adding a new netdev api to be used for offloading flows

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/automake.mk | 2 + lib/netdev-bsd.c | 2 + lib/netdev-dpdk.c| 1 + lib/netdev-dummy.c | 2 + lib/netdev-linux.c |

[ovs-dev] [PATCH ovs V5 07/24] netdev-tc-offloads: Implement netdev flow flush using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c

[ovs-dev] [PATCH ovs V5 06/24] dpif-netlink: Flush added ports using netdev flow api

2017-03-22 Thread Roi Dayan
From: Paul Blakey If netdev flow offloading is enabled, flush all added ports using netdev flow api. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 5 + lib/netdev.c | 12

[ovs-dev] [PATCH ovs V5 05/24] dpif: Save added ports in a port map for netdev flow api use

2017-03-22 Thread Roi Dayan
From: Paul Blakey To use netdev flow offloading api, dpifs needs to iterate over added ports. This addition inserts the added dpif ports in a hash map, The map will also be used to translate dpif ports to netdevs. Signed-off-by: Paul Blakey Reviewed-by:

[ovs-dev] [PATCH ovs V5 04/24] other-config: Add tc-policy switch to control tc flower flag

2017-03-22 Thread Roi Dayan
From: Paul Blakey Add a new configuration tc-policy option that controls tc flower flag. Possible options are none, skip_sw, skip_hw. The default is none which is to insert the rule both to sw and hw. This option is only relevant if hw-offload is enabled. Signed-off-by: Paul

[ovs-dev] [PATCH ovs V5 03/24] other-config: Add hw-offload switch to control netdev flow offloading

2017-03-22 Thread Roi Dayan
From: Paul Blakey Add a new configuration option - hw-offload that enables netdev flow api. Enabling this option will allow offloading flows using netdev implementation instead of the kernel datapath. This configuration option defaults to false - disabled. Signed-off-by:

[ovs-dev] How does ovs userpsace+dpdk handle the original data packet when it needs ARP request and waits for ARP reply?

2017-03-22 Thread Joo Kim
Hello, In OVS2.6 userspace datapath code, when there is no ARP cache, looks like it sends ARP request on the fly (via tnl_send_arp_request() ). Then, until ARP reply is received later, what happens to the original data packet? Is it queued and sent later once ARP resolution is made ? Or just

Re: [ovs-dev] Checksum Incorrect IPv6

2017-03-22 Thread Aqsa Malik
I upgraded to 2.7 but still no luck with the checksum. On Fri, Mar 17, 2017 at 4:20 PM, Ben Pfaff wrote: > I recommend upgrading, because that version is 3 1/2 years old. > > On Fri, Mar 17, 2017 at 09:21:03AM +0100, Aqsa Malik wrote: > > I am using 2.0.2 at the moment. > > > > On

[ovs-dev] [PATCH 1/1] dpif-netdev: The pmd-*-show commands will show info in core order

2017-03-22 Thread Eelco Chaudron
The "ovs-appctl dpif-netdev/pmd-rxq-show" and "ovs-appctl dpif-netdev/pmd-stats-show" commands show their output per core_id, sorted on the hash location. My OCD was kicking in when using these commands, hence this change to display them in natural core_id order. In addition I had to change a