[ovs-dev] Urgent,

2018-09-13 Thread Camila Cave
Hello Dear , It’s my urgent need For a foreign partner that made me to contact you. Watch Autism-Live.com . Live Daily Web Show on Autism This e-mail message and any documents attached to it are confidential and may contain information that is protected from disclosure by various federal and

Re: [ovs-dev] gre: Rename fallback devices to avoid udev's interference

2018-09-13 Thread Yifeng Sun
Sure, will do. On Thu, Sep 13, 2018 at 4:05 PM Justin Pettit wrote: > Great. By the way, the indication of how far back it should be backported > is very helpful. However, can you please add it after the “---“ so that > it’s not part of the commit message. > > Thanks! > > --Justin > > > On Sep

Re: [ovs-dev] [PATCH] ovsdb-idlc: Use ALIGNED_CAST to avoid spurious warnings for index rows.

2018-09-13 Thread Han Zhou
On Thu, Sep 13, 2018 at 2:54 PM Ben Pfaff wrote: > > On Wed, Sep 12, 2018 at 03:47:05PM -0700, Han Zhou wrote: > > On Wed, Sep 12, 2018 at 2:18 PM Ben Pfaff wrote: > > > > > > On Tue, Sep 11, 2018 at 11:32:25PM -0700, Han Zhou wrote: > > > > On Mon, Sep 10, 2018 at 1:01 PM Ben Pfaff wrote: > >

Re: [ovs-dev] [PATCH] ovs-save: Don't always include the default flow during restore

2018-09-13 Thread Guru Shetty
On Sun, 9 Sep 2018 at 05:20, Timothy Redaelli wrote: > Currently the default flow (actions=NORMAL) is present in the flow table > after > the flow table is restored also when the default flow is removed. > > This commit changes the behaviour of the "ovs-save save-flows" command to > use >

Re: [ovs-dev] gre: Rename fallback devices to avoid udev's interference

2018-09-13 Thread Justin Pettit
Great. By the way, the indication of how far back it should be backported is very helpful. However, can you please add it after the “---“ so that it’s not part of the commit message. Thanks! --Justin > On Sep 13, 2018, at 3:52 PM, Yifeng Sun wrote: > > Thanks Justin for the review, I will

Re: [ovs-dev] gre: Rename fallback devices to avoid udev's interference

2018-09-13 Thread Yifeng Sun
Thanks Justin for the review, I will look at it and create v2. Yifeng On Thu, Sep 13, 2018 at 3:19 PM Justin Pettit wrote: > > > > On Sep 11, 2018, at 6:52 AM, Yifeng Sun wrote: > > > > Please backport this patch to upstream OVS down to 2.9. > > > > On certain kernel versions, when

Re: [ovs-dev] gre: Rename fallback devices to avoid udev's interference

2018-09-13 Thread Justin Pettit
> On Sep 11, 2018, at 6:52 AM, Yifeng Sun wrote: > > Please backport this patch to upstream OVS down to 2.9. > > On certain kernel versions, when openvswitch kernel module creates > a gre0 interface, the kernel’s gre module will jump out and compete > to control the gre0 interface. This will

Re: [ovs-dev] [PATCH] ovsdb-idlc: Use ALIGNED_CAST to avoid spurious warnings for index rows.

2018-09-13 Thread Ben Pfaff
On Wed, Sep 12, 2018 at 03:47:05PM -0700, Han Zhou wrote: > On Wed, Sep 12, 2018 at 2:18 PM Ben Pfaff wrote: > > > > On Tue, Sep 11, 2018 at 11:32:25PM -0700, Han Zhou wrote: > > > On Mon, Sep 10, 2018 at 1:01 PM Ben Pfaff wrote: > > > > > > > > The *_index_init_row() function casts a generic

[ovs-dev] [PATCH] ofp-print: Fix a memory leak reported by fuzz

2018-09-13 Thread Yifeng Sun
When ofputil_decode_port_stats returns error, it is possible that custom_stats_counters is valid and still need freed. The fuzz report link is https://oss-fuzz.com/testcase?key=5739356233400320 Signed-off-by: Yifeng Sun --- lib/ofp-print.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [ovs-dev] [PATCH] ovs-save: Don't always include the default flow during restore

2018-09-13 Thread Guru Shetty
On Thu, 13 Sep 2018 at 10:38, Timothy Redaelli wrote: > On Wed, 12 Sep 2018 10:36:22 -0700 > Guru Shetty wrote: > > > On Tue, 11 Sep 2018 at 09:07, Flavio Leitner wrote: > > > > > On Sun, Sep 09, 2018 at 02:20:02PM +0200, Timothy Redaelli wrote: > > > > Currently the default flow

Re: [ovs-dev] [PATCH] ovs-save: Don't always include the default flow during restore

2018-09-13 Thread Timothy Redaelli
On Wed, 12 Sep 2018 10:36:22 -0700 Guru Shetty wrote: > On Tue, 11 Sep 2018 at 09:07, Flavio Leitner wrote: > > > On Sun, Sep 09, 2018 at 02:20:02PM +0200, Timothy Redaelli wrote: > > > Currently the default flow (actions=NORMAL) is present in the > > > flow table > > after > > > the

Re: [ovs-dev] ERSPAN type 3 not working in OVS

2018-09-13 Thread Gregory Rose
On 9/12/2018 12:28 AM, ram kumar wrote: Hi All, I am trying to use ovs switch for decaping the ERSPAN header(GRE+ERSPAN removal). The ovs version 2.10 has support to ERSPAN termination, but currently it works only for ERSPAN type 2. I am not able to make it work for

Re: [ovs-dev] Revert "Revert "utilities/ovs-ctl: Force removal of ip_gre/gre""

2018-09-13 Thread Yifeng Sun
Looks good to me and testing shows no problem. Thanks Greg. Tested-by: Yifeng Sun Reviewed-by: Yifeng Sun On Thu, Sep 13, 2018 at 9:42 AM Gregory Rose wrote: > On 9/12/2018 3:09 PM, Ben Pfaff wrote: > > On Tue, Sep 11, 2018 at 06:52:35AM -0700, Yifeng Sun wrote: > >> Please backport this

[ovs-dev] Obtención y análisis de la información

2018-09-13 Thread Director de Marketing
--- Este correo electrónico ha sido comprobado en busca de virus por AVG. http://www.avg.com ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] Revert "Revert "utilities/ovs-ctl: Force removal of ip_gre/gre""

2018-09-13 Thread Gregory Rose
On 9/12/2018 3:09 PM, Ben Pfaff wrote: On Tue, Sep 11, 2018 at 06:52:35AM -0700, Yifeng Sun wrote: Please backport this patch to upstream OVS down to 2.10. Author: Greg Rose Date: Wed Jun 6 15:34:44 2018 -0700 This reverts commit a94f9524dbc11c78c83d1a49959497f5e73bf949. Signed-off-by:

Re: [ovs-dev] [PATCH branch-2.10] lib/tc: treat vlan id and prio as independent fields

2018-09-13 Thread Simon Horman
On Thu, Sep 13, 2018 at 08:57:48AM +0100, Pieter Jansen van Vuuren wrote: > Previously the key was used to check the presence of vlan id and > prio fields instead of using the mask. Additionally the vlan id > field was considered to be present if only the prio field was set, > and vice versa. f.e.

Re: [ovs-dev] [PATCH 2/2] dpctl: Expand the flow dump type filter

2018-09-13 Thread Simon Horman
On Wed, Sep 12, 2018 at 09:27:37AM +, Gavi Teitz wrote: > On Thursday, August 30, 2018 at 14:06, Simon Horman wrote: > >On Fri, Aug 10, 2018 at 11:30:08AM +0300, Gavi Teitz wrote: > >> Added new types to the flow dump filter, and allowed multiple filter > >> types to be passed at once, as a

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Fix translation of groups with no buckets.

2018-09-13 Thread Eelco Chaudron
On 2 Sep 2018, at 18:30, Ben Pfaff wrote: A group can have no buckets, in which case ovs_list_back() assert-fails. This fixes the problem. Found by OFTest. Fixes: a04e58881e25 ("ofproto-dpif-xlate: Simplify translation for groups.") Signed-off-by: Ben Pfaff The change looks fine to

[ovs-dev] [PATCH branch-2.7] dpdk: Use DPDK 16.11.8 release.

2018-09-13 Thread Ian Stokes
Modify travis linux build script to use the latest DPDK stable release 16.11.8. Update docs for latest DPDK stable releases. Signed-off-by: Ian Stokes --- .travis/linux-build.sh | 2 +- Documentation/faq/releases.rst | 2 +- Documentation/intro/install/dpdk.rst |

Re: [ovs-dev] [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0

2018-09-13 Thread Chandran, Sugesh
Regards _Sugesh > >> -2.10.x 17.11.3 > >> +2.10.x 18.08.0 > > > > Hi - on updating versions in docs, this one should not change as the > > released/to be released versions of OVS 2.10.x stick with 17.11.x. We > > can just leave this unmodified for now and add an

Re: [ovs-dev] [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0

2018-09-13 Thread Ian Stokes
On 9/13/2018 1:57 PM, Kevin Traynor wrote: On 09/13/2018 01:45 PM, Ian Stokes wrote: On 9/13/2018 1:34 PM, Ophir Munk wrote: Hi Sugesh, I can update documentation. Is it enough to update dpdk.rst.txt only? If not - can you please send a link to additional files with references to 17.11 to be

Re: [ovs-dev] [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0

2018-09-13 Thread Kevin Traynor
On 09/13/2018 01:45 PM, Ian Stokes wrote: > On 9/13/2018 1:34 PM, Ophir Munk wrote: >> Hi Sugesh, >> I can update documentation. Is it enough to update dpdk.rst.txt only? >> If not - can you please send a link to additional files with >> references to 17.11 to be updated? >> I will wait 1-2 days

Re: [ovs-dev] [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0

2018-09-13 Thread Ian Stokes
On 9/13/2018 1:34 PM, Ophir Munk wrote: Hi Sugesh, I can update documentation. Is it enough to update dpdk.rst.txt only? If not - can you please send a link to additional files with references to 17.11 to be updated? I will wait 1-2 days to get more reviews then will send v3. Hi Ophir, I've

[ovs-dev] [PATCH 1/1] dpdk: Use DPDK 17.11.4 release.

2018-09-13 Thread Ian Stokes
Modify travis linux build script to use the latest DPDK stable release 17.11.4. Update docs for latest DPDK stable releases. Signed-off-by: Ian Stokes --- .travis/linux-build.sh | 2 +- Documentation/faq/releases.rst | 6 +++--- Documentation/intro/install/dpdk.rst

Re: [ovs-dev] [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0

2018-09-13 Thread Ophir Munk
Hi Sugesh, I can update documentation. Is it enough to update dpdk.rst.txt only? If not - can you please send a link to additional files with references to 17.11 to be updated? I will wait 1-2 days to get more reviews then will send v3. Regards, Ophir > -Original Message- > From:

Re: [ovs-dev] [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0

2018-09-13 Thread Chandran, Sugesh
Hi Ophir, Also a quick grep show there are number of places in documentation still pointing to DPDK17.11, I guess its worth while to change them as well. This will avoid confusion for people who wanted to use this branch, What do you think? Regards _Sugesh > -Original Message- >

Re: [ovs-dev] [ovs-dev, branch-2.10] lib of tc: treat vlan id and prio as independent fields

2018-09-13 Thread 0-day Robot
Bleep bloop. Greetings Pieter Jansen van Vuuren, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Too many signoffs; are you missing Co-authored-by lines? Lines checked: 88,

Re: [ovs-dev] [PATCH] conntrack: invalid packet should not modify ct state

2018-09-13 Thread Zang MingJie
On Thu, Sep 13, 2018 at 2:55 AM Darrell Ball wrote: > > Thanks for looking MingJie > > > On Wed, Sep 12, 2018 at 2:16 AM, Zang MingJie wrote: >> >> When encounter an invalid packet, all changes made by the packet should >> be reverted. Currently an invalid packet can change the seq number while

[ovs-dev] [PATCH branch-2.10] lib/tc: treat vlan id and prio as independent fields

2018-09-13 Thread Pieter Jansen van Vuuren
Previously the key was used to check the presence of vlan id and prio fields instead of using the mask. Additionally the vlan id field was considered to be present if only the prio field was set, and vice versa. f.e. setting the following: ovs-ofctl -OOpenFlow13,OpenFlow15 add-flow br0 \