> On Oct 12, 2015, at 11:20 AM, Ben Pfaff <b...@nicira.com> wrote: > > On Wed, Sep 30, 2015 at 01:48:21PM -0700, Jarno Rajahalme wrote: >> >>> On Sep 30, 2015, at 1:54 AM, Daniele Venturino <daniele.ventur...@m3s.it> >>> wrote: >>> >>> I tried the commands you listed but I was not able to reproduce this in a >>> simpler setup only with RSTP enabled on the bridge. >>> >>> I’m attaching a patch I’d like you to try. I’m not really expecting much >>> from it, but can you give it a try since I can’t reproduce this? >>> >>> Daniele >>> >>> <0001-ofproto-dpif-Add-check-in-rstp_run.patch> >>> >> >> Yes, NULLing the aux’es does not help without checking for NULL :-) >> >> Acked-by: Jarno Rajahalme <jrajaha...@nicira.com> > > Jarno, is this patch ready to go? Daniele Venturino is not an OVS > committer so if you're happy with the patch then you should apply it > yourself.
This patch avoids a crash, so I just applied it. However, this leaves the question open that whether bundle_move is properly called in all cases before the rest port is removed. I also applied my patch that disables an RSTP port before it is deleted, as Daniel Swahn tested these patches together. Regards, Jarno _______________________________________________ discuss mailing list discuss@openvswitch.org http://openvswitch.org/mailman/listinfo/discuss
