[ovs-dev] [PATCH ovn 3/6] northd: Add support for Logical Datapath Groups.

2020-11-13 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- lib/ovn-util.c | 14 +--- lib/ovn-util.h | 3 +- northd/ovn-northd.c | 166 ++-- 3 files changed, 133 insertions(+), 50 deletions(-) diff --git a/lib/ovn-util.c b/lib/ovn-util.c index 18aac8da3..a7fdd431d 100

[ovs-dev] [PATCH ovn 5/6] utilities: Add support for Logical Datapath Groups.

2020-11-13 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- utilities/ovn-sbctl.c | 57 ++- utilities/ovn-trace.c | 35 ++ 2 files changed, 65 insertions(+), 27 deletions(-) diff --git a/utilities/ovn-sbctl.c b/utilities/ovn-sbctl.c index 85e448ec0..bd8c00918

[ovs-dev] [PATCH ovn 6/6] northd: Use same Logical Datapath Group for different flows.

2020-11-13 Thread Ilya Maximets
This significantly reduces space wasted on storing duplicated datapath groups. Signed-off-by: Ilya Maximets --- northd/ovn-northd.c | 102 ++-- 1 file changed, 88 insertions(+), 14 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index

[ovs-dev] [PATCH ovn 0/6] Combine Logical Flows by Logical Datapath.

2020-11-13 Thread Ilya Maximets
This series is aimed to reduce size of a Southbound DB. In modern OVN cluster there is a big probability that same logical flows are installed for several logical switches. Currently, OVN will create a separate logical flow for each logical switch (datapath) filled with exact same match, actions a

[ovs-dev] [PATCH ovn 4/6] controller: Add support for Logical Datapath Groups.

2020-11-13 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- controller/lflow.c | 105 +++- controller/lflow.h | 1 - controller/ovn-controller.c | 23 ++-- 3 files changed, 73 insertions(+), 56 deletions(-) diff --git a/controller/lflow.c b/controller/lflow.c index

[ovs-dev] [PATCH ovn 2/6] ovn-sb.ovsschema: Add Logical Datapath Groups.

2020-11-13 Thread Ilya Maximets
New table 'Logical_Datapath_Group'. Each row of this table will store a set of logical datapaths. Each logical flow now has a reference to a logical datapath group, so the same logical flow could be applied to many different logical datapaths in a group. Signed-off-by: Ilya Maximets --- ovn-sb

[ovs-dev] [PATCH ovn 1/6] tests: Sort flow and database dumps.

2020-11-13 Thread Ilya Maximets
The order in which flows appears in the dump is not strictly determined. Need to sort the output to make tests independent from the system and implementation details. Signed-off-by: Ilya Maximets --- tests/ovn-northd.at | 6 +++--- tests/ovn.at| 2 +- 2 files changed, 4 insertions(+), 4

[ovs-dev] Zoom para reuniones y clases

2020-11-13 Thread Desde cero hasta ser un experto
Buenas tardes Quise aprovechar la oportunidad de hacerte una invitación para tomar nuestro curso: Nombre: Uso de Zoom para reuniones y clases virtuales Horario: de 16:00 am a 19:00 pm. ¿Cuándo?: 31 de Octubre Precio: $ 290,00 + IVA Formato: En línea con interacción en vivo. Lugar: En Vivo desde

Re: [ovs-dev] Bug#974588: openvswitch: DPDK 20.11 support and transition for bullseye

2020-11-13 Thread Thomas Goirand
On 11/13/20 6:54 PM, Ilya Maximets wrote: > On 11/13/20 1:47 PM, Thomas Goirand wrote: >> On 11/12/20 5:09 PM, Luca Boccassi wrote: >>> Source: openvswitch >>> Version: 2.13.0+dfsg1-12 >>> Severity: normal >>> X-Debbugs-CC: pkg-dpdk-de...@lists.alioth.debian.org >>> christian.ehrha...@canonical.co

[ovs-dev] [PATCH v3] net: openvswitch: use core API to update/provide stats

2020-11-13 Thread Lev Stipakov
Commit d3fd65484c781 ("net: core: add dev_sw_netstats_tx_add") has added function "dev_sw_netstats_tx_add()" to update net device per-cpu TX stats. Use this function instead of own code. While on it, remove internal_get_stats() and replace it with dev_get_tstats64(). Signed-off-by: Lev Stipakov

Re: [ovs-dev] [PATCH v4 ovn 1/2] northd: Fair ACL log meters.

2020-11-13 Thread 0-day Robot
Bleep bloop. Greetings Flavio Fernandes, 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. git-am: error: Failed to merge in the changes. hint: Use 'git am --show-current-patch' to see the faile

[ovs-dev] [PATCH v4 ovn 2/2] northd: Fair ACL log meters.

2020-11-13 Thread Flavio Fernandes
ddlog for ACL log meters. Implement fair meters for acl logs in ovn_northd.dl. Enable fair Meter test to also exercise ovn-northd-ddlog. This is a small variation of blp's implementation of ddlog for ACL log meters, now using the northbound schema changes [1]. The changes address overall design i

[ovs-dev] [PATCH v4 ovn 0/2] northd: Fair ACL log meters.

2020-11-13 Thread Flavio Fernandes
Using meters is a great way to keep the ovn-controllers from getting overwhelmed with ACL log events. Since multiple ACL rows with logging enabled can refer to the same meter name, I ran a little experiment to better understand how that behaves [1]. >From that experiment, we see that a 'noisy' ACL

Re: [ovs-dev] [PATCH V2 2/3] compat: Fix build issue on RHEL 7.7

2020-11-13 Thread Yi-Hung Wei
On Thu, Nov 12, 2020 at 3:10 PM Greg Rose wrote: > > RHEL 7.2 introduced a KABI fixup in struct sk_buff for the name > change of l4_rxhash to l4_hash. Then patch > 9ba57fc7 ("datapath: Add hash info to upcall") introduced a > compile error by using l4_hash and not fixing up the HAVE_L4_RXHAS

Re: [ovs-dev] [PATCH V2 3/3] compat: Fix compile warning

2020-11-13 Thread Yi-Hung Wei
On Thu, Nov 12, 2020 at 3:10 PM Greg Rose wrote: > > In ../compat/nf_conntrack_reasm.c nf_frags_cache_name is declared > if OVS_NF_DEFRAG6_BACKPORT is defined. However, later in the patch > it is only used if HAVE_INET_FRAGS_WITH_FRAGS_WORK is defined and > HAVE_INET_FRAGS_RND is not defined. Th

Re: [ovs-dev] [PATCH V2 1/3] compat: Remove stale code

2020-11-13 Thread Yi-Hung Wei
On Thu, Nov 12, 2020 at 3:10 PM Greg Rose wrote: > > Remove stale and unused code left over after support for kernels > older than 3.10 was removed. > > Fixes: 8063e0958780 ("datapath: Drop support for kernel older than 3.10") > Signed-off-by: Greg Rose > > --- > V2 Separate out stale code remova

[ovs-dev] [PATCH ovn] Provide the option to pin ovn-controller and ovn-northd to a specific version.

2020-11-13 Thread numans
From: Numan Siddique OVN recommends updating/upgrading ovn-controllers first and then ovn-northd and OVN DB ovsdb-servers. This is to ensure that new any functionality specified by the database or logical flows created by ovn-northd is understood by ovn-controller. However certain deployments ma

Re: [ovs-dev] Bug#974588: openvswitch: DPDK 20.11 support and transition for bullseye

2020-11-13 Thread Ilya Maximets
On 11/13/20 1:47 PM, Thomas Goirand wrote: > On 11/12/20 5:09 PM, Luca Boccassi wrote: >> Source: openvswitch >> Version: 2.13.0+dfsg1-12 >> Severity: normal >> X-Debbugs-CC: pkg-dpdk-de...@lists.alioth.debian.org >> christian.ehrha...@canonical.com >> Tags: bullseye >> >> Dear Openvswitch Maintai

[ovs-dev] Excel - Estados Financieros

2020-11-13 Thread BUEN FIN
Webinar en Vivo: Taller de elaboración de estados financieros con Excel. Viernes 27 de Noviembre - Horario de 10:00 a 17:00 Hrs. BUEN FIN: 3 x 2 en todos nuestros cursos. Promoción válida durante noviembre y diciembre 202 así como enero 2021 El objetivo es que el participante cuente con las he

Re: [ovs-dev] [PATCH] odp-util: Fix overflow of nested netlink attributes.

2020-11-13 Thread Flavio Leitner
Hi Ilya, On Mon, Oct 19, 2020 at 10:03:00PM +0200, Ilya Maximets wrote: > Length of nested attributes must be checked before storing to the > header. If current length exceeds the maximum value parsing should > fail, otherwise the length value will be truncated leading to > corrupted netlink me

Re: [ovs-dev] [PATCH v2] ovsdb: raft: add some debugging information to cluster/status command

2020-11-13 Thread Dumitru Ceara
On 9/3/20 12:10 PM, Lorenzo Bianconi wrote: > Introduce the following info useful for cluster debugging to > cluster/status command: > - time elapsed from last start/complete election > - election trigger (e.g. timeout) > - number of disconnections > - time elapsed from last raft messaged received

Re: [ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread Ilya Maximets
Cc: netdev On 11/13/20 3:28 PM, Eelco Chaudron wrote: > > > On 13 Nov 2020, at 13:06, Ilya Maximets wrote: > >> On 11/13/20 11:04 AM, Eelco Chaudron wrote: >>> Add support for the dec_ttl action. Instead of programming the datapath with >>> a flow that matches the packet TTL and an IP set, use

Re: [ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread Eelco Chaudron
On 13 Nov 2020, at 13:53, Ilya Maximets wrote: On 11/13/20 11:04 AM, Eelco Chaudron wrote: Add support for the dec_ttl action. Instead of programming the datapath with a flow that matches the packet TTL and an IP set, use a single dec_ttl action. The old behavior is kept if the new action

Re: [ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread Matteo Croce
On Fri, Nov 13, 2020 at 3:28 PM Eelco Chaudron wrote: > > It is already in the mainline kernel, so changing it now would break the > UAPI. Don't think this is allowed from the kernel side. > > > What I'm suggesting is to send a bugfix to kernel > > to accept only format with nested OVS_DEC_TTL_ATT

Re: [ovs-dev] Bug#974588: openvswitch: DPDK 20.11 support and transition for bullseye

2020-11-13 Thread Luca Boccassi
Control: clone -1 -2 Control: retitle -2 transition: dpdk Control: reassign -2 release.debian.org Control: affects -2 -1 On Fri, 2020-11-13 at 13:47 +0100, Thomas Goirand wrote: > On 11/12/20 5:09 PM, Luca Boccassi wrote: > > Source: openvswitch > > Version: 2.13.0+dfsg1-12 > > Severity: normal >

Re: [ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread Eelco Chaudron
On 13 Nov 2020, at 13:06, Ilya Maximets wrote: On 11/13/20 11:04 AM, Eelco Chaudron wrote: Add support for the dec_ttl action. Instead of programming the datapath with a flow that matches the packet TTL and an IP set, use a single dec_ttl action. The old behavior is kept if the new action

Re: [ovs-dev] [PATCH] Fix SHA-1 algorithm for data bigger than 512 megabytes.

2020-11-13 Thread Ilya Maximets
On 11/12/20 4:51 PM, Renat Nurgaliyev wrote: > Missing cast to 32 bits in overflow check makes this conditional never > be true. Because of this, computation of SHA-1 checksum will be wrong > for any data that is bigger than 512 megabytes, which in bits is the > boundary of 32 bits integer. > > In

Re: [ovs-dev] Bug#974588: openvswitch: DPDK 20.11 support and transition for bullseye

2020-11-13 Thread Thomas Goirand
On 11/12/20 5:09 PM, Luca Boccassi wrote: > Source: openvswitch > Version: 2.13.0+dfsg1-12 > Severity: normal > X-Debbugs-CC: pkg-dpdk-de...@lists.alioth.debian.org > christian.ehrha...@canonical.com > Tags: bullseye > > Dear Openvswitch Maintainers, > > We are scoping the src:dpdk 19.11 -> 20.1

Re: [ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread Ilya Maximets
On 11/13/20 11:04 AM, Eelco Chaudron wrote: > Add support for the dec_ttl action. Instead of programming the datapath with > a flow that matches the packet TTL and an IP set, use a single dec_ttl action. > > The old behavior is kept if the new action is not supported by the datapath. > > # ovs-

Re: [ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread Ilya Maximets
On 11/13/20 11:04 AM, Eelco Chaudron wrote: > Add support for the dec_ttl action. Instead of programming the datapath with > a flow that matches the packet TTL and an IP set, use a single dec_ttl action. > > The old behavior is kept if the new action is not supported by the datapath. > > # ovs-

[ovs-dev] KARTE

2020-11-13 Thread Mitnick
Grüße, ich bin Mitnick, ein amerikanischer Berater für Cybersicherheit, Autor und verurteilter Hacker. Meine Gruppe hat eine Karte generiert, die als geklonte Geldautomatenkarte bekannt ist. Mit dieser Karte können Sie Rechnungen online bezahlen, im Supermarkt verwenden und sogar täglich abheben

Re: [ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread 0-day Robot
Bleep bloop. Greetings Eelco Chaudron, 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_CONFIG_H -I. -I ./include -

Re: [ovs-dev] [PATCH] datapath: Add a new action dec_ttl

2020-11-13 Thread 0-day Robot
Bleep bloop. Greetings Eelco Chaudron, 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_CONFIG_H -I. -I ./include -

Re: [ovs-dev] [PATCH v2 2/3] net: openvswitch: use core API to update/provide stats

2020-11-13 Thread Heiner Kallweit
Am 13.11.2020 um 10:02 schrieb Lev Stipakov: > Commit d3fd65484c781 ("net: core: add dev_sw_netstats_tx_add") has added > function "dev_sw_netstats_tx_add()" to update net device per-cpu TX > stats. > > Use this function instead of own code. > > While on it, remove internal_get_stats() and replac

Re: [ovs-dev] [PATCH] datapath: Add a new action dec_ttl

2020-11-13 Thread Eelco Chaudron
Please ignore this one, as I forgot to add the v2 tag. A new one was sent, and patchwork is updated! //Eelco On 13 Nov 2020, at 10:58, Eelco Chaudron wrote: Add support for the dec_ttl action. Instead of programming the datapath with a flow that matches the packet TTL and an IP set, use a si

[ovs-dev] [PATCH v2] datapath: Add a new action dec_ttl

2020-11-13 Thread Eelco Chaudron
Add support for the dec_ttl action. Instead of programming the datapath with a flow that matches the packet TTL and an IP set, use a single dec_ttl action. The old behavior is kept if the new action is not supported by the datapath. # ovs-ofctl dump-flows br0 cookie=0x0, duration=12.538s, ta

[ovs-dev] [PATCH] datapath: Add a new action dec_ttl

2020-11-13 Thread Eelco Chaudron
Add support for the dec_ttl action. Instead of programming the datapath with a flow that matches the packet TTL and an IP set, use a single dec_ttl action. The old behavior is kept if the new action is not supported by the datapath. # ovs-ofctl dump-flows br0 cookie=0x0, duration=12.538s, ta

[ovs-dev] [PATCH v2 2/3] net: openvswitch: use core API to update/provide stats

2020-11-13 Thread Lev Stipakov
Commit d3fd65484c781 ("net: core: add dev_sw_netstats_tx_add") has added function "dev_sw_netstats_tx_add()" to update net device per-cpu TX stats. Use this function instead of own code. While on it, remove internal_get_stats() and replace it with dev_get_tstats64(). Signed-off-by: Lev Stipakov