New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Periodically synchronize kernel state #141
base: main
Are you sure you want to change the base?
Conversation
configure.ac
Outdated
@@ -120,6 +120,10 @@ AC_ARG_WITH([path-manager], | |||
|
|||
AC_SUBST([mptcpd_default_pm],[$with_path_manager]) | |||
|
|||
dnl Default kernel sync interval in seconds. | |||
default_sync_interval=5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems fast to me. Maybe 60 seconds? (I'm not super attached to that number, just trying to think of a less aggressive rate)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Bumped to 60 seconds in 4f1b183.
src/path_manager.c
Outdated
Periodically synchronize kernel state with mptcpd, such as | ||
address IDs managed by the in-kernel path manager. | ||
*/ | ||
pm->sync_timeout = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think about using sync_interval == 0
to turn off polling?
That seems to be what happens with l_timeout_create()
is given a 0 time value anyway, but it goes ahead and sets up the timerfd
and ELL watch anyway. Could skip that work if the sync interval is 0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree. Implemented in 69b1e6e.
No description provided.