From: Dave Barach <dave@barachs.net>
Date: Thu, 12 Mar 2020 18:43:04 +0000 (-0400)
Subject: vppinfra: remove time jump workaround
X-Git-Tag: v20.09-rc0~419
X-Git-Url: https://gerrit.fd.io/r/gitweb?p=vpp.git;a=commitdiff_plain;h=7a91b0e264d1dfd911eec18a265ead2ab8a9ad82

vppinfra: remove time jump workaround

A partial revert of gerrit 25729. The last_run_time == 0.0 check is
necessary and remains in place.

Type: fix
Fixes: 3d9f134

Signed-off-by: Dave Barach <dave@barachs.net>
Change-Id: I3d2c9f90b2bc867f02c4749a5b19f997b84185b9
---

diff --git a/src/vppinfra/tw_timer_template.c b/src/vppinfra/tw_timer_template.c
index e78c309ebaa..174ffaceb47 100644
--- a/src/vppinfra/tw_timer_template.c
+++ b/src/vppinfra/tw_timer_template.c
@@ -533,20 +533,6 @@ static inline
       return callback_vector_arg;
     }
 
-  /*
-   * Refuse to do anything if we're about to process way too many slots.
-   * Should never come anywhere close to happening, with the possible exception
-   * of cases involving a large forward jump in the timebase.
-   */
-  if (nticks > (1 << (TW_RING_SHIFT + 1)))
-    {
-      if (nticks / tw->ticks_per_second > 100e-3)
-	clib_warning ("Excessive nticks %u at %.6f last run %.6f",
-		      nticks, now, tw->last_run_time);
-      tw->last_run_time = now;
-      return callback_vector_arg;
-    }
-
   if (callback_vector_arg == 0)
     {
       _vec_len (tw->expired_timer_handles) = 0;