Re: [ovs-dev] [PATCH net v3] net: openvswitch: fix race on port output

2023-04-07 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Wed, 5 Apr 2023 07:53:41 + you wrote: > assume the following setup on a single machine: > 1. An openvswitch instance with one bridge and default flows > 2. two network namespaces "server" and "client" > 3. two ovs i

[ovs-dev] [PATCH v2 ovn] northd: revert ct.inv drop flows

2023-04-07 Thread Lorenzo Bianconi
Partially revert the following commit since it introduces a regression when we want to directly connect to a backend ip from a client outside the cluster for gw-router-port scenario. For this kind of traffic we do not commit to CT the 'original' incoming packet but we send the reply one to CT in un

Re: [ovs-dev] [PATCH ovn branch-22.09] ci: ovn-kubernetes: Pin ovn-k8s version.

2023-04-07 Thread Simon Horman
On Thu, Apr 06, 2023 at 11:40:20PM +0200, Ilya Maximets wrote: > On 4/6/23 21:57, Dumitru Ceara wrote: > > Recently, ovn-k8s started using the new OVN Northbound database table > > Chassis_Template_Var that was added in v22.12.0. Unfortunately, > > ovn-k8s (libovsdb) became incompatible with OVN 2

Re: [ovs-dev] [PATCH] ovsdb-cs: fix 'cs_db' lock flag not updated error.

2023-04-07 Thread Simon Horman
On Fri, Apr 07, 2023 at 02:59:04PM +0800, wanghanlin wrote: > When 'cs' is not connected to a server, then > ovsdb_cs_db_compose_lock_request__ will update > 'cs_db' lock flag, but the lock flag will not > be updated next. > > Signed-off-by: wanghanlin Hi, Are there also error conditions where

Re: [ovs-dev] [PATCH v3] learning-switch: Fix coredump of OpenFlow15 learning-switch

2023-04-07 Thread Simon Horman
On Fri, Apr 07, 2023 at 02:30:22PM +0800, Faicker Mo wrote: > The OpenFlow15 Packet-Out message contains the match instead of the in_port. > The flow.tunnel.metadata.tab is not inited but used in the loop of > tun_metadata_to_nx_match. > > The coredump gdb backtrace is: > #0 memcpy_from_metadat

Re: [ovs-dev] [PATCH v3] util: fix an issue that thread name cannot be set

2023-04-07 Thread Simon Horman
On Fri, Apr 07, 2023 at 10:06:39AM +0800, Songtao Zhan wrote: > To: d...@openvswitch.org, > i.maxim...@ovn.org > > The name of the current thread consists of a name with a maximum > length of 16 bytes and a thread ID. The final name may be longer > than 16 bytes. If the name is longer than 16

Re: [ovs-dev] [PATCH ovn v3] controller: Clear tunnels from old integration bridge

2023-04-07 Thread Simon Horman
On Thu, Apr 06, 2023 at 05:00:18PM +0200, Ales Musil wrote: > After integration bridge change the tunnels would > stay on the old bridge preventing new tunnels creation > and disrupting traffic. Detect the bridge change > and clear the tunnels from the old integration bridge. > > Reported-at: http

Re: [ovs-dev] [PATCH net v3] net: openvswitch: fix race on port output

2023-04-07 Thread Simon Horman
On Fri, Apr 07, 2023 at 10:44:35AM +0200, Simon Horman wrote: > On Thu, Apr 06, 2023 at 07:05:13PM -0700, Jakub Kicinski wrote: > > On Wed, 5 Apr 2023 07:53:41 + Felix Huettner wrote: > > > assume the following setup on a single machine: > > > 1. An openvswitch instance with one bridge and defa

Re: [ovs-dev] [PATCH net v3] net: openvswitch: fix race on port output

2023-04-07 Thread Eric Dumazet via dev
On Wed, Apr 5, 2023 at 9:53 AM Felix Huettner wrote: > > assume the following setup on a single machine: > 1. An openvswitch instance with one bridge and default flows > 2. two network namespaces "server" and "client" > 3. two ovs interfaces "server" and "client" on the bridge > 4. for each ovs in

Re: [ovs-dev] [PATCH net v3] net: openvswitch: fix race on port output

2023-04-07 Thread Simon Horman
On Thu, Apr 06, 2023 at 07:05:13PM -0700, Jakub Kicinski wrote: > On Wed, 5 Apr 2023 07:53:41 + Felix Huettner wrote: > > assume the following setup on a single machine: > > 1. An openvswitch instance with one bridge and default flows > > 2. two network namespaces "server" and "client" > > 3. t

Re: [ovs-dev] [PATCH v4] OpenFlow: Add extn to set conntrack entries limit per zone.

2023-04-07 Thread Simon Horman
On Thu, Mar 30, 2023 at 08:17:18AM +, Naveen Yerramneni wrote: > Add OpenFlow extn to set conntrack entries limit per zone. > This extn will be used in future to set the zone level limit for > drop zones used by OVN. > > Signed-off-by: Naveen Yerramneni > Reviewed-by: Simon Horman > --- > No

Re: [ovs-dev] [PATCH ovn] northd: take into account qos_min_rate in port_has_qos_params

2023-04-07 Thread Simon Horman
On Thu, Apr 06, 2023 at 05:25:06PM +0200, Dumitru Ceara wrote: > On 4/6/23 14:45, Simon Horman wrote: > > On Thu, Apr 06, 2023 at 02:07:00PM +0200, Dumitru Ceara wrote: > >> On 4/5/23 14:43, Simon Horman wrote: > >>> On Tue, Apr 04, 2023 at 11:01:26PM +0200, Lorenzo Bianconi wrote: > Similar t