On Thu, Mar 05, 2020 at 07:21:41AM +0100, Damijan Skvarc wrote:
> ovnfield_by_name is hash of strings which is used to quickly find
> field by name. This hash is initialized from ovn_init_symtab(). In case
> the latter function is called multiple times then also ovnfield_by_name is
> initialized mu
Bleep bloop. Greetings Gregory Rose, 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 failed pa
On Fri, Mar 06, 2020 at 02:48:39PM -0800, Han Zhou wrote:
> On Fri, Mar 6, 2020 at 2:39 PM Ben Pfaff wrote:
> >
> > On Thu, Mar 05, 2020 at 11:48:46PM -0800, Han Zhou wrote:
> > > During election, there shouldn't be any leader. This change makes sure
> > > that a server in candidate role always re
On Fri, Mar 06, 2020 at 02:47:50PM -0800, Han Zhou wrote:
> On Fri, Mar 6, 2020 at 2:32 PM Ben Pfaff wrote:
> >
> > On Fri, Feb 28, 2020 at 06:07:10PM -0800, Han Zhou wrote:
> > > When a leader handles install_snapshot reply, the next_index for
> > > the follower should be log_start instead of log
On Wed, Mar 04, 2020 at 05:13:54PM -0800, Jean Tourrilhes wrote:
> Hi all,
>
> I'm facing a puzzling bug [...]
One more detail I did not mention.
The vswitchd log does not show very much. It kinds of confirm
that the packets are processed in software : I see way less u
On 3/6/2020 2:41 PM, Ben Pfaff wrote:
On Fri, Mar 06, 2020 at 02:37:12PM -0800, Greg Rose wrote:
The patch series enables OVS kernel datapath support up to
release 5.5. Updates the faq and NEWS as well.
Passes Travis here:
https://travis-ci.org/gvrose8192/ovs-experimental/builds/658424074
Pas
On Fri, Mar 6, 2020 at 2:39 PM Ben Pfaff wrote:
>
> On Thu, Mar 05, 2020 at 11:48:46PM -0800, Han Zhou wrote:
> > During election, there shouldn't be any leader. This change makes sure
> > that a server in candidate role always report leader as "unknown".
> >
> > Signed-off-by: Han Zhou
>
> Thank
On Fri, Mar 6, 2020 at 2:32 PM Ben Pfaff wrote:
>
> On Fri, Feb 28, 2020 at 06:07:10PM -0800, Han Zhou wrote:
> > When a leader handles install_snapshot reply, the next_index for
> > the follower should be log_start instead of log_end, because there
> > can be new entries added in leader's log aft
Sounds good.
On Fri, Mar 06, 2020 at 10:36:28PM +, Anton Ivanov wrote:
> I have a cleaner version in the works which should be ready some times next
> week.
>
> I will try to release the next version ~ Tue/Wed along with some level of
> async IO as well as reduction of copying in the json rpc
On Fri, Mar 06, 2020 at 02:37:12PM -0800, Greg Rose wrote:
> The patch series enables OVS kernel datapath support up to
> release 5.5. Updates the faq and NEWS as well.
>
> Passes Travis here:
> https://travis-ci.org/gvrose8192/ovs-experimental/builds/658424074
>
> Passes check-kmod with no regr
On Thu, Mar 05, 2020 at 11:48:46PM -0800, Han Zhou wrote:
> During election, there shouldn't be any leader. This change makes sure
> that a server in candidate role always report leader as "unknown".
>
> Signed-off-by: Han Zhou
Thanks for the patches. I applied them to master.
_
Acked-by: Yi-Hung Wei
Signed-off-by: Greg Rose
---
V4 - Fix up the error message indicating that kernels greater than
5.5 are not supported.
---
acinclude.m4 | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/acinclude.m4 b/acinclude.m4
index 9338af9..03d3484 100644
---
From: Pankaj Bharadiya
Upstream commit:
commit c593642c8be046915ca3a4a300243a68077cd207
Author: Pankaj Bharadiya
Date: Mon Dec 9 10:31:43 2019 -0800
treewide: Use sizeof_field() macro
Replace all the occurrences of FIELD_SIZEOF() with sizeof_field() except
at places w
From: Johannes Berg
Upstream commit:
commit 8cb081746c031fb164089322e2336a0bf5b3070c
Author: Johannes Berg
Date: Fri Apr 26 14:07:28 2019 +0200
netlink: make validation more configurable for future strictness
We currently have two levels of strict validation:
1) lib
Since Linux kernel release 5.1 the fragments field of the inet_frag_queue
structure is removed and now only the rb_fragments structure with an
rb_node pointer is used for both ipv4 and ipv6. In addition, the
atomic_sub and atomic_add functions are replaced with their
equivalent long counterparts.
The patch series enables OVS kernel datapath support up to
release 5.5. Updates the faq and NEWS as well.
Passes Travis here:
https://travis-ci.org/gvrose8192/ovs-experimental/builds/658424074
Passes check-kmod with no regressions on Ubuntu 16.04 and Ubuntu 18.04
distros.
Greg Rose (5):
acinc
From: Johannes Berg
Upstream commit:
commit 3b0f31f2b8c9fb348e4530b88f6b64f9621f83d6
Author: Johannes Berg
Date: Thu Mar 21 22:51:02 2019 +0100
genetlink: make policy common to family
Since maxattr is common, the policy can't really differ sanely,
so make it common as
From: "Gustavo A. R. Silva"
Upstream commit:
commit 279badc2a85be83e0187b8c566e3b476b76a87a2
Author: Gustavo A. R. Silva
Date: Thu Oct 19 12:55:03 2017 -0500
openvswitch: conntrack: mark expected switch fall-through
In preparation to enabling -Wimplicit-fallthrough, mark
Flex array support is removed since kernel 5.1. Convert to use
kvmalloc_array instead.
Acked-by: Yi-Hung Wei
Signed-off-by: Greg Rose
---
datapath/linux/compat/stt.c | 28 ++--
1 file changed, 10 insertions(+), 18 deletions(-)
diff --git a/datapath/linux/compat/stt.c b
We are adding support for Linux kernels up to 5.5 so update the
Travis test list, NEWS and FAQ.
Signed-off-by: Greg Rose
---
V4 -- Fix up release note to state the 5.5 support is added to branch
2.14.
---
.travis.yml| 2 +-
Documentation/faq/releases.rst | 1 +
NEWS
Since this commit in the Linux upstream kernel:
'commit 9b9a3f20cbe0 ("kbuild: split final module linking out into
Makefile.modfinal")'
The openvswitch kernel module fails to build against the upstream
Linux kernel. The cause of the build failure is that the include of the
KBUILD_EXTMOD variable w
I have a cleaner version in the works which should be ready some times
next week.
I will try to release the next version ~ Tue/Wed along with some level
of async IO as well as reduction of copying in the json rpc TX path.
A.
On 06/03/2020 21:38, Ben Pfaff wrote:
On Wed, Feb 26, 2020 at 03:2
On Fri, Feb 28, 2020 at 06:07:10PM -0800, Han Zhou wrote:
> When a leader handles install_snapshot reply, the next_index for
> the follower should be log_start instead of log_end, because there
> can be new entries added in leader's log after initiating the
> install_snapshot procedure. Also, it s
On Fri, Feb 28, 2020 at 06:07:09PM -0800, Han Zhou wrote:
> When a follower needs to "catch up", leader can send N entries in
> a single append_request instead of only one entry by each message.
>
> The function raft_send_append_request() already supports this, so
> this patch just calculate the c
On Wed, Feb 12, 2020 at 08:38:51AM +, Vishal Deep Ajmera wrote:
> > To me it looks like we will need an equivalent of flow_wc_map() which is
> > less stricter than
> > flow_wildcards_init_for_packet() if we want to add this check in
> > xlate_wc_finish().
> >
> > Let me know if the current sol
On Wed, Feb 26, 2020 at 03:24:17PM +, anton.iva...@cambridgegreys.com wrote:
> From: Anton Ivanov
>
> 1. Adds "persistent" behaviour where feasible streams.
> These are waited upon in the same thread where they are created. This
> allows them to be registered persistently with the OS (if poss
On Mon, Feb 03, 2020 at 11:32:46AM +0100, Vishal Deep Ajmera via dev wrote:
> Currently fast-failover group does not support checking liveness of controller
> port (OFPP_CONTROLLER). However this feature can be useful for selecting
> alternate pipeline when controller connection itself is down for
On 3/6/2020 11:34 AM, Ben Pfaff wrote:
On Wed, Mar 04, 2020 at 03:04:11PM -0800, Greg Rose wrote:
The patch series enables OVS kernel datapath support up to
release 5.5. Updates the faq and NEWS as well.
I'm happy to apply this in v4 if you'll add the acks.
Sure, will do!
Thanks,
- Greg
__
On Fri, Mar 06, 2020 at 01:07:34PM -0800, Gregory Rose wrote:
> Maintainers, if you could fix that up on commit please let me know,
> otherwise I'm fine with sending an updated patch series.
I'm enthusiatic about seeing a v4 that fixes that and adds the acks.
Thanks,
Ben.
___
On Mon, Mar 02, 2020 at 11:21:56AM +0800, Tao YunXiang wrote:
> Commit a529e3cd1f (ovsdb-server: Allow OVSDB clients to specify the
> UUID for inserted rows) solves ovsdb-client specifing the UUID for
> insert operation. OVSDB now can support directly using uuid to identify
> a row. But for xxxctl
On 3/6/2020 10:26 AM, Yi-Hung Wei wrote:
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
Signed-off-by: Greg Rose
---
acinclude.m4 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/acinclude.m4 b/acinclude.m4
index 1212a46..db64267 100644
--- a/acinclude.m4
+++ b/acinclu
On 3/6/2020 11:18 AM, Yi-Hung Wei wrote:
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
We are adding support for Linux kernels up to 5.5 so update the
Travis test list, NEWS and FAQ.
Signed-off-by: Greg Rose
---
V3 - Squash separate commits for Travis, NEWS and FAQ
---
.travis.yml
On Fri, Mar 06, 2020 at 09:05:55PM +0800, guohongzhi wrote:
> ASAN report use-after-free when destroy ofproto_rule, the rule->ofproto
> has freed in ofproto_destroy.
> Add ref_count for ofproto to avoid use-after-free when destroy
> ofproto_rule adn group.
>
> Signed-off-by: guohongzhi
Why isn't
Can't hurt. I added a pointer and applied this to master. Thanks!
On Thu, Mar 05, 2020 at 04:25:40PM -0800, Justin Pettit wrote:
> Small point, but do you think it's worth pointing to the website repo in the
> instructions? (I suppose anyone who's doing branching probably knows it.)
>
> Acked
On Fri, Mar 06, 2020 at 08:41:09PM +0530, Numan Siddique wrote:
> On Fri, Mar 6, 2020 at 1:07 AM Ben Pfaff wrote:
> >
> > I found the documentation for gateways, especially HA gateways, to
> > be unclear and insufficient. This commit improves it.
> >
> > CC: Numan Siddique
> > CC: Gurucharan She
On Wed, Mar 04, 2020 at 03:04:11PM -0800, Greg Rose wrote:
> The patch series enables OVS kernel datapath support up to
> release 5.5. Updates the faq and NEWS as well.
I'm happy to apply this in v4 if you'll add the acks.
___
dev mailing list
d...@open
On Fri, Mar 6, 2020 at 7:31 AM Ilya Maximets wrote:
>
> On 2/20/20 7:59 PM, William Tu wrote:
> > Similar to using veth across namespaces, this patch creates
> > tap devices, assigns to namespaces, and allows traffic to
> > go through different test cases.
> >
> > Signed-off-by: William Tu
> > --
> > diff --git a/lib/dp-packet.h b/lib/dp-packet.h
> > index 9f8991faad52..6b90cec2afb4 100644
> > --- a/lib/dp-packet.h
> > +++ b/lib/dp-packet.h
> > @@ -53,7 +53,25 @@ enum OVS_PACKED_ENUM dp_packet_source {
> > enum dp_packet_offload_mask {
> > DP_PACKET_OL_RSS_HASH_MASK = 0x1, /* Is the
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> We are adding support for Linux kernels up to 5.5 so update the
> Travis test list, NEWS and FAQ.
>
> Signed-off-by: Greg Rose
>
> ---
> V3 - Squash separate commits for Travis, NEWS and FAQ
> ---
> .travis.yml| 2 +-
> Docu
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> From: "Gustavo A. R. Silva"
>
> Upstream commit:
> commit 279badc2a85be83e0187b8c566e3b476b76a87a2
> Author: Gustavo A. R. Silva
> Date: Thu Oct 19 12:55:03 2017 -0500
>
> openvswitch: conntrack: mark expected switch fall-thro
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> From: Johannes Berg
>
> Upstream commit:
> commit 8cb081746c031fb164089322e2336a0bf5b3070c
> Author: Johannes Berg
> Date: Fri Apr 26 14:07:28 2019 +0200
>
> netlink: make validation more configurable for future strictness
>
>
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> Since this commit in the Linux upstream kernel:
> 'commit 9b9a3f20cbe0 ("kbuild: split final module linking out into
> Makefile.modfinal")'
> The openvswitch kernel module fails to build against the upstream
> Linux kernel. The cause of the buil
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> From: Pankaj Bharadiya
>
> Upstream commit:
> commit c593642c8be046915ca3a4a300243a68077cd207
> Author: Pankaj Bharadiya
> Date: Mon Dec 9 10:31:43 2019 -0800
>
> treewide: Use sizeof_field() macro
>
> Replace all the occu
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> Flex array support is removed since kernel 5.1. Convert to use
> kvmalloc_array instead.
>
> Signed-off-by: Greg Rose
> ---
Acked-by: Yi-Hung Wei
___
dev mailing list
d...@openvswitch.org
https://mai
On 3/6/2020 10:26 AM, Yi-Hung Wei wrote:
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
Signed-off-by: Greg Rose
---
acinclude.m4 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/acinclude.m4 b/acinclude.m4
index 1212a46..db64267 100644
--- a/acinclude.m4
+++ b/acinclu
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> From: Johannes Berg
>
> Upstream commit:
> commit 3b0f31f2b8c9fb348e4530b88f6b64f9621f83d6
> Author: Johannes Berg
> Date: Thu Mar 21 22:51:02 2019 +0100
>
> genetlink: make policy common to family
>
> Since maxattr is com
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> Since Linux kernel release 5.1 the fragments field of the inet_frag_queue
> structure is removed and now only the rb_fragments structure with an
> rb_node pointer is used for both ipv4 and ipv6. In addition, the
> atomic_sub and atomic_add funct
On Wed, Mar 4, 2020 at 3:04 PM Greg Rose wrote:
>
> Signed-off-by: Greg Rose
> ---
> acinclude.m4 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/acinclude.m4 b/acinclude.m4
> index 1212a46..db64267 100644
> --- a/acinclude.m4
> +++ b/acinclude.m4
> @@ -151,7 +151,7 @@ A
On 3/4/20 7:09 PM, William Tu wrote:
> On Fri, Feb 28, 2020 at 12:12 AM Ilya Maximets wrote:
>>
>> On 2/27/20 8:52 PM, William Tu wrote:
>>> On Thu, Feb 27, 2020 at 11:10 AM William Tu wrote:
On Thu, Feb 27, 2020 at 9:54 AM Ilya Maximets wrote:
>
> On 2/27/20 6:51 PM, William T
On 3/2/20 8:51 PM, Flavio Leitner wrote:
> On Mon, Mar 02, 2020 at 01:11:35PM +0100, Ilya Maximets wrote:
>> On 2/28/20 11:58 PM, Flavio Leitner wrote:
>>> The kernel needs to be at least 4.19-rc7 to include the commit
>>> 9d2f67e43b73 ("net/packet: fix packet drop as of virtio gso")
>>> otherwise
On 2/20/20 7:59 PM, William Tu wrote:
> Similar to using veth across namespaces, this patch creates
> tap devices, assigns to namespaces, and allows traffic to
> go through different test cases.
>
> Signed-off-by: William Tu
> ---
> tests/atlocal.in | 3 +++
> tests/automake.mk
On 2/20/20 7:59 PM, William Tu wrote:
> This patch enables TSO support for non-DPDK use cases, and
> also add check-system-tso testsuite. Before TSO, we have to
> disable checksum offload, allowing the kernel to calculate the
> TCP/UDP packet checsum. With TSO, we can skip the checksum
> validation
On Fri, Mar 6, 2020 at 1:07 AM Ben Pfaff wrote:
>
> I found the documentation for gateways, especially HA gateways, to
> be unclear and insufficient. This commit improves it.
>
> CC: Numan Siddique
> CC: Gurucharan Shetty
> Signed-off-by: Ben Pfaff
Thanks for improving the documentation. This
On 2/26/20 4:10 PM, Dumitru Ceara wrote:
> On 2/26/20 1:57 PM, Ilya Maximets wrote:
>> Since OVS and OVN are split but still using the same mailing list for
>> development, subject line should contain 'ovn' word. This helps
>> to identify to which project patch should be applied. Especially,
>> i
On 3/6/20 3:17 PM, Ilya Maximets wrote:
> Since OVS and OVN are split but still using the same mailing list for
> development, subject line should contain 'ovn' word. This helps
> to identify to which project patch should be applied. Especially,
> it helps to ovsrobot to not try and fail OVN patc
Hi, William
After I checked some doubtable points, I think performance issue lies in
tpacket_v3 kernel side implementation, so I fall back it to original one in
case userspace-tso-enabled is set to true. V6 has been sent out, please review
v6.
-邮件原件-
发件人: William Tu [mailto:u9012...@gm
From: Yi Yang
We can avoid high system call overhead by using TPACKET_V3
and using DPDK-like poll to receive and send packets (Note: send
still needs to call sendto to trigger final packet transmission).
>From Linux kernel 3.10 on, TPACKET_V3 has been supported,
so all the Linux kernels current
Since OVS and OVN are split but still using the same mailing list for
development, subject line should contain 'ovn' word. This helps
to identify to which project patch should be applied. Especially,
it helps to ovsrobot to not try and fail OVN patches on OVS repository.
Additionally removed unw
Bleep bloop. Greetings guohongzhi, 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 89 characters long (recommended limit is 79)
#72 FILE: ofproto/ofproto.c:1744:
On 3/2/20 1:49 PM, Ilya Maximets wrote:
> Ilya Maximets (2):
> dpdk: Remove deprecated pdump support.
> netdev-dpdk: Remove deprecated ring port type.
>
> .travis/linux-build.sh| 6 -
> Documentation/automake.mk | 2 -
> Documentation/topics/dpd
ASAN report use-after-free when destroy ofproto_rule, the rule->ofproto
has freed in ofproto_destroy.
Add ref_count for ofproto to avoid use-after-free when destroy
ofproto_rule adn group.
Signed-off-by: guohongzhi
---
ofproto/ofproto-dpif.c | 1 +
ofproto/ofproto-provider.h | 3 +++
ofpro
Ich bin Frau Maria Ricardo, ich bin Verwaltungsleiterin in einem Tresor eines
Finanz- und Sicherheitsinstituts hier in Barcelona.
Ich kontaktiere Sie auf der Grundlage einer finanziellen Chance, die ich hier
in unserer Bank entdeckt habe. Es geht um eine aufgegebene Summe von 26,7 Mio.
Euro (zwa
On Fri, Mar 6, 2020 at 7:14 AM Ihar Hrachyshka wrote:
>
> As per RFC2131, section 4.1:
>A server or relay agent sending or relaying a DHCP message directly
>to a DHCP client (i.e., not to a relay agent specified in the
>'giaddr' field) SHOULD examine the BROADCAST bit in the 'flags'
>
Atención: receptor del fondo,
Hemos sido autorizados por el banco de inversión europeo London y el Fondo
internacional de monitoreo (FMI) para investigar la razón del fondo de
aplazamiento innecesario que ha sido aprobado por ley. Durante Nuestra
investigación descubrió que su pago fue retrasado
64 matches
Mail list logo
