-
Notifications
You must be signed in to change notification settings - Fork 184
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
Add package to {cpus,uncores} matching functions #610
Conversation
Is it ready for merge, or still WIP (because it has WIP flag)? From my point of view it's ready. I added few comments that could improve it, but I can see no problem with this PR. |
ce66e10
to
620d960
Compare
Addressed the comments and remove WIP flag. Please review. |
/packit build |
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.
It's not blocker, but I think it's worth fixing.
CI failure seems unrelated to this PR. |
Allow to specify CPUs and uncores using package (socket) number. type=cpu devices=${f:package2cpus:2} # cpus from package 2 type=cpu devices=${f:package2cpus:1*} # cpus from any package started from 1 i.e. 11, 12, 13 ... type=uncore devices=${f:package2uncores:1:5:9} # uncores from package 1, 5 and 9 Signed-off-by: Stanislaw Gruszka <[email protected]>
620d960
to
9312ac2
Compare
Addressed the comments, please review. |
/packit build |
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, thanks.
CI failures are unrelated:
- Centos 7 CI failure is caused by dropped EPEL-7, it's being worked on.
- Fedora rawhide failure is caused by systemd rate limiting, workaround will be enabled in beakerlib/tuned library.
Allow to specify cpus and uncores based on package (socket) number.