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

Fallback if the counts of num processors via affinity mask is 0 #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kubo39
Copy link
Contributor

@kubo39 kubo39 commented Jun 14, 2018

No description provided.

@seanmonstar
Copy link
Owner

What exactly does it mean if the affinity mask was set to 0? Is the intent to try to say that you can't pick any CPU at all? It'd seem like that would mean 0 cpus (which really is 1 because the code is executing).

@kubo39 kubo39 changed the title Fallback if num processors via affinity mask is 0 Fallback if the counts of num processors via affinity mask is 0 Jun 15, 2018
@kubo39
Copy link
Contributor Author

kubo39 commented Jun 15, 2018

Ah, sorry, this means 0 cpus.

@seanmonstar
Copy link
Owner

So, if the mask is trying to say "you have 0 cores to use", it seems we should respect that instead of saying "the mask is wrong, you actually have 32 cores!", right?

@kubo39
Copy link
Contributor Author

kubo39 commented Jun 24, 2018

I'm sorry my late reply. My suggestion just brought from Ruby, we should do a deeper discussion.

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.

2 participants