Aaron Conole via dev <[email protected]> writes:

> 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.
>
> For all cases, call bundle_update after the Spanning Tree changes have
> been calculated to make sure that final port details are correctly
> applied (in the case of port state transitions which can occur without
> subsequent port reconfiguration calls).
>
> 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).")
> Acked-by: Mike Pattrick <[email protected]>
> Signed-off-by: Aaron Conole <[email protected]>
> ---

Please ignore this version.  I forgot to stage some changes in the test
cases.

v4 at:
https://patchwork.ozlabs.org/project/openvswitch/patch/[email protected]/

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

Reply via email to