From: Neale Ranns <nranns@cisco.com>
Date: Sat, 10 Jun 2017 08:43:44 +0000 (-0700)
Subject: FIB walk process - wake-up rate unnecessarily high
X-Git-Tag: odp_vpp~5
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=aba8be617c1c716b55b8b2919ec3d4b61e69dff1;p=odp4vpp.git

FIB walk process - wake-up rate unnecessarily high

Change-Id: I7dedf283c83c7f0e0b7642f095b68bc0b40898cf
Signed-off-by: Neale Ranns <nranns@cisco.com>
---

diff --git a/src/vnet/fib/fib_walk.c b/src/vnet/fib/fib_walk.c
index 70180137..94297442 100644
--- a/src/vnet/fib/fib_walk.c
+++ b/src/vnet/fib/fib_walk.c
@@ -399,7 +399,17 @@ typedef enum fib_walk_sleep_type_t_
  * @brief Durations for the sleep types
  */
 static f64 fib_walk_sleep_duration[] = {
-    [FIB_WALK_LONG_SLEEP] = 1e-3,
+    /**
+     * Long sleep when there is no more work, i.e. the queues are empty.
+     * This is a sleep (as opposed to a wait for event) just to be sure we
+     * are not missing events by sleeping forever.
+     */
+    [FIB_WALK_LONG_SLEEP] = 2,
+
+    /**
+     * Short sleep. There is work left in the queues. We are yielding the CPU
+     * momentarily.
+     */
     [FIB_WALK_SHORT_SLEEP] = 1e-8,
 };