On Thu, Mar 02, 2023 at 03:57:42PM +0800, Faicker Mo wrote:
> The device may be deleted and added with ifindex changed.
> The tc rules on the device will be deleted if the device is deleted.
> The func tc_del_filter will fail when flow del. The mapping of
> ufid to tc will not be deleted.
> The traffic will trigger the same flow(with same ufid) to put to tc
> on the new device. Duplicated ufid mapping will be added.
> If the hashmap is expanded, the old mapping entry will be the first entry,
> and now the dp flow can't be deleted.
> 
> 
> Signed-off-by: Faicker Mo <faicker...@ucloud.cn>

FWIIW, I exercised the test added by this patch.

It seems to fail reliably without the C-code change.
And I was able to run it successfully (~2400 times) with the C-code change.

...
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to