[ovs-dev] Un prestito a basso tasso

2018-09-27 Thread Loan Office
Stai cercando un prestito legittimo? Stanco di cercare prestiti e mutui? Sei stato rifiutato dalle tue banche? Avete bisogno di un prestito per cancellare i vostri debiti / fatture? il tuo trauma finanziario รจ finito. Offriamo prestiti da 5.000,00 ? a 1.000.000,00 ? con un tasso di interesse de

Re: [ovs-dev] ovsdb-client: Fix a bug that uses wrong index

2018-09-27 Thread 0-day Robot
Bleep bloop. Greetings Yifeng Sun, 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: WARNING: Line is 81 characters long (recommended limit is 79) #22 FILE: ovsdb/ovsdb-client.c:1764

[ovs-dev] [PATCH] ovsdb-client: Fix a bug that uses wrong index

2018-09-27 Thread Yifeng Sun
This patch fixes the incorrect index to argv. Signed-off-by: Yifeng Sun --- ovsdb/ovsdb-client.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ovsdb/ovsdb-client.c b/ovsdb/ovsdb-client.c index ab3aa7c93307..7c8a59d0e749 100644 --- a/ovsdb/ovsdb-client.c +++ b/ovsdb/ovsd

Re: [ovs-dev] [PATCH v2 2/2] Makes stylistic change suggested by 0-day robot.

2018-09-27 Thread Ben Pfaff
On Thu, Sep 27, 2018 at 02:07:42PM +0200, bshas...@sect.tu-berlin.de wrote: > From: Bhargava Shastry > > Wraps overly long line in expr_parse_target.c > > Signed-off-by: Bhargava Shastry I folded this into the first patch. Thanks! ___ dev mailing li

Re: [ovs-dev] [PATCH v2 1/2] This patch adds a new oss-fuzz target for OVN

2018-09-27 Thread Ben Pfaff
On Thu, Sep 27, 2018 at 02:07:41PM +0200, bshas...@sect.tu-berlin.de wrote: > From: Bhargava Shastry > > The fuzzer target, expr_parse_target.c, comprises test cases adapted > from test-ovn.c. > > In addition, this patch contains configuration files for oss-fuzz, > including a dictionary, ex

Re: [ovs-dev] [patch v4] conntrack: Add rcu support.

2018-09-27 Thread Ben Pfaff
OK. I'll wait for the 3-patch series. On Thu, Sep 27, 2018 at 08:14:41PM +, Darrell Ball wrote: > Below is the recent private commit message I have for the patch: > > However, I plan to split out to 3 main patches, with probably a few minor > cosmetic patches. > > 1/ cmap performance and co

Re: [ovs-dev] [patch v4] conntrack: Add rcu support.

2018-09-27 Thread Darrell Ball
Below is the recent private commit message I have for the patch: However, I plan to split out to 3 main patches, with probably a few minor cosmetic patches. 1/ cmap performance and code simplification. 2/ Eliminating exporting private datastructures to outside modules 3/ Eliminating passing 'ct'

Re: [ovs-dev] [PATCH v7 4/4] Replace router internal MAC with gateway MAC for reply packets

2018-09-27 Thread Guru Shetty
On Mon, 24 Sep 2018 at 03:10, Miguel Angel Ajo Pelayo wrote: > > > No worries Guru, I understand your feeling, I worked with Anil on > developing this feature, and it's indeed rather complex (we are actually > replacing keepalived + VRRP with openflow and BFD). > > I'm happy to work on a more det

Re: [ovs-dev] [patch v4] conntrack: Add rcu support.

2018-09-27 Thread Ben Pfaff
On Tue, Sep 18, 2018 at 01:22:50PM -0700, Darrell Ball wrote: > Add in rcu support for conntrack. At the same time, the array of > hmaps is replaced by a cmap. Using a single map also simplifies > the handling of nat and allows the removal of the nat_conn map. > There is still some cleanup to do a

Re: [ovs-dev] [dpdk-howl PATCH v4] netdev-dpdk: Upgrade to dpdk v18.08

2018-09-27 Thread Kevin Traynor
On 09/26/2018 11:05 PM, Ophir Munk wrote: > Hi Kevin, > I have reviews to be addressed in v5 which may change v4. > Therefore I suggest sending v5 (hopefully) as the final version for this > PATCH. > Please let me know if you still want me to resend v4 to the ML. No need - makes sense to just se

Re: [ovs-dev] ofproto: Handle OpenFlow version mismatch for requestforward with groups.

2018-09-27 Thread Aaron Conole
Ben Pfaff writes: > On Thu, Sep 27, 2018 at 08:57:23AM -0400, Aaron Conole wrote: >> I think the issue here on the bot side is that the compiler is not able >> to deal with this initialization semantic. I'll follow up and get it >> fixed. > > I sent a fix yesterday: > https://patchwork.o

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate.c: Fix uninitialized variable warning.

2018-09-27 Thread Flavio Leitner
On Thu, Sep 27, 2018 at 10:34:06AM -0700, Justin Pettit wrote: > With gcc 7.3.0 a warning is given about two variables possibly being > uninitialized in compose_sample_action(). The code path only allows the > variables to be used if they've been initialized, so this warning is > incorrect. Howev

Re: [ovs-dev] [PATCH] dpif-netdev-perf: Clarify frequency number.

2018-09-27 Thread Flavio Leitner
On Tue, Sep 25, 2018 at 07:00:40PM +0300, Ilya Maximets wrote: > 'dpif-netdev/pmd-perf-show' command prints the frequency number > calculated from the total number of cycles spent for iterations > for the measured period. This number could be confusing, because > users may think that it should be e

[ovs-dev] [PATCH] ofproto-dpif-xlate.c: Fix uninitialized variable warning.

2018-09-27 Thread Justin Pettit
With gcc 7.3.0 a warning is given about two variables possibly being uninitialized in compose_sample_action(). The code path only allows the variables to be used if they've been initialized, so this warning is incorrect. However, this change allows a clean build. Signed-off-by: Justin Pettit --

Re: [ovs-dev] [PATCH] treewide: Fix spelling of "receive".

2018-09-27 Thread Ben Pfaff
On Wed, Sep 26, 2018 at 11:03:26PM -0700, Justin Pettit wrote: > > > On Sep 26, 2018, at 4:12 PM, Ben Pfaff wrote: > > > > Signed-off-by: Ben Pfaff > > Ha. > > Acked-by: Justin Pettit Thanks, applied to master. ___ dev mailing list d...@openvswitc

Re: [ovs-dev] [PATCH] ofproto: Fix build with some GCC versions.

2018-09-27 Thread Ben Pfaff
On Wed, Sep 26, 2018 at 11:04:13PM -0700, Justin Pettit wrote: > > > On Sep 26, 2018, at 4:11 PM, Ben Pfaff wrote: > > > > GCC 4.8.x and possibly other versions don't like a designated initializer > > for an anonymous struct, see e.g. > > https://travis-ci.org/openvswitch/ovs/jobs/433747674 > >

Re: [ovs-dev] ofproto: Handle OpenFlow version mismatch for requestforward with groups.

2018-09-27 Thread Ben Pfaff
On Thu, Sep 27, 2018 at 08:57:23AM -0400, Aaron Conole wrote: > I think the issue here on the bot side is that the compiler is not able > to deal with this initialization semantic. I'll follow up and get it > fixed. I sent a fix yesterday: https://patchwork.ozlabs.org/patch/975494/ __

Re: [ovs-dev] [PATCH 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-09-27 Thread Numan Siddique
On Sat, Sep 22, 2018 at 6:08 AM wrote: > When starting OVN DBs in HA using pacemaker with ssl, we need to pass ssl > certs for starting standby DBs. Hence, we need this change. > > Signed-off-by: aginwala > Hi Aliasgar, I will try this out and get back to you with any comments. Thanks Numa

Re: [ovs-dev] [PATCH 2/2] dpif-netdev-perf: Print SMC statistics.

2018-09-27 Thread Wang, Yipeng1
Thank you Ilya for adding this! Besides the small comment, otherwise Acked-by: Yipeng Wang >-Original Message- >From: Ilya Maximets [mailto:i.maxim...@samsung.com] >Sent: Wednesday, September 26, 2018 10:19 AM >To: ovs-dev@openvswitch.org >Cc: Jan Scheurich ; Stokes, Ian >; Wang, Yipen

Re: [ovs-dev] Makes stylistic change suggested by 0-day robot.

2018-09-27 Thread 0-day Robot
Bleep bloop. Greetings Bhargava Shastry, 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: Author Bhargava Shastry needs to sign off. WARNING: Unexpected sign-offs from devel

Re: [ovs-dev] ofproto: Handle OpenFlow version mismatch for requestforward with groups.

2018-09-27 Thread Aaron Conole
0-day Robot writes: > Bleep bloop. Greetings Ben Pfaff, 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. > > > build: > /bin/sh ./libtool --tag=CC --mode=compile gcc -std=gnu99 -DHAVE

Re: [ovs-dev] This patch adds a new oss-fuzz target for OVN

2018-09-27 Thread 0-day Robot
Bleep bloop. Greetings Bhargava Shastry, 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: Author Bhargava Shastry needs to sign off. WARNING: Unexpected sign-offs from devel

Re: [ovs-dev] This patch adds a new oss-fuzz target for OVN lexer

2018-09-27 Thread Aaron Conole
0-day Robot writes: > Bleep bloop. Greetings Bhargava Shastry, 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: Author Bhargava Shastry needs to sign off. > W

[ovs-dev] [PATCH v2 2/2] Makes stylistic change suggested by 0-day robot.

2018-09-27 Thread bshastry
From: Bhargava Shastry Wraps overly long line in expr_parse_target.c Signed-off-by: Bhargava Shastry --- tests/oss-fuzz/expr_parse_target.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/oss-fuzz/expr_parse_target.c b/tests/oss-fuzz/expr_parse_target.c index e7dee

[ovs-dev] [PATCH v2 1/2] This patch adds a new oss-fuzz target for OVN

2018-09-27 Thread bshastry
From: Bhargava Shastry The fuzzer target, expr_parse_target.c, comprises test cases adapted from test-ovn.c. In addition, this patch contains configuration files for oss-fuzz, including a dictionary, expr.dict, to aid quick path discovery and a fuzzer configuration file that customises fuzz

Re: [ovs-dev] This patch adds a new oss-fuzz target for OVN lexer

2018-09-27 Thread 0-day Robot
Bleep bloop. Greetings Bhargava Shastry, 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: Author Bhargava Shastry needs to sign off. WARNING: Unexpected sign-offs from devel

[ovs-dev] [PATCH] This patch adds a new oss-fuzz target for OVN lexer

2018-09-27 Thread bshastry
From: Bhargava Shastry The fuzzer target, expr_parse_target.c, comprises test cases adapted from test-ovn.c. In addition, this patch contains configuration files for oss-fuzz, including a dictionary, expr.dict, to aid quick path discovery and a fuzzer configuration file that customises fuzz

Re: [ovs-dev] [PATCH v5] Improved Packet Drop Statistics in OVS

2018-09-27 Thread Gowrishankar Muthukrishnan
Hi Keshav, Thank you for the patches. They certainly help us debugging packet drops more meaningfully. Following your changes, some of the places where additional reasons for drop could also be added as I hope helpful. Could you please check below items as well. __netdev_dpdk_vhost_send() AND netd

Re: [ovs-dev] OVSDB IDL - tracked columns gives out of sequenced updates sometimes

2018-09-27 Thread Arunkumar Rg
Thanks Ben. Let me check the newer version. Regards, Arun. On Thu, Sep 27, 2018 at 2:43 AM Ben Pfaff wrote: > There have been fixed to column tracking since then. Try a newer > version. > > On Thu, Sep 20, 2018 at 05:10:46PM +0530, Arunkumar Rg wrote: > > Forgot to mention that I'm using OVS 2

Re: [ovs-dev] [PATCH v3] ovn: Support configuring the BFD params for the tunnel interfaces

2018-09-27 Thread Miguel Angel Ajo Pelayo
I believe northbound database is the right place although it adds a change to ovn-northd, but it's more natural, and in such way the admin would not need to do this chassis by chassis, something specially important since those can be dynamically added. On Thu, Sep 27, 2018 at 9:58 AM Numan Siddi

Re: [ovs-dev] [PATCH v3] ovn: Support configuring the BFD params for the tunnel interfaces

2018-09-27 Thread Numan Siddique
On Thu, Sep 27, 2018 at 12:53 PM Miguel Angel Ajo Pelayo < majop...@redhat.com> wrote: > > > On Wed, Sep 26, 2018 at 11:08 PM Ben Pfaff wrote: > >> On Wed, Sep 26, 2018 at 04:14:18PM +0530, nusid...@redhat.com wrote: >> > From: Numan Siddique >> > >> > With this commit the users can override the

Re: [ovs-dev] [PATCH v3] ovn: Support configuring the BFD params for the tunnel interfaces

2018-09-27 Thread Miguel Angel Ajo Pelayo
On Wed, Sep 26, 2018 at 11:08 PM Ben Pfaff wrote: > On Wed, Sep 26, 2018 at 04:14:18PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique > > > > With this commit the users can override the default values of > > the BFD params - min_rx, min_tx, decay_min_rx and mult if desired. > > This