From 7a91b0e264d1dfd911eec18a265ead2ab8a9ad82 Mon Sep 17 00:00:00 2001 From: Dave Barach Date: Thu, 12 Mar 2020 14:43:04 -0400 Subject: [PATCH] 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 Change-Id: I3d2c9f90b2bc867f02c4749a5b19f997b84185b9 --- src/vppinfra/tw_timer_template.c | 14 -------------- 1 file changed, 14 deletions(-) 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; -- 2.16.6