bgpd: Fix holdtime not working properly when busy

Commit:  cc9f21da2218d95567eff1501482ce58e6600f54

Modified the bgp_fsm code to dissallow the extension
of the hold time when the system is under extremely
heavy load.  This was a attempt to remove the return
code but it was too aggressive and messed up this bit
of code.

Put the behavior back that was introduced in:
d0874d195d0127009a7d9c06920c52c95319eff9

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

diff --git a/bgpd/bgp_fsm.c b/bgpd/bgp_fsm.c
index 607e34e8d3f9b2860a3eee821d2c8e9087d8b26a..6fdb121837d3054994674ba080117fa525bfdbb1 100644 (file)
--- a/bgpd/bgp_fsm.c
+++ b/bgpd/bgp_fsm.c
@@ -529,9 +529,11 @@ static void bgp_holdtime_timer(struct event *thread)
        frr_with_mutex (&connection->io_mtx) {
                inq_count = atomic_load_explicit(&connection->ibuf->count, memory_order_relaxed);
        }
-       if (inq_count)
+       if (inq_count) {
                BGP_TIMER_ON(connection->t_holdtime, bgp_holdtime_timer,
                             peer->v_holdtime);
+               return;
+       }
 
        EVENT_VAL(thread) = Hold_Timer_expired;
        bgp_event(thread); /* bgp_event unlocks peer */