-
-
Notifications
You must be signed in to change notification settings - Fork 7k
ROCm-on-WSL: support discrete Radeon (RDNA 3/4) in WSL, not just Strix Halo #6915
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
9a7615f
WSL ROCm: generalize ROCm-on-WSL bootstrap from Strix-only to any RDN…
LeoBorcherding 7e974c8
WSL ROCm: trigger the ROCm-on-WSL bootstrap for discrete Radeon GPUs too
LeoBorcherding 8b03188
WSL ROCm: address review -- filter generic ISA in bootstrap, bound th…
LeoBorcherding 07af417
WSL ROCm: harden arch-detect + librocdxg copy under set -eo pipefail …
LeoBorcherding d1834af
Merge branch 'main' into amd-wsl-fix
LeoBorcherding 0f37e70
WSL ROCm: address Codex review (gfx000, 24.04 reroute for discrete, t…
LeoBorcherding 03ff51c
Condense ROCm-on-WSL comments in install.sh and bootstrap helper
danielhanchen be8cf07
Guard WSL reroute from NVIDIA hybrid hosts and fix GFX-override pipef…
danielhanchen 9b7a358
Honor CUDA_VISIBLE_DEVICES-hidden NVIDIA in the WSL reroute guard
danielhanchen 4dcbb74
Reuse _has_usable_nvidia_gpu in the WSL reroute guard
danielhanchen d6e036d
Merge remote-tracking branch 'origin/main' into amd-wsl-fix
danielhanchen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When a WSL host has an AMD integrated adapter alongside a usable NVIDIA GPU, this WMI fallback returns true before the installer ever reaches the NVIDIA guard in
_maybe_bootstrap_rocm_wslorget_torch_index_url. On a non-24.04 distro with anUbuntu-24.04distro present, the installer reroutes there and exits after the child install, so the current distro never gets the CUDA path even though NVIDIA would work. Please skip this AMD reroute when a usable NVIDIA GPU is present, or make the WMI fallback distinguish the intended AMD-only target.Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in be8cf07: _maybe_reroute_strixhalo_to_2404 now probes nvidia-smi -L (awk-consumed, so pipefail-safe) before the AMD/WMI signal and returns early when a usable NVIDIA GPU is present, so a hybrid AMD+NVIDIA host keeps the CUDA path on this distro instead of rerouting for AMD. Added a regression test.