Re: [ovs-dev] [patch_v3 1/2] ovn: Fix receive from vxlan inovn-controller.

2016-07-06 Thread Darrell Ball
On Tue, Jul 5, 2016 at 7:39 PM, Ryan Moats wrote: > "dev" wrote on 06/26/2016 08:34:01 PM: > > > From: Darrell Ball > > To: dlu...@gmail.com, d...@openvswitch.com > > Date: 06/26/2016 08:34 PM > > Subject: [ovs-dev] [patch_v3

[ovs-dev] [patch_v4 2/2] ovn: Reformat some ovn design documentation.

2016-07-06 Thread Darrell Ball
Some design micro-details (e.g.) register assignments) that may change over time were moved from the ovn-architecture.7.xml document to the OVN-DESIGN.md document. A table is added to summarize and quantify MFF logical metadata and register usage. The OVN-DESIGN.md file was tested using the

[ovs-dev] [patch_v4 1/2] ovn: Fix receive from vxlan in ovn-controller.

2016-07-06 Thread Darrell Ball
OVN only supports source_node replication and previously vtep interaction, which used service node replication by default for multicast/broadcast/unknown unicast traffic worked by happenstance. Because of limited vxlan encapsulation metadata, received packets were resubmitted to find the egress

[ovs-dev] [patch_v4 0/2] ovn: Fix receive from vxlan in ovn-controller.

2016-07-06 Thread Darrell Ball
OVN only supports source_node replication and previously vtep interaction, which used service node replication by default for multicast/broadcast/unknown unicast traffic worked by happenstance.  Because of limited vxlan encapsulation metadata, received packets were resubmitted to find the egress

[ovs-dev] Delivery reports about your e-mail

2016-07-06 Thread MAILER-DAEMON
The message was undeliverable due to the following reason(s): Your message was not delivered because the destination server was unreachable 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 there

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Support user-defined socket attribs

2016-07-06 Thread Aaron Conole
Daniele Di Proietto writes: > On 06/07/2016 11:26, "Aaron Conole" wrote: > >>Ansis Atteka writes: >> >>> On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote: Aaron Conole writes:

Re: [ovs-dev] [PATCH v3 04/28] datapath: backport: net: add dst_cache support

2016-07-06 Thread pravin shelar
On Wed, Jul 6, 2016 at 4:34 PM, Jesse Gross wrote: > On Wed, Jul 6, 2016 at 3:44 PM, pravin shelar wrote: >> On Tue, Jul 5, 2016 at 6:54 PM, Jesse Gross wrote: >>> On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote:

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Support user-defined socket attribs

2016-07-06 Thread Daniele Di Proietto
On 06/07/2016 11:26, "Aaron Conole" wrote: >Ansis Atteka writes: > >> On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote: >>> Aaron Conole writes: >>> Daniele Di Proietto writes:

[ovs-dev] [patch_v1] ovn: Remove unreferenced patched datapaths

2016-07-06 Thread Darrell Ball
Patched datapaths that are no longer referenced should be removed from the patched_datapaths map; otherwise incorrect state references for a patched datapath may be used and also datapaths that are absent will be interpreted as present. Signed-off-by: Darrell Ball ---

Re: [ovs-dev] [PATCH] Makefile.am: Add INSTALL.DPDK-ADVANCED.md to EXTRA_DIST.

2016-07-06 Thread Daniele Di Proietto
On 06/07/2016 17:28, "Aaron Conole" wrote: >Daniele Di Proietto writes: > >> Fixes: c9b9d6dfc06c("INSTALL.DPDK: Refactor DPDK install guide, add >> ADVANCED doc") >> >> Signed-off-by: Daniele Di Proietto >> --- >>

Re: [ovs-dev] [PATCH] netdev-linux: Do not log a warning if the device is down.

2016-07-06 Thread Daniele Di Proietto
I took your version of the code (much clearer!) and pushed this to master. Thanks, Daniele On 24/06/2016 21:38, "Ben Pfaff" wrote: >On Fri, Jun 10, 2016 at 03:52:16PM -0700, Daniele Di Proietto wrote: >> In the userspace datapath we use tap devices as internal netdev. The >>

Re: [ovs-dev] [RFC 3/3] Red Hat Systemd Integration

2016-07-06 Thread Aaron Conole
Flavio Leitner writes: > On Wed, Jul 06, 2016 at 04:28:26PM -0400, Aaron Conole wrote: >> Flavio Leitner writes: >> >> > On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote: >> >> +ConditionPathIsReadWrite=/var/run/openvswitch/db.sock >> > >> >

Re: [ovs-dev] [PATCH] Makefile.am: Add INSTALL.DPDK-ADVANCED.md to EXTRA_DIST.

2016-07-06 Thread Aaron Conole
Daniele Di Proietto writes: > Fixes: c9b9d6dfc06c("INSTALL.DPDK: Refactor DPDK install guide, add > ADVANCED doc") > > Signed-off-by: Daniele Di Proietto > --- > Makefile.am | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Makefile.am

Re: [ovs-dev] [PATCH RFC v2 0/6] dpif-netdev: Manual pinning of RX queues + XPS.

2016-07-06 Thread Daniele Di Proietto
Hi Ilya, apologies for the delay On 20/06/2016 07:22, "Ilya Maximets" wrote: >On 11.06.2016 02:53, Daniele Di Proietto wrote: >> On 02/06/2016 06:55, "Ilya Maximets" wrote: >> >>> Hi, Daniele. >>> Thanks for review. >>> >>> On 02.06.2016

Re: [ovs-dev] [PATCH] vlog.py: Remove redundant setLevel() if "/dev/log" doesn't exist

2016-07-06 Thread Daniele Di Proietto
On 06/07/2016 16:49, "Guru Shetty" wrote: > > >On 6 July 2016 at 16:41, Daniele Di Proietto > wrote: > >Also update a comment. > >Signed-off-by: Daniele Di Proietto > > >Acked-by: Gurucharan > Shetty Pushed to

[ovs-dev] Error

2016-07-06 Thread The Post Office
The original message was received at Thu, 7 Jul 2016 06:57:05 +0700 from [161.112.192.13] - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org

Re: [ovs-dev] [PATCH] vlog.py: Remove redundant setLevel() if "/dev/log" doesn't exist

2016-07-06 Thread Guru Shetty
On 6 July 2016 at 16:41, Daniele Di Proietto wrote: > Also update a comment. > > Signed-off-by: Daniele Di Proietto > Acked-by: Gurucharan Shetty > --- > python/ovs/vlog.py | 5 ++--- > 1 file changed, 2 insertions(+), 3

Re: [ovs-dev] [PATCH V6 17/17] vlog test: Disable default syslog logger

2016-07-06 Thread Daniele Di Proietto
2016-07-06 15:32 GMT-07:00 Guru Shetty : > On 6 July 2016 at 05:38, Paul Boca wrote: > > > Disable the syslog logger in case on Windows, '/dev/log' doesn't exist. > > Seems like on Python34 a default handler is added to the logger and it > > prints > >

[ovs-dev] [PATCH] vlog.py: Remove redundant setLevel() if "/dev/log" doesn't exist

2016-07-06 Thread Daniele Di Proietto
Also update a comment. Signed-off-by: Daniele Di Proietto --- python/ovs/vlog.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/python/ovs/vlog.py b/python/ovs/vlog.py index 49210b8..48d52ad 100644 --- a/python/ovs/vlog.py +++

[ovs-dev] [PATCH] netdev-dummy: fix crash with more than one passive connection

2016-07-06 Thread Lance Richardson
Investigation found that Some of the occasional failures in the "ovn -- vtep: 3 HVs, 1 VIFs/HV, 1 GW, 1 LS" test case are caused by ovs-vswitchd crashing with SIGSEGV. It turns out that the crash occurrs when the number of netdev-dummy passive connections transitions from 1 to 2. When xrealloc()

Re: [ovs-dev] [PATCH v3 04/28] datapath: backport: net: add dst_cache support

2016-07-06 Thread Jesse Gross
On Wed, Jul 6, 2016 at 3:44 PM, pravin shelar wrote: > On Tue, Jul 5, 2016 at 6:54 PM, Jesse Gross wrote: >> On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote: >>> diff --git a/acinclude.m4 b/acinclude.m4 >>> index 263c31d..05b5f48 100644

Re: [ovs-dev] [RFC 3/3] Red Hat Systemd Integration

2016-07-06 Thread Flavio Leitner
On Wed, Jul 06, 2016 at 04:28:26PM -0400, Aaron Conole wrote: > Flavio Leitner writes: > > > On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote: > >> +ConditionPathIsReadWrite=/var/run/openvswitch/db.sock > > > > Interesting, so we start ovs-vswitchd only when

[ovs-dev] [PATCH] Makefile.am: Add INSTALL.DPDK-ADVANCED.md to EXTRA_DIST.

2016-07-06 Thread Daniele Di Proietto
Fixes: c9b9d6dfc06c("INSTALL.DPDK: Refactor DPDK install guide, add ADVANCED doc") Signed-off-by: Daniele Di Proietto --- Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/Makefile.am b/Makefile.am index b2c1126..be42921 100644 --- a/Makefile.am +++

Re: [ovs-dev] [PATCH v3 1/2] netdev-dpdk: Use instant sending instead of queueing of packets.

2016-07-06 Thread Daniele Di Proietto
I applied this to master, thanks On 27/06/2016 06:28, "Ilya Maximets" wrote: >Current implementarion of TX packet's queueing is broken in several ways: > > * TX queue flushing implemented on receive assumes that all > core_id-s are sequential and starts

Re: [ovs-dev] [PATCH v3 18/28] datapath: compat: Use dst-cache for Geneve and VxLAN tunnels.

2016-07-06 Thread pravin shelar
On Tue, Jul 5, 2016 at 7:57 PM, Jesse Gross wrote: > On Fri, Jul 1, 2016 at 5:59 PM, Pravin B Shelar wrote: >> diff --git a/datapath/linux/compat/geneve.c b/datapath/linux/compat/geneve.c >> index 2ed9ad1..dbb220f 100644 >> --- a/datapath/linux/compat/geneve.c

Re: [ovs-dev] [PATCH v1] INSTALL.DPDK: Update vhost multiqueue instructions.

2016-07-06 Thread Daniele Di Proietto
I applied this to branch-2.5, thanks 2016-07-04 1:10 GMT-07:00 Bodireddy, Bhanuprakash < bhanuprakash.bodire...@intel.com>: > >-Original Message- > >From: Stokes, Ian > >Sent: Wednesday, June 29, 2016 2:32 PM > >To: dev@openvswitch.org > >Cc: Bodireddy, Bhanuprakash

Re: [ovs-dev] [PATCH v3 04/28] datapath: backport: net: add dst_cache support

2016-07-06 Thread pravin shelar
On Tue, Jul 5, 2016 at 6:54 PM, Jesse Gross wrote: > On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote: >> diff --git a/acinclude.m4 b/acinclude.m4 >> index 263c31d..05b5f48 100644 >> --- a/acinclude.m4 >> +++ b/acinclude.m4 >> @@ -556,6 +556,7 @@

Re: [ovs-dev] [PATCH v3 13/28] datapath: backport: ip_tunnel: Move stats update to iptunnel_xmit()

2016-07-06 Thread pravin shelar
On Tue, Jul 5, 2016 at 6:07 PM, Jesse Gross wrote: > On Fri, Jul 1, 2016 at 5:59 PM, Pravin B Shelar wrote: >> diff --git a/datapath/linux/compat/include/net/ip6_tunnel.h >> b/datapath/linux/compat/include/net/ip6_tunnel.h >> index ce65087..565b69b 100644 >>

Re: [ovs-dev] [PATCH V6 17/17] vlog test: Disable default syslog logger

2016-07-06 Thread Guru Shetty
On 6 July 2016 at 05:38, Paul Boca wrote: > Disable the syslog logger in case on Windows, '/dev/log' doesn't exist. > Seems like on Python34 a default handler is added to the logger and it > prints > even if no handler is set by us. > > Signed-off-by: Paul-Daniel

Re: [ovs-dev] [PATCH v9 0/2] INSTALL.DPDK: Refactor DPDK install documentation

2016-07-06 Thread Daniele Di Proietto
I applied the series to master, thanks 2016-07-06 6:06 GMT-07:00 Bhanuprakash Bodireddy < bhanuprakash.bodire...@intel.com>: > This patchset refactors the present INSTALL.DPDK.md guide. > > The INSTALL guide is split in to two documents named INSTALL.DPDK and > INSTALL.DPDK-ADVANCED. The former

[ovs-dev] Welcoming speech

2016-07-06 Thread dev
Compliments I am the personnel department manager and I am appealing to you in the name of the large-scale and first-rate partnership. Our company is engaged in different areas of activity, such as: - real estate - companies setting-up and winding-up - supporting business in Europe and

Re: [ovs-dev] [PATCH V6 17/17] vlog test: Disable default syslog logger

2016-07-06 Thread Daniele Di Proietto
I'm not a windows expert, but this also solves a problem I was having on a linux system chroot where /dev/log is absent. I pushed this patch to master, thanks, Daniele 2016-07-06 5:38 GMT-07:00 Paul Boca : > Disable the syslog logger in case on Windows, '/dev/log'

Re: [ovs-dev] [PATCH v3 10/28] datapath: compat: Update udp_sock_create

2016-07-06 Thread pravin shelar
On Tue, Jul 5, 2016 at 4:54 PM, Jesse Gross wrote: > On Fri, Jul 1, 2016 at 5:59 PM, Pravin B Shelar wrote: >> diff --git a/datapath/linux/compat/include/linux/udp.h >> b/datapath/linux/compat/include/linux/udp.h >> new file mode 100644 >> index

Re: [ovs-dev] [PATCH v3 05/28] datapath: backport: ip_tunnel_core: iptunnel_handle_offloads returns int and doesn't free skb

2016-07-06 Thread pravin shelar
On Tue, Jul 5, 2016 at 4:40 PM, Jesse Gross wrote: > On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote: >> diff --git a/datapath/linux/compat/geneve.c b/datapath/linux/compat/geneve.c >> index db1c713..a7229c8 100644 >> --- a/datapath/linux/compat/geneve.c

Re: [ovs-dev] [RFC 2/3] rhel/ovsdb-server.service: Rename the nonetwork service

2016-07-06 Thread Flavio Leitner
On Wed, Jul 06, 2016 at 02:40:01PM -0400, Aaron Conole wrote: > Flavio Leitner writes: > > > On Fri, Jul 01, 2016 at 05:26:22PM -0400, Aaron Conole wrote: > >> diff --git a/rhel/etc_sysconfig_network-scripts_ifup-ovs > >> b/rhel/etc_sysconfig_network-scripts_ifup-ovs > >>

Re: [ovs-dev] [PATCH V6 10/17] python tests: Fixed abs_file_name function for Windows

2016-07-06 Thread Alin Serdean
Just add a comment in the code so people know. Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Wednesday, July 6, 2016 3:38 PM > Către: dev@openvswitch.org > Subiect:

Re: [ovs-dev] [RFC 3/3] Red Hat Systemd Integration

2016-07-06 Thread Aaron Conole
Flavio Leitner writes: > On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote: >> This commit builds upon some of the recent ovs-ctl changes to build a >> more integrated systemd setup. A new service (openvswitch-network) is > > I think you renamed to 'ovs-vswitchd'

Re: [ovs-dev] [RFC 3/3] Red Hat Systemd Integration

2016-07-06 Thread Flavio Leitner
On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote: > This commit builds upon some of the recent ovs-ctl changes to build a > more integrated systemd setup. A new service (openvswitch-network) is I think you renamed to 'ovs-vswitchd' after we talked offline :-) > added to track the

Re: [ovs-dev] [PATCH V6 08/17] python tests: Skip python tests on Windows where POSIX pid is used

2016-07-06 Thread Alin Serdean
It's ok with me to skip the tests for the moment, but please add a comment why are you skipping them i.e. "echo $! gives shell pid instead of parent process" Otherwise: Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: dev

Re: [ovs-dev] [RFC 1/3] utilities/ovs-ctl.in: Allow non-monitoring daemons

2016-07-06 Thread Aaron Conole
Ben Pfaff writes: > On Fri, Jul 01, 2016 at 05:26:21PM -0400, Aaron Conole wrote: >> This commit allows the ovs-ctl command to spawn daemons without the >> internal process monitor. This is useful when integrating with, >> ex. systemd, which provides its own monitoring facilities.

Re: [ovs-dev] [RFC 1/3] utilities/ovs-ctl.in: Allow non-monitoring daemons

2016-07-06 Thread Aaron Conole
Flavio Leitner writes: > On Fri, Jul 01, 2016 at 09:32:24PM -0700, Ben Pfaff wrote: >> On Fri, Jul 01, 2016 at 05:26:21PM -0400, Aaron Conole wrote: >> > This commit allows the ovs-ctl command to spawn daemons without the >> > internal process monitor. This is useful when

Re: [ovs-dev] [RFC 2/3] rhel/ovsdb-server.service: Rename the nonetwork service

2016-07-06 Thread Aaron Conole
Flavio Leitner writes: > On Fri, Jul 01, 2016 at 05:26:22PM -0400, Aaron Conole wrote: >> Currently, openvswitch.service calls out to start >> openvswitch-nonetwork.service. However, openvswitch-nonetwork.service >> is better called ovsdb-server, since that is truly

Re: [ovs-dev] [PATCH V6 14/17] python tests: Ignore stderr output

2016-07-06 Thread Alin Serdean
+ AT_CHECK([$3 $srcdir/test-unixctl.py --log-file=`pwd`/log --pidfile \ + --detach 2>/dev/null], [0], [], [ignore]) Either ignore stderr or 2>/dev/null. In case [ignore] doesn't work please add a comment so others will be aware of the problem Alin. > -Mesaj original- > De la: dev

Re: [ovs-dev] [PATCH V6 07/17] python tests: Skip TCP6 idl tests on Windows

2016-07-06 Thread Alin Serdean
We discussed this issue offline root cause seems to be: https://bugs.python.org/issue6926 https://github.com/openvswitch/ovs/blob/master/python/ovs/socket_util.py#L309 Needs a commit message update. Otherwise: Acked-by: Alin Gabriel Serdean > -Mesaj

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Support user-defined socket attribs

2016-07-06 Thread Aaron Conole
Ansis Atteka writes: > On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote: >> Aaron Conole writes: >> >>> Daniele Di Proietto writes: >>> On 10/06/2016 10:51, "Aaron Conole" wrote:

[ovs-dev] [PATCH net-next v11 5/6] openvswitch: add layer 3 flow/port support

2016-07-06 Thread Simon Horman
From: Lorand Jakab Implementation of the pop_eth and push_eth actions in the kernel, and layer 3 flow support. This doesn't actually do anything yet as no layer 2 tunnel ports are supported yet. The original patch by Lorand was against the Open vSwitch tree which has L2 LISP

[ovs-dev] [PATCH net-next v11 3/6] openvswitch: set skb protocol and mac_len when receiving on internal device

2016-07-06 Thread Simon Horman
* Set skb protocol based on contents of packet. I have observed this is necessary to get actual protocol of a packet when it is injected into an internal device e.g. by libnet in which case skb protocol will be set to ETH_ALL. * Set the mac_len which has been observed to not be set up

[ovs-dev] [PATCH net-next v11 4/6] openvswitch: add support to push and pop mpls for layer3 packets

2016-07-06 Thread Simon Horman
Allow push and pop mpls actions to act on layer 3 packets by teaching them not to access non-existent L2 headers of such packets. Signed-off-by: Simon Horman --- v11 * group l2 code in pop_mpls() v10 * Limit scope of hdr in {push,pop}_mpls() v9 * New Patch ---

[ovs-dev] [PATCH net-next v11 1/6] net: introduce skb_transport_header_was_set()

2016-07-06 Thread Simon Horman
This helper resets the mac_header of an skb to a state where skb_transport_header_was_set() will return false. This is intended to be used with packets received on ARPHRD_NONE devices without an Ethernet header in the inner packet. It allows skb_transport_header_was_set to be subsequently used to

[ovs-dev] [PATCH net-next v11 0/6] openvswitch: support for layer 3 encapsulated packets

2016-07-06 Thread Simon Horman
At the core of this patch set is removing the assumption in Open vSwitch datapath that all packets have Ethernet header. Support for layer 3 GRE tunnels is also added by this patchset. The implementation relies on the presence of pop_eth and push_eth actions in datapath flows to facilitate adding

[ovs-dev] [PATCH net-next v11 2/6] gre: unset mac header for non-TEB packets received by ipgre device

2016-07-06 Thread Simon Horman
unset rather than reset mach header for non-TEB packets received by an ipgre device. This allows skb_transport_header_was_set to be subsequently used to differentiate TEB and non-TEB packets recieved on an ipgre device. Signed-off-by: Simon Horman --- v11 * New patch

[ovs-dev] [PATCH v12 1/3] userspace: add support for pop_eth and push_eth actions

2016-07-06 Thread Simon Horman
From: Lorand Jakab These actions will allow L2->L3 and L3->L2 switching, and are supposed to be added to flows installed in the datapath transparently by ovs-vswitchd. Signed-off-by: Lorand Jakab Signed-off-by: Simon Horman

[ovs-dev] [PATCH v12 3/3] userspace: add non-tap (l3) support to GRE vports

2016-07-06 Thread Simon Horman
Add support for layer 3 GRE vports (non-tap aka non-VTEP). This makes use of a vport mode configuration for the existing (tap/VTEP) GRE vports. In order to differentiate packets for two different types of GRE vports a new flow key attribute, OVS_KEY_ATTR_NEXT_BASE_LAYER, is used. It is intended

[ovs-dev] [PATCH v12 2/3] userspace: add layer 3 flow and switching support

2016-07-06 Thread Simon Horman
From: Lorand Jakab This commit relaxes the assumption that all packets have an Ethernet header, and adds support for layer 3 flows. For each packet received on the Linux kernel datapath the l2 and l3 members of struct ofpbuf are intialized appropriately, and some functions

[ovs-dev] [PATCH v12 0/3] userspace: Support for layer 3 encapsulated packets

2016-07-06 Thread Simon Horman
This series implements support for layer 3 encapsulated packets. At the core of this change is removing the assumption that all packets/flows have Ethernet header. Support for layer 3 packets in GRE tunnels is also added by this patchset. The implementation automatically adds appropriate pop_eth

Re: [ovs-dev] [RFC 2/3] rhel/ovsdb-server.service: Rename the nonetwork service

2016-07-06 Thread Flavio Leitner
On Fri, Jul 01, 2016 at 05:26:22PM -0400, Aaron Conole wrote: > Currently, openvswitch.service calls out to start > openvswitch-nonetwork.service. However, openvswitch-nonetwork.service > is better called ovsdb-server, since that is truly nonetwork. This > commit does make the file a bit of a

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Support user-defined socket attribs

2016-07-06 Thread Ansis Atteka
On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote: > Aaron Conole writes: > >> Daniele Di Proietto writes: >> >>> On 10/06/2016 10:51, "Aaron Conole" wrote: >>> Aaron Conole writes:

Re: [ovs-dev] [PATCH V6 05/17] python tests: Added fcntl module for Windows

2016-07-06 Thread Paul Boca
> -Original Message- > From: Alin Serdean > Sent: Wednesday, July 6, 2016 8:42 PM > To: Paul Boca; dev@openvswitch.org > Subject: RE: [ovs-dev] [PATCH V6 05/17] python tests: Added fcntl module for > Windows > > > -Mesaj original- > > De la: dev

Re: [ovs-dev] [PATCH V6 05/17] python tests: Added fcntl module for Windows

2016-07-06 Thread Alin Serdean
> -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Wednesday, July 6, 2016 3:38 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH V6 05/17] python tests: Added fcntl module for > Windows > > This is needed for lockf function used

Re: [ovs-dev] [RFC 1/3] utilities/ovs-ctl.in: Allow non-monitoring daemons

2016-07-06 Thread Flavio Leitner
On Fri, Jul 01, 2016 at 09:32:24PM -0700, Ben Pfaff wrote: > On Fri, Jul 01, 2016 at 05:26:21PM -0400, Aaron Conole wrote: > > This commit allows the ovs-ctl command to spawn daemons without the > > internal process monitor. This is useful when integrating with, > > ex. systemd, which provides

Re: [ovs-dev] [PATCH V6 03/17] python tests: Fixed ctl file name for Windows

2016-07-06 Thread Paul Boca
Hi Alin! Comments inlined. Thanks, Paul > -Original Message- > From: Alin Serdean > Sent: Wednesday, July 6, 2016 8:18 PM > To: Paul Boca; dev@openvswitch.org > Subject: RE: [ovs-dev] [PATCH V6 03/17] python tests: Fixed ctl file name for > Windows > > > -Mesaj original- > > De

Re: [ovs-dev] [PATCH V6 04/17] python tests: Fixed unixctl python tests for Windows

2016-07-06 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Wednesday, July 6, 2016 3:38 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH V6 04/17] python tests: Fixed

Re: [ovs-dev] [PATCH V6 03/17] python tests: Fixed ctl file name for Windows

2016-07-06 Thread Alin Serdean
> -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Wednesday, July 6, 2016 3:38 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH V6 03/17] python tests: Fixed ctl file name for > Windows > > On Windows the CTL file doesn't contain

Re: [ovs-dev] [PATCH V6 02/17] python tests: Register signal handlers only on supported types on Windows

2016-07-06 Thread Alin Serdean
The code is sound but looks a bit messy. What do you think about? @@ -15,6 +15,7 @@ import atexit import os import signal +import sys import ovs.vlog @@ -129,8 +130,12 @@ def _init(): if not _inited: _inited = True -for signr in (signal.SIGTERM, signal.SIGINT, -

Re: [ovs-dev] [PATCH] bridge: open_type should be used for netdev_open

2016-07-06 Thread Jesse Gross
On Mon, Jul 4, 2016 at 11:19 AM, Thadeu Lima de Souza Cascardo wrote: > ofproto_port_open_type should be used for netdev_open, but not for other > tests. > For example, STP/RSTP check for interfaces of internal type, but that check > will > fail when the netdev datapath is

Re: [ovs-dev] [PATCH V6 15/17] python tests: Skip ovn-controller-vtep tests on Windows

2016-07-06 Thread Paul Boca
Hi Alin! Please see my comment inline. Thanks, Paul > -Original Message- > From: Alin Serdean > Sent: Wednesday, July 6, 2016 6:08 PM > To: Paul Boca; dev@openvswitch.org > Subject: RE: [PATCH V6 15/17] python tests: Skip ovn-controller-vtep tests on > Windows > > Just out of curiosity

Re: [ovs-dev] [PATCH V6 15/17] python tests: Skip ovn-controller-vtep tests on Windows

2016-07-06 Thread Alin Serdean
Just out of curiosity why is this test is skipped on Windows? Since this patch was skipped thus far I will acknowledge the patch and we can fix the test if we can in another. Acked-by: Alin Gabriel Serdean Thanks, Alin. > -Mesaj original- > De la: dev

[ovs-dev] 849529F5898CFFEE

2016-07-06 Thread dev
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] Bridge IPFIX statistics check question

2016-07-06 Thread Daniel Ye
Hi Paul, Thanks for your information. I will fix it. Bests, Daniel On Jul 6, 2016, at 8:37 PM, Paul Boca > wrote: Hi Daniel, Thanks for the fast reply! I thought that it is a problem in Windows implementation or some sort of

Re: [ovs-dev] ovs+dpdk hang? (was: Re: c)

2016-07-06 Thread Charlie Li
Hi Mark, I observed the same issue using Ubuntu 16.04 server with 4.4.0-28-generic kernel. Thanks for looking into this. Charlie On Tue, Jul 5, 2016 at 5:04 PM, Kavanagh, Mark B wrote: > Hi Charlie, > > > > Thanks for providing the setup scripts and steps to

[ovs-dev] 403DF4C2CF02B4B7

2016-07-06 Thread dev
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] CV

2016-07-06 Thread dev
Speech of welcome I am the personnel department manager and I am appealing to you in the name of the large-scale and first-rate partnership. Our company is met in many departments, such as: - real estate - companies setting-up and winding-up - supporting business in Europe and other

[ovs-dev] B83FC9997B83625F

2016-07-06 Thread dev
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Support user-defined socket attribs

2016-07-06 Thread Aaron Conole
Aaron Conole writes: > Daniele Di Proietto writes: > >> On 10/06/2016 10:51, "Aaron Conole" wrote: >> >>>Aaron Conole writes: >>> Christian Ehrhardt writes: >

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

2016-07-06 Thread linux-uvc-devel
The original message was received at Wed, 6 Jul 2016 22:16:07 +0800 from lists.berlios.de [72.135.101.196] - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org

Re: [ovs-dev] [PATCH v2] ovsdb: Expose vhost-user socket directory in ovsdb

2016-07-06 Thread Aaron Conole
"Wojciechowicz, RobertX" writes: > Hi Ben, > > >> -Original Message- >> From: Ben Pfaff [mailto:b...@ovn.org] >> Sent: Tuesday, July 5, 2016 5:07 PM >> To: Wojciechowicz, RobertX >> Cc: dev@openvswitch.org >> Subject: Re:

Re: [ovs-dev] [PATCH V6 16/17] tests: Skip vlog tests that try to move opened file

2016-07-06 Thread Alin Serdean
As a personal preference I think the comment: > + # This test won't work as-is on Windows because Windows doesn't allow > + # files that are open to be renamed. is sufficient. Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: dev

Re: [ovs-dev] [PATCH v8 1/2] INSTALL.DPDK: Refactor DPDK install documentation

2016-07-06 Thread Bodireddy, Bhanuprakash
>-   With SMT enabled, one physical core appears as two logical cores >-   which can improve performance. >+  - OVS current development can be clone using 'git' tool > >-   SMT can be utilized to add additional pmd threads without consuming >-   additional physical cores. Additional pmd threads

Re: [ovs-dev] [PATCH v8 2/2] INSTALL.DPDK: Refactor DPDK install guide, add ADVANCED doc

2016-07-06 Thread Bodireddy, Bhanuprakash
>-Original Message- >From: Daniele Di Proietto [mailto:diproiet...@ovn.org] >Sent: Wednesday, July 6, 2016 7:05 AM >To: Bodireddy, Bhanuprakash >Cc: Lance Richardson ; dev@openvswitch.org >Subject: Re: [ovs-dev] [PATCH v8 2/2]

[ovs-dev] [PATCH v9 2/2] INSTALL.DPDK: Refactor DPDK install guide, add ADVANCED doc

2016-07-06 Thread Bhanuprakash Bodireddy
Add INSTALL.DPDK-ADVANCED document that is forked off from original INSTALL.DPDK guide. This document is targeted at users looking for optimum performance on OVS using dpdk datapath. Signed-off-by: Bhanuprakash Bodireddy Acked-by: Flavio Leitner

[ovs-dev] [PATCH v9 1/2] INSTALL.DPDK: Refactor DPDK install documentation

2016-07-06 Thread Bhanuprakash Bodireddy
Refactor the INSTALL.DPDK in to two documents named INSTALL.DPDK and INSTALL.DPDK-ADVANCED. While INSTALL.DPDK document shall facilitate the novice user in setting up the OVS DPDK and running it out of box, the ADVANCED document is targeted at expert users looking for the optimum performance

[ovs-dev] [PATCH v9 0/2] INSTALL.DPDK: Refactor DPDK install documentation

2016-07-06 Thread Bhanuprakash Bodireddy
This patchset refactors the present INSTALL.DPDK.md guide. The INSTALL guide is split in to two documents named INSTALL.DPDK and INSTALL.DPDK-ADVANCED. The former document is simplified with emphasis on installation, basic testcases and targets novice users. Sections on system configuration,

Re: [ovs-dev] [PATCH V6 13/17] at tests: Allow Python tests to be run on Windows

2016-07-06 Thread Alin Serdean
This patch needs to be applied last. Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: Paul Boca > Trimis: Wednesday, July 6, 2016 3:39 PM > Către: dev@openvswitch.org > Cc: Alin Serdean > Subiect: [PATCH V6

Re: [ovs-dev] [PATCH 1/2] tests: Ignore proxy configuration in kernel testsuite.

2016-07-06 Thread Jarno Rajahalme
> On Jul 1, 2016, at 8:30 PM, Ben Pfaff wrote: > > On Tue, Jun 21, 2016 at 09:57:15AM -0700, Jarno Rajahalme wrote: >> Add '--noproxy' parameter to 'wget' so that the test cases do not try >> to use a proxy that may be configured in the test environment. >> >> Signed-off-by:

[ovs-dev] [PATCH V6 17/17] vlog test: Disable default syslog logger

2016-07-06 Thread Paul Boca
Disable the syslog logger in case on Windows, '/dev/log' doesn't exist. Seems like on Python34 a default handler is added to the logger and it prints even if no handler is set by us. Signed-off-by: Paul-Daniel Boca --- V3: Initial commit V4: No changes V5: No

[ovs-dev] [PATCH V6 15/17] python tests: Skip ovn-controller-vtep tests on Windows

2016-07-06 Thread Paul Boca
The tests are not intended to run on Windows. Until now they were skipped based on $HAVE_PYTHON only. Signed-off-by: Paul-Daniel Boca --- V3: Initial commit V4: No changes V5: No changes V6: No changes --- tests/ovn-controller-vtep.at | 1 + 1 file changed, 1

[ovs-dev] [PATCH V6 16/17] tests: Skip vlog tests that try to move opened file

2016-07-06 Thread Paul Boca
On Windows if a file is opened by an application for writing, we cannot move it until all handles to that file are closed. Signed-off-by: Paul-Daniel Boca --- V3: Initial commit V4: No changes V5: No changes V6: Removed code that disables 'vlog/close' tests. ---

[ovs-dev] [PATCH V6 13/17] at tests: Allow Python tests to be run on Windows

2016-07-06 Thread Paul Boca
This patch removes the code which disables Python tests to be run on Windows. Signed-off-by: Alin Gabriel Serdean Signed-off-by: Paul-Daniel Boca --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes ---

[ovs-dev] [PATCH V6 14/17] python tests: Ignore stderr output

2016-07-06 Thread Paul Boca
test-unixctl.py and test-vlog.py outputs on stderr and on Windows stderr is not overriden by every AT_CHECK call, the logs are only apended to the file and subsequent AT_CHECKs get errors from previous call. Signed-off-by: Paul-Daniel Boca --- V3: Initial commit V4:

[ovs-dev] [PATCH V6 11/17] python tests: Ported UNIX sockets to Windows

2016-07-06 Thread Paul Boca
AF_UNIX sockets are not supported on Windows. Instead of an AF_UNIX socket use localhost tcp connections to communicate between components. This makes the python sockets compatible with the ones used in Windows applications. In case the socket returns WSAEWOULDBLOCK, it is replaced by EAGAIN

[ovs-dev] [PATCH V6 12/17] python tests: Ported Python daemon to Windows

2016-07-06 Thread Paul Boca
Used subprocess.Popen instead os.fork (not implemented on windows) and repaced of os.pipe with Windows pipes. To be able to identify the child process I added an extra parameter to daemon process '--pipe-handle', this parameter also contains the parent Windows pipe handle, used by the child to

[ovs-dev] [PATCH V6 10/17] python tests: Fixed abs_file_name function for Windows

2016-07-06 Thread Paul Boca
On windows a path containint ':' is considered an absolute path. Signed-off-by: Paul-Daniel Boca --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes --- python/ovs/util.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[ovs-dev] [PATCH V6 07/17] python tests: Skip TCP6 idl tests on Windows

2016-07-06 Thread Paul Boca
TCP6 sockets are not supported in python on Windows. Signed-off-by: Paul-Daniel Boca --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes --- tests/ovsdb-idl.at | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/ovsdb-idl.at

[ovs-dev] [PATCH V6 08/17] python tests: Skip python tests on Windows where POSIX pid is used

2016-07-06 Thread Paul Boca
There is a difference between POSIX pid and Windows pid, not all the time are equal. On Windows when a python script is started, a sh command is triggered as the parent for script. So when we try to get the daemon pid with 'echo $!', this will get the pid of sh not of python.exe as expected.

[ovs-dev] [PATCH V6 09/17] python tests: Fixed OSError not iterable on Windows

2016-07-06 Thread Paul Boca
On Windows if this exception is triggered then it will raise an exception while in the exception handler. Signed-off-by: Paul-Daniel Boca --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes --- python/ovs/poller.py | 5 + 1 file

[ovs-dev] [PATCH V6 06/17] python tests: Skip IPsec test on Windows

2016-07-06 Thread Paul Boca
IPsec is not used on Windows yet, and the services used by this test are non-existent on Windows. Also the paths in the test do not exist on Windows. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: No changes

[ovs-dev] [PATCH V6 05/17] python tests: Added fcntl module for Windows

2016-07-06 Thread Paul Boca
This is needed for lockf function used to lock the PID file on Windows. ioctl and fcntl functions are not implemented at this time because they are not used by any script. Signed-off-by: Paul-Daniel Boca --- V2: No changes V3: No changes V4: No changes V5: No

[ovs-dev] [PATCH V6 02/17] python tests: Register signal handlers only on supported types on Windows

2016-07-06 Thread Paul Boca
SIGHUP and SIGALRM are not available on Windows. Signed-off-by: Paul-Daniel Boca --- V2: Fixed Python function inet_open_active, treat WSAEWOULDBLOCK error as EINPROGRESS on Windows V3: No changes V4: No changes V5: No changes V6: No changes ---

[ovs-dev] [PATCH V6 03/17] python tests: Fixed ctl file name for Windows

2016-07-06 Thread Paul Boca
On Windows the CTL file doesn't contain the pid of the process. Signed-off-by: Paul-Daniel Boca --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes --- python/ovs/unixctl/__init__.py | 10 -- 1 file changed, 8 insertions(+), 2

[ovs-dev] [PATCH V6 04/17] python tests: Fixed unixctl python tests for Windows

2016-07-06 Thread Paul Boca
For bogus pid file path, use a windows-like file path. Signed-off-by: Paul-Daniel Boca --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes --- tests/unixctl-py.at | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff

  1   2   >