Re: [ovs-dev] [PATCH v2] lib/classifier: Lockless lookups.

2014-07-07 Thread Jarno Rajahalme
Yes, I figured it might be better pipeline the revision process to get things through faster :-) Jarno On Jul 7, 2014, at 9:31 AM, Ben Pfaff wrote: > It looks like you posted a v2 of just the beginning of the series also? > I'll review those, then we can look at the rest of the series. > >

Re: [ovs-dev] [PATCH v2] lib/classifier: Lockless lookups.

2014-07-07 Thread Ben Pfaff
It looks like you posted a v2 of just the beginning of the series also? I'll review those, then we can look at the rest of the series. On Mon, Jun 30, 2014 at 04:40:06PM -0700, Jarno Rajahalme wrote: > Just to be clear, this is a v2 on the patch 12/12 of the series, due to the > batching changes

Re: [ovs-dev] [PATCH v2] lib/classifier: Lockless lookups.

2014-06-30 Thread Jarno Rajahalme
Just to be clear, this is a v2 on the patch 12/12 of the series, due to the batching changes Ethan just pushed. The 1-11 seem to still apply. Jarno On Jun 30, 2014, at 4:37 PM, Jarno Rajahalme wrote: > Now that all the relevant classifier structures use RCU and internal > mutual exclusion fo

[ovs-dev] [PATCH v2] lib/classifier: Lockless lookups.

2014-06-30 Thread Jarno Rajahalme
Now that all the relevant classifier structures use RCU and internal mutual exclusion for modifications, we can remove the fat-rwlock and thus make the classifier lookups lockless. As the readers are operating concurrently with the writers, a concurrent reader may or may not see a new rule being a