On Mar 19, 2014, at 8:45 AM, Alex Wang <[email protected]> wrote:

> I think we should also remove the 'Note' in the comment.
> 

It still seems valid to me:

“Note: While hmap in general is not thread-safe without additional locking, 
hmap_is_empty() is.”

  Jarno

> 
> On Tue, Mar 18, 2014 at 9:00 PM, Ben Pfaff <[email protected]> wrote:
> On Tue, Mar 18, 2014 at 04:42:55PM -0700, Jarno Rajahalme wrote:
> > The fence made classifier_lookup() slower.  Access to a size_t 'n' is
> > safe without synchonizing, and if racing with writers matters,
> > additional syncronization primitives are used anyway.
> >
> > Signed-off-by: Jarno Rajahalme <[email protected]>
> 
> I think it's useful for documentation but it's not worth emitting code.
> 
> Acked-by: Ben Pfaff <[email protected]>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
> 

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to