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

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

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

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

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

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

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

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

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

[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

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

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

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

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.

Re: [ovs-dev] [PATCH ovn v3 6/8] utilities/docker: Fix up container build.

2024-03-27 Thread Ales Musil
On Tue, Mar 26, 2024 at 7:09 PM Mark Michelson wrote: > On 3/25/24 05:09, Ales Musil wrote: > > > > > > On Mon, Mar 25, 2024 at 9:47 AM Dumitru Ceara > > wrote: > > > > On 3/22/24 19:34, Mark Michelson wrote: > > > On 3/21/24 19:15, Dumitru Ceara wrote: > >

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

2024-03-27 Thread Han Zhou
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. This allows more fine grained >> control for I-P processing of address