On 7/2/25 4:21 AM, Jakub Kicinski wrote: > On Sat, 28 Jun 2025 00:01:33 +0200 Ilya Maximets wrote: >> @@ -616,6 +618,7 @@ static int ovs_packet_cmd_execute(struct sk_buff *skb, >> struct genl_info *info) >> struct sw_flow_actions *sf_acts; >> struct datapath *dp; >> struct vport *input_vport; >> + u32 upcall_pid = 0; >> u16 mru = 0; >> u64 hash; >> int len; >> @@ -651,6 +654,10 @@ static int ovs_packet_cmd_execute(struct sk_buff *skb, >> struct genl_info *info) >> !!(hash & OVS_PACKET_HASH_L4_BIT)); >> } >> >> + if (a[OVS_PACKET_ATTR_UPCALL_PID]) >> + upcall_pid = nla_get_u32(a[OVS_PACKET_ATTR_UPCALL_PID]); >> + OVS_CB(packet)->upcall_pid = upcall_pid; > > sorry for a late nit: > > OVS_CB(packet)->upcall_pid = > nla_get_u32_default(a[OVS_PACKET_ATTR_UPCALL_PID], 0); > > ?
No worries. I was actually looking for ways to collapse this block, but somehow missed these "new" accessors. I'll send a v2 a bit later today in case there will be no further feedback. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev