X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=src%2Fvnet%2Fl2%2Fl2_bd.c;h=9c81b0f81a0ba11c84833d9b75b9522fb8ca564e;hb=662c59a80a886f9a6741eb95229ee2aca2e06fad;hp=a8429170788634f6dde2b86587fed0776e28be10;hpb=10e5b4a016061f2d9485ff309d1239abb012aae0;p=vpp.git

diff --git a/src/vnet/l2/l2_bd.c b/src/vnet/l2/l2_bd.c
index a8429170788..9c81b0f81a0 100644
--- a/src/vnet/l2/l2_bd.c
+++ b/src/vnet/l2/l2_bd.c
@@ -1608,28 +1608,25 @@ VLIB_CLI_COMMAND (bd_create_cli, static) = {
  * Returns an unused bridge domain id, and ~0 if it can't find one.
  */
 u32
-bd_get_unused_id ()
+bd_get_unused_id (void)
 {
   bd_main_t *bdm = &bd_main;
   int i, j;
-  int is_seed_low = 0;
   static u32 seed = 0;
+
   /* limit to 1M tries */
   for (j = 0; j < 1 << 10; j++)
     {
-      seed = random_u32 (&seed) & L2_BD_ID_MAX;
-      if (seed == 0)
-	continue;
-      if (seed < L2_BD_ID_MAX % 2)
-	is_seed_low = 1;
-      for (i = 0; i < L2_BD_ID_MAX % 2; i++)
+      seed = random_u32 (&seed);
+      for (i = 0; i < 1 << 10; i++)
 	{
-	  /* look around randomly generated id */
-	  if (is_seed_low)
-	    seed += (2 * (i % 2) - 1) * i;
-	  else
-	    seed -= (2 * (i % 2) - 1) * i;
-	  if (seed == ~0 || seed == 0)
+	  /*
+	   * iterate seed+0, seed+1, seed-1, seed+2, seed-2, ... to generate id
+	   */
+	  seed += (2 * (i % 2) - 1) * i;
+	  /* bd_id must be (1 <= bd_id <= L2_BD_ID_MAX) */
+	  seed &= L2_BD_ID_MAX;
+	  if (seed == 0)
 	    continue;
 	  if (bd_find_index (bdm, seed) == ~0)
 	    return seed;