[ovs-git] Open vSwitch: lib/classifier: Clarify subtable skipping. (master)

dev at openvswitch.org dev at openvswitch.org
Fri Jul 18 10:04:06 UTC 2014


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, master has been updated
       via  1817dceab16d33406abf17a4696b2584a38fe5fd (commit)
       via  c0bfb6502c2ac085717ec381f77a30c6ebaa9157 (commit)
       via  27ce650ff6c67c8e184aacefb06bdecd09ff8dc5 (commit)
       via  e48eccd1337fdb288d01510119da36e360cedfc0 (commit)
      from  033e9df25fd3828959e332caf086706e91111a55 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 1817dceab16d33406abf17a4696b2584a38fe5fd
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=1817dceab16d33406abf17a4696b2584a38fe5fd
Author: Jarno Rajahalme <jrajahalme at nicira.com>
		
lib/classifier: Clarify subtable skipping.
		
Clarify comments for trie-based subtable skipping.

Perform the cheaper check first.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Acked-by: YAMAMOTO Takashi <yamamoto at valinux.co.jp>


commit c0bfb6502c2ac085717ec381f77a30c6ebaa9157
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=c0bfb6502c2ac085717ec381f77a30c6ebaa9157
Author: Jarno Rajahalme <jrajahalme at nicira.com>
		
lib/classifier: Return all matching prefix lengths from trie lookup.
		
Previously we only returned the last matching prefix length
encountered during a trie lookup, and skipped subtables that had
prefixes longer than that.  This patch changes the trie lookup
functions to return all matching prefix lengths seen, so that all
non-matching prefix lengths can be skipped.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


commit 27ce650ff6c67c8e184aacefb06bdecd09ff8dc5
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=27ce650ff6c67c8e184aacefb06bdecd09ff8dc5
Author: Jarno Rajahalme <jrajahalme at nicira.com>
		
lib/classifier: Change local variable names.
		
These stylistic changes makes the following patch a bit simpler.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: YAMAMOTO Takashi <yamamoto at valinux.co.jp>


commit e48eccd1337fdb288d01510119da36e360cedfc0
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e48eccd1337fdb288d01510119da36e360cedfc0
Author: Jarno Rajahalme <jrajahalme at nicira.com>
		
lib/classifier: Unify struct classifier and cls_classifier.
		
Now that it is clear that struct cls_classifier itself does not
need RCU indirection and pvector is defined in its own header, it
is possible get rid of the indirection from struct classifier to
struct cls_classifier.

Suggested-by: YAMAMOTO Takashi <yamamoto at valinux.co.jp>
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 lib/classifier.c        |  241 ++++++++++++++++++++---------------------------
 lib/classifier.h        |   29 +++++-
 tests/test-classifier.c |   35 ++++---
 3 files changed, 142 insertions(+), 163 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list