Skip to content

Account for cgroups constraints in the NUMA task arena conformance test#2014

Merged
isaevil merged 3 commits intomasterfrom
dev/dnmokhov/create_numa_task_arenas
Mar 19, 2026
Merged

Account for cgroups constraints in the NUMA task arena conformance test#2014
isaevil merged 3 commits intomasterfrom
dev/dnmokhov/create_numa_task_arenas

Conversation

@dnmokhov
Copy link
Copy Markdown
Contributor

Description

Fix the NUMA task arena conformance test to account for CPU-constrained environments (e.g., cgroups/containers) where worker threads may not fully occupy an arena, similar to the existing single-NUMA-node case. Broaden the workers_cannot_fully_occupy_arena condition accordingly, preventing barrier deadlock in containerized settings.

Type of change

  • bug fix - change that fixes an issue
  • new feature - change that adds functionality
  • tests - change in tests
  • infrastructure - change in infrastructure and CI
  • documentation - documentation update

Tests

  • added - required for new features and some bug fixes
  • not needed

Documentation

  • updated in # - add PR number
  • needs to be updated
  • not needed

Breaks backward compatibility

  • Yes
  • No
  • Unknown

@github-actions github-actions bot added the tests label Mar 17, 2026
@dnmokhov dnmokhov changed the title Check for cgroups constraints Accomodate cgroups constraints in the NUMA task arena conformance test Mar 17, 2026
@dnmokhov
Copy link
Copy Markdown
Contributor Author

@isaevil and @aleksei-fedotov, it seems that this test did not take cgroups into account. Feel free to refine or correct the added logic as needed.

@dnmokhov dnmokhov changed the title Accomodate cgroups constraints in the NUMA task arena conformance test Account for cgroups constraints in the NUMA task arena conformance test Mar 17, 2026
Copy link
Copy Markdown
Contributor

@aleksei-fedotov aleksei-fedotov left a comment

Choose a reason for hiding this comment

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

Consider this PR as a way to improve readability of the test.

dnmokhov and others added 2 commits March 18, 2026 12:27
Co-authored-by: Aleksei Fedotov <fedotov.aleksei@gmail.com>
Co-authored-by: Ilya Isaev <ilya.isaev@intel.com>
@isaevil isaevil merged commit ef6f9b5 into master Mar 19, 2026
29 checks passed
@isaevil isaevil deleted the dev/dnmokhov/create_numa_task_arenas branch March 19, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants