Hi Team,
could anyone please suggest me how to make my sch_taprio scheduling
algorithm work with OVS switch? any changes in code is required or any
specific configuration?
Thanks
vikash
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.o
-Original Message-
From: Ilya Maximets
Sent: 09 August 2019 17:44
To: ovs-dev@openvswitch.org
Cc: Ian Stokes ; Kevin Traynor ;
David Marchand ; Sriram Vatala
; Ilya Maximets
Subject: [PATCH 2/2] netdev-dpdk: Refactor vhost custom stats for
extensibility.
vHost interfaces currently ha
On 19.08.2019 12:39, Sriram Vatala wrote:
>
> -Original Message-
> From: Ilya Maximets
> Sent: 09 August 2019 17:44
> To: ovs-dev@openvswitch.org
> Cc: Ian Stokes ; Kevin Traynor ;
> David Marchand ; Sriram Vatala
> ; Ilya Maximets
> Subject: [PATCH 2/2] netdev-dpdk: Refactor vhost cus
There is a big code duplication issue with DPDK xstats that led to
missed "rx_oversize_errors" statistics. It's defined but not used.
Fix that by actually using this stat along with code refactoring that
will allow us to not make same mistakes in the future.
Macro definitions are perfectly suitable
Version 2:
* Fixed incorrect handling of vhost stats if there are more than
one of them.
Ilya Maximets (2):
netdev-dpdk: Fix not reporting rx_oversize_errors in stats.
netdev-dpdk: Refactor vhost custom stats for extensibility.
lib/netdev-dpdk.c | 130 ++
vHost interfaces currently has only one custom statistic, but there
might be others in the near future. This refactoring makes the code
work in the same way as it done for dpdk and afxdp stats to keep the
common style over the different code places and makes it easily
extensible for the new stats a
On Tue, Aug 13, 2019 at 6:28 PM wrote:
>
> From: Numan Siddique
>
> Until now, OVN uses the openvswitch rundirs (rundir, logdir, etcdir).
> The commit [1] changed the package name from openvswitch to ovn, but
> it didn't take into the account the effects of it. When "make install"
> is run ovn-ct
Bleep bloop. Greetings Mark Michelson, 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 Mark Michelson needs to sign off.
Lines checked: 29, Warnings: 0, Errors: 1
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: Unexpected sign-offs from developers who are not authors or co-authors
or committers:
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 lacks whitespace around operator
#249 FILE: utilities/ovn-nbctl.c:670:
lrp-set-
Introduce the capability to associate a meter to each controller event
type in order to not overload the pinctrl thread under heavy load.
For each event type we need to define a naming convention for the related
meter:
- empty_lb_backends: event-elb
Changes since v2:
- fold patch 'Repair memory le
From: Mark Michelson
From: Mark Michelson
Controller event action is leaking its genopts. This corrects the error.
Signed-off-by: Mark Michelson
Signed-off-by: Lorenzo Bianconi
---
lib/actions.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lib/actions.c b/lib/action
Introduce meter support to trigger_event action in order to not
overload pinctrl thread under heavy load
Signed-off-by: Lorenzo Bianconi
---
include/ovn/actions.h | 1 +
lib/actions.c | 43 +--
2 files changed, 38 insertions(+), 6 deletions(-)
di
Introduce the capability to associate a meter to each controller event
type in order to not overload the pinctrl thread under heavy load.
Each event type relies on a meter with a defined name:
- empty_lb_backends: event-elb
Acked-by: Mark Michelson
Signed-off-by: Lorenzo Bianconi
---
northd/ovn
Bleep bloop. Greetings Dumitru Ceara, 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:
Failed to merge in the changes.
Patch failed at 0001 ovn-northd: Add IGMP Relay support
The copy o
Bleep bloop. Greetings Dumitru Ceara, 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: Unexpected sign-offs from developers who are not authors or co-authors
or committers
Thanks for the patch
On Sun, Aug 18, 2019 at 11:01 PM Li RongQing wrote:
> the result of extract_l3_ipv4/6 should be checked in reverse_nat_packet
> when it is false, meaning this packet is wrong, should not do handle it
> continually
>
> Signed-off-by: Li RongQing
> ---
> lib/conntrack.c | 17
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 lacks whitespace around operator
#249 FILE: utilities/ovn-nbctl.c:670:
lrp-set-
Bleep bloop. Greetings Aliasgar Ginwala, 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 Aliasgar Ginwala needs to sign off.
WARNING: Unexpected sign-offs from devel
From: Han Zhou
Fixes: commit 5a9b53a5 ("ovsdb raft: Fix duplicated transaction execution when
leader failover.")
Signed-off-by: Han Zhou
---
ovsdb/raft.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ovsdb/raft.c b/ovsdb/raft.c
index c60ef41..1c38b3b 100644
--- a/ovsdb/ra
From: Han Zhou
When clustered mode is used, the client needs to retry connecting
to new servers when certain failures happen. Today it is allowed to
retry new connection only if multiple remotes are used, which prevents
using LB VIP with clustered nodes. This patch makes sure the retry
logic work
From: Han Zhou
As mentioned in RAFT paper, section 6.2:
Leaders: A server might be in the leader state, but if it isn’t the current
leader, it could be needlessly delaying client requests. For example, suppose a
leader is partitioned from the rest of the cluster, but it can still
communicate wit
From: Han Zhou
candiate_retrying is used to determine if the current node is disconnected
from the cluster when the node is in candiate role. However, a node
can flap between candidate and follower role before a leader is elected
when majority of the cluster is down, so is_connected() will flap,
From: Han Zhou
A new unixctl command cluster/change-election-timer is implemented to
change leader election timeout base value according to the scale needs.
The change takes effect upon consensus of the cluster, implemented through
the append-request RPC. A new field "election-timer" is added t
Bleep bloop. Greetings Aliasgar Ginwala, 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 Aliasgar Ginwala needs to sign off.
WARNING: Unexpected sign-offs from devel
I just sent V3 of the series which mainly adds the 5th patch to the series
for changing election timer online. In addition, there are just minor fixes
in test cases.
https://patchwork.ozlabs.org/project/openvswitch/list/?series=126000
I hope at least 1/5 - 4/5 can get in branch 2.12. 5/5 is optio
On Fri, Aug 16, 2019 at 10:46 PM Mark Michelson wrote:
> On 8/13/19 12:28 PM, nusid...@redhat.com wrote:
> > From: Numan Siddique
> >
> > "make rpm-fedora" is broken and this patch fixes it. Previous patch
> > in this series supported building OVN from external OVS sources.
> >
> > Before runnin
On Fri, Aug 16, 2019 at 5:07 PM Darrell Ball wrote:
>
> Thanks for the patch
>
> Pls let me know if this incremental works for you.
> Main change is logging fix for timeout policy deletion.
>
> Darrell
>
> diff --git a/lib/dpif-netlink.c b/lib/dpif-netlink.c
> index 1d4ee60..00d957b 100644
> --- a
Bleep bloop. Greetings Lorenzo Bianconi, 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: Unexpected sign-offs from developers who are not authors or co-authors
or committ
On Fri, Aug 16, 2019 at 5:10 PM Darrell Ball wrote:
>
> Thanks for the patch
>
> Pls let me know if the following incremental works for you.
>
> diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> index 244155a..cb8b51e 100644
> --- a/ofproto/ofproto-dpif.c
> +++ b/ofproto/ofproto-dpif.
From: Numan Siddique
This patch series adds support for building OVN from external OVS
sources.
The first patch adds the support to compile OVN from external OVS sources.
The following configuration options are added when configuring OVN
* --with-ovs-source (mandatory)
* --with-ovs-build (op
From: Numan Siddique
With this patch we have to configure OVN to refer to external OVS source/build
directory instead of the ovs subtree.
The new configuration options added are:
* --with-ovs-source=/path/to/ovs/source/dir
* --with-ovs-build=/path/to/ovs/build/dir
Before configuring OVN, user
From: Numan Siddique
Until now, OVN uses the openvswitch rundirs (rundir, logdir, etcdir).
The commit [1] changed the package name from openvswitch to ovn, but
it didn't take into the account the effects of it. When "make install"
is run ovn-ctl utility is copied to /usr/local/share/ovn/scripts f
From: Numan Siddique
"make rpm-fedora" is broken and this patch fixes it. Previous patch
in this series supported building OVN from external OVS sources.
Before running "make rpm-fedora", it is expected that the developer has run
"make dist" in the OVS source folder to generate the
openvswitch-
From: Numan Siddique
This patch could have created a new user 'ovn' for ovn services instead
of using 'openvswitch' user. But this would require some amount of work and
proper testing since the new user 'ovn' should be part of 'openvswitch'
group (to access /var/run/openvswitch/db.sock.). If ovs
On Fri, Aug 16, 2019 at 10:45 PM Mark Michelson wrote:
> On 8/13/19 12:27 PM, nusid...@redhat.com wrote:
> > From: Numan Siddique
> >
> > Until now, OVN uses the openvswitch rundirs (rundir, logdir, etcdir).
> > The commit [1] changed the package name from openvswitch to ovn, but
> > it didn't t
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.
checkpatch:
WARNING: Line is 81 characters long (recommended limit is 79)
#141 FILE: .travis/osx-build.sh
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.
checkpatch:
WARNING: Line is 99 characters long (recommended limit is 79)
#2123 FILE: utilities/ovn-ctl:3
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.
checkpatch:
WARNING: Line is 108 characters long (recommended limit is 79)
#284 FILE: rhel/usr_lib_system
On Mon, Aug 19, 2019 at 10:52 AM Yi-Hung Wei wrote:
> On Fri, Aug 16, 2019 at 5:07 PM Darrell Ball wrote:
> >
> > Thanks for the patch
> >
> > Pls let me know if this incremental works for you.
> > Main change is logging fix for timeout policy deletion.
> >
> > Darrell
> >
> > diff --git a/lib/d
On Mon, Aug 19, 2019 at 11:12 AM Yi-Hung Wei wrote:
> On Fri, Aug 16, 2019 at 5:10 PM Darrell Ball wrote:
> >
> > Thanks for the patch
> >
> > Pls let me know if the following incremental works for you.
> >
> > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> > index 244155a..cb8b5
From: Han Zhou
When a client uses monitor-cond-since with a non-zero last-id but the
server is not in cluster mode for the DB being monitored, it leads to
segmentation fault because the txn_history list is not initialized in
this case.
Program terminated with signal SIGSEGV, Segmentation fault.
Current code of create-cluster from standalone db takes backup of existing
standalone db and then generates a new clustered dbs from backup dbs. Hence,
during migration if nb and sb dbs are still present, create-cluster will fail
saying file exists and will not really convert dbs to clustered dbs
Es, para mí, un placer poder invitarte a nuestro Curso en Línea "¿Cómo realizar
una DNC efectiva?", que se estará llevando a cabo el
día Jueves 05 de Septiembre con un horario de 10:00 a 14:00 hrs. (hora del
centro de México)
El Diagnóstico de las Necesidades de Capacitación (DNC) es un proceso
Bleep bloop. Greetings Aliasgar Ginwala, 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 Aliasgar Ginwala needs to sign off.
WARNING: Unexpected sign-offs from devel
On Mon, Aug 19, 2019 at 12:42 PM Darrell Ball wrote:
>
>
> On Mon, Aug 19, 2019 at 10:52 AM Yi-Hung Wei wrote:
>
>> On Fri, Aug 16, 2019 at 5:07 PM Darrell Ball wrote:
>> >
>> > Thanks for the patch
>> >
>> > Pls let me know if this incremental works for you.
>> > Main change is logging fix for
46 matches
Mail list logo