> I'm pretty sure that the above will self-deadlock on > ofproto->facets.rwlock, because facet_remove() write-locks it. > > My understanding is that the locking on this classifier is superfluous > anyway since only a single thread accesses it--it's only there to > suppress Clang lock warnings--so I just moved the unlock just past the > cls_cursor_init().
Oops, good catch. Thanks Ethan _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
