On 5/25/26 9:13 AM, Ales Musil wrote:
> When LRP is configured with multiple IPs we could have MAC Bindings
> with IP addresses corresponding to any of the LRP subnets. However,
> the refresh mechanism would just choose the first IP as source.
> That could cause confusion for the switches along the way and it
> could also be dropped along the way. Make sure we always select
> the right IP. The linear lookup shouldn't cause any performance
> issues as in the practice there isn't too many IPs per LRP.
> 
> Reported-at: https://redhat.atlassian.net/browse/FDP-3784
> Fixes: 1e4d4409f391 ("controller: Send ARP/ND for stale mac_bindings 
> entries.")
> Assisted-by: Claude Opus 4.6, Claude Code
> Signed-off-by: Ales Musil <[email protected]>
> ---
> v2: Rebase on top of latest main.
>     Update the test according to suggestion from Dumitru.
>     Replace AT_CHECK with check_column.
> ---
>  controller/mac-cache.c |  28 +++++++---
>  tests/ovn.at           | 117 +++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 139 insertions(+), 6 deletions(-)
> 

Hi Ales,

Thanks for v2, this looks good to me:

Acked-by: Dumitru Ceara <[email protected]>

Regards,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to