Skip to content

Fix nvexec scheduler equality across stream priorities - #2200

Merged
ericniebler merged 2 commits into
NVIDIA:mainfrom
fallintoplace:fix/nvexec-scheduler-priority-equality
Aug 15, 2026
Merged

Fix nvexec scheduler equality across stream priorities#2200
ericniebler merged 2 commits into
NVIDIA:mainfrom
fallintoplace:fix/nvexec-scheduler-priority-equality

Conversation

@fallintoplace

@fallintoplace fallintoplace commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • include stream priority when comparing stream schedulers
  • apply the same check to multi-GPU stream schedulers
  • add regression coverage for equal resources, different priorities, and different contexts
  • avoid constructing a second multi-GPU context in the regression test

Schedulers with the same hub but different priorities use different stream pools. They should not compare equal.

Tests:

  • cmake --build build-cuda --target test.nvexec -j 8
  • ./build-cuda/test/nvexec/test.nvexec --reporter compact

@copy-pr-bot

copy-pr-bot Bot commented Aug 15, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ericniebler

Copy link
Copy Markdown
Collaborator

/ok to test 6cf193a

@ericniebler
ericniebler merged commit 035858e into NVIDIA:main Aug 15, 2026
67 of 68 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