-
Notifications
You must be signed in to change notification settings - Fork 0
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
lightway-app-utils: iouring: Make sqpoll idle time configurable #125
Conversation
Code coverage summary for 2755fac:
✅ Region coverage 57% passes |
This is a useful tunable to play with when trading CPU use vs performance.
665e62c
to
2755fac
Compare
/// IO-uring sqpoll idle time. If non-zero use a kernel thread to | ||
/// perform submission queue polling. After the given idle time | ||
/// the thread will go to sleep. | ||
#[clap(long, default_value = "100ms")] |
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.
Think it is better if we should do a sanity check for this value for upper bound like 1 or 2 seconds.
Setting this as large value will make CPU run 100% for longer time.
What do you think ?
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 think sysadmins need to understand the consequences of fiddling with these kinds of low level tunables.
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.
LGTM except boundary check
Description
Make it configurable and plumb it all the way through to the config file. The default remains 100ms.
Motivation and Context
This is a useful tunable to play with when trading CPU use vs performance.
How Has This Been Tested?
Observing the behaviour using htop and in particular observing that with it set to 0 no kernel thread is created.
Types of changes
Checklist:
main