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.

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

(#1740657) core: add support for setting CPUAffinity= to special "numa" value #119

Merged
merged 2 commits into from Nov 2, 2020
Merged

(#1740657) core: add support for setting CPUAffinity= to special "numa" value #119

merged 2 commits into from Nov 2, 2020

Conversation

msekletar
Copy link
Member

No description provided.

msekletar and others added 2 commits August 28, 2020 17:35
@msekletar
dbus-execute: make transfer of CPUAffinity endian safe (#12711)
2657e5e 
We store the affinity mask in the native endian. However, over D-Bus we
must transfer the mask in little endian byte order.

This is the second part of c367f99.

(cherry picked from commit 75e4011)

Related: #1740657
@msekletar
core: add support for setting CPUAffinity= to special "numa" value
a6dc853 
systemd will automatically derive CPU affinity mask from NUMA node
mask.

Fixes #13248

(cherry picked from commit e2b2fb7)

Resolves: #1740657
@msekletar msekletar changed the title #(1740657) cpuaffinity numa #(1740657) core: add support for setting CPUAffinity= to special "numa" value Aug 28, 2020
@msekletar msekletar changed the title #(1740657) core: add support for setting CPUAffinity= to special "numa" value (#1740657) core: add support for setting CPUAffinity= to special "numa" value Aug 28, 2020
@systemd-rhel-bot systemd-rhel-bot added pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review tracker/unapproved Formerly needs-acks and removed pr/needs-ci Formerly needs-ci labels Aug 28, 2020
@systemd-rhel-bot systemd-rhel-bot removed the tracker/unapproved Formerly needs-acks label Nov 2, 2020
dtardon
dtardon approved these changes Nov 2, 2020
View reviewed changes
Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@systemd-rhel-bot systemd-rhel-bot removed the pr/needs-review Formerly needs-review label Nov 2, 2020
@systemd-rhel-bot systemd-rhel-bot merged commit 1a822db into redhat-plumbers:master Nov 2, 2020
@mergify mergify bot added the pr/needs-ci Formerly needs-ci label Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dtardon dtardon dtardon approved these changes

Assignees
No one assigned
Labels
pr/needs-ci Formerly needs-ci released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@msekletar @dtardon @jamacku @systemd-rhel-bot