2 * Copyright (c) 2009, 2010 Nicira Networks.
4 * Licensed under the Apache License, Version 2.0 (the "License");
5 * you may not use this file except in compliance with the License.
6 * You may obtain a copy of the License at:
8 * http://www.apache.org/licenses/LICENSE-2.0
10 * Unless required by applicable law or agreed to in writing, software
11 * distributed under the License is distributed on an "AS IS" BASIS,
12 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13 * See the License for the specific language governing permissions and
14 * limitations under the License.
18 #include "classifier.h"
21 #include <netinet/in.h>
25 const struct cls_field cls_fields[CLS_N_FIELDS + 1] = {
26 #define CLS_FIELD(WILDCARDS, MEMBER, NAME) \
27 { offsetof(flow_t, MEMBER), \
28 sizeof ((flow_t *)0)->MEMBER, \
33 { sizeof(flow_t), 0, 0, "exact" },
36 static uint32_t hash_fields(const flow_t *, int table_idx);
37 static bool equal_fields(const flow_t *, const flow_t *, int table_idx);
39 static int table_idx_from_wildcards(uint32_t wildcards);
40 static struct cls_rule *table_insert(struct hmap *, struct cls_rule *);
41 static struct cls_rule *insert_exact_rule(struct classifier *,
43 static struct cls_bucket *find_bucket(struct hmap *, size_t hash,
44 const struct cls_rule *);
45 static struct cls_rule *search_table(const struct hmap *table, int field_idx,
46 const struct cls_rule *);
47 static struct cls_rule *search_exact_table(const struct classifier *,
48 size_t hash, const flow_t *);
49 static bool rules_match_1wild(const struct cls_rule *fixed,
50 const struct cls_rule *wild, int field_idx);
52 /* Converts the flow in 'flow' into a cls_rule in 'rule', with the given
53 * 'wildcards' and 'priority'.*/
55 cls_rule_from_flow(struct cls_rule *rule, const flow_t *flow,
56 uint32_t wildcards, unsigned int priority)
58 assert(flow->reserved == 0);
60 flow_wildcards_init(&rule->wc, wildcards);
61 rule->priority = priority;
62 rule->table_idx = table_idx_from_wildcards(rule->wc.wildcards);
65 /* Converts the ofp_match in 'match' into a cls_rule in 'rule', with the given
68 cls_rule_from_match(struct cls_rule *rule, const struct ofp_match *match,
69 unsigned int priority)
72 flow_from_match(&rule->flow, &wildcards, match);
73 flow_wildcards_init(&rule->wc, wildcards);
74 rule->priority = rule->wc.wildcards ? priority : UINT16_MAX;
75 rule->table_idx = table_idx_from_wildcards(rule->wc.wildcards);
78 /* Prints cls_rule 'rule', for debugging.
80 * (The output could be improved and expanded, but this was good enough to
81 * debug the classifier.) */
83 cls_rule_print(const struct cls_rule *rule)
85 printf("wildcards=%x priority=%u ", rule->wc.wildcards, rule->priority);
86 flow_print(stdout, &rule->flow);
90 /* Adjusts pointers around 'old', which must be in classifier 'cls', to
91 * compensate for it having been moved in memory to 'new' (e.g. due to
94 * This function cannot be realized in all possible flow classifier
95 * implementations, so we will probably have to change the interface if we
96 * change the implementation. Shouldn't be a big deal though. */
98 cls_rule_moved(struct classifier *cls, struct cls_rule *old,
102 if (new->wc.wildcards) {
103 list_moved(&new->node.list);
105 hmap_node_moved(&cls->exact_table,
106 &old->node.hmap, &new->node.hmap);
111 /* Replaces 'old', which must be in classifier 'cls', by 'new' (e.g. due to
112 * realloc()); that is, after calling this function 'new' will be in 'cls' in
115 * 'new' and 'old' must be exactly the same: wildcard the same fields, have the
116 * same fixed values for non-wildcarded fields, and have the same priority.
118 * The caller takes ownership of 'old' and is thus responsible for freeing it,
119 * etc., as necessary.
121 * This function cannot be realized in all possible flow classifier
122 * implementations, so we will probably have to change the interface if we
123 * change the implementation. Shouldn't be a big deal though. */
125 cls_rule_replace(struct classifier *cls, const struct cls_rule *old,
126 struct cls_rule *new)
129 assert(old->wc.wildcards == new->wc.wildcards);
130 assert(old->priority == new->priority);
132 if (new->wc.wildcards) {
133 list_replace(&new->node.list, &old->node.list);
135 hmap_replace(&cls->exact_table, &old->node.hmap, &new->node.hmap);
139 /* Initializes 'cls' as a classifier that initially contains no classification
142 classifier_init(struct classifier *cls)
147 for (i = 0; i < ARRAY_SIZE(cls->tables); i++) {
148 hmap_init(&cls->tables[i]);
150 hmap_init(&cls->exact_table);
153 /* Destroys 'cls'. Rules within 'cls', if any, are not freed; this is the
154 * caller's responsibility. */
156 classifier_destroy(struct classifier *cls)
159 struct cls_bucket *bucket, *next_bucket;
162 for (tbl = &cls->tables[0]; tbl < &cls->tables[CLS_N_FIELDS]; tbl++) {
163 HMAP_FOR_EACH_SAFE (bucket, next_bucket,
164 struct cls_bucket, hmap_node, tbl) {
169 hmap_destroy(&cls->exact_table);
173 /* Returns true if 'cls' does not contain any classification rules, false
176 classifier_is_empty(const struct classifier *cls)
178 return cls->n_rules == 0;
181 /* Returns the number of rules in 'classifier'. */
183 classifier_count(const struct classifier *cls)
188 /* Returns the number of rules in 'classifier' that have no wildcards. */
190 classifier_count_exact(const struct classifier *cls)
192 return hmap_count(&cls->exact_table);
195 /* Inserts 'rule' into 'cls'. Transfers ownership of 'rule' to 'cls'.
197 * If 'cls' already contains an identical rule (including wildcards, values of
198 * fixed fields, and priority), replaces the old rule by 'rule' and returns the
199 * rule that was replaced. The caller takes ownership of the returned rule and
200 * is thus responsible for freeing it, etc., as necessary.
202 * Returns NULL if 'cls' does not contain a rule with an identical key, after
203 * inserting the new rule. In this case, no rules are displaced by the new
204 * rule, even rules that cannot have any effect because the new rule matches a
205 * superset of their flows and has higher priority. */
207 classifier_insert(struct classifier *cls, struct cls_rule *rule)
209 struct cls_rule *old;
210 assert((rule->wc.wildcards == 0) == (rule->table_idx == CLS_F_IDX_EXACT));
211 old = (rule->wc.wildcards
212 ? table_insert(&cls->tables[rule->table_idx], rule)
213 : insert_exact_rule(cls, rule));
220 /* Inserts 'rule' into 'cls'. Transfers ownership of 'rule' to 'cls'.
222 * 'rule' must be an exact-match rule (rule->wc.wildcards must be 0) and 'cls'
223 * must not contain any rule with an identical key. */
225 classifier_insert_exact(struct classifier *cls, struct cls_rule *rule)
227 hmap_insert(&cls->exact_table, &rule->node.hmap,
228 flow_hash(&rule->flow, 0));
232 /* Removes 'rule' from 'cls'. It is caller's responsibility to free 'rule', if
233 * this is desirable. */
235 classifier_remove(struct classifier *cls, struct cls_rule *rule)
237 if (rule->wc.wildcards) {
238 /* Remove 'rule' from bucket. If that empties the bucket, remove the
239 * bucket from its table. */
240 struct hmap *table = &cls->tables[rule->table_idx];
241 struct list *rules = list_remove(&rule->node.list);
242 if (list_is_empty(rules)) {
243 /* This code is a little tricky. list_remove() returns the list
244 * element just after the one removed. Since the list is now
245 * empty, this will be the address of the 'rules' member of the
246 * bucket that was just emptied, so pointer arithmetic (via
247 * CONTAINER_OF) can find that bucket. */
248 struct cls_bucket *bucket;
249 bucket = CONTAINER_OF(rules, struct cls_bucket, rules);
250 hmap_remove(table, &bucket->hmap_node);
254 /* Remove 'rule' from cls->exact_table. */
255 hmap_remove(&cls->exact_table, &rule->node.hmap);
260 /* Finds and returns the highest-priority rule in 'cls' that matches 'flow'.
261 * Returns a null pointer if no rules in 'cls' match 'flow'. If multiple rules
262 * of equal priority match 'flow', returns one arbitrarily.
264 * (When multiple rules of equal priority happen to fall into the same bucket,
265 * rules added more recently take priority over rules added less recently, but
266 * this is subject to change and should not be depended upon.) */
268 classifier_lookup(const struct classifier *cls, const flow_t *flow)
270 struct cls_rule *rule = classifier_lookup_exact(cls, flow);
272 rule = classifier_lookup_wild(cls, flow);
278 classifier_lookup_exact(const struct classifier *cls, const flow_t *flow)
280 return (!hmap_is_empty(&cls->exact_table)
281 ? search_exact_table(cls, flow_hash(flow, 0), flow)
286 classifier_lookup_wild(const struct classifier *cls, const flow_t *flow)
288 struct cls_rule *best = NULL;
289 if (cls->n_rules > hmap_count(&cls->exact_table)) {
290 struct cls_rule target;
293 cls_rule_from_flow(&target, flow, 0, 0);
294 for (i = 0; i < CLS_N_FIELDS; i++) {
295 struct cls_rule *rule = search_table(&cls->tables[i], i, &target);
296 if (rule && (!best || rule->priority > best->priority)) {
305 classifier_find_rule_exactly(const struct classifier *cls,
306 const flow_t *target, uint32_t wildcards,
307 unsigned int priority)
309 struct cls_bucket *bucket;
314 /* Ignores 'priority'. */
315 return search_exact_table(cls, flow_hash(target, 0), target);
318 assert(wildcards == (wildcards & OFPFW_ALL));
319 table_idx = table_idx_from_wildcards(wildcards);
320 hash = hash_fields(target, table_idx);
321 HMAP_FOR_EACH_WITH_HASH (bucket, struct cls_bucket, hmap_node, hash,
322 &cls->tables[table_idx]) {
323 if (equal_fields(&bucket->fixed, target, table_idx)) {
324 struct cls_rule *pos;
325 LIST_FOR_EACH (pos, struct cls_rule, node.list, &bucket->rules) {
326 if (pos->priority < priority) {
328 } else if (pos->priority == priority &&
329 pos->wc.wildcards == wildcards &&
330 flow_equal(target, &pos->flow)) {
339 /* Ignores target->priority.
341 * 'callback' is allowed to delete the rule that is passed as its argument, but
342 * it must not delete (or move) any other rules in 'cls' that are in the same
343 * table as the argument rule. Two rules are in the same table if their
344 * cls_rule structs have the same table_idx; as a special case, a rule with
345 * wildcards and an exact-match rule will never be in the same table. */
347 classifier_for_each_match(const struct classifier *cls,
348 const struct cls_rule *target,
349 int include, cls_cb_func *callback, void *aux)
351 if (include & CLS_INC_WILD) {
352 const struct hmap *table;
354 for (table = &cls->tables[0]; table < &cls->tables[CLS_N_FIELDS];
356 struct cls_bucket *bucket, *next_bucket;
358 HMAP_FOR_EACH_SAFE (bucket, next_bucket,
359 struct cls_bucket, hmap_node, table) {
360 /* XXX there is a bit of room for optimization here based on
361 * rejecting entire buckets on their fixed fields, but it will
362 * only be worthwhile for big buckets (which we hope we won't
363 * get anyway, but...) */
364 struct cls_rule *prev_rule, *rule;
366 /* We can't just use LIST_FOR_EACH_SAFE here because, if the
367 * callback deletes the last rule in the bucket, then the
368 * bucket itself will be destroyed. The bucket contains the
369 * list head so that's a use-after-free error. */
371 LIST_FOR_EACH (rule, struct cls_rule, node.list,
373 if (rules_match_1wild(rule, target, 0)) {
375 callback(prev_rule, aux);
381 callback(prev_rule, aux);
387 if (include & CLS_INC_EXACT) {
388 if (target->wc.wildcards) {
389 struct cls_rule *rule, *next_rule;
391 HMAP_FOR_EACH_SAFE (rule, next_rule, struct cls_rule, node.hmap,
393 if (rules_match_1wild(rule, target, 0)) {
398 /* Optimization: there can be at most one match in the exact
400 size_t hash = flow_hash(&target->flow, 0);
401 struct cls_rule *rule = search_exact_table(cls, hash,
410 /* 'callback' is allowed to delete the rule that is passed as its argument, but
411 * it must not delete (or move) any other rules in 'cls' that are in the same
412 * table as the argument rule. Two rules are in the same table if their
413 * cls_rule structs have the same table_idx; as a special case, a rule with
414 * wildcards and an exact-match rule will never be in the same table. */
416 classifier_for_each(const struct classifier *cls, int include,
417 void (*callback)(struct cls_rule *, void *aux),
420 if (include & CLS_INC_WILD) {
421 const struct hmap *tbl;
423 for (tbl = &cls->tables[0]; tbl < &cls->tables[CLS_N_FIELDS]; tbl++) {
424 struct cls_bucket *bucket, *next_bucket;
426 HMAP_FOR_EACH_SAFE (bucket, next_bucket,
427 struct cls_bucket, hmap_node, tbl) {
428 struct cls_rule *prev_rule, *rule;
430 /* We can't just use LIST_FOR_EACH_SAFE here because, if the
431 * callback deletes the last rule in the bucket, then the
432 * bucket itself will be destroyed. The bucket contains the
433 * list head so that's a use-after-free error. */
435 LIST_FOR_EACH (rule, struct cls_rule, node.list,
438 callback(prev_rule, aux);
443 callback(prev_rule, aux);
449 if (include & CLS_INC_EXACT) {
450 struct cls_rule *rule, *next_rule;
452 HMAP_FOR_EACH_SAFE (rule, next_rule,
453 struct cls_rule, node.hmap, &cls->exact_table) {
459 static struct cls_bucket *create_bucket(struct hmap *, size_t hash,
460 const flow_t *fixed);
461 static struct cls_rule *bucket_insert(struct cls_bucket *, struct cls_rule *);
463 static inline bool equal_bytes(const void *, const void *, size_t n);
465 /* Returns a hash computed across the fields in 'flow' whose field indexes
466 * (CLS_F_IDX_*) are less than 'table_idx'. (If 'table_idx' is
467 * CLS_F_IDX_EXACT, hashes all the fields in 'flow'). */
469 hash_fields(const flow_t *flow, int table_idx)
471 /* I just know I'm going to hell for writing code this way.
473 * GCC generates pretty good code here, with only a single taken
474 * conditional jump per execution. Now the question is, would we be better
475 * off marking this function ALWAYS_INLINE and writing a wrapper that
476 * switches on the value of 'table_idx' to get rid of all the conditional
477 * jumps entirely (except for one in the wrapper)? Honestly I really,
478 * really hope that it doesn't matter in practice.
480 * We could do better by calculating hashes incrementally, instead of
481 * starting over from the top each time. But that would be even uglier. */
486 a = b = c = 0xdeadbeef + table_idx;
489 #define CLS_FIELD(WILDCARDS, MEMBER, NAME) \
490 if (table_idx == CLS_F_IDX_##NAME) { \
492 memset((uint8_t *) tmp + n, 0, sizeof tmp - n); \
495 const size_t size = sizeof flow->MEMBER; \
496 const uint8_t *p1 = (const uint8_t *) &flow->MEMBER; \
497 const size_t p1_size = MIN(sizeof tmp - n, size); \
498 const uint8_t *p2 = p1 + p1_size; \
499 const size_t p2_size = size - p1_size; \
501 /* Append to 'tmp' as much data as will fit. */ \
502 memcpy((uint8_t *) tmp + n, p1, p1_size); \
505 /* If 'tmp' is full, mix. */ \
506 if (n == sizeof tmp) { \
514 /* Append to 'tmp' any data that didn't fit. */ \
515 memcpy(tmp, p2, p2_size); \
529 /* Compares the fields in 'a' and 'b' whose field indexes (CLS_F_IDX_*) are
530 * less than 'table_idx'. (If 'table_idx' is CLS_F_IDX_EXACT, compares all the
531 * fields in 'a' and 'b').
533 * Returns true if all the compared fields are equal, false otherwise. */
535 equal_fields(const flow_t *a, const flow_t *b, int table_idx)
537 /* XXX The generated code could be better here. */
538 #define CLS_FIELD(WILDCARDS, MEMBER, NAME) \
539 if (table_idx == CLS_F_IDX_##NAME) { \
541 } else if (!equal_bytes(&a->MEMBER, &b->MEMBER, sizeof a->MEMBER)) { \
551 table_idx_from_wildcards(uint32_t wildcards)
554 return CLS_F_IDX_EXACT;
556 #define CLS_FIELD(WILDCARDS, MEMBER, NAME) \
557 if (wildcards & WILDCARDS) { \
558 return CLS_F_IDX_##NAME; \
565 /* Inserts 'rule' into 'table'. Returns the rule, if any, that was displaced
566 * in favor of 'rule'. */
567 static struct cls_rule *
568 table_insert(struct hmap *table, struct cls_rule *rule)
570 struct cls_bucket *bucket;
573 hash = hash_fields(&rule->flow, rule->table_idx);
574 bucket = find_bucket(table, hash, rule);
576 bucket = create_bucket(table, hash, &rule->flow);
579 return bucket_insert(bucket, rule);
582 /* Inserts 'rule' into 'bucket', given that 'field' is the first wildcarded
585 * Returns the rule, if any, that was displaced in favor of 'rule'. */
586 static struct cls_rule *
587 bucket_insert(struct cls_bucket *bucket, struct cls_rule *rule)
589 struct cls_rule *pos;
590 LIST_FOR_EACH (pos, struct cls_rule, node.list, &bucket->rules) {
591 if (pos->priority <= rule->priority) {
592 if (pos->priority == rule->priority
593 && pos->wc.wildcards == rule->wc.wildcards
594 && rules_match_1wild(pos, rule, rule->table_idx))
596 list_replace(&rule->node.list, &pos->node.list);
602 list_insert(&pos->node.list, &rule->node.list);
606 static struct cls_rule *
607 insert_exact_rule(struct classifier *cls, struct cls_rule *rule)
609 struct cls_rule *old_rule;
612 hash = flow_hash(&rule->flow, 0);
613 old_rule = search_exact_table(cls, hash, &rule->flow);
615 hmap_remove(&cls->exact_table, &old_rule->node.hmap);
617 hmap_insert(&cls->exact_table, &rule->node.hmap, hash);
621 /* Returns the bucket in 'table' that has the given 'hash' and the same fields
622 * as 'rule->flow' (up to 'rule->table_idx'), or a null pointer if no bucket
624 static struct cls_bucket *
625 find_bucket(struct hmap *table, size_t hash, const struct cls_rule *rule)
627 struct cls_bucket *bucket;
628 HMAP_FOR_EACH_WITH_HASH (bucket, struct cls_bucket, hmap_node, hash,
630 if (equal_fields(&bucket->fixed, &rule->flow, rule->table_idx)) {
637 /* Creates a bucket and inserts it in 'table' with the given 'hash' and 'fixed'
638 * values. Returns the new bucket. */
639 static struct cls_bucket *
640 create_bucket(struct hmap *table, size_t hash, const flow_t *fixed)
642 struct cls_bucket *bucket = xmalloc(sizeof *bucket);
643 list_init(&bucket->rules);
644 bucket->fixed = *fixed;
645 hmap_insert(table, &bucket->hmap_node, hash);
649 /* Returns true if the 'n' bytes in 'a' and 'b' are equal, false otherwise. */
650 static inline bool ALWAYS_INLINE
651 equal_bytes(const void *a, const void *b, size_t n)
654 /* For some reason GCC generates stupid code for memcmp() of small
655 * constant integer lengths. Help it out.
657 * This function is always inlined, and it is always called with 'n' as a
658 * compile-time constant, so the switch statement gets optimized out and
659 * this whole function just expands to an instruction or two. */
662 return *(uint8_t *) a == *(uint8_t *) b;
665 return *(uint16_t *) a == *(uint16_t *) b;
668 return *(uint32_t *) a == *(uint32_t *) b;
671 return (*(uint32_t *) a == *(uint32_t *) b
672 && ((uint16_t *) a)[2] == ((uint16_t *) b)[2]);
678 /* I hope GCC is smarter on your platform. */
679 return !memcmp(a, b, n);
683 /* Returns the 32-bit unsigned integer at 'p'. */
684 static inline uint32_t
685 read_uint32(const void *p)
687 /* GCC optimizes this into a single machine instruction on x86. */
689 memcpy(&x, p, sizeof x);
693 /* Compares the specified field in 'a' and 'b'. Returns true if the fields are
694 * equal, or if the ofp_match wildcard bits in 'wildcards' are set such that
695 * non-equal values may be ignored. 'nw_src_mask' and 'nw_dst_mask' must be
696 * those that would be set for 'wildcards' by cls_rule_set_masks().
698 * The compared field is the one with wildcard bit or bits 'field_wc', offset
699 * 'rule_ofs' within cls_rule's "fields" member, and length 'len', in bytes. */
700 static inline bool ALWAYS_INLINE
701 field_matches(const flow_t *a_, const flow_t *b_,
702 uint32_t wildcards, uint32_t nw_src_mask, uint32_t nw_dst_mask,
703 uint32_t field_wc, int ofs, int len)
705 /* This function is always inlined, and it is always called with 'field_wc'
706 * as a compile-time constant, so the "if" conditionals here generate no
708 const void *a = (const uint8_t *) a_ + ofs;
709 const void *b = (const uint8_t *) b_ + ofs;
710 if (!(field_wc & (field_wc - 1))) {
711 /* Handle all the single-bit wildcard cases. */
712 return wildcards & field_wc || equal_bytes(a, b, len);
713 } else if (field_wc == OFPFW_NW_SRC_MASK ||
714 field_wc == OFPFW_NW_DST_MASK) {
715 uint32_t a_ip = read_uint32(a);
716 uint32_t b_ip = read_uint32(b);
717 uint32_t mask = (field_wc == OFPFW_NW_SRC_MASK
718 ? nw_src_mask : nw_dst_mask);
719 return ((a_ip ^ b_ip) & mask) == 0;
725 /* Returns true if 'a' and 'b' match, ignoring fields for which the wildcards
726 * in 'wildcards' are set. 'nw_src_mask' and 'nw_dst_mask' must be those that
727 * would be set for 'wildcards' by cls_rule_set_masks(). 'field_idx' is the
728 * index of the first field to be compared; fields before 'field_idx' are
729 * assumed to match. (Always returns true if 'field_idx' is CLS_N_FIELDS.) */
731 rules_match(const struct cls_rule *a, const struct cls_rule *b,
732 uint32_t wildcards, uint32_t nw_src_mask, uint32_t nw_dst_mask,
735 /* This is related to Duff's device (see
736 * http://en.wikipedia.org/wiki/Duff's_device). */
738 #define CLS_FIELD(WILDCARDS, MEMBER, NAME) \
739 case CLS_F_IDX_##NAME: \
740 if (!field_matches(&a->flow, &b->flow, \
741 wildcards, nw_src_mask, nw_dst_mask, \
742 WILDCARDS, offsetof(flow_t, MEMBER), \
743 sizeof a->flow.MEMBER)) { \
753 /* Returns true if 'fixed' and 'wild' match. All fields in 'fixed' must have
754 * fixed values; 'wild' may contain wildcards.
756 * 'field_idx' is the index of the first field to be compared; fields before
757 * 'field_idx' are assumed to match. Always returns true if 'field_idx' is
760 rules_match_1wild(const struct cls_rule *fixed, const struct cls_rule *wild,
763 return rules_match(fixed, wild, wild->wc.wildcards, wild->wc.nw_src_mask,
764 wild->wc.nw_dst_mask, field_idx);
767 /* Searches 'bucket' for a rule that matches 'target'. Returns the
768 * highest-priority match, if one is found, or a null pointer if there is no
771 * 'field_idx' must be the index of the first wildcarded field in 'bucket'. */
772 static struct cls_rule *
773 search_bucket(struct cls_bucket *bucket, int field_idx,
774 const struct cls_rule *target)
776 struct cls_rule *pos;
778 if (!equal_fields(&bucket->fixed, &target->flow, field_idx)) {
782 LIST_FOR_EACH (pos, struct cls_rule, node.list, &bucket->rules) {
783 if (rules_match_1wild(target, pos, field_idx)) {
790 /* Searches 'table' for a rule that matches 'target'. Returns the
791 * highest-priority match, if one is found, or a null pointer if there is no
794 * 'field_idx' must be the index of the first wildcarded field in 'table'. */
795 static struct cls_rule *
796 search_table(const struct hmap *table, int field_idx,
797 const struct cls_rule *target)
799 struct cls_bucket *bucket;
801 switch (hmap_count(table)) {
802 /* In these special cases there's no need to hash. */
806 bucket = CONTAINER_OF(hmap_first(table), struct cls_bucket, hmap_node);
807 return search_bucket(bucket, field_idx, target);
810 HMAP_FOR_EACH_WITH_HASH (bucket, struct cls_bucket, hmap_node,
811 hash_fields(&target->flow, field_idx), table) {
812 struct cls_rule *rule = search_bucket(bucket, field_idx, target);
820 static struct cls_rule *
821 search_exact_table(const struct classifier *cls, size_t hash,
822 const flow_t *target)
824 struct cls_rule *rule;
826 HMAP_FOR_EACH_WITH_HASH (rule, struct cls_rule, node.hmap,
827 hash, &cls->exact_table) {
828 if (flow_equal(&rule->flow, target)) {