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 <b...@nicira.com> wrote: > On Thu, Aug 08, 2013 at 12:58:30PM -0700, Ethan Jackson wrote: >> Signed-off-by: Ethan Jackson <et...@nicira.com> > > 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: dev@openvswitch.org _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
