[ovs-dev] Darlehensangebot jetzt bewerben

2018-11-12 Thread Marco Tulio Rafael. Fuentes Linares
Befinden Sie sich in einer Finanzkrise? Benötigen Sie einen Kredit, um Ihr Zuhause zu refinanzieren oder Ihr Geschäft auszubauen? Benötigen Sie einen Kredit, um dringende Schulden zu begleichen und bei einer Bank zu bleiben? Benötigen Sie einen Kredit, um Ihre Rechnungen zu bezahlen?

[ovs-dev] [PATCH] ovn-sb.ovsschema: Avoid duplicated IPs in Encap table.

2018-11-12 Thread Han Zhou
From: Han Zhou When adding a new chassis, if there is an old chassis with same IP existed in Encap table, it is allowed to be added today. However, allowing it to be added results in problems: 1. The new chassis cannot work because none of the other chassises are able to create tunnel to it,

Re: [ovs-dev] About static libraries on openvswitch-fedora.spec

2018-11-12 Thread Martin Xu
Hi all, That commit was only to get the shared libraries. It doesn't need the static libraries. Martin On Mon, Nov 12, 2018 at 2:22 PM Flavio Leitner wrote: > On Sun, Nov 11, 2018 at 11:04:01AM -0800, Ben Pfaff wrote: > > On Sun, Nov 11, 2018 at 04:32:40PM +0100, Timothy Redaelli wrote: > > >

Re: [ovs-dev] About static libraries on openvswitch-fedora.spec

2018-11-12 Thread Flavio Leitner
On Sun, Nov 11, 2018 at 11:04:01AM -0800, Ben Pfaff wrote: > On Sun, Nov 11, 2018 at 04:32:40PM +0100, Timothy Redaelli wrote: > > Hi, > > from commit bc4fd439586f ("rhel: Ship ovs shared libraries, fedora") > > the openvswitch-devel subpackage (in openvswitch-fedora.spec) > > stated to include

[ovs-dev] [PATCH v1] Documentation: Fixing some minor spelling mistakes and consistent usage of certain keywords.

2018-11-12 Thread Ashish Varma
Signed-off-by: Ashish Varma --- Documentation/tutorials/ovs-conntrack.rst | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/Documentation/tutorials/ovs-conntrack.rst b/Documentation/tutorials/ovs-conntrack.rst index 07ea998..a86288f 100644 ---

Re: [ovs-dev] 8021q module is not loaded when adding vlan through ovs (ovs-vsctl)

2018-11-12 Thread Michael Shteinbok
Calling it from net_dev_init doesn't look clean. If you will see who is calling 'dev_add_offload' you will find out that there are couple of modules that doing that except of the 8021q. so either all modules should add offload from net_dev_init or stay with the current model where each module adds

Re: [ovs-dev] [PATCH] [RFC][windows]: Remap VPORT socket pool

2018-11-12 Thread Sairam Venugopal
Hi Alin, Thanks for the patch. Shashank is giving it a run. How do we prevent this issue in the future? For starters, I think we need to consolidate the usage under nl_* functions to ensure it's not doing something differently in vport_*. We need to look into investing in the test framework

Re: [ovs-dev] [PATCH] docs: Fix cross-references that referred to discussions that have moved.

2018-11-12 Thread Yifeng Sun
Looks good to me, thanks. Reviewed-by: Yifeng Sun On Thu, Nov 8, 2018 at 9:39 PM Ben Pfaff wrote: > Signed-off-by: Ben Pfaff > --- > include/openvswitch/meta-flow.h | 4 ++-- > ovn/utilities/ovn-sbctl.8.in| 2 +- > ovn/utilities/ovn-trace.8.xml | 2 +- > vswitchd/vswitch.xml

Re: [ovs-dev] VLAN tenant network patches

2018-11-12 Thread Guru Shetty
On Sun, 11 Nov 2018 at 21:02, Ankur Sharma wrote: > Hi Guru, > > Thanks for spending time in understanding the proposal and drafting your > understanding as well. > Thanks Numan for pitching in. > > Some comments (trying to keep them as brief as possible). > > a. On a high level, we are trying

[ovs-dev] [PATCH] [RFC][windows]: Remap VPORT socket pool

2018-11-12 Thread Alin Gabriel Serdean
Fixes: https://github.com/openvswitch/ovs-issues/issues/164 Signed-off-by: Alin Gabriel Serdean --- lib/dpif-netlink.c | 58 +- 1 file changed, 57 insertions(+), 1 deletion(-) diff --git a/lib/dpif-netlink.c b/lib/dpif-netlink.c index

Re: [ovs-dev] [PATCH v2 1/3] Routing policies, add routing-policy commands in ovn-nbctl

2018-11-12 Thread Ben Pfaff
Thanks for working to make OVN better. I've only glanced at this series. At a glance, it looks to me like it should be a single patch, because we usually change the schema, implement the change in ovn-northd, and add commands to ovn-nbctl to make it useful in a single patch. Please add a NEWS

Re: [ovs-dev] Error at ovsdb-cluster.at:204

2018-11-12 Thread Ben Pfaff
I'm having some trouble reproducing this... I'd like to, though, because the Raft implementation needs to be reliable. Do you have any more advice? On Mon, Nov 05, 2018 at 04:28:22PM -0800, Ashish Varma wrote: > Not sure if this is reported before. I am seeing failure (not always) at > the

Re: [ovs-dev] [PATCH 1/2] netdev-dpdk: Drop offload API for vhost ports.

2018-11-12 Thread Stokes, Ian
> On Thu, Nov 08, 2018 at 02:40:12PM +, Stokes, Ian wrote: > > > On Thu, Nov 08, 2018 at 02:16:15PM +, Stokes, Ian wrote: > > > > > On 06.11.2018 17:31, Stokes, Ian wrote: > > > > > >> On 18.10.2018 16:29, Ilya Maximets wrote: > > > > > >>> vhost ports are not DPDK eth ports and has no

Re: [ovs-dev] [PATCH] ipsec: Install ovs-monitor-ipsec in script directory

2018-11-12 Thread Ben Pfaff
On Sun, Nov 11, 2018 at 11:04:17AM +0100, Timothy Redaelli wrote: > In commit d5cc46e3d185 ("ipsec: Use @PYTHON@ directly instead of > "/usr/bin/env python"") ovs-monitor-ipsec is installed in bin directory, > but it's supposed to be installed in script directory. > > This commit removes also the

Re: [ovs-dev] [PATCH] gitignore: Ignore ovs-monitor-ipsec

2018-11-12 Thread Ben Pfaff
On Sun, Nov 11, 2018 at 11:04:33AM +0100, Timothy Redaelli wrote: > Commit d5cc46e3d185 ("ipsec: Use @PYTHON@ directly instead of "/usr/bin/env > python"") introduced ovs-monitor-ipsec.in that generates > ovs-monitor-ipsec. > > This commit adds ovs-monitor-ipsec to ipsec/.gitignore. > > Fixes:

Re: [ovs-dev] [PATCH] rhel: Rename the IPsec package as openvswitch-ipsec

2018-11-12 Thread Ben Pfaff
On Sun, Nov 11, 2018 at 11:13:50AM +0100, Timothy Redaelli wrote: > Currently the split package for ipsec is named > openvswitch-openvswitch-ipsec, but it should be named openvswitch-ipsec. > > This commit changes the spec file in order to have the IPsec package as > openvswitch-ipsec instead of

Re: [ovs-dev] [PATCH] build: Clean up ovs-monitor-ipsec.

2018-11-12 Thread Ben Pfaff
On Mon, Nov 12, 2018 at 01:34:11PM +0300, Ilya Maximets wrote: > This fixes travis distcheck: > > ERROR: files left in build directory after distclean: > ./ipsec/ovs-monitor-ipsec > make[1]: *** [distcleancheck] Error 1 > > CC: Timothy Redaelli > Fixes: d5cc46e3d185 ("ipsec: Use @PYTHON@

Re: [ovs-dev] [PATCH] pinctrl: Fix dp_packet structure leak.

2018-11-12 Thread Ben Pfaff
On Mon, Nov 12, 2018 at 03:54:12PM +0100, Lorenzo Bianconi wrote: > > Buffered packets are always packets created by 'dp_packet_clone_data()' > > i.e. they are malloced. It's not enough to free the packet data, > > dp_packet structure must be freed too. 'dp_packet_delete()' will take > > care of

Re: [ovs-dev] [PATCH] pinctrl: Fix crash on buffered packets hmap double remove.

2018-11-12 Thread Ben Pfaff
On Mon, Nov 12, 2018 at 04:04:20PM +0100, Lorenzo Bianconi wrote: > > 'destroy_buffered_packets()' removes the hmap node which was > > already removed by 'HMAP_FOR_EACH_POP()' producing following > > crash log: > > > > Invalid read of size 8 > > at 0x134EDB: hmap_remove (hmap.h:287) >

Re: [ovs-dev] [PATCH v5] netdev-dpdk: Bring link down when NETDEV_UP is not set

2018-11-12 Thread Eelco Chaudron
Thanks a lot Ian! On 12 Nov 2018, at 17:16, Stokes, Ian wrote: >> On 12.11.2018 12:26, Eelco Chaudron wrote: >>> When the netdev link flags are changed, !NETDEV_UP, the DPDK ports are >>> not actually going down. This is causing problems for people trying to >>> bring down a bond member. The

Re: [ovs-dev] [PATCH v5] netdev-dpdk: Bring link down when NETDEV_UP is not set

2018-11-12 Thread Stokes, Ian
> On 12.11.2018 12:26, Eelco Chaudron wrote: > > When the netdev link flags are changed, !NETDEV_UP, the DPDK ports are > > not actually going down. This is causing problems for people trying to > > bring down a bond member. The bond link is no longer being used to > > receive or transmit traffic,

[ovs-dev] Ik heb dringend uw antwoord nodig

2018-11-12 Thread N.Ca
Beste vriend , Ik neem contact met u op om te helpen bij het claimen van de borg van (veertien miljoen en vijfhonderdduizend Amerikaanse dollars) om inbeslagname te voorkomen. Mijn overleden cliënt, die dezelfde achternaam met u deelt, heeft een borg van veertien miljoen dollar,

Re: [ovs-dev] [PATCH] pinctrl: Fix dp_packet structure leak.

2018-11-12 Thread Lorenzo Bianconi
> Buffered packets are always packets created by 'dp_packet_clone_data()' > i.e. they are malloced. It's not enough to free the packet data, > dp_packet structure must be freed too. 'dp_packet_delete()' will take > care of that. > > CC: Lorenzo Bianconi > Fixes: d7abfe39cfd2 ("OVN: add buffering

Re: [ovs-dev] [PATCH] build: Clean up ovs-monitor-ipsec.

2018-11-12 Thread Ilya Maximets
On 12.11.2018 14:04, Timothy Redaelli wrote: > On Mon, 12 Nov 2018 13:34:11 +0300 > Ilya Maximets wrote: > >> This fixes travis distcheck: >> >> ERROR: files left in build directory after distclean: >> ./ipsec/ovs-monitor-ipsec >> make[1]: *** [distcleancheck] Error 1 >> >> CC: Timothy

[ovs-dev] [PATCH] pinctrl: Fix dp_packet structure leak.

2018-11-12 Thread Ilya Maximets
Buffered packets are always packets created by 'dp_packet_clone_data()' i.e. they are malloced. It's not enough to free the packet data, dp_packet structure must be freed too. 'dp_packet_delete()' will take care of that. CC: Lorenzo Bianconi Fixes: d7abfe39cfd2 ("OVN: add buffering support for

[ovs-dev] [PATCH] pinctrl: Fix crash on buffered packets hmap double remove.

2018-11-12 Thread Ilya Maximets
'destroy_buffered_packets()' removes the hmap node which was already removed by 'HMAP_FOR_EACH_POP()' producing following crash log: Invalid read of size 8 at 0x134EDB: hmap_remove (hmap.h:287) by 0x134EDB: destroy_buffered_packets (pinctrl.c:237) by 0x13AB3B:

Re: [ovs-dev] [PATCH v5] netdev-dpdk: Bring link down when NETDEV_UP is not set

2018-11-12 Thread Ilya Maximets
On 12.11.2018 12:26, Eelco Chaudron wrote: > When the netdev link flags are changed, !NETDEV_UP, the DPDK ports are not > actually going down. This is causing problems for people trying to bring > down a bond member. The bond link is no longer being used to receive or > transmit traffic,

Re: [ovs-dev] [PATCH v5] netdev-dpdk: Bring link down when NETDEV_UP is not set

2018-11-12 Thread Flavio Leitner
On Mon, Nov 12, 2018 at 04:26:22AM -0500, Eelco Chaudron wrote: > When the netdev link flags are changed, !NETDEV_UP, the DPDK ports are not > actually going down. This is causing problems for people trying to bring > down a bond member. The bond link is no longer being used to receive or >

[ovs-dev] [PATCH] netdev-dpdk: Flow validation refactoring.

2018-11-12 Thread Ilya Maximets
* Dropped 'is_all_zero' function, which is equal to 'is_all_zeros' from util.h . * util.h added to includes. Includes re-sorted within their blocks. (it's hard to figure out where to put new one if there is no order.) Note: linux/if.h depends on sys/socket.h . * 'ovs_u128_is_zero' used

Re: [ovs-dev] [PATCH v4] netdev-dpdk: Bring link down when NETDEV_UP is not set

2018-11-12 Thread Eelco Chaudron
On 9 Nov 2018, at 17:19, Flavio Leitner wrote: On Fri, Nov 09, 2018 at 07:24:55AM -0500, Eelco Chaudron wrote: When the netdev link flags are changed, !NETDEV_UP, the DPDK ports are not actually going down. This is causing problems for people trying to bring down a bond member. The bond

[ovs-dev] [PATCH v5] netdev-dpdk: Bring link down when NETDEV_UP is not set

2018-11-12 Thread Eelco Chaudron
When the netdev link flags are changed, !NETDEV_UP, the DPDK ports are not actually going down. This is causing problems for people trying to bring down a bond member. The bond link is no longer being used to receive or transmit traffic, however, the other end keeps sending data as the link