On Tue, May 20, 2025 at 1:22 AM Dumitru Ceara <[email protected]> wrote: > > On 12/11/24 12:56 PM, Dumitru Ceara wrote: > > On 11/28/24 9:48 AM, Dumitru Ceara wrote: > >> On 11/28/24 12:24 AM, Han Zhou wrote: > >>> On Wed, Nov 27, 2024 at 3:27 AM Dumitru Ceara <[email protected]> wrote: > >>>> > >>>> Hi Han, > >>>> > >>>> On 11/27/24 7:14 AM, Han Zhou wrote: > >>>>> The current implementation prefers dst routes over src routes only if > >>>>> they have the same prefix length. This is not very useful for real world > >>>>> use cases, because it doesn't make much sense to compare the prefix > >>>>> length between different fields of the IP header (dst IP v.s. src IP). > >>>>> The prefix length should make sense only when comparing for the same > >>>>> field, either dst or src. > >>>>> > >>>>> This patch changes the behavior by always prefering dst routes over src > >>>>> routes, regardless of the prefix length, and comparing prefix length > >>>>> only within the same type of routes. > >>>> > > [...] > > > > > Assuming silence means people are OK with this behavior change: > > Acked-by: Dumitru Ceara <[email protected]> > > > > Hi Han, > > I just noticed in patchwork that this patch is still not applied. Are > you still interested in having this merged? Based on the previous > discussion it seems OK to me. > > Regards, > Dumitru >
Hi Dumitru, Sorry I missed this one and your email until someone reminded me that a new ovn-k8s patch depends on this change. I rebased it on top of the latest main and pushed. Thanks, Han _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
