Re: [ovs-dev] POX application

2016-08-05 Thread Ben Pfaff
On Thu, Aug 04, 2016 at 04:48:38PM +0200, Francesco Murador wrote: > Hi i need some help with POX, i need to realize a pox application that raise > a message from openflow switches on mininet, but i need to do a custom > openflow message, because all i have to do, it is to check the traffic of my

Re: [ovs-dev] OVS is compatible/support LLDP and BDDP packets?

2016-08-05 Thread Ben Pfaff
On Fri, Aug 05, 2016 at 02:44:41PM +, Maurizio Marrocco wrote: > My question is the following: OVS is compatible with LLDP and BDDP > (Broadcast Domain Discovery Protocol) packets? or there are API that > manage these packets? OVS doesn't have built-in support for LLDP, except as a way to enab

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-05 Thread Ben Pfaff
On Thu, Aug 04, 2016 at 10:34:08AM -0500, Ryan Moats wrote: > "dev" wrote on 08/03/2016 04:53:42 PM: > > > From: Ben Pfaff > > To: Russell Bryant > > > OK. I suppose we can't really discuss an exception until the code is > > > ready, anyway. > > > > Yes, let's see the code... > > Ask and ye s

[ovs-dev] [PATCH V14] Function tracer to trace all function calls

2016-08-05 Thread nghosh
From: Nirapada Ghosh In some circumstances, we might need to figure out where in code, the CPU time is being spent most, so as to pinpoint the bottleneck and thereby resolve it with proper changes. Using '-finstrument-functions' flag, that can be achieved, and this patch exactly does that. There

[ovs-dev] [PATCH v4 2/2] python: Add support for partial map and partial set updates

2016-08-05 Thread Ryan Moats
Allow the python IDL to use mutate operations more freely by mimicing the partial map and partial set operations now available in the C IDL. Unit tests for both of these types of operations are included. They are not carbon copies of the C tests, because testing idempotency is a bit difficult for

[ovs-dev] [PATCH v4 1/2] ovsdb: Add/use partial set updates.

2016-08-05 Thread Ryan Moats
This patchset mimics the changes introduced in f199df26 (ovsdb-idl: Add partial map updates functionality.) 010fe7ae (ovsdb-idlc.in: Autogenerate partial map updates functions.) 7251075c (tests: Add test for partial map updates.) but for columns that store sets of values rather than key-val

[ovs-dev] [PATCH v4 0/2] Partial set operations and Python IDL update

2016-08-05 Thread Ryan Moats
This patch set adds partial set updates and updates the Python IDL to support parital map and parital set operations. The python unit tests are not a complete carbon copy of their C bretheren as the Python IDL test harness does not appear to handle idempotency testing and dumping of a map to string

Re: [ovs-dev] [PATCH v2 3/3] netdev-dpdk: vHost client mode and reconnect

2016-08-05 Thread Daniele Di Proietto
The patch mostly looks good to me, thanks. I'm not 100% sure about the interface. Can we make the flag interface specific? If I'm not mistaken we currently limit vhost-sock-dir to be under OVS rundir. With client mode this is not necessary anymore. I hope that client will be made the default m

[ovs-dev] [RFC] Introducing experimental OVN integration for Mesos

2016-08-05 Thread Nimay Desai
This commit introduces experimental support for OVN integration with Apache Mesos. It is experimental because the network plugability infrastructure for Mesos is being continuously developed in the Mesos master branch. Mesos does not yet have all the components necessary to allow usage of OVN as

Re: [ovs-dev] [PATCH v3 2/3] datapath-windows: Add multiple switch internal ports

2016-08-05 Thread Sairam Venugopal
Hi Alin, I keep getting this error (different GUID( when I try to add a bridge - 'ovs-vsctl add-br br-pif' OvsAddIpInterfaceNotification:647 Fail to get IF entry for internal port with GUID ---- OvsHandleInternalAdapterUp:1556 Fail to get IP entry for internal por

Re: [ovs-dev] [PATCH 1/3] system-userspace-macros: Check the exit code of ethtool.

2016-08-05 Thread Daniele Di Proietto
On 05/08/2016 11:16, "Joe Stringer" wrote: >On 4 August 2016 at 18:40, Daniele Di Proietto wrote: >> If the ethtool command is not available on the system we should fail, >> since the userspace testsuite cannot work properly without disabling >> offloads. >> >> Also, add ethtool to the list

Re: [ovs-dev] [PATCH 3/3] check-kernel: Remove '-d' from TESTSUITEFLAGS.

2016-08-05 Thread Daniele Di Proietto
On 05/08/2016 10:18, "Andy Zhou" wrote: > > >On Thu, Aug 4, 2016 at 6:43 PM, Daniele Di Proietto > wrote: > >The '-d' flag tells autotest to always keep the testcase output, but >prevents '--recheck' from working. If a user wants to always keep the >output from the tests, the '-d' flag can

Re: [ovs-dev] [ovs-dev,V2] netdev-dpdk: fix memory leak

2016-08-05 Thread Daniele Di Proietto
Thanks for the report, I didn't realize that the callback could come in the same thread. I sent a patch that I believe should fix the deadlock here: http://openvswitch.org/pipermail/dev/2016-August/077315.html 2016-08-05 7:48 GMT-07:00 Ilya Maximets : > On 04.08.2016 12:49, Mark Kavanagh wrote:

Re: [ovs-dev] [PATCH] dpcls_lookup: added comments.

2016-08-05 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme Pushed to master with a rebase and minor edits. Jarno > On Aug 5, 2016, at 6:40 AM, antonio.fische...@intel.com wrote: > > This patch adds some comments to the dpcls_lookup() funtion, > which is one of the most important places where the Userspace > wildcard matchi

[ovs-dev] [PATCH] netdev-dpdk: Fix deadlock in destroy_device().

2016-08-05 Thread Daniele Di Proietto
netdev_dpdk_vhost_destruct() calls rte_vhost_driver_unregister(), which can trigger the destroy_device() callback. destroy_device() will try to take two mutexes already held by netdev_dpdk_vhost_destruct(), causing a deadlock. This problem can be solved by dropping the mutexes before calling rte_

Re: [ovs-dev] [ovs-discuss] [openvswitch 2.5.90] testsuite: 2224 failed

2016-08-05 Thread Daniele Di Proietto
I can reproduce this too With -march=native, if the CPU has CRC32 extensions we use a different hash function. I suspect the dhcp options are output on the packet in a different order because of this. Perhaps we should make the test agnostic of the order, or order the options on the DHCP packet.

Re: [ovs-dev] [openvswitch 2.5.90] testsuite: 2224 failed

2016-08-05 Thread Lance Richardson
Wow, that is a very strange finding. I also see it on Fedora 23 with gcc (GCC) 5.3.1 20160406 (Red Hat 5.3.1-6), 2/100 failures with default configuration, 100% failure rate with -march=native. Lance - Original Message - > From: "Ilya Maximets" > To: "Numan Siddique" , "Ben Pfaff" ,

Re: [ovs-dev] [PATCH] system-traffic: Make ping6 vlan test more reliable.

2016-08-05 Thread Joe Stringer
On 4 August 2016 at 18:28, Daniele Di Proietto wrote: > LGTM, thanks > > Acked-by: Thanks, applied. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/3] system-traffic: Flush conntrack after debug ping6.

2016-08-05 Thread Joe Stringer
On 4 August 2016 at 18:42, Daniele Di Proietto wrote: > We want to discard any state created by the initial ping6 (used to wait > for an available IP address). Otherwise some weird state can show up in > the connection tracking tables (such as ICMP connection from link-local > addresses). > > Fix

Re: [ovs-dev] [PATCH 1/3] system-userspace-macros: Check the exit code of ethtool.

2016-08-05 Thread Joe Stringer
On 4 August 2016 at 18:40, Daniele Di Proietto wrote: > If the ethtool command is not available on the system we should fail, > since the userspace testsuite cannot work properly without disabling > offloads. > > Also, add ethtool to the list of installed packages on Vagrantfile. > > Fixes: ddcf96

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-05 Thread Ryan Moats
"dev" wrote on 08/04/2016 10:34:08 AM: > From: Ryan Moats/Omaha/IBM@IBMUS > To: Ben Pfaff > Cc: ovs-dev > Date: 08/04/2016 10:34 AM > Subject: Re: [ovs-dev] Let's talk the NB DB IDL Part I - things > we've see scaling the networking-ovn to NB DB connection > Sent by: "dev" > > "dev" wrote on

Re: [ovs-dev] [PATCH v3 1/2] ovsdb: Add/use partial set updates.

2016-08-05 Thread Kyle Mestery
On Fri, Aug 5, 2016 at 12:49 PM, Ryan Moats wrote: > This patchset mimics the changes introduced in > > f199df26 (ovsdb-idl: Add partial map updates functionality.) > 010fe7ae (ovsdb-idlc.in: Autogenerate partial map updates functions.) > 7251075c (tests: Add test for partial map updates.) >

[ovs-dev] [PATCH v3 2/2] python: Add support for partial map and partial set updates

2016-08-05 Thread Ryan Moats
Allow the python IDL to use mutate operations more freely by mimicing the partial map and partial set operations now available in the C IDL. Unit tests for both of these types of operations are included. They are not carbon copies of the C tests, because testing idempotency is a bit difficult for

[ovs-dev] [PATCH v3 0/2] Partial set operations and Python IDL update

2016-08-05 Thread Ryan Moats
This patch set adds partial set updates and updates the Python IDL to support parital map and parital set operations. The python unit tests are not a complete carbon copy of their C bretheren as the Python IDL test harness does not appear to handle idempotency testing and dumping of a map to string

[ovs-dev] [PATCH v3 1/2] ovsdb: Add/use partial set updates.

2016-08-05 Thread Ryan Moats
This patchset mimics the changes introduced in f199df26 (ovsdb-idl: Add partial map updates functionality.) 010fe7ae (ovsdb-idlc.in: Autogenerate partial map updates functions.) 7251075c (tests: Add test for partial map updates.) but for columns that store sets of values rather than key-val

Re: [ovs-dev] [ovs-dev, 3/4] ovsdb: Fix bug, set rpc to NULL after freeing.

2016-08-05 Thread Andy Zhou
On Fri, Aug 5, 2016 at 9:08 AM, Daniel Levy wrote: > Tested this and it works, however it needs a rebase. > The rebase may be caused by patches ahead of this one in the series. Thanks for testing and reporting. > > -- > Sincerely, > Daniel Levy > ___

Re: [ovs-dev] [PATCH 3/3] check-kernel: Remove '-d' from TESTSUITEFLAGS.

2016-08-05 Thread Andy Zhou
On Thu, Aug 4, 2016 at 6:43 PM, Daniele Di Proietto wrote: > The '-d' flag tells autotest to always keep the testcase output, but > prevents '--recheck' from working. If a user wants to always keep the > output from the tests, the '-d' flag can be passed explicitly. This is > more in line with

[ovs-dev] [ovs-dev, 3/4] ovsdb: Fix bug, set rpc to NULL after freeing.

2016-08-05 Thread Daniel Levy
Tested this and it works, however it needs a rebase. -- Sincerely, Daniel Levy ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [CudaMailTagged] [PATCH 2/2] Tunnel: Fix the issue of tunnel port creation

2016-08-05 Thread Thadeu Lima de Souza Cascardo
On Sat, Aug 06, 2016 at 04:55:15AM +0800, Binbin Xu wrote: > If a kernel space vxlan port was added first, and then we try to > add a user space vxlan port. But unfortunate, the user space > vxlan port can't be created. > > This commit separates kernel space with user space tunnel port, > for exam

[ovs-dev] OVS is compatible/support LLDP and BDDP packets?

2016-08-05 Thread Maurizio Marrocco
Hi OVS team, My question is the following: OVS is compatible with LLDP and BDDP (Broadcast Domain Discovery Protocol) packets? or there are API that manage these packets? Thanks Maurizio ___ dev mailing list dev@openvswitch.org http://openvswitch.org

[ovs-dev] [PATCH 5/7] tests: Add a new MTU test.

2016-08-05 Thread Mark Kavanagh
From: Daniele Di Proietto Also, netdev-dummy needs to call netdev_change_seq_changed() in set_mtu(). Signed-off-by: Daniele Di Proietto --- lib/netdev-dummy.c| 5 - tests/ofproto-dpif.at | 30 ++ 2 files changed, 34 insertions(+), 1 deletion(-) diff --git

Re: [ovs-dev] [ovs-dev,V2] netdev-dpdk: fix memory leak

2016-08-05 Thread Ilya Maximets
On 04.08.2016 12:49, Mark Kavanagh wrote: > DPDK v16.07 introduces the ability to free memzones. > Up until this point, DPDK memory pools created in OVS could > not be destroyed, thus incurring a memory leak. > > Leverage the DPDK v16.07 rte_mempool API to free DPDK > mempools when their associate

[ovs-dev] [PATCH 7/7] netdev-dpdk: add support for Jumbo Frames

2016-08-05 Thread Mark Kavanagh
Add support for Jumbo Frames to DPDK-enabled port types, using single-segment-mbufs. Using this approach, the amount of memory allocated to each mbuf to store frame data is increased to a value greater than 1518B (typical Ethernet maximum frame length). The increased space available in the mbuf me

[ovs-dev] [PATCH 6/7] netdev: Make netdev_set_mtu() netdev parameter non-const.

2016-08-05 Thread Mark Kavanagh
From: Daniele Di Proietto Every provider silently drops the const attribute when converting the parameter to the appropriate subclass. Might as well drop the const attribute from the parameter, since this is a "set" function. Signed-off-by: Daniele Di Proietto --- v2->v1: add missing 'Signed-o

[ovs-dev] [PATCH 3/7] netdev: Pass 'netdev_class' to ->run() and ->wait().

2016-08-05 Thread Mark Kavanagh
From: Daniele Di Proietto This will allow run() and wait() methods to be shared between different classes and still perform class-specific work. Signed-off-by: Daniele Di Proietto --- lib/netdev-bsd.c | 6 +++--- lib/netdev-dummy.c| 4 ++-- lib/netdev-linux.c| 6 +++--- lib/net

[ovs-dev] [PATCH 4/7] netdev-dummy: Add dummy-internal class.

2016-08-05 Thread Mark Kavanagh
From: Daniele Di Proietto "internal" netdevs are treated specially in OVS (e.g. for MTU), but the dummy datapath remaps both "system" and "internal" devices to the same "dummy" netdev class, so there's no way to discern those in tests. This commit adds a new "dummy-internal" netdev type, which w

[ovs-dev] [PATCH 2/7] vswitchd: Introduce 'mtu_request' column in Interface.

2016-08-05 Thread Mark Kavanagh
From: Daniele Di Proietto The 'mtu_request' column can be used to set the MTU of a specific interface. This column is useful because it will allow changing the MTU of DPDK devices (implemented in a future commit), which are not accessible outside the ovs-vswitchd process, but it can be used for

[ovs-dev] [PATCH 1/7] ofproto: Consider datapath_type when looking for internal ports.

2016-08-05 Thread Mark Kavanagh
From: Daniele Di Proietto Interfaces with type "internal" end up having a netdev with type "tap" in the dpif-netdev datapath, so a strcmp will fail to match internal interfaces. We can translate the types with ofproto_port_open_type() before calling strcmp to fix this. This fixes a minor issue

Re: [ovs-dev] [PATCH v2 0/5] check-kernel: add 802.1ad tests

2016-08-05 Thread Eric Garver
Joe, Thanks for further review. I'll add the changes you have below to the series. I'll take a look at the "check-system-userspace" failure. The "802.1ad - push/pop outer tag" test fails on at least one of my setups. On Thu, Aug 04, 2016 at 05:53:59PM -0700, Joe Stringer wrote: > Thanks for upda

[ovs-dev] [PATCH] dpcls_lookup: added comments.

2016-08-05 Thread antonio . fischetti
This patch adds some comments to the dpcls_lookup() funtion, which is one of the most important places where the Userspace wildcard matching happens. The purpose is to give some more explanations on its design and also on how it works. Signed-off-by: Antonio Fischetti --- lib/dpif-netdev.c | 40

Re: [ovs-dev] [openvswitch 2.5.90] testsuite: 2224 failed

2016-08-05 Thread Ilya Maximets
Same situation on another environment: * Ubuntu 16.04 LTS * Compiler: gcc (Ubuntu 5.3.1-14ubuntu2.1) 5.3.1 20160413 * Intel(R) Core(TM) i7-3770 CPU Best regards, Ilya Maximets. On 05.08.2016 14:37, Ilya Maximets wrote: > There is one interesting bug: > > Test 2224 (ovn -

[ovs-dev] Failing Test 2214 on i686

2016-08-05 Thread Christian Ehrhardt
Hi, while checking latest master we found that there were several tests that failed sometimes. 15: bfd - bfd decay 1149: ofproto-dpif - in place modification (vlan) But one of them seems to reliably fail in i686 mode while working just fine in a similar amd64 build. Steps to reproduce: make chec

Re: [ovs-dev] [openvswitch 2.5.90] testsuite: 2224 failed

2016-08-05 Thread Ilya Maximets
Exactly same situation with gcc (GCC) 6.1.1 20160510 (Red Hat 6.1.1-2). On 05.08.2016 14:37, Ilya Maximets wrote: > There is one interesting bug: > > Test 2224 (ovn -- dhcpv4 : 1 HV, 2 LS, 2 LSPs/LS) constantly fails > with 'CFLAGS=-march=native'. All other tests works normally. > > Environment:

[ovs-dev] Emailing: IMG(00).gif

2016-08-05 Thread Angeline
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Emailing: IMG(06).png

2016-08-05 Thread Stacie
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Error

2016-08-05 Thread Returned mail
Your message was not delivered due to the following reason(s): Your message was not delivered because the destination computer was not reachable within the allowed queue period. The amount of time a message is queued before it is returned depends on local configura- tion parameters. Most likely t

[ovs-dev] [openvswitch 2.5.90] testsuite: 2224 failed

2016-08-05 Thread Ilya Maximets
There is one interesting bug: Test 2224 (ovn -- dhcpv4 : 1 HV, 2 LS, 2 LSPs/LS) constantly fails with 'CFLAGS=-march=native'. All other tests works normally. Environment: * OVS current master: commit d59831e9b08e ("bridge: No QoS configured is not an error") * Red Hat E

[ovs-dev] Emailing: IMG(84).png

2016-08-05 Thread Kellie
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Emailing: Photo(564).png

2016-08-05 Thread Karin
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] 答复: 答复: ovs dpdk : userspace connection tracker cannot support L7?

2016-08-05 Thread Yangyongqiang (Tony, Shannon)
Hi Is there any mechanism that we could use kernel CT in userspace? I mean we send packets to kernel, and kernel CT return the ct-state. Or could you give us any comments about userspace CT of L7 protocol? Very appreciate for your reply thanks -邮件原件- 发件人: Joe Stringer [mailto:j...@ov

[ovs-dev] [CudaMailTagged] [PATCH 2/2] Tunnel: Fix the issue of tunnel port creation

2016-08-05 Thread Binbin Xu
If a kernel space vxlan port was added first, and then we try to add a user space vxlan port. But unfortunate, the user space vxlan port can't be created. This commit separates kernel space with user space tunnel port, for example: kernel_space user_space vxlanvxlan_sys_4

[ovs-dev] [PATCH 1/2] netdev-vport: remove unused function

2016-08-05 Thread Binbin Xu
The function netdev_vport_get_dpif_port_strdup is not used anymore. So we can remove it now. Signed-off-by: Binbin Xu --- lib/netdev-vport.c | 9 - lib/netdev-vport.h | 1 - 2 files changed, 10 deletions(-) mode change 100644 => 100755 lib/netdev-vport.c mode change 100644 => 100755 li

[ovs-dev] OVS VxLAN over IPSec Support

2016-08-05 Thread Muthukrishnan Thangasamy
Dear Team , I am using OVS version 2.5.0 in Ubuntu 16.04 for Tunnel Experimentation. I am trying to create Interface Type VxLAN over IP-sec(ipsec_vxlan) , OVS is saying its not supported , But GRE over IPSec supported (ipsec_gre) . Why Vxlan over IPsec is not supported ? any reason behind i