Re: [ovs-dev] OVN L3-HA request for feedback

2017-05-24 Thread Miguel Angel Ajo Pelayo
I wanted to share a small status update: Anil and I have been working on this [1] and we expect to post some preliminary patches before the end of the week. I can confirm that the BFD + bundle(active_backup) strategy works well from the hypervisors point of view. With 1 sec BFD pings we get a ~2

[ovs-dev] Query for missing function

2017-05-24 Thread 王志克
Hi All, Reading the release note of DPDK section for OVS2.6, I note below: * Basic connection tracking for the userspace datapath (no ALG, fragmentation or NAT support yet) I am wondering for the missing part (no ALG, fragmentation, NAT), can I have the release plan for such feature

[ovs-dev] ifconfig not installed on minimal Debian Stretch

2017-05-24 Thread Raymond Burkholder
This probably isn’t an optimal solution, as openvswitch runs on a number of different platforms. I am not sure how to perform a proper selection of ifconfig vs ip link. But i think I noticed some other openvswitch scripts use ip link only. The first change below relates to my previous messa

Re: [ovs-dev] ifconfig not installed on minimal Debian Stretch

2017-05-24 Thread Raymond Burkholder
ok, sorry, this message can be ignored, I see github changes on Mar 17 to reflect the ‘ip link’ change, but I think that the '/etc/init.d/openvswitch-switch status’ may still be a problem. > On 24 May 2017, at 08:42, Raymond Burkholder wrote: > > This probably isn’t an optimal solution, as op

Re: [ovs-dev] ifconfig not installed on minimal Debian Stretch

2017-05-24 Thread Guru Shetty
Does your '/etc/init.d/openvswitch-switch have the following line: https://github.com/openvswitch/ovs/blob/master/debian/openvswitch-switch.init#L30 That line should prevent any redirects to systemd. On 24 May 2017 at 08:05, Raymond Burkholder wrote: > ok, sorry, this message can be ignored, I

[ovs-dev] Sync on PTAP, EXT-382 and NSH - Tue 2017-06-07 17:00 CET

2017-05-24 Thread Jan Scheurich
Hi, Let's have a look at the progress of the different patch series with respect to submission, review and merging. Target is still to upstream these changes in time for OVS 2.8. Thank you, Jan Link to the Google design doc: https://docs.google.com/document/d/1oWMYUH8sjZJzWa72o2q9kU0N6pNE-rwZcL

Re: [ovs-dev] ifconfig not installed on minimal Debian Stretch

2017-05-24 Thread Raymond Burkholder
> On 24 May 2017, at 09:31, Guru Shetty wrote: > > Does your '/etc/init.d/openvswitch-switch have the following line: > https://github.com/openvswitch/ovs/blob/master/debian/openvswitch-switch.init#L30 > > That line should prevent any redirects to systemd. > yes it does have: _SYSTEMCTL_SKI

Re: [ovs-dev] ifconfig not installed on minimal Debian Stretch

2017-05-24 Thread Guru Shetty
On 24 May 2017 at 09:06, Raymond Burkholder wrote: > > > On 24 May 2017, at 09:31, Guru Shetty wrote: > > > > Does your '/etc/init.d/openvswitch-switch have the following line: > > https://github.com/openvswitch/ovs/blob/master/ > debian/openvswitch-switch.init#L30 > > > > That line should prev

Re: [ovs-dev] [RFC PATCH 0/6] Change dpdk rxq scheduling to incorporate rxq processing cycles.

2017-05-24 Thread Kevin Traynor
On 05/22/2017 04:43 PM, Jan Scheurich wrote: > Hi Kevin, > > Thanks a lot for addressing this very important limitation of OVS-DPDK > multi-core scalability in cloud contexts such as OpenStack. This is highly > appreciated! > > We have not started testing this, so for the time being just some h

Re: [ovs-dev] ifconfig not installed on minimal Debian Stretch

2017-05-24 Thread Raymond Burkholder
> > Can you do a 'sh -x /etc/init.d/openvswitch-switch status' to see whether it > looks at _SYSTEMCTL_SKIP_REDIRECT in any of the system libraries. > > For e.g, on a ubuntu15.04, when I run the above command, I get a match for > _SYSTEMCTL_SKIP_REDIRECT variable in /lib/lsb/init-functions.d/4

Re: [ovs-dev] ifconfig not installed on minimal Debian Stretch

2017-05-24 Thread Guru Shetty
On 24 May 2017 at 10:10, Raymond Burkholder wrote: > > > > Can you do a 'sh -x /etc/init.d/openvswitch-switch status' to see > whether it looks at _SYSTEMCTL_SKIP_REDIRECT in any of the system libraries. > > > > For e.g, on a ubuntu15.04, when I run the above command, I get a match > for _SYSTEMC

Re: [ovs-dev] OVN L3-HA request for feedback

2017-05-24 Thread Russell Bryant
On Wed, May 24, 2017 at 7:19 AM, Miguel Angel Ajo Pelayo wrote: > I wanted to share a small status update: > > Anil and I have been working on this [1] and we expect to post > some preliminary patches before the end of the week. > > I can confirm that the BFD + bundle(active_backup) strategy > wo

Re: [ovs-dev] [PATCH 1/3] datapath-windows: Remove Strsafe usage from datapath

2017-05-24 Thread Anand Kumar
Thank you for the patches. Acked-By: Anand Kumar Thanks, Anand Kumar On 5/23/17, 10:59 AM, "ovs-dev-boun...@openvswitch.org on behalf of Alin Serdean" wrote: The removal is mandatory to use the VStudio 2013 static code analyzer. The only function that was used from the include

Re: [ovs-dev] [PATCH 2/3] datapath-windows: add two new build targets for code analysis

2017-05-24 Thread Anand Kumar
Acked-by: Anand Kumar Thanks, Anand Kumar On 5/23/17, 10:59 AM, "ovs-dev-boun...@openvswitch.org on behalf of Alin Serdean" wrote: Add two new build targets: 'Win8Analyze' and 'Win8.1Analyze'. The new build targets have the static code analyzer (built in Visual Studio feature).

Re: [ovs-dev] [PATCH 3/3] appveyor: Add new make target

2017-05-24 Thread Anand Kumar
Acked-by: Anand Kumar Thanks, Anand Kumar On 5/23/17, 10:59 AM, "ovs-dev-boun...@openvswitch.org on behalf of Alin Serdean" wrote: This patch adds the new make target 'datapath_windows_analyze' (static analysis over the windows datapath code) to the appveyor build. Signed-off

Re: [ovs-dev] [PATCH 0/5] Fix unaligned pointer values reported by clang 4.0.

2017-05-24 Thread Greg Rose
On Tue, 2017-05-23 at 16:02 -0700, Joe Stringer wrote: > Clang 4.0 has added some new warnings around taking the address of packed > members of structures which may result in unaligned pointer values. This > series > addresses the resulting compilation failures (reported via -Werror). > > Joe Str

[ovs-dev] [PATCH] Add SYSTEMCTL_SKIP_REDIRECT to init files

2017-05-24 Thread root
Current versions of systemd in Debian Stretch use SYSTEMCTL_SKIP_REDIRECT instead of _SYSTEMCTL_SKIP_REDIRECT. Provide both variables in the .init files. Signed-off-by: Raymond Burkholder Suggested-by: Guru Shetty --- debian/openvswitch-switch.init | 1 + debian/ovn-central.init | 1 +

[ovs-dev] [PATCH] debian: Add SYSTEMCTL_SKIP_REDIRECT to init files

2017-05-24 Thread root
From: Raymond Burkholder Current versions of systemd in Debian Stretch use SYSTEMCTL_SKIP_REDIRECT instead of _SYSTEMCTL_SKIP_REDIRECT. Provide both variables in the .init files. Signed-off-by: Raymond Burkholder Suggested-by: Guru Shetty --- debian/openvswitch-switch.init | 1 + debian/ovn-

[ovs-dev] BOOKING

2017-05-24 Thread Jerry Williams
Dear Sir/Madam, I write to make inquiry if it is possible to make reservation for 4 guests in your accommodation facility for the period of 7 days stay from 23rd of September, 2017 till 30th of September, 2017. We will be needing either 2 double rooms or 4 Single rooms or an apartment for 4 pe

Re: [ovs-dev] [PATCH] debian: Add SYSTEMCTL_SKIP_REDIRECT to init files

2017-05-24 Thread Guru Shetty
On 24 May 2017 at 12:56, root wrote: > From: Raymond Burkholder > > Current versions of systemd in Debian Stretch use > SYSTEMCTL_SKIP_REDIRECT instead of _SYSTEMCTL_SKIP_REDIRECT. > Provide both variables in the .init files. > > Signed-off-by: Raymond Burkholder > Suggested-by: Guru Shetty >

Re: [ovs-dev] [PATCH] debian: Add SYSTEMCTL_SKIP_REDIRECT to init files

2017-05-24 Thread Greg Rose
On Wed, 2017-05-24 at 19:56 +, root wrote: > From: Raymond Burkholder > > Current versions of systemd in Debian Stretch use > SYSTEMCTL_SKIP_REDIRECT instead of _SYSTEMCTL_SKIP_REDIRECT. > Provide both variables in the .init files. > > Signed-off-by: Raymond Burkholder > Suggested-by: Guru

Re: [ovs-dev] [PATCH] debian: Add SYSTEMCTL_SKIP_REDIRECT to init files

2017-05-24 Thread Raymond Burkholder
> On 24 May 2017, at 14:27, Greg Rose wrote: > > On Wed, 2017-05-24 at 19:56 +, root wrote: >> From: Raymond Burkholder >> >> Current versions of systemd in Debian Stretch use >> SYSTEMCTL_SKIP_REDIRECT instead of _SYSTEMCTL_SKIP_REDIRECT. >> Provide both variables in the .init files. >>

Re: [ovs-dev] [PATCH] debian: Add SYSTEMCTL_SKIP_REDIRECT to init files

2017-05-24 Thread Greg Rose
On Wed, 2017-05-24 at 14:41 -0600, Raymond Burkholder wrote: > > On 24 May 2017, at 14:27, Greg Rose wrote: > > > > On Wed, 2017-05-24 at 19:56 +, root wrote: > >> From: Raymond Burkholder > >> > >> Current versions of systemd in Debian Stretch use > >> SYSTEMCTL_SKIP_REDIRECT instead of _S

Re: [ovs-dev] [PATCH RFC] Drop support for RHEL 5 and 6

2017-05-24 Thread Flavio Leitner
On Tue, May 16, 2017 at 11:05:24AM -0700, Guru Shetty wrote: > On 15 May 2017 at 16:49, Joe Stringer wrote: > > > On 12 May 2017 at 07:45, Timothy Redaelli wrote: > > > RHEL 6 is not supported anymore since it uses Python 2.6 and GCC 4.4.x, > > > but Open vSwitch needs, at least, Python 2.7 and

Re: [ovs-dev] [PATCH 0/5] Fix unaligned pointer values reported by clang 4.0.

2017-05-24 Thread Pravin Shelar
On Wed, May 24, 2017 at 12:18 PM, Greg Rose wrote: > On Tue, 2017-05-23 at 16:02 -0700, Joe Stringer wrote: >> Clang 4.0 has added some new warnings around taking the address of packed >> members of structures which may result in unaligned pointer values. This >> series >> addresses the resulting

Re: [ovs-dev] [PATCH 1/5] odp-execute: Fix unaligned eth_addr pointers.

2017-05-24 Thread Ben Pfaff
On Tue, May 23, 2017 at 04:02:12PM -0700, Joe Stringer wrote: > Clang 4.0 complains: > > ../lib/odp-execute.c:61:37: error: taking address of packed member 'eth_dst' > of > class or structure 'eth_header' may result in an unaligned pointer value > [-Werror,-Waddress-of-packed-member] >

Re: [ovs-dev] [PATCH 2/5] ofproto-dpif: Fix unaligned eth_addr pointers.

2017-05-24 Thread Ben Pfaff
On Tue, May 23, 2017 at 04:02:13PM -0700, Joe Stringer wrote: > Clang 4.0 complains: > > ../ofproto/ofproto-dpif.c:2291:46: error: taking address of packed member > 'eth_src' of class or structure 'eth_header' may result in an unaligned > pointer > value > [-Werror,-Waddress-of-packed-membe

[ovs-dev] [PATCH] ofproto-dpif: Include EthType in ct_orig_tuple feature probe

2017-05-24 Thread Sairam Venugopal
Set flow->dl_type to either ETH_TYPE_IP or ETH_TYPE_IPV6 when probing for ct_orig_tuple feature support. This can be expanded later on to check for both IPv4 and IPv6 support. Signed-off-by: Sairam Venugopal --- ofproto/ofproto-dpif.c | 13 - 1 file changed, 8 insertions(+), 5 deleti