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

Reply via email to