On Mon, Nov 10, 2014 at 01:47:49PM +0900, Simon Horman wrote:
> id_pool_alloc_id() was created by breaking out the recirculation
> allocation code. As it is now a library call it makes sense to remove
> the restriction that id 0 is reserved.
> 
> Signed-off-by: Simon Horman <simon.hor...@netronome.com>

Applied, thanks, with the following fix:

diff --git a/lib/id-pool.c b/lib/id-pool.c
index f30b650..0f46b7f 100644
--- a/lib/id-pool.c
+++ b/lib/id-pool.c
@@ -145,7 +145,7 @@ id_pool_free_id(struct id_pool *pool, uint32_t id)
     struct id_node *id_node;
     if (id > pool->base && (id <= pool->base + pool->n_ids)) {
         id_node = id_pool_find(pool, id);
-        if (id) {
+        if (id_node) {
             hmap_remove(&pool->map, &id_node->node);
         }
     }
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to