On Sep 25, 2014, at 7:52 AM, Daniele Venturino <daniele.ventur...@m3s.it> wrote:
> After some testing, here’s my ack. > > Acked-by: Daniele Venturino <daniele.ventur...@m3s.it > Pushed to master, thanks! Jarno >>>> >>>> diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c >>>> index d3e527a..d3973e5 100644 >>>> --- a/ofproto/ofproto-dpif.c >>>> +++ b/ofproto/ofproto-dpif.c >>>> @@ -2386,6 +2386,8 @@ set_rstp_port(struct ofport *ofport_, >>>> rstp_port_set(rp, s->port_num, s->priority, s->path_cost, >>>> s->admin_edge_port, s->auto_edge, s->mcheck, ofport); >>>> update_rstp_port_state(ofport); >>>> + /* Synchronize operational status. */ >>>> + rstp_port_set_mac_operational(rp, ofport->may_enable); >>>> } >>>> >>>> static void >>> >>> This helps, physical interfaces are no longer blocked in role disabled if >>> RSTP is enabled after they are added to a bridge. >>> I’ll test this more thoroughly soon. >>> >> >> I’ll merge this when I get your “Acked-by:”, >> >> Jarno > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
