Hi,
I had tested 2.8.1/2 earlier which uses 17.05.01 or 17.05.02 and found around
10% drop for udp traffic. OVS 2.7.4 gave the similar result as OVS 2.6.2 (DPDK
16.11.4). I was using Intel Niantic 82599 for testing.
Regards,
Nitin
-Original Message-
From: Ilya Maximets [mailto:i.maxim..
Compliments!!
I have access to very vital information that can be used to move huge amount of
money. I have done my homework very well and I have the machineries in place to
get it done since I am still in active service.
If it was possible for me to do it alone I would not have bothered contac
Bleep bloop. Greetings Qiuyu Xiao, 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: No signatures found.
Lines checked: 237, Warnings: 0, Errors: 1
Please check this out.
This patch adds IPsec support for OVN tunnel. Basically, OVN offers a
binary option to its user for encryption configuration. If the IPsec
option is turned on, all tunnels will be encrypted. Otherwise, no tunnel
will be encrypted.
The changes are summarized as below:
1) Added a ipsec column on the
The last few OVS releases have included a "soft freeze" stage in the
release process, but this stage has never been formalized in the
documentation. This adds a description.
Signed-off-by: Ben Pfaff
---
Documentation/internals/release-process.rst | 87 -
1 file chang
This patch series is preparatory work for turning any of the tools from db-ctl
family (ovs-vsctl, vtep-ctl, ovn-nbctl, or ovn-sbctl) into a long-lived
server/daemon that processes database commands on client request.
The goal of this patch set it to prevent dying (terminating the process) when
run
This patch series is preparatory work for turning any of the tools from db-ctl
family (ovs-vsctl, vtep-ctl, ovn-nbctl, or ovn-sbctl) into a long-lived
server/daemon that processes database commands on client request.
The goal of this patch set it to prevent dying (terminating the process) when
run
On Wed, Jun 20, 2018 at 10:39:46AM +0100, Ian Stokes wrote:
> Modify travis linux build script to use the latest
> DPDK stable release 17.11.3. Update docs for latest
> DPDK stable releases.
>
> Signed-off-by: Ian Stokes
> ---
> .travis/linux-build.sh | 2 +-
> Documentation/fa
From: Numan Siddique
Calling ovs.stream.open_block(ovs.stream.open("tcp:127.0.0.1:6641")) returns
success even if there is no server listening on 6641. To check if the connection
is established or not, Stream class makes use of
ovs.socket_util.check_connection_completion().
This function returns
On 25/06/2018 13:15, Anju Thomas wrote:
> Hi Ben,
>
> We are facing multiple such crashes on different computes in our deployments.
> Seems to be a pretty common problem in our setup. As you suggested, it would
> be good if we can make the below changes as well.How do you suggest we
> move
On Mon, Jul 2, 2018, 10:25 PM 0-day Robot wrote:
> Bleep bloop. Greetings Numan Siddique, 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.
>
Thanks Robot.
I thought I fixed this issue. L
Bleep bloop. Greetings Numan Siddique, 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
/var/lib/jenkins/jobs/upstream_build_from_pw/workspace/build-aux/missing
autom4te --lang
Sure, you need to collect perf records for the same binary, i.e. built with
the same compiler options (and on the same machine), to make them useful.
Unfortunately, I have no setup to test your case right now.
Data for 2.8 could help bisecting the issue.
On 02.07.2018 18:04, Shahaji Bhosle wrote:
According to our release process, we should fork branch-2.10 from master
July 1 (yesterday), then release on August 15. I'm going to propose
that we modify this in the same way that has been successful in the
past, by calling for an approximately 2-week "soft freeze". During the
freeze period, we
Hi,
Awaiting comments on "dpif-netdev: Avoid reordering of packets in a batch with
same megaflow". Did anyone tried and see any issues ?
Warm Regards,
Vishal Ajmera
> -Original Message-
> From: ovs-dev-boun...@openvswitch.org boun...@openvswitch.org> On Behalf Of Kevin Traynor
> Sent:
From: Numan Siddique
Calling ovs.stream.open_block(ovs.stream.open("tcp:127.0.0.1:6641")) returns
success even if there is no server listening on 6641. To check if the connection
is established or not, Stream class makes use of
ovs.socket_util.check_connection_completion().
This function returns
Using icmp6 action, send an ICMPv6 time exceeded frame whenever
an OVN logical router receives an IPv6 packets whose TTL has
expired (ip.ttl == {0, 1})
Signed-off-by: Lorenzo Bianconi
---
ovn/northd/ovn-northd.8.xml | 12 +++-
ovn/northd/ovn-northd.c | 31
Hi Ilya,
Thanks for the reply.
For performance traffic testing we are running with -O2. You are right
about the perf report, when were running with perf record we had set "-g
-O0". Do you need us to run with just "-g -O2" and give you the profile, or
any other optimization setting.
Do you have a te
Hi.
Sorry for late response.
Looking at your perf data, I see functions like "dp_packet_batch_size"
consuming ~0.5 - 0.7 % of time. Are you building with all compiler
optimizations disabled? Otherwise where should be no such symbols in
perf report. They should be completely inlined.
Best regards,
Hi Jakub,
Your cover letter still hasn't come through, so feel free to tell me if
my comments here are misguided.
I'm guessing that you did not remove ctl_fatal() calls from
parse_commands() and ctl_parse_commands() because there is no
ctl_context present. However, it's not clear why the ctl
I'm not sure why you went with a different approach on this compared to
the other functions. I would have expected you would change
get_row_by_id() to return a string (NULL if no error, otherwise an error
string). Then return the row as an output parameter.
On 07/02/2018 06:50 AM, Jakub Sitnic
On Mon, Jul 02, 2018 at 12:07:58PM +0300, Roi Dayan wrote:
> When adding TC rules we save the prio so can reuse same prio
> for same mask since different mask will have to use different prio.
> The multi mask per prio probe broke this by using a prio but
> get_prio_for_tc_flower() didn't know about
So, what about this patch?
Best regards, Ilya Maximets.
On 20.06.2018 10:44, Ilya Maximets wrote:
> Recent assertion failure fix changed rconn workflow for unreliable
> connections (such as connections from ovs-ofctl) from
>
> |rconn|DBG|br-int<->unix#151: entering ACTIVE
> |rconn|DBG|br
This commit adds ovs_dump_udpif_keys and ovs_show_upcall commands to the GDB
script.
Here are some examples of the output:
(gdb) ovs_show_upcall
netdev@ovs-netdev:
flows : (current 0) (avg 0) (max 0) (limit 1)
dump duration : 1ms
ufid enabled : true
39: (keys 0)
42: (keys
On Mon, 2 Jul 2018 06:55:57 -0400
0-day Robot wrote:
> Bleep bloop. Greetings Jakub Sitnicki, 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 80
Bleep bloop. Greetings Jakub Sitnicki, 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 80 characters long (recommended limit is 79)
#62 FILE: lib/db-ctl-base.c:988
Return the error via the context instead of calling ctl_fatal() so that
the caller can decide how to handle it.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index f5492aadb..b
Return the error via the context instead of calling ctl_fatal() so that
the caller can decide how to handle it.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index 585cf67a0..f549
Return the error via the context instead of calling ctl_fatal() so that
the caller can decide how to handle it.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index 7c1e10373..
Return the error via the context instead of calling ctl_fatal() so that
the caller can decide how to handle it.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
ind
All users are gone.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 8
1 file changed, 8 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index d3ca4ca70..c670236da 100644
--- a/lib/db-ctl-base.c
+++ b/lib/db-ctl-base.c
@@ -186,14 +186,6 @@ print_command_arguments(
Release resources now that we are returning to the caller on error.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index 2fff7127b..8d4799433 100644
--- a/lib/db-ctl-base.c
+++ b/lib/db-ctl-base.c
@@ -
Release resources now that we are returning to the caller on error.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index 5f386cab5..d3ca4ca70 100644
--- a/lib/db-ctl-base.c
+++ b/lib/db-ctl-base.c
@@ -
Release resources now that we are returning to the caller on error.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index 8d4799433..5f386cab5 100644
--- a/lib/db-ctl-base.c
+++ b/lib/db-ctl-base.c
@@ -
Prepare for the command handlers (pre_cmd_*() cmd_*() functions) to
report errors by storing them in the context.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 22 ++
lib/db-ctl-base.h | 3 +++
ovn/utilities/ovn-nbctl.c | 9 +
ovn/utilities/o
Propagate the error via the context for the caller to handle it.
Result of applying the following semantic patch (Coccinelle):
@@
expression s;
@@
- die_if_error(s);
+ ctx->error = s;
+ if (ctx->error) {
+ return;
+ }
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 294 ++
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 19 ++-
lib/db-ctl-base.h | 5 ++---
ovn/utilities/ov
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/lib/db
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Also, we no longer return the column as it was not used by any of
existing callers.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 2
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 33 -
1 file changed, 20 insertions(+), 13 deletions(-
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/li
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 49 +++
lib/db-ctl-base.h
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 24
1 file changed, 16 insertions(+), 8 deletions(-)
diff --
Signal that multiple rows match the record identifier via a new output
parameter instead of reporting the problem and dying, so that the caller
can handle the error without terminating the process if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 23 ++-
1 file
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 61 +--
1 file changed, 41 inserti
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 17 ++---
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/l
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 51 +--
1 file changed, 29 insertions(
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Also, rename the function as it is no longer a typical predicate, so
that the users don't assume that the result is passed in return value.
S
Return the error message to the caller instead of reporting it and dying
so that the caller can handle the error without terminating the process
if needed.
Signed-off-by: Jakub Sitnicki
---
lib/db-ctl-base.c | 19 +--
1 file changed, 13 insertions(+), 6 deletions(-)
diff --git a
When adding TC rules we save the prio so can reuse same prio
for same mask since different mask will have to use different prio.
The multi mask per prio probe broke this by using a prio but
get_prio_for_tc_flower() didn't know about it.
Also multi mask per prio support changes the hash calculation.
Hi，by our talking before the memory issue
,could you provide the fixed patch?
孙文杰 于2018年6月19日 周二10:25写道：
> We use ovs 2.9.0.
>
> Ben Pfaff 于2018年6月16日周六 上午2:24写道：
>
>> [Dropping invalid email address findtheonly...@example.com.]
>>
>> Thanks.
>>
>> I believe that we've found and fixed related me
51 matches
Mail list logo
