Skip to content
This repository was archived by the owner on Mar 16, 2026. It is now read-only.

cpu affinity: consider --cpu-offset as override to auto-detection#27

Merged
il-steffen merged 1 commit intoIntelLabs:masterfrom
il-steffen:fix/cpu_offset_semantics
Nov 2, 2022
Merged

cpu affinity: consider --cpu-offset as override to auto-detection#27
il-steffen merged 1 commit intoIntelLabs:masterfrom
il-steffen:fix/cpu_offset_semantics

Conversation

@il-steffen
Copy link
Copy Markdown
Contributor

The --cpu-offset arugment was incompatible with automated detection of free/busy CPUs. The reduced cpu set is not obvious during startup time and we may fail to launch workers if cpu set + cpu offset < workers.

This patch disables auto-detection of free vcpus if a --cpu-offset is given. This restores the original semantics of --cpu-offset.

The --cpu-offset arugment was incompatible with automated detection of
free/busy CPUs. The reduced cpu set is not obvious during startup time
and we may fail to launch workers if cpu set + cpu offset < workers.

This patch disables auto-detection of free vcpus if a --cpu-offset
is given. This restores the original semantics of --cpu-offset.
@il-steffen il-steffen merged commit 68fb0ff into IntelLabs:master Nov 2, 2022
@il-steffen il-steffen deleted the fix/cpu_offset_semantics branch November 25, 2022 15:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants