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

[sa-submission-q1-2026] Add numa core binding for all gb200 and gb300 configs#164

Open
trevor-m wants to merge 1 commit into
sa-submission-q1-2026from
trevor-m/numa
Open

[sa-submission-q1-2026] Add numa core binding for all gb200 and gb300 configs#164
trevor-m wants to merge 1 commit into
sa-submission-q1-2026from
trevor-m/numa

Conversation

@trevor-m

@trevor-m trevor-m commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Grace-blackwell systems are composed of two numa nodes. Node 0 contains GPUs 0,1 and cpu cores 0-71. Node 1 contains GPUs 2,3 and cpu cores 72-144. Each numa node also has a unified memory between its cpus and gpus. This PR ensures that worker processes are bound to the correct numa node for their gpu that they are assigned to.

@trevor-m trevor-m changed the title [sa-submussion-q1-2026] Add numa core binding for all gb200 and gb300 configs [sa-submission-q1-2026] Add numa core binding for all gb200 and gb300 configs Feb 9, 2026
@coderabbitai

coderabbitai Bot commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch trevor-m/numa

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nvcastet

Copy link
Copy Markdown
Contributor

@ishandhanani Shall we merge that one?

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