Skip to content
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

Disable RT throttling #28

Closed
shuhaowu opened this issue Apr 3, 2022 · 1 comment · Fixed by #56
Closed

Disable RT throttling #28

shuhaowu opened this issue Apr 3, 2022 · 1 comment · Fixed by #56
Labels

Comments

@shuhaowu
Copy link
Member

shuhaowu commented Apr 3, 2022

See https://wiki.linuxfoundation.org/realtime/documentation/technical_basics/sched_rt_throttling.

Basically, the default /proc/sys/kernel/sched_rt_runtime_us means that there's a chance that the kernel would throttle/sleep a RT process once it uses up 950ms out of 1s of CPU time. This could result in deadline misses. The feature was implemented before the widespread availability of SMP and thus needed to avoid complete system lockup/denial-of-service. With multicore systems (especially with isolcpu), this shouldn't be needed.

Basically, we need a startup service that writes -1 to /proc/sys/kernel/sched_rt_runtime_us.

FYI: @carlossvg @LanderU

@carlossvg
Copy link
Collaborator

@shuhaowu I agree to disable it by default and add it to the configuration list in the documentation. I would put the script in the rt-configuration-scripts directory described in #26

@carlossvg carlossvg added the enhancement New feature or request label Apr 6, 2022
@carlossvg carlossvg moved this to In Progress in RTWG roadmap Apr 12, 2022
@carlossvg carlossvg moved this from In Progress to Todo in RTWG roadmap Apr 12, 2022
This was referenced Aug 31, 2022
Repository owner moved this from In Progress to Done in ROS 2 Raspberry PI real-time image release Sep 1, 2022
Repository owner moved this from Todo to Done in RTWG roadmap Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants