Skip to content

nvbug6084457: Fix device architecture handling and NVLink link count query#1937

Open
mdboom wants to merge 4 commits intoNVIDIA:mainfrom
mdboom:nvbug6084457
Open

nvbug6084457: Fix device architecture handling and NVLink link count query#1937
mdboom wants to merge 4 commits intoNVIDIA:mainfrom
mdboom:nvbug6084457

Conversation

@mdboom
Copy link
Copy Markdown
Contributor

@mdboom mdboom commented Apr 17, 2026

Filing as a draft because this is still only a partial fix for the reported bug. The final fix requires coordination with upstream NVML.

@copy-pr-bot
Copy link
Copy Markdown
Contributor

copy-pr-bot bot commented Apr 17, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions bot added the Needs-Restricted-Paths-Review PR touches cuda_bindings or cuda_python; only NVIDIA employees may modify these paths; see LICENSEs label Apr 17, 2026
@mdboom mdboom self-assigned this Apr 17, 2026
@mdboom mdboom added bug Something isn't working test Improvements or additions to tests cuda.bindings Everything related to the cuda.bindings module labels Apr 17, 2026
@mdboom mdboom added this to the cuda.bindings next milestone Apr 17, 2026
Comment thread cuda_core/cuda/core/system/_device.pyx Outdated
Co-authored-by: Phillip Cloud <417981+cpcloud@users.noreply.github.com>
@mdboom mdboom marked this pull request as ready for review April 20, 2026 12:34
@mdboom mdboom requested a review from cpcloud April 20, 2026 12:34
@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 20, 2026

Marking this as "ready to review". As a follow-on when we do the 13.3 bring up, we will need to add logic to make NVML_NVLINK_MAX_LINK dynamic based on the CTK version, but no point in doing that now.

@github-actions
Copy link
Copy Markdown

@rwgk rwgk removed the Needs-Restricted-Paths-Review PR touches cuda_bindings or cuda_python; only NVIDIA employees may modify these paths; see LICENSEs label Apr 21, 2026
@github-actions github-actions bot added the cuda.core Everything related to the cuda.core module label Apr 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cuda.bindings Everything related to the cuda.bindings module cuda.core Everything related to the cuda.core module test Improvements or additions to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants