I was having a lot of trouble getting the thread safety analysis to work out with it, and I figured that since only the main thread iterates or needs to take the write lock, we're safe. Let me revist this though, I agree it's a wart.
Ethan On Thu, Aug 8, 2013 at 3:14 PM, Ben Pfaff <[email protected]> wrote: > On Thu, Aug 08, 2013 at 12:58:30PM -0700, Ethan Jackson wrote: >> Signed-off-by: Ethan Jackson <[email protected]> > > The bits of code in ofproto.c that iterate over classifier rules do > not take any classifier locks, but every other use of a classifier > (e.g. even classifier_is_empty()) does take the lock. I don't > understand this. X-CudaMail-Whitelist-To: [email protected] _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
