[ovs-dev] [PATCH] tc: Set action flags for tunnel_key release
Ilya Maximets
i.maximets at ovn.org
Mon Aug 16 20:06:30 UTC 2021
On 8/11/21 8:31 PM, Marcelo Ricardo Leitner wrote:
> On Wed, Aug 11, 2021 at 01:39:41PM +0200, Ilya Maximets wrote:
>> On 8/9/21 9:26 AM, Roi Dayan via dev wrote:
>>> From: Vlad Buslov <vladbu at nvidia.com>
>>>
>>> The commit that enabled 'no_percpu' flag for compatible actions missed the
>>> tunnel_key release action code. Add missing call to nl_msg_put_act_flags().
>>>
>>> Fixes: 292d5bd9bb34 ("tc: Set 'no_percpu' flag for compatible actions")
>>> Reported-by: Marcelo Ricardo Leitner <mleitner at redhat.com>
>>> Signed-off-by: Vlad Buslov <vladbu at nvidia.com>
>>> Reviewed-by: Roi Dayan <roid at nvidia.com>
>>> ---
>>
>> Hi, Marcelo.
>> Can you, please, take a look at this patch?
>
> Hi,
>
> Just tested it. It works:
>
> # tc filter show dev vxlan_sys_4789 parent ffff: | head -n 30
> filter ingress protocol ip pref 3 flower chain 0
> filter ingress protocol ip pref 3 flower chain 0 handle 0x1
> dst_mac 6a:66:2d:48:92:c2
> src_mac 00:00:00:00:00:0a
> eth_type ipv4
> enc_dst_ip 1.1.1.1
> enc_src_ip 1.1.1.2
> enc_key_id 0
> enc_dst_port 4789
> enc_tos 0
> ip_flags nofrag
> in_hw in_hw_count 1
> action order 1: tunnel_key unset pipe
> index 5 ref 1 bind 1
> no_percpu <---- [A]
> used_hw_stats delayed
>
> action order 2: mirred (Egress Redirect to device enp130s0f0_0) stolen
> index 5 ref 1 bind 1
> cookie 722d78c06e4b7c21285241b7a4654607
> no_percpu
> used_hw_stats delayed
> ....
>
> [A] is the one that was missing.
>
> Tested-by: Marcelo Ricardo Leitner <mleitner at redhat.com>
Thanks! Applied to master and backported down to 2.14.
Best regards, Ilya Maximets.
More information about the dev
mailing list