-
Notifications
You must be signed in to change notification settings - Fork 6
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
(RHEL-55746) cgroup-util: Don't try to open pidfd for kernel threads #34
Conversation
Similar to the implementation of cgroup.kill in the kernel, let's skip kernel threads in cg_kill_items() as trying to kill kernel threads as an unprivileged process will fail with EPERM and doesn't do anything when running privileged. (cherry picked from commit 0fbb569de1dcc06118dba006cf7a40caf6cd94d0) Resolves: RHEL-55746
The kernel might start returning -EINVAL when trying to open pidfd's for kernel threads so let's not try to open pidfd's for kernel threads. (cherry picked from commit ead48ec35c863650944352a3455f26ce3b393058) Resolves: RHEL-55746
Commit validationTracker - RHEL-55746 The following commits meet all requirements
Follow-up detectionReverts
Success🟠 Mentions, Follow-ups and Revert commits - Waived Tracker validationSuccess🟢 Tracker RHEL-55746 has set desired product: Pull Request validationSuccess🟢 CI - All checks have passed Auto MergeSuccess🟢 Pull Request is not marked as draft and it's not blocked by |
Follow-up for 0fbb569de1dcc06118dba006cf7a40caf6cd94d0. (cherry picked from commit ec4964692ae0e080c596610adee2ddb83008c839) Resolves: RHEL-55746
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
This was reverted upstream because it is a workaround for kernel bugs. The kernel fixed this upstream, but we don't have that in el10 yet. |
No description provided.