On 9/1/26 8:49 AM, Han Zhou wrote:
> ovn-ic typically interconnects zones by attaching LRPs to a transit
> switch. On that switch, remote LRPs appear as LSPs of type "remote", so
> OVN treats them as regular L2 ports and floods IP multicast to them via
> _MC_flood_l2. Multicast to real LSPs on the transit switch is still
> desired, but flooding toward remote LRPs is useless (they drop it) and
> scales poorly.
> 
> Add Logical_Switch_Port options:is_router so the CMS can mark a
> type=remote port as representing a logical router port. lsp_is_router()
> now returns true for such ports, so they are handled like type=router
> ports; in particular they are omitted from _MC_flood_l2. The option is
> ignored for any other port type.
> 
> Assisted-by: Claude Opus 5, Cursor
> Signed-off-by: Han Zhou <[email protected]>
> ---
> 
> Notes:
>     v2:
>     - Extend lsp_is_router() for type=remote + options:is_router instead of
>       adding a multicast-specific helper (Dumitru).
>     - Add missing OVN_CLEANUP_NORTHD in the new test.
> 

Hi Han,

Thanks for the v2, it looks good to me.  I applied it to main.

Regards,
Dumitru

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

Reply via email to