[ovs-dev] [PATCH ovn v2] ofctrl: Wait at S_WAIT_BEFORE_CLEAR only once.

2024-03-27 Thread Han Zhou
The ovn-ofctrl-wait-before-clear setting is designed to minimize downtime during the initial start-up of the ovn-controller. For this purpose, the ovn-controller should wait only once upon entering the S_WAIT_BEFORE_CLEAR state for the first time. Subsequent reconnections to the OVS, such as those

Re: [ovs-dev] [PATCH] netdev-dpdk: Fallback to non tunnel offloading API.

2024-03-27 Thread junwan...@cestc.cn
I validated this modification on my x710 network card, but I found that the outer UDP checksum of the transmitted packets is incorrect, leading to communication abnormalities. I think it's necessary to disable the outer UDP checksum because although the capability reported by DPDK indicates suppo

Re: [ovs-dev] [PATCH v2 0/5] ovsdb: raft: Fixes for cluster joining state.

2024-03-27 Thread Ilya Maximets
On 3/26/24 18:27, Ilya Maximets wrote: > > Issues discovered while working on: > https://github.com/ovn-org/ovn/issues/235 > > > Version 2: > * Replaced the first patch with an implementation of leadership > transfer to servers that applied the most changes. [Felix, Han] > * Fixed some

Re: [ovs-dev] [PATCH v2 1/5] ovsdb: raft: Avoid transferring leadership to unavailable servers.

2024-03-27 Thread Ilya Maximets
On 3/26/24 21:10, Han Zhou wrote: > > > On Tue, Mar 26, 2024 at 10:26 AM Ilya Maximets > wrote: >> >> Current implementation of the leadership transfer just shoots the >> leadership in the general direction of the first stable server in the >> configuration.  It doesn'

Re: [ovs-dev] [PATCH] Userspace: Software fallback for UDP encapsulated TCP segmentation.

2024-03-27 Thread Ilya Maximets
On 3/27/24 22:07, Mike Pattrick wrote: > On Wed, Mar 27, 2024 at 1:39 PM Simon Horman wrote: >> >> On Tue, Feb 20, 2024 at 11:08:55PM -0500, Mike Pattrick wrote: >>> When sending packets that are flagged as requiring segmentation to an >>> interface that doens't support this feature, send the pack

Re: [ovs-dev] [PATCH] Userspace: Software fallback for UDP encapsulated TCP segmentation.

2024-03-27 Thread Mike Pattrick
On Wed, Mar 27, 2024 at 1:39 PM Simon Horman wrote: > > On Tue, Feb 20, 2024 at 11:08:55PM -0500, Mike Pattrick wrote: > > When sending packets that are flagged as requiring segmentation to an > > interface that doens't support this feature, send the packet to the TSO > > software fallback instead

Re: [ovs-dev] [PATCH] ovs-tcpdump:Solve the problem of residual mirror port

2024-03-27 Thread Simon Horman
On Thu, Mar 21, 2024 at 08:33:37PM +0800, yangchang wrote: > Before creating an mirror port, it is necessary to first check whether the > original port exists. > Otherwise, when the original port does not exist, the created mirror port > will remain. > > Signed-off-by: yangchang Hi, Thanks fo

Re: [ovs-dev] [PATCH] Userspace: Software fallback for UDP encapsulated TCP segmentation.

2024-03-27 Thread Simon Horman
On Tue, Feb 20, 2024 at 11:08:55PM -0500, Mike Pattrick wrote: > When sending packets that are flagged as requiring segmentation to an > interface that doens't support this feature, send the packet to the TSO > software fallback instead of dropping it. > > Signed-off-by: Mike Pattrick Hi Mike,

Re: [ovs-dev] [PATCH v5] tunnel: Allow UDP zero checksum with IPv6 tunnels.

2024-03-27 Thread Simon Horman
On Wed, Mar 27, 2024 at 11:55:14AM -0400, Mike Pattrick wrote: > This patch adopts the proposed RFC 6935 by allowing null UDP checksums > even if the tunnel protocol is IPv6. This is already supported by Linux > through the udp6zerocsumtx tunnel option. It is disabled by default and > IPv6 tunnels

Re: [ovs-dev] [PATCH] conntrack: Do clean instead of forece expire.

2024-03-27 Thread Aaron Conole
Cheng Li writes: > Force expire a connection and then create new connection of the same > tuple(cmap hash). This makes ct->conns cmap operation expensive( > within ct->ct_lock). > > This patch cover the scenario by doing the clean immediately instead > of setting expire. Also this patch fix ct_cl

Re: [ovs-dev] [PATCH ovn] controller: Track individual address set constants.

2024-03-27 Thread Han Zhou
On Wed, Mar 27, 2024 at 12:46 AM Ales Musil wrote: > > On Wed, Mar 27, 2024 at 7:14 AM Han Zhou wrote: > > > > > > > On Tue, Mar 19, 2024 at 9:45 AM Ales Musil wrote: > > > > > > > > > > > > On Tue, Mar 19, 2024 at 5:43 PM Ales Musil wrote: > > >> > > >> Instead of tracking address set per stru

Re: [ovs-dev] [PATCH net] openvswitch: Set the skbuff pkt_type for proper pmtud support.

2024-03-27 Thread Aaron Conole
Eelco Chaudron writes: > On 25 Mar 2024, at 13:37, Ilya Maximets wrote: > >> On 3/25/24 13:22, Aaron Conole wrote: >>> Eelco Chaudron writes: >>> On 22 Mar 2024, at 20:06, Aaron Conole wrote: > Open vSwitch is originally intended to switch at layer 2, only dealing > with >

[ovs-dev] [PATCH] netdev-dpdk: Fallback to non tunnel offloading API.

2024-03-27 Thread David Marchand
The outer checksum offloading API in DPDK is ambiguous and was added by Intel folks with the assumption that any outer offloading always goes with an inner offloading request. With net/i40e and net/ice drivers, requesting outer ip checksum with a tunnel context but no inner offloading request trig

[ovs-dev] [PATCH v5] tunnel: Allow UDP zero checksum with IPv6 tunnels.

2024-03-27 Thread Mike Pattrick
This patch adopts the proposed RFC 6935 by allowing null UDP checksums even if the tunnel protocol is IPv6. This is already supported by Linux through the udp6zerocsumtx tunnel option. It is disabled by default and IPv6 tunnels are flagged as requiring a checksum, but this patch enables the user to

Re: [ovs-dev] [PATCH v2] ofproto-dpif-xlate: Fix continuations with associated metering.

2024-03-27 Thread Aaron Conole
Aaron Conole writes: > Ilya Maximets writes: > >> On 3/22/24 14:40, Aaron Conole wrote: >>> Open vSwitch supports the ability to invoke a controller action by way >>> of a sample action with a specified meter. In the normal case, this >>> sample action is transparently generated during xlate pr

Re: [ovs-dev] [PATCH v4] tunnel: Allow UDP zero checksum with IPv6 tunnels.

2024-03-27 Thread Simon Horman
On Thu, Mar 21, 2024 at 05:08:06PM -0400, Mike Pattrick wrote: > This patch adopts the proposed RFC 6935 by allowing null UDP checksums > even if the tunnel protocol is IPv6. This is already supported by Linux > through the udp6zerocsumtx tunnel option. It is disabled by default and > IPv6 tunnels

[ovs-dev] [PATCH ovn] northd: Support routing over other address families.

2024-03-27 Thread Felix Huettner via dev
In most cases IPv4 packets are routed only over other IPv4 networks and IPv6 packets are routed only over IPv6 networks. However there is no interent reason for this limitation. Routing IPv4 packets over IPv6 networks just requires the router to contain a route for an IPv4 network with an IPv6 next

Re: [ovs-dev] [PATCH v2 1/5] ovsdb: raft: Avoid transferring leadership to unavailable servers.

2024-03-27 Thread Felix Huettner via dev
On Tue, Mar 26, 2024 at 06:27:09PM +0100, Ilya Maximets wrote: > Current implementation of the leadership transfer just shoots the > leadership in the general direction of the first stable server in the > configuration. It doesn't check if the server was active recently or > even that the connecti

Re: [ovs-dev] [PATCH ovn] controller: Track individual address set constants.

2024-03-27 Thread Ales Musil
On Wed, Mar 27, 2024 at 7:14 AM Han Zhou wrote: > > > On Tue, Mar 19, 2024 at 9:45 AM Ales Musil wrote: > > > > > > > > On Tue, Mar 19, 2024 at 5:43 PM Ales Musil wrote: > >> > >> Instead of tracking address set per struct expr_constant_set track it > >> per individual struct expr_constant. Thi