Skip to content

Mask Linux thermal interrupt info in /proc and /sys #25634

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.

Sign up for GitHub

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

Jump to bottom
Closed
giuseppe opened this issue Mar 20, 2025 · 0 comments · Fixed by #25635
Closed

Mask Linux thermal interrupt info in /proc and /sys #25634

giuseppe opened this issue Mar 20, 2025 · 0 comments · Fixed by #25635
Assignees
@giuseppe
Labels
jira kind/bug Categorizes issue or PR as related to a bug.

Comments

@giuseppe
Copy link
Member

giuseppe commented Mar 20, 2025 •
edited
Loading

Issue Description

we need the equivalent of moby/moby#49560 for Moby.

The paths /proc/interrupts and /sys/devices/system/cpu/<cpuX>/thermal_throttle must be masked by default.

The main difficulty is that /sys/devices/system/cpu/<cpuX>/thermal_throttle is not a static value but it depends on the number of cpus on the current system, so we need to make sure it doesn't break the current unmasking mechanism we have in place.
@giuseppe giuseppe added kind/bug Categorizes issue or PR as related to a bug. jira labels Mar 20, 2025
@giuseppe giuseppe self-assigned this Mar 20, 2025
giuseppe added a commit to giuseppe/libpod that referenced this issue Mar 20, 2025
@giuseppe
vendor: test new common and buildah
1447a50 
vendor the following dependencies:

- containers/common#2375
- containers/buildah#6074

Closes: containers#25634

Signed-off-by: Giuseppe Scrivano <[email protected]>
giuseppe added a commit to giuseppe/libpod that referenced this issue Mar 20, 2025
@giuseppe
vendor: test new common and buildah
9d63675 
vendor the following dependencies:

- containers/common#2375
- containers/buildah#6074

Closes: containers#25634

Signed-off-by: Giuseppe Scrivano <[email protected]>
@giuseppe giuseppe mentioned this issue Mar 20, 2025
Merged
giuseppe added a commit to giuseppe/libpod that referenced this issue Mar 20, 2025
@giuseppe
vendor: test new common and buildah
c281e69 
vendor the following dependencies:

- containers/common#2375
- containers/buildah#6074

Closes: containers#25634

Signed-off-by: Giuseppe Scrivano <[email protected]>
giuseppe added a commit to giuseppe/libpod that referenced this issue Mar 20, 2025
@giuseppe
vendor: test new common and buildah
6305eaf 
vendor the following dependencies:

- containers/common#2375
- containers/buildah#6074

Closes: containers#25634

Signed-off-by: Giuseppe Scrivano <[email protected]>
giuseppe added a commit to giuseppe/libpod that referenced this issue Mar 21, 2025
@giuseppe
vendor: update common and buildah
5bbcafa 
vendor the following dependencies:

- containers/common#2375
- containers/buildah#6074

Closes: containers#25634

Signed-off-by: Giuseppe Scrivano <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@giuseppe giuseppe

Labels
jira kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Mask thermal paths giuseppe/libpod
1 participant
@giuseppe