Hi All,
Please review the update patch.
Changes since v10 :
Corrected the spelling mistake in the commit message.
Thanks & Regards,
Sriram.
-Original Message-
From: Sriram Vatala
Sent: 29 October 2019 20:20
To: ovs-dev@openvswitch.org; ktray...@redhat.com; i.maxim...@ovn.org
Cc: Ilya M
Sometimes, analysing the drop statistics of the ports
will be helpful in debugging. This patch adds script
to collect all supported port stats which also includes
the drop counters in userspace datapath. The output of
this script is included in the bugtool output.
Signed-off-by: Sriram Vatala
---
OVS may be unable to transmit packets for multiple reasons on
the userspace datapath and today there is a single counter to
track packets dropped due to any of those reasons. This patch
adds custom software stats for the different reasons packets
may be dropped during tx/rx on the userspace datapat
From: Ilya Maximets
This is yet another refactoring for upcoming detailed drop stats.
It allows to use single function for all the software calculated
statistics in netdev-dpdk for both vhost and ETH ports.
UINT64_MAX used as a marker for non-supported statistics in a
same way as it's done in br
Thanks Kevin for reviewing and acknowledging the patch. I will correct the
spelling and send the updated patch.
Thanks & Regards,
Sriram.
-Original Message-
From: Kevin Traynor
Sent: 25 October 2019 20:36
To: Sriram Vatala ; ovs-dev@openvswitch.org;
i.maxim...@ovn.org
Cc: Ilya Maximets
Sending again with a meaningful subject:)
Dear ovs-dev,
I'm trying to see if I can increase the number of flows supported in
kernel cache. I found two relevant config options
(other_config:flow-limit and upcall/flow limit). My understanding was
that other_config:flow-limit is for the normal openf
Date: Sat, 19 Oct 2019 16:08:34 +0800
The date on your postings is in the past.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Dear ovs-dev,
I'm trying to see if I can increase the number of flows supported in
kernel cache. I found two relevant config options
(other_config:flow-limit and upcall/flow limit). My understanding was
that other_config:flow-limit is for the normal openflow flows while
upcall is for the kernel fl
From: Tonghao Zhang
use the specified functions to init resource.
Signed-off-by: Tonghao Zhang
Tested-by: Greg Rose
---
net/openvswitch/datapath.c | 60 +-
1 file changed, 38 insertions(+), 22 deletions(-)
diff --git a/net/openvswitch/datapath.c b/
From: Tonghao Zhang
When we destroy the flow tables which may contain the flow_mask,
so release the flow mask struct.
Signed-off-by: Tonghao Zhang
Tested-by: Greg Rose
---
net/openvswitch/flow_table.c | 185 +++
1 file changed, 97 insertions(+), 88 dele
From: Tonghao Zhang
Unlocking of a not locked mutex is not allowed.
Other kernel thread may be in critical section while
we unlock it because of setting user_feature fail.
Fixes: 95a7233c4 ("net: openvswitch: Set OvS recirc_id from tc chain index")
Cc: Paul Blakey
Signed-off-by: Tonghao Zhang
From: Tonghao Zhang
Simplify the code and remove the unnecessary BUILD_BUG_ON.
Signed-off-by: Tonghao Zhang
Tested-by: Greg Rose
Acked-by: William Tu
---
net/openvswitch/flow_table.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/net/openvswitch/flow_table.c b/ne
From: Tonghao Zhang
The most case *index < ma->max, and flow-mask is not NULL.
We add un/likely for performance.
Signed-off-by: Tonghao Zhang
Tested-by: Greg Rose
Acked-by: William Tu
---
net/openvswitch/flow_table.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net
From: Tonghao Zhang
Port the codes to linux upstream and with little changes.
Pravin B Shelar, says:
| In case hash collision on mask cache, OVS does extra flow
| lookup. Following patch avoid it.
Link:
https://github.com/openvswitch/ovs/commit/0e6efbe2712da03522532dc5e84806a96f6a0dd1
Signed-o
From: Tonghao Zhang
The full looking up on flow table traverses all mask array.
If mask-array is too large, the number of invalid flow-mask
increase, performance will be drop.
One bad case, for example: M means flow-mask is valid and NULL
of flow-mask means deleted.
+---
From: Tonghao Zhang
When creating and inserting flow-mask, if there is no available
flow-mask, we realloc the mask array. When removing flow-mask,
if necessary, we shrink mask array.
Signed-off-by: Tonghao Zhang
Tested-by: Greg Rose
Acked-by: William Tu
---
net/openvswitch/flow_table.c | 33
From: Tonghao Zhang
Port the codes to linux upstream and with little changes.
Pravin B Shelar, says:
| mask caches index of mask in mask_list. On packet recv OVS
| need to traverse mask-list to get cached mask. Therefore array
| is better for retrieving cached mask. This also allows better
| cac
From: Tonghao Zhang
This series patch optimize openvswitch for performance or simplify
codes.
Patch 1, 2, 4: Port Pravin B Shelar patches to
linux upstream with little changes.
Patch 5, 6, 7: Optimize the flow looking up and
simplify the flow hash.
Patch 8, 9: are bugfix.
The performance test
From: Tonghao Zhang
The idea of this optimization comes from a patch which
is committed in 2014, openvswitch community. The author
is Pravin B Shelar. In order to get high performance, I
implement it again. Later patches will use it.
Pravin B Shelar, says:
| On every packet OVS needs to lookup f
Hi all:
I have been testing OVS-DPDK Rate limit using meter table these days, but I
have found the rate deviation is larger than kernel meter. My test result is as
below:
case 1. No Rate limit : Iperf between two VMs(on the same compute node) TCP
bandwidth is above 1G
case 2. Rate lim
Hi Eelco,
Thanks for your review.
On Tue, Oct 29, 2019 at 8:28 AM Eelco Chaudron wrote:
>
> See comments inline…
>
> On 5 Oct 2019, at 2:12, William Tu wrote:
>
> > Now netdev-afxdp always forwards all packets to userspace because
> > it is using libbpf's default XDP program, see 'xsk_load_xdp_p
Bleep bloop. Greetings Russell Bryant, 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 82 characters long (recommended limit is 79)
#86 FILE: northd/ovn-northd.c:6
Bleep bloop. Greetings Russell Bryant, 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)
#154 FILE: utilities/ovn-nbctl.
Bleep bloop. Greetings Russell Bryant, 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 85 characters long (recommended limit is 79)
#172 FILE: utilities/ovn-trace.
These tests failed prior to the changes leading up to this one.
Signed-off-by: Russell Bryant
---
tests/system-ovn.at | 862 +++-
1 file changed, 860 insertions(+), 2 deletions(-)
diff --git a/tests/system-ovn.at b/tests/system-ovn.at
index f88ad31e4..b3f
Signed-off-by: Russell Bryant
---
tests/ovn-nbctl.at| 41
utilities/ovn-nbctl.c | 48 ---
2 files changed, 68 insertions(+), 21 deletions(-)
diff --git a/tests/ovn-nbctl.at b/tests/ovn-nbctl.at
index 01091dd99..43a9
Signed-off-by: Russell Bryant
---
NEWS | 6 ++
1 file changed, 6 insertions(+)
diff --git a/NEWS b/NEWS
index 73045d65f..ab2f13318 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,9 @@
+Post-OVS-v2.12.0
+-
+ - OVN was split out from the OVS repository and is now released
+ i
Signed-off-by: Russell Bryant
---
northd/ovn-northd.c | 376
1 file changed, 278 insertions(+), 98 deletions(-)
diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c
index ae81a6944..a10017ba1 100644
--- a/northd/ovn-northd.c
+++ b/northd/ovn-northd.
I noticed that this table number was outdated. This is now table 3.
There are a few other sections of code for this table that were all
correctly referencing table 3.
Signed-off-by: Russell Bryant
---
northd/ovn-northd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/northd
Signed-off-by: Russell Bryant
---
include/ovn/actions.h | 6 +-
lib/actions.c | 35 +++
tests/ovn.at | 18 --
utilities/ovn-trace.c | 14 +-
4 files changed, 53 insertions(+), 20 deletions(-)
diff --git a/include/o
This came up with reviewing the usage of ovn-kubernetes with IPv6.
It's more straight forward to start with using IPv6 NAT, matching the
IPv4 network topology with Kubernetes. Eventually, we'd want to allow
routable IPv6 addresses everywhere, but this gives us the option of
using NAT where it's he
Thanks Han for the patches. OVN-ctl doesn't seem to accept inb/sb args.
On Sun, Oct 20, 2019 at 5:55 PM Han Zhou wrote:
> Add support for managing IC-NB and IC-SB DBs, and ovn-ic daemon.
>
> Signed-off-by: Han Zhou
> ---
> utilities/ovn-ctl | 362
> +++
Thanks Han for the patches.
On Sun, Oct 20, 2019 at 5:54 PM Han Zhou wrote:
> This patch introduces interconnection controller, ovn-ic, and
> implements the basic AZ registration feature: taking the AZ
> name from NB DB and create an Availability_Zone entry in
> IC-SB DB.
>
> Signed-off-by: Han
Thanks Han for the correction. Just one more minor typo in the tutorial
below.
I hit some roadblocks to start ic controller on different az but got my
setup running with workarounds in current code and have posted in comments
which needs fix for sure. I tried with 2 different ovn setups with 2 AZs
Bleep bloop. Greetings Ankur Sharma, 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 220 characters long (recommended limit is 79)
#121 FILE: utilities/ovn-nbctl.8
Good Day,
Please do check our new order list and send quote us.
Get back to us with best price confirmation.
Thanks Regards,
Sui
Purchase Manager,___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
For dnat_and_snat rules which are meant to be stateless
instead of using ct_snat/dnat OVN actions, we will use
ip4.src/ip4.dst.
This actions will do 1:1 mapping to inner ip to external ip,
while recalculating the checksums.
Signed-off-by: Ankur Sharma
---
northd/ovn-northd.8.xml | 34 --
n
Adding ovn-nbctl to mark a dnat_and_snat rule as stateless.
This configuration will added to "options" column of NAT table.
Signed-off-by: Ankur Sharma
---
ovn-nb.ovsschema | 6 --
ovn-nb.xml| 5 +
tests/ovn-nbctl.at| 29 +
u
NAT implementation in OVN uses connection tracker to replace
source and dest ips. This implementation works fine and
is the right approach for cases where external ips are shared
(i.e. SNAT) or where we replace ip only when relevant flow is there
(i.e. DNAT).
However, it opens the possibility of D
Add support for travis-ci ppc64le builds.
- Updated matrix in .travis.yml to include a ppc64le build.
- Move package install needed for 32bit builds to .travis/linux-prepare.sh.
To keep the total build time at an acceptable level only a single build
job is included in the matrix for ppc64le.
A b
On Tue, Oct 29, 2019 at 4:31 AM Tonghao Zhang wrote:
>
> On Tue, Oct 29, 2019 at 3:38 PM Pravin Shelar wrote:
> >
> > On Sun, Oct 27, 2019 at 11:49 PM Tonghao Zhang
> > wrote:
> > >
> > > On Thu, Oct 24, 2019 at 3:14 PM Pravin Shelar wrote:
> > > >
> > > > On Tue, Oct 22, 2019 at 7:35 PM Tongh
26 de Noviembre | Horario de 10:00 a 17:00 hrs. | (hora del centro de México)
- Coaching y liderazgo efectivo. -
¿De qué hablaremos?
Este webinar tiene como finalidad llevar a los participantes a ser capaces de
intervenir positivamente en otros individuos en
funciones de liderazgo, compren
On 29.10.2019 0:32, William Tu wrote:
On Mon, Oct 28, 2019 at 12:11 PM Ilya Maximets wrote:
On 28.10.2019 11:46, Eelco Chaudron wrote:
On 23 Oct 2019, at 23:06, William Tu wrote:
The patch adds support for using need_wakeup flag in AF_XDP rings.
A new option, use_need_wakeup, is added. W
On 2019-10-29 09:52, Ilya Maximets wrote:
On 28.10.2019 22:22, David Wilder wrote:
Add support for travis-ci ppc64le builds.
- Updated matrix in .travis.yml to include a ppc64le build.
- Added support to install packages needed by specific architectures.
To keep the total build time at an acce
On 28.10.2019 22:22, David Wilder wrote:
Add support for travis-ci ppc64le builds.
- Updated matrix in .travis.yml to include a ppc64le build.
- Added support to install packages needed by specific architectures.
To keep the total build time at an acceptable level only a single build
job is inc
Me da mucho gusto saludarte.
Es, para mí, un placer poder invitarte a nuestro Curso en Línea "¿Cómo
presentar resultados a nivel profesional?", que se estará llevando a cabo el
día Miércoles 13 de Noviembre con un horario de 10:00 a 14:00 hrs. (hora del
centro de México)
Presentar un proyecto
On 23.10.2019 23:08, William Tu wrote:
The patch detects the numa node id from the name of the netdev,
by reading the '/sys/class/net//device/numa_node'.
If not available, ex: virtual device, or any error happens,
return numa id 0. Currently only the afxdp netdev type uses it,
other linux netdev
On Tue, Oct 29, 2019 at 8:16 PM Russell Bryant wrote:
>
> Thanks, I just hit this and the patch fixes it for me.
>
> Acked-by: Russell Bryant
Thanks for the reviews. I applied this patch to master.
Numan
>
> On Tue, Oct 29, 2019 at 9:56 AM Han Zhou wrote:
> >
> > Acked-by: hz...@ovn.org
> >
>
See comments inline…
On 5 Oct 2019, at 2:12, William Tu wrote:
Now netdev-afxdp always forwards all packets to userspace because
it is using libbpf's default XDP program, see 'xsk_load_xdp_prog'.
There are some cases when users want to keep packets in kernel instead
of sending to userspace, for
The incremental processing engine might stop a run before the
en_runtime_data node is processed. In such cases the ed_runtime_data
fields might contain pointers to already deleted SB records. For
example, if a port binding corresponding to a patch port is removed from
the SB database and the increm
Thanks, I just hit this and the patch fixes it for me.
Acked-by: Russell Bryant
On Tue, Oct 29, 2019 at 9:56 AM Han Zhou wrote:
>
> Acked-by: hz...@ovn.org
>
> On Tue, Oct 29, 2019 at 5:26 AM wrote:
>
> > From: Numan Siddique
> >
> > The commit b740928656a1("testsuite: Use ovn-macros instead
Acked-by: hz...@ovn.org
On Tue, Oct 29, 2019 at 5:26 AM wrote:
> From: Numan Siddique
>
> The commit b740928656a1("testsuite: Use ovn-macros instead of
> ofproto-macros.")
> missed updating the system test suite files to include ovn-macros.at. This
> patch adds it.
>
> CC: Han Zhou
> Signed-of
Could this patch be reviewed please.
Thanks,
Gowrishankar
On Tue, Oct 22, 2019 at 3:06 PM Gowrishankar Muthukrishnan <
gmuth...@redhat.com> wrote:
> There is no log about isolated rxq assignment in a pmd today, which
> sometimes could be useful to trace rxq/pmd pinning, when debugging
> with log
Signed-off-by: Damijan Skvarc
---
ovsdb/execution.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/ovsdb/execution.c b/ovsdb/execution.c
index c55a0b7..f2cf3d7 100644
--- a/ovsdb/execution.c
+++ b/ovsdb/execution.c
@@ -562,7 +562,6 @@ ovsdb_execute_mutate(struct ovsdb_execution *x, struct
On Tue, Oct 29, 2019 at 5:49 PM Lorenzo Bianconi
wrote:
>
> Introduce the possibility to specify a DNSSL option to Router
> Advertisement packets. DNS Search list can be specified using
> 'dnssl' tag in the ipv6_ra_configs column of logical router
> port table
>
> Signed-off-by: Lorenzo Bianconi
On Tue, Oct 29, 2019 at 4:39 PM Lorenzo Bianconi
wrote:
>
> > On Tue, Oct 29, 2019 at 3:29 PM Lorenzo Bianconi
> > wrote:
> > >
> > > Introduce the possibility to specify a DNSSL option to Router
> > > Advertisement packets. DNS Search list can be specified using
> > > 'dnssl' tag in the ipv6_ra_
From: Numan Siddique
The commit b740928656a1("testsuite: Use ovn-macros instead of ofproto-macros.")
missed updating the system test suite files to include ovn-macros.at. This
patch adds it.
CC: Han Zhou
Signed-off-by: Numan Siddique
---
tests/system-kmod-testsuite.at | 1 +
tests/system
Introduce the possibility to specify a DNSSL option to Router
Advertisement packets. DNS Search list can be specified using
'dnssl' tag in the ipv6_ra_configs column of logical router
port table
Signed-off-by: Lorenzo Bianconi
---
Changes since v5:
- add comment describing how domain names are en
On Mon, Oct 28, 2019 at 10:37:44AM +0200, Roi Dayan wrote:
> Add missing slash.
>
> Fixes: 0447019df7c6 ("fedora-spec: added systemd post/postun/pre/preun
> sections")
> Signed-off-by: Roi Dayan
Thanks Roi,
I've applied this to master and backported it as far back as branch-2.5.
> ---
> rhel
On Tue, Oct 29, 2019 at 3:38 PM Pravin Shelar wrote:
>
> On Sun, Oct 27, 2019 at 11:49 PM Tonghao Zhang
> wrote:
> >
> > On Thu, Oct 24, 2019 at 3:14 PM Pravin Shelar wrote:
> > >
> > > On Tue, Oct 22, 2019 at 7:35 PM Tonghao Zhang
> > > wrote:
> > > >
> > > > On Tue, Oct 22, 2019 at 2:58 PM
> -Original Message-
> From: William Tu [mailto:u9012...@gmail.com]
> Sent: Monday, October 28, 2019 10:46 PM
> To: Noa Levy
> Cc: ovs-dev@openvswitch.org; Oz Shlomo ; Majd
> Dibbiny ; Ameer Mahagneh
> ; Eli Britstein
> Subject: Re: [ovs-dev] [PATCH ovs v3 2/2] netdev-dpdk: Add dpdkvdp
> On Tue, Oct 29, 2019 at 3:29 PM Lorenzo Bianconi
> wrote:
> >
> > Introduce the possibility to specify a DNSSL option to Router
> > Advertisement packets. DNS Search list can be specified using
> > 'dnssl' tag in the ipv6_ra_configs column of logical router
> > port table
> >
> > Signed-off-by:
On Tue, Oct 29, 2019 at 3:29 PM Lorenzo Bianconi
wrote:
>
> Introduce the possibility to specify a DNSSL option to Router
> Advertisement packets. DNS Search list can be specified using
> 'dnssl' tag in the ipv6_ra_configs column of logical router
> port table
>
> Signed-off-by: Lorenzo Bianconi
Introduce the possibility to specify a DNSSL option to Router
Advertisement packets. DNS Search list can be specified using
'dnssl' tag in the ipv6_ra_configs column of logical router
port table
Signed-off-by: Lorenzo Bianconi
---
Changes since v4:
- update documentation
- fix possible array out
> -Original Message-
> From: Roni Bar Yanai
> Sent: Monday, October 28, 2019 10:25 PM
> To: Ilya Maximets ; Noa Levy ;
> ovs-dev@openvswitch.org
> Cc: Oz Shlomo ; Majd Dibbiny
> ; Ameer Mahagneh ; Eli
> Britstein ; William Tu ; Simon
> Horman
> Subject: RE: [ovs-dev] [PATCH ovs v3 0/2]
Hi Ben,
Idea is to log lacp specific tx failure. A one step further as in
ofproto_dpif_send_packet has to make the debug info more generic, even
though it would cater all of its calling functions.
For example, bundle_send_learning_packets reports such tx failures for
gratuitous packets. I think, we
On Tue, Oct 29, 2019 at 1:31 PM Numan Siddique wrote:
>
> On Mon, Oct 28, 2019 at 10:57 PM Ben Pfaff wrote:
> >
> > On Mon, Oct 28, 2019 at 10:25:34AM -0700, Ben Pfaff wrote:
> > > On Mon, Oct 28, 2019 at 08:04:15PM +0530, num...@ovn.org wrote:
> > > > From: Numan Siddique
> > > >
> > > > CC: Al
> On Mon, Oct 28, 2019 at 8:10 PM Lorenzo Bianconi
> wrote:
> >
> > Introduce the possibility to specify a DNSSL option to Router
> > Advertisement packets. DNS Search list can be specified using
> > 'dnssl' tag in the ipv6_ra_configs column of logical router
> > port table
> >
> > Signed-off-by:
On Mon, Oct 28, 2019 at 10:57 PM Ben Pfaff wrote:
>
> On Mon, Oct 28, 2019 at 10:25:34AM -0700, Ben Pfaff wrote:
> > On Mon, Oct 28, 2019 at 08:04:15PM +0530, num...@ovn.org wrote:
> > > From: Numan Siddique
> > >
> > > CC: Aliasgar Ginwala
> > > Signed-off-by: Numan Siddique
> >
> > Should we
On Sun, Oct 27, 2019 at 11:49 PM Tonghao Zhang wrote:
>
> On Thu, Oct 24, 2019 at 3:14 PM Pravin Shelar wrote:
> >
> > On Tue, Oct 22, 2019 at 7:35 PM Tonghao Zhang
> > wrote:
> > >
> > > On Tue, Oct 22, 2019 at 2:58 PM Pravin Shelar wrote:
> > > >
> > ...
> >
...
> > > struct sw_flow *ovs_fl
On Mon, Oct 28, 2019 at 8:10 PM Lorenzo Bianconi
wrote:
>
> Introduce the possibility to specify a DNSSL option to Router
> Advertisement packets. DNS Search list can be specified using
> 'dnssl' tag in the ipv6_ra_configs column of logical router
> port table
>
> Signed-off-by: Lorenzo Bianconi
On Mon, Oct 28, 2019 at 8:09 PM Lorenzo Bianconi
wrote:
>
> Introduce the possibility to specify a RDNSS option to Router
> Advertisement packets. DNS IPv6 address can be specified using
> 'rdnss' tag in the ipv6_ra_configs column of logical router
> port table
>
> Acked-by: Mark Michelson
> Sign
72 matches
Mail list logo
