On Tue, May 26, 2026 at 3:01 PM Aaron Conole <[email protected]> wrote:

> When STP/RSTP are in a disabled transition on a bridge, the sets will call
> bundle_update() for each port as it should transition from FORWARDING to
> DISABLED.  However, this happens while the states are still non-NULL,
> which causes bundle_update() to assume that the STP/RSTP status will
> remain as forwarding.  After this happens, the bundle->floodable flag
> will remain 'false' and NORMAL action flood attempts will skip over
> the ports.
>
> In the case of RSTP, the state processing is a bit more complex, so we
> do a final bundle_update pass to reset the floodable flag.  We could
> choose to do a more simplistic fix in STP case by releasing the
> ofproto->stp object early; however to keep it consistent, the fix will
> adopt the same approach of doing a final pass for bundle updates.
>
> Reported-at: https://redhat.atlassian.net/browse/FDP-3852
> Fixes: 21f7563cef5f ("ovs-vswitchd: Add support for 802.1D STP.")
> Fixes: 9efd308e957c ("Rapid Spanning Tree Protocol (IEEE 802.1D).")
> Signed-off-by: Aaron Conole <[email protected]>
> ---
>


Thanks for this Aaron.

Acked-by: Mike Pattrick <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to