ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

acl-plugin: fix the high cpu usage caused by the connection cleaner

Browse Source 
The commit 4bc1796b34 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 <ayourtch@gmail.com>
This commit is contained in:
Andrew Yourtchenko
2018-06-14 18:36:41 +02:00
committed by Damjan Marion
parent e0b6c8a96d
commit a0dfa8b158
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/plugins/acl/sess_mgmt_node.c
+2 -1
View File
@@ -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])
{