Re: [ovs-dev] [PATCH v4 1/5] classifier: Do not insert duplicate rules in indices.

2014-11-14 Thread Jarno Rajahalme
On Nov 13, 2014, at 3:50 PM, Ben Pfaff wrote: > On Thu, Nov 13, 2014 at 11:56:13AM -0800, Jarno Rajahalme wrote: >> There is no point in adding duplicate information into prefix tries. >> >> Also, since the lower-priority duplicate rules are not visible to >> lookups, they do not need to be in

Re: [ovs-dev] [PATCH v4 1/5] classifier: Do not insert duplicate rules in indices.

2014-11-13 Thread Ben Pfaff
On Thu, Nov 13, 2014 at 11:56:13AM -0800, Jarno Rajahalme wrote: > There is no point in adding duplicate information into prefix tries. > > Also, since the lower-priority duplicate rules are not visible to > lookups, they do not need to be in staged lookup indices directly > either (the head rule

[ovs-dev] [PATCH v4 1/5] classifier: Do not insert duplicate rules in indices.

2014-11-13 Thread Jarno Rajahalme
There is no point in adding duplicate information into prefix tries. Also, since the lower-priority duplicate rules are not visible to lookups, they do not need to be in staged lookup indices directly either (the head rule is). Finally, now that cmap operations return the number of elements in th