Re: [ovs-dev] [PATCH] lib/classifier: Simplify subtable array.

2014-05-28 Thread Jarno Rajahalme
Ben, Thanks for the review! In working towards making the classifier use RCU instead of locking I noticed that we still need the priority value copied to the priority list itself. For example, when a rule is deleted that was the sole highest priority rule in the subtable, the subtable’s max_pr

Re: [ovs-dev] [PATCH] lib/classifier: Simplify subtable array.

2014-05-28 Thread Ben Pfaff
On Thu, May 22, 2014 at 06:38:53PM -0700, Ethan Jackson wrote: > From: Jarno Rajahalme > > Do not cache the 'tag' and 'max_priority' in the subtable array. This > makes later changes to classifier easier. > > Change CLS_SUBTABLES_FOR_EACH to iterate with a regular array index. > > Makes the 'c

[ovs-dev] [PATCH] lib/classifier: Simplify subtable array.

2014-05-22 Thread Ethan Jackson
From: Jarno Rajahalme Do not cache the 'tag' and 'max_priority' in the subtable array. This makes later changes to classifier easier. Change CLS_SUBTABLES_FOR_EACH to iterate with a regular array index. Makes the 'cls_subtables*' functions to always leave the subtables array in a consistent st