Re: [ovs-dev] [PATCH net] net: ovs: fix ovs_drop_reasons error

2024-08-22 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Wed, 21 Aug 2024 20:32:52 +0800 you wrote: > There is something wrong with ovs_drop_reasons. ovs_drop_reasons[0] is > "OVS_DROP_LAST_ACTION", but OVS_DROP_LAST_ACTION == __OVS_DROP_REASON + 1, > which means that ovs_dro

Re: [ovs-dev] [PATCH v1 net-next] net: openvswitch: Use ERR_CAST() to return

2024-08-30 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski : On Thu, 29 Aug 2024 17:55:09 +0800 you wrote: > Using ERR_CAST() is more reasonable and safer, When it is necessary > to convert the type of an error pointer and return it. > > Signed-off-by: Yan Zhen > --- > net/o

Re: [ovs-dev] [PATCH net-next] openvswitch: Pass on secpath details for internal port rx.

2024-11-05 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski : On Fri, 1 Nov 2024 16:47:32 -0400 you wrote: > Clearing the secpath for internal ports will cause packet drops when > ipsec offload or early SW ipsec decrypt are used. Systems that rely > on these will not be able t

Re: [ovs-dev] [PATCH net] MAINTAINERS: list openvswitch docs under its entry

2025-02-03 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Sat, 1 Feb 2025 16:50:24 -0800 you wrote: > Submissions to the docs seem to not get properly CCed. > > Signed-off-by: Jakub Kicinski > --- > CC: pshe...@ovn.org > CC: d...@openvswitch.org > > [...] Here is the summ

Re: [ovs-dev] [PATCH net] selftests: openvswitch: fix tcpdump execution

2024-12-18 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Tue, 17 Dec 2024 22:16:51 +0100 you wrote: > Fix the way tcpdump is executed by: > - Using the right variable for the namespace. Currently the use of the > empty "ns" makes the command fail. > - Waiting until it start

Re: [ovs-dev] [PATCH net] openvswitch: fix lockup on tx to unregistering netdev with carrier

2025-01-10 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Thu, 9 Jan 2025 13:21:24 +0100 you wrote: > Commit in a fixes tag attempted to fix the issue in the following > sequence of calls: > > do_output > -> ovs_vport_send >-> dev_queue_xmit > -> __

Re: [ovs-dev] [PATCH net-next] net: openvswitch: fix kernel-doc warnings in internal headers

2025-03-24 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski : On Thu, 20 Mar 2025 23:42:07 +0100 you wrote: > Some field descriptions were missing, some were not very accurate. > Not touching the uAPI header or .c files for now. > > Formatting of those comments isn't great in g

Re: [ovs-dev] [PATCH net] net: openvswitch: fix nested key length validation in the set() action

2025-04-14 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Sat, 12 Apr 2025 12:40:18 +0200 you wrote: > It's not safe to access nla_len(ovs_key) if the data is smaller than > the netlink header. Check that the attribute is OK first. > > Fixes: ccb1352e76cf ("net: Add Open vSw

Re: [ovs-dev] [PATCH net-next] selftests: openvswitch: don't hardcode the drop reason subsys

2025-03-06 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski : On Tue, 4 Mar 2025 10:06:15 -0800 you wrote: > WiFi removed one of their subsys entries from drop reasons, in > commit 286e69677065 ("wifi: mac80211: Drop cooked monitor support") > SKB_DROP_REASON_SUBSYS_OPENVSWITCH

Re: [ovs-dev] [PATCH net] net: openvswitch: remove misbehaving actions length check

2025-03-13 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Paolo Abeni : On Sat, 8 Mar 2025 01:45:59 +0100 you wrote: > The actions length check is unreliable and produces different results > depending on the initial length of the provided netlink attribute and > the composition of the actual act

Re: [ovs-dev] [PATCH net] Revert "openvswitch: switch to per-action label counting in conntrack"

2025-03-13 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Paolo Abeni : On Sat, 8 Mar 2025 13:05:43 -0500 you wrote: > Currently, ovs_ct_set_labels() is only called for confirmed conntrack > entries (ct) within ovs_ct_commit(). However, if the conntrack entry > does not have the labels_ext exten

Re: [ovs-dev] [PATCH net-next v4 0/2] Flexible array for ip tunnel options

2025-02-20 Thread patchwork-bot+netdevbpf--- via dev
Hello: This series was applied to netdev/net-next.git (main) by Jakub Kicinski : On Wed, 19 Feb 2025 16:32:54 +0200 you wrote: > Remove the hidden assumption that options are allocated at the end of > the struct, and teach the compiler about them using a flexible array. > > First patch is conver

Re: [ovs-dev] [PATCH net] tunnels: Accept PACKET_HOST in skb_tunnel_check_pmtu().

2025-04-03 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Sat, 29 Mar 2025 01:33:44 +0100 you wrote: > Because skb_tunnel_check_pmtu() doesn't handle PACKET_HOST packets, > commit 30a92c9e3d6b ("openvswitch: Set the skbuff pkt_type for proper > pmtud support.") forced skb->pkt

Re: [ovs-dev] [PATCH net] openvswitch: Fix unsafe attribute parsing in output_userspace()

2025-05-07 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski : On Tue, 6 May 2025 16:28:54 +0200 you wrote: > This patch replaces the manual Netlink attribute iteration in > output_userspace() with nla_for_each_nested(), which ensures that only > well-formed attributes are processed.

Re: [ovs-dev] [PATCH net v4] net: openvswitch: Fix the dead loop of MPLS parse

2025-05-28 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Paolo Abeni : On Fri, 23 May 2025 03:41:43 + you wrote: > The unexpected MPLS packet may not end with the bottom label stack. > When there are many stacks, The label count value has wrapped around. > A dead loop occurs, soft lockup/CPU

Re: [ovs-dev] [PATCH net-next v2] openvswitch: Stricter validation for the userspace action

2025-05-14 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski : On Mon, 12 May 2025 10:08:24 +0200 you wrote: > This change enhances the robustness of validate_userspace() by ensuring > that all Netlink attributes are fully contained within the parent > attribute. The previous use

Re: [ovs-dev] [PATCH net] openvswitch: Allocate struct ovs_pcpu_storage dynamically

2025-06-17 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net.git (main) by Paolo Abeni : On Fri, 13 Jun 2025 14:36:29 +0200 you wrote: > PERCPU_MODULE_RESERVE defines the maximum size that can by used for the > per-CPU data size used by modules. This is 8KiB. > > Commit 035fcdc4d240c ("openvswitch: Merge three p

Re: [ovs-dev] [PATCH net-next v2] net: openvswitch: allow providing upcall pid for the 'execute' command

2025-07-07 Thread patchwork-bot+netdevbpf--- via dev
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski : On Wed, 2 Jul 2025 17:50:34 +0200 you wrote: > When a packet enters OVS datapath and there is no flow to handle it, > packet goes to userspace through a MISS upcall. With per-CPU upcall > dispatch mechanism, we're u