Skip to content

Fix CPU_AFFINITY flag#475

Merged
mdboom merged 1 commit intofaster-cpython:mainfrom
diegorusso:affinity
Apr 22, 2026
Merged

Fix CPU_AFFINITY flag#475
mdboom merged 1 commit intofaster-cpython:mainfrom
diegorusso:affinity

Conversation

@diegorusso
Copy link
Copy Markdown
Collaborator

This is really an escape string bug.

you can see it is working here: https://github.com/diegorusso/pyperf-bench/actions/runs/24757827257/job/72434698676

Perf event: Max sample rate set to 1 per second
  CPU Frequency: Minimum frequency of CPU 175-191 set to the maximum frequency
  IRQ affinity: Set default affinity to CPU 0-174
  IRQ affinity: Set affinity of IRQ 13-15,26-31,39-62,64-71,77,85-86,345-440 to CPU 0-174
  CPU scaling governor: CPU scaling governor of CPUs 175-191 set to performance

Earlier instead it was failing in tuning the system when cpu affinity was set

https://github.com/diegorusso/pyperf-bench/actions/runs/24646323352/job/72059601480 the tuning system fails:

Tuning system
  usage: -m pyperf system [-h] [--affinity CPU_LIST] [{show,tune,reset}]
  -m pyperf system: error: argument --affinity: invalid CPU list: '"183-191"'

Copy link
Copy Markdown
Contributor

@mdboom mdboom left a comment

Choose a reason for hiding this comment

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

LGTM.

@mdboom mdboom merged commit 3c91adf into faster-cpython:main Apr 22, 2026
6 checks passed
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