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

numactl: Fix shm verfiy for preferred policy #156

Merged
merged 2 commits into from
Dec 14, 2022
Merged

numactl: Fix shm verfiy for preferred policy #156

merged 2 commits into from
Dec 14, 2022

Conversation

luochenglcs
Copy link
Collaborator

@luochenglcs luochenglcs commented Dec 10, 2022

The following command can report "numactl: Need a policy first to verify"

numactl --length=4096 --shm abc -p0 --verify

when policy uses perferred, set_policy isn't set.

Fixes: 0c844d8("Update to support multiple nodes")

#155

'numactl --length=xxx --shm xxx -px' doesn't work

When preferred is used for shm, no value is assigned to the variable node.
As a result, it does not take effect.

fixes: 5862e0e("numactl: Simplify preferred selection")
#158

Chunsheng Luo added 2 commits December 10, 2022 22:54
The following command can report verify bug:

    numactl --length=4096 --shm abc -p0 --verify

when policy uses perferred, set_policy isn't set.

Fixes: 0c844d8("Update to support multiple nodes")
When preferred is used for shm, no value is assigned to the variable node,
As a result, it does not take effect.

fixes: 5862e0e("numactl: Simplify preferred selection")
@luochenglcs
Copy link
Collaborator Author

@andikleen please review.

@andikleen andikleen merged commit afa2545 into numactl:master Dec 14, 2022
@andikleen
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants