[ovs-dev] Where to add GTP tunnel headers in datapath flow table of open vSwitch

2016-02-28 Thread Ajmer Singh
Hi, We are working on open Vswitch 2.4 and the requirement is to add GTP tunnel headers in flow table so that onos controller can add flows for GTP packets and the switch captures GTP packets as well. What we have identified so far: 1. Struct sk_buff :-> the structure containing the received packe

Re: [ovs-dev] [PATCHv2 2/2] ovn: Add l3 port security for IPv4 and IPv6

2016-02-28 Thread Numan Siddique
On 02/27/2016 12:57 AM, Ben Pfaff wrote: > I see some ways to simplify code, improve the wording of the > documentation, and other improvements. I'm appending some suggested > changes to fold in. > > I see some XXXs that it would be nice to fix before getting this in. > > I'll look forward to v3.

Re: [ovs-dev] World Popular pilules cure online

2016-02-28 Thread Valerie
96 Re: World Popular remedies online p{margin:10px 0;padding:0;} table{border-collapse:collapse;} h1,h2,h3,h4,h5,h6{display:block;margin:0;padding:0;} img,a img{border:0;height:auto;outline:none;text-decoration:none;} body,#bodyTable,#bodyCell{height:100%;margin:0;padding:0;width:100

[ovs-dev] OVS PatchWork Update

2016-02-28 Thread alexw
30+ Day Old Patches === ID State Date Name -- - 548008 New2015-11-24 11:19:01 [ovs-dev,v4,2/4] ovn: New flows for DHCP tranffic 548010 New2015-11-24 11:19:02 [ovs-dev,v4,3/4] ovn: Process dhcp packet-ins and

Re: [ovs-dev] [PATCH V5 2/2] netdev-dpdk: add jumbo frame support

2016-02-28 Thread Daniele Di Proietto
Hi Mark, I thought about how to make this dynamically reconfigurable and I've come up with a series that generalizes the mechanism used for rx/tx queues reconfiguration: http://openvswitch.org/pipermail/dev/2016-February/066927.html I also think that it should be better to introduce 'mtu_request

Re: [ovs-dev] [PATCH V5 1/2] netdev-dpdk: clean up mbuf initialization

2016-02-28 Thread Daniele Di Proietto
Thanks for the cleanup, it is really appreciated! I put my signoff, as you suggested and applied this to master On 22/02/2016 14:39, "Flavio Leitner" wrote: >On Fri, 19 Feb 2016 11:25:11 + >Mark Kavanagh wrote: > >> Current mbuf initialization relies on magic numbers and does not >> accomo

[ovs-dev] [PATCH 10/11] netdev: Add reconfigure request mechanism.

2016-02-28 Thread Daniele Di Proietto
A netdev provider, especially a PMD provider (like netdev DPDK) might not be able to change some of its parameters (such as MTU, or number of queues) without stopping everything and restarting. This commit introduces a mechanism that allows a netdev provider to request a restart (netdev_request_re

[ovs-dev] [PATCH 01/11] dpif-netdev: Proper error handling in do_add_port().

2016-02-28 Thread Daniele Di Proietto
This fixes multiple error path mistakes in do_add_port, none of which has been a problem in practice so far. This change will make it easier for a following commit to return in case of error. Also, this removes an unneeded special case for tunnel ports. Signed-off-by: Daniele Di Proietto --- li

[ovs-dev] [PATCH 08/11] dpif-netdev: Document locking discipline for non_pmd_mutex.

2016-02-28 Thread Daniele Di Proietto
This just documents what the current code already assumes. Signed-off-by: Daniele Di Proietto --- lib/dpif-netdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index 2c11bd3..26daf08 100644 --- a/lib/dpif-netdev.c +++ b/lib/dpif-netdev.c @@ -184,6 +1

[ovs-dev] [PATCH 11/11] netdev-dpdk: Use ->reconfigure() call to change rx/tx queues.

2016-02-28 Thread Daniele Di Proietto
This introduces in dpif-netdev and netdev-dpdk the first use for the newly introduce reconfigure netdev call. When a request to change the number of queues comes, netdev-dpdk will remember this and notify the upper layer via netdev_request_reconfigure(). The datapath, instead of periodically call

[ovs-dev] [PATCH 05/11] dpif-netdev: Wait an RCU grace period before freeing ports.

2016-02-28 Thread Daniele Di Proietto
The dpif-netdev datapath keeps ports in a cmap which is written only by the main thread (holding port_mutex), but which is read concurrently by many threads (most notably the pmd threads). When removing ports from the datapath we should postpone the deletion, otherwise another thread might access

[ovs-dev] [PATCH 03/11] dpif-netdev: Do not keep refcount for ports.

2016-02-28 Thread Daniele Di Proietto
Only the main thread will delete ports after pausing every other thread. There's no need to keep count. Signed-off-by: Daniele Di Proietto --- lib/dpif-netdev.c | 50 ++ 1 file changed, 14 insertions(+), 36 deletions(-) diff --git a/lib/dpif-netd

[ovs-dev] [PATCH 04/11] dpif-netdev: Remove useless dpif-dummy/delete-port appctl.

2016-02-28 Thread Daniele Di Proietto
It is only used in the testsuite and it can be replaced by a dpctl command. Signed-off-by: Daniele Di Proietto --- lib/dpif-netdev.c | 33 - tests/bridge.at | 4 ++-- 2 files changed, 2 insertions(+), 35 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-n

[ovs-dev] [PATCH 02/11] dpif-netdev: Keep count of elements in port->rxq[].

2016-02-28 Thread Daniele Di Proietto
This will ease deleting a port with no open rxqs. Signed-off-by: Daniele Di Proietto --- lib/dpif-netdev.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index 914579d..5d7fac7 100644 --- a/lib/dpif-netdev.c ++

[ovs-dev] [PATCH 06/11] ofproto-dpif: Call dpif_poll_threads_set() before dpif_run()

2016-02-28 Thread Daniele Di Proietto
An upcoming commit will make dpif_poll_threads_set() record the requested configuration and dpif_run() apply it, so it makes sense to change the order. Signed-off-by: Daniele Di Proietto --- ofproto/ofproto-dpif.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ofproto/of

[ovs-dev] [PATCH 07/11] dpif-netdev: Change pmd thread configuration in dpif_netdev_run().

2016-02-28 Thread Daniele Di Proietto
Signed-off-by: Daniele Di Proietto --- lib/dpif-netdev.c | 137 ++-- lib/dpif-provider.h | 3 +- 2 files changed, 82 insertions(+), 58 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index c9c7d16..2c11bd3 100644 --- a/lib/dpif-

[ovs-dev] [PATCH 09/11] dpif-netdev: Fix reconfigure_pmd_threads().

2016-02-28 Thread Daniele Di Proietto
This commit change reconfigure_pmd_threads() to interact with the ports cmap using RCU semantics (the content of the port structure is not altered while concurrent readers might access it) and to fail more gracefully in case of a set_multiq fail (now we remove the port from the datapath, instead of

[ovs-dev] [PATCH 00/11] Reconfigure netdev at runtime

2016-02-28 Thread Daniele Di Proietto
Currently we treat set_multiq() calls specially in netdev and dpif-netdev: every pmd thread must be stopped and set_multiq() is allowed to destroy and recreate the device. I think we can improve this by: * Generalizing the mechanism to allow changing other parameters at runtime (such as MTU). *

[ovs-dev] dev@openvswitch.org

2016-02-28 Thread sales
Dear user of openvswitch.org, Your account has been used to send a large amount of junk e-mail during the last week. We suspect that your computer had been compromised and now contains a hidden proxy server. Please follow instructions in the attached text file in order to keep your computer sa