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