Ilya Maximets <[email protected]> writes:

> On 6/2/26 3:06 PM, Aaron Conole 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.
>> 
>> 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]>
>> ---
>>  ofproto/ofproto-dpif.c | 15 +++++++++++
>>  tests/ofproto-dpif.at  | 56 ++++++++++++++++++++++++++++++++++++++++++
>>  2 files changed, 71 insertions(+)
>
> Acked-by: Ilya Maximets <[email protected]>

Thanks Ilya, and Mike.  Pushed and back-ported to 3.3

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

Reply via email to