On Mon, Jun 09, 2014 at 11:53:55AM -0700, Jarno Rajahalme wrote:
> Use cmap instead of hmap & hindex in classifier.  Performance impact
> with current locking strategy is not yet tested.  Later patches will
> introduce RCU into the classifer.

into the "classifier".

> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com>

As long as this passes the tests (I assume you ran them) it seems OK
to me.  The tricky review will be when we start actually making the
classifier RCU.  (I don't know if it makes sense to push this patch
before then.  It might be better as the first patch of a series that
makes the classifier RCU.  But I'll leave that decision to you.)

Acked-by: Ben Pfaff <b...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to