On 12/18/24 23:34, Han Zhou wrote: > It doesn't make much sense to use higher priority for user-added routes. > This patch sets the priority only based on prefix length. Originally the > increased priority was used to distinguish if it is a cached route or > user route. Now add a flag in the entry to explicitly tell.
Hi, Han. Thanks for the patches! Though, there is a reason for different priorities - the classifier that we're using for route lookups can't store duplicate entries. So, if there is a cached route that is the same as the user-specified one, the user-specified route will be replaced. Then when the cached route is remove, we'll no longer have the route that was manually set by the user. You can see the PTAP test fails in CI with this change, because the manually configured route is replaced by the cached one and we no longer have it. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
