Re: [ovs-dev] [PATCH 4/8] lib/pvector: Non-intrusive RCU priority vector.

2014-06-25 Thread Ben Pfaff
Thanks! This series has dragged on a while so I'll try to expedite review. On Wed, Jun 25, 2014 at 03:58:00AM -0700, Jarno Rajahalme wrote: > Thanks for the review. I have addressed all the comments, there is > quite a nice additional clean-up in classifier_lookup(). In addition > to the lookahea

Re: [ovs-dev] [PATCH 4/8] lib/pvector: Non-intrusive RCU priority vector.

2014-06-25 Thread Jarno Rajahalme
Thanks for the review. I have addressed all the comments, there is quite a nice additional clean-up in classifier_lookup(). In addition to the lookahead, I also integrated the priority handling to an iterator. Will send v2 right away. Jarno On Jun 13, 2014, at 10:06 AM, Ben Pfaff wrote: > O

Re: [ovs-dev] [PATCH 4/8] lib/pvector: Non-intrusive RCU priority vector.

2014-06-13 Thread Ben Pfaff
On Mon, Jun 09, 2014 at 11:53:51AM -0700, Jarno Rajahalme wrote: > Factor out the priority vector code from the classifier. > > Making the classifier use RCU instead of locking requires parallel > access to the priority vector, pointing to subtables in descending > priority order. When a new subt

[ovs-dev] [PATCH 4/8] lib/pvector: Non-intrusive RCU priority vector.

2014-06-09 Thread Jarno Rajahalme
Factor out the priority vector code from the classifier. Making the classifier use RCU instead of locking requires parallel access to the priority vector, pointing to subtables in descending priority order. When a new subtable is added, a new copy of the priority vector is allocated, while the cu