X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=lib%2Ftag.c;h=13d182925eb3f5a0f4570a8fb717efb3035a5bcf;hb=HEAD;hp=0430224c82b402d7e4d40eaeaae0fdfae5f6e036;hpb=c393047875e2c9dc76b2f19092fff6bbf69ec4f9;p=sliver-openvswitch.git

diff --git a/lib/tag.c b/lib/tag.c
index 0430224c8..13d182925 100644
--- a/lib/tag.c
+++ b/lib/tag.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2008, 2009, 2010 Nicira Networks.
+ * Copyright (c) 2008, 2009, 2010, 2011, 2013 Nicira, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,30 +16,10 @@
 
 #include <config.h>
 #include "tag.h"
-#include <limits.h>
-#include "random.h"
-#include "type-props.h"
-#include "util.h"
-
-#define N_TAG_BITS (CHAR_BIT * sizeof(tag_type))
-BUILD_ASSERT_DECL(IS_POW2(N_TAG_BITS));
 
 #define LOG2_N_TAG_BITS (N_TAG_BITS == 32 ? 5 : N_TAG_BITS == 64 ? 6 : 0)
 BUILD_ASSERT_DECL(LOG2_N_TAG_BITS > 0);
 
-/* Returns a randomly selected tag. */
-tag_type
-tag_create_random(void)
-{
-    int x, y;
-    do {
-        uint16_t r = random_uint16();
-        x = r & (N_TAG_BITS - 1);
-        y = r >> (16 - LOG2_N_TAG_BITS);
-    } while (x == y);
-    return (1u << x) | (1u << y);
-}
-
 /* Returns a tag deterministically generated from 'seed'.
  *
  * 'seed' should have data in all of its bits; if it has data only in its
@@ -54,29 +34,31 @@ tag_create_deterministic(uint32_t seed)
     return (1u << x) | (1u << y);
 }
 
-/* Initializes 'set' as an empty tag set. */
+/* Initializes 'tracker'. */
 void
-tag_set_init(struct tag_set *set)
+tag_tracker_init(struct tag_tracker *tracker)
 {
-    memset(set, 0, sizeof *set);
+    memset(tracker, 0, sizeof *tracker);
 }
 
-/* Adds 'tag' to 'set'. */
+/* Adds 'add' to '*tags' and records the bits added in 'tracker'. */
 void
-tag_set_add(struct tag_set *set, tag_type tag)
+tag_tracker_add(struct tag_tracker *tracker, tag_type *tags, tag_type add)
 {
-    if (tag && (!tag_is_valid(tag) || !tag_set_intersects(set, tag))) {
-        /* XXX We could do better by finding the set member to which we would
-         * add the fewest number of 1-bits.  This would reduce the amount of
-         * ambiguity, since e.g. three 1-bits match 3 * 2 / 2 = 3 unique tags
-         * whereas four 1-bits match 4 * 3 / 2 = 6 unique tags. */
-        tag_type *t = &set->tags[set->n++ % TAG_SET_SIZE];
-        *t |= tag;
-        if (*t == TYPE_MAXIMUM(tag_type)) {
-            set->tags[0] = *t;
-        }
-
-        set->total |= tag;
+    *tags |= add;
+    for (; add; add = zero_rightmost_1bit(add)) {
+        tracker->counts[rightmost_1bit_idx(add)]++;
     }
 }
 
+/* Removes 'sub' from 'tracker' and unsets any bits in '*tags' that no
+ * remaining tag includes. */
+void
+tag_tracker_subtract(struct tag_tracker *tracker, tag_type *tags, tag_type sub)
+{
+    for (; sub; sub = zero_rightmost_1bit(sub)) {
+        if (!--tracker->counts[rightmost_1bit_idx(sub)]) {
+            *tags &= ~rightmost_1bit(sub);
+        }
+    }
+}