-
-
Notifications
You must be signed in to change notification settings - Fork 17.2k
rocmPackages: 6.3.3 -> 6.4.3 #427944
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
rocmPackages: 6.3.3 -> 6.4.3 #427944
Conversation
e8872b0 to
f034276
Compare
This comment was marked as outdated.
This comment was marked as outdated.
ed91505 to
8d8c252
Compare
8d8c252 to
11a7af6
Compare
29e7d0c to
2758d52
Compare
This comment was marked as outdated.
This comment was marked as outdated.
8089e29 to
87e341b
Compare
6715162 to
8b7c3c1
Compare
8b7c3c1 to
251034d
Compare
This comment was marked as outdated.
This comment was marked as outdated.
286e46c to
29c2885
Compare
| rocm-core = self.callPackage ./rocm-core { | ||
| inherit stdenv; | ||
| }; |
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.
Shouldn’t need to inherit it if it’s in the scope. (Applies to a bunch of stuff in this file.)
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.
so this actually changes the stdenv because this is inheriting the original normal stdenv from pkgs from outside the package set. I need to do some renaming.
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.
renamed but it's still not very clear.
9127b6c to
9c6d9ff
Compare
| # "950" # MI350X TODO: Expected in ROCm 7.x | ||
| # "10-1-generic" # fine for all RDNA1 cards | ||
| "1010" | ||
| "1012" |
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.
It was an error for us to build 1012 by default in previous versions, our ISA compat patches allow 1010 to load for 1012 so this just wasted resources.
gfx1151 support in torch was fixed by the ROCm 6.4 bump
… supported targets are set Allows building torchWithRocm with clr's localGpuTargets set to gfx906 and other arches which composable_kernel does not support.
Will be required for CMake 4 bump which is hopefully happening in nixpkgs soon
f7734fc to
62a423d
Compare
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.
@emilazy sadly migraphx still needs a patch. going to try to upstream this.
f5eacbb to
ec546ee
Compare
ec546ee to
d844a2f
Compare
|
llama-cpp failure to investigate: unsupported insn in a very recently added llama-cpp feature ggml-org/llama.cpp#15884 |
|
Fixes #412640
Fixes #418825
Fixes #434280
TODO
Test gfx11-generic handles strix halo if anyone is available to test with one of the new ryzen AI max boxesTest gfx9-generic handles early vegaThings done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.