From a0dfa8b158e327303fccc1f93bf05e948c93e93c Mon Sep 17 00:00:00 2001 From: Andrew Yourtchenko Date: Thu, 14 Jun 2018 18:36:41 +0200 Subject: acl-plugin: fix the high cpu usage caused by the connection cleaner The commit 4bc1796b346efd10f3fb19b176ff089179263a24 had incorrect calculation of the session lists minimal timeout, resulting in returned value of 0 which resulted in existing sessions constantly requeued, taking up the CPU. Fix this calculation. Change-Id: I9a789739f96a1f01522c68f91b0a02db2417837f Signed-off-by: Andrew Yourtchenko --- src/plugins/acl/sess_mgmt_node.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/plugins') diff --git a/src/plugins/acl/sess_mgmt_node.c b/src/plugins/acl/sess_mgmt_node.c index bfc357f667f..abb1e7cf0e4 100644 --- a/src/plugins/acl/sess_mgmt_node.c +++ b/src/plugins/acl/sess_mgmt_node.c @@ -38,7 +38,8 @@ fa_session_get_shortest_timeout (acl_main_t * am) { int timeout_type; u64 timeout = ~0LL; - for (timeout_type = 0; timeout_type <= ACL_N_USER_TIMEOUTS; timeout_type++) + for (timeout_type = ACL_TIMEOUT_UDP_IDLE; + timeout_type < ACL_N_USER_TIMEOUTS; timeout_type++) { if (timeout > am->session_timeout_sec[timeout_type]) { -- cgit 1.2.3-korg