On Tue, Apr 8, 2025 at 6:31 AM Dumitru Ceara <dce...@redhat.com> wrote:
>
> On 4/7/25 6:32 PM, num...@ovn.org wrote:
> > From: Numan Siddique <num...@ovn.org>
> >
> > Commit c48ed1736a58 added the logical flow in "ls_in_l2_lkup"
> > stage to flood ARP/ND request packets if these packets are self
> > originated. i,e generated by ovn-controller. As a side effect,
> > Unicast ARP packets are also flooded if these packets are
> > generated by VIF logical ports and the same MAC is used for both
> > the logical ports and their dnat_and_snat entries in the router
> > (as these macs are considered as router owned IP macs.)
> >
> > This patch fixes this behavior by also matching on the broadcast
> > eth.dst. Note that ovn-controller always generates GARPs with
> > broadcast eth.dst.
> >
> > This issue got resurfaced due to [1] which disabled unicast ARP
> > responder flows in OVN.
> >
> > [1] - c48ed1736a58("Do not reply on unicast arps for IPv4 targets.")
> > Fixes: 32f5ebb06226 ("ovn-northd: Limit ARP/ND broadcast domain whenever
> > possible.")
> >
> > Signed-off-by: Numan Siddique <num...@ovn.org>
> > ---
>
> Hi Numan,
>
> Thanks for the fix!
>
> Acked-by: Dumitru Ceara <dce...@redhat.com>
Thanks Dumitru. I applied the patch to main and backported to
branches 25.03, 24.09 and 24.03.
Numan
>
> Regards,
> Dumitru
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
