rocmPackages: 7.1.1 -> 7.2.0#484792
Conversation
410df74 to
a74ad69
Compare
a74ad69 to
dbca05e
Compare
|
Thanks @GZGavinZhao for providing continued support in the form of the ISA compat patch set. Co-Authored-By: Gavin Zhao <git@gzgz.dev>
Reverting a patch that caused terrible performance for some large langle mangle workloads. See also rocm-systems issue 2865 and rocm llvm-project pull 1349
Files were moved to DVC in rocm-libraries PR 1594
06kellyjac
left a comment
There was a problem hiding this comment.
2 packages with an update but no hash change, just need to double check nothing actually changed there & is a bump in name only.
A few nits on sha256 -> hash but not very important.
Looks very good. I'll get a build going later 🙂
There was a problem hiding this comment.
Specifically calling out this DVC stuff as odd, is there a nicer approach I'm missing than this script and the generated kdbs file full of s3 paths?
Flakebi
left a comment
There was a problem hiding this comment.
Changes look good to me (though @06kellyjac left some good comments).
Thanks for your tremendous effort on these updates!
|
Applied a rocm-modules wide swap from |
There was a problem hiding this comment.
I built some of the packages on my system with no errors
nix build 'github:LunNova/nixpkgs/3bc0e6a4c5fc9c8652694b0eaccebd8b1c419f5e#rocmPackages.amdsmi'
nix build 'github:LunNova/nixpkgs/3bc0e6a4c5fc9c8652694b0eaccebd8b1c419f5e#rocmPackages.clr'thanks for the titanic work 💪🏻 🎉
|
|
|
It seems that the error is propagated from: https://hydra.nixos.org/build/323309073. So it's a problem with the |
If it's OOM again this is a pretty annoying issue that should be fixed tbh. My laptop can't handle building hipblaslt :( |
|
it's not an oom in hipblaslt this time. the source tree exceeds the output limit. new hipblaslt failure mode! :( |
About the OOMs: hipblaslt's tensilelite build system needs a rewrite or significant rearchitecture, upstream are working on it but no timeline, I've done the best that I can with the resources I have available in the patch set we carry. I'm sorry about the trouble it causes. |
Issues encountered so far:
Missing tags for some reposEval diff
5f83595 (+0 -0 ~79 ) rocmPackages.llvm: use clang config file instead of deprecated GCC_INSTALL_PREFIX
691e492 (+0 -0 ~92 ) rocmPackages: 7.1.1 -> 7.2.0
331070f (+0 -0 ~82 ) rocmPackages.llvm: revert loop unrolling pessimisation
3376d1e (+0 -0 ~0 ) rocmPackages.miopen: add updateScript for DVC S3 kdb files
3bc0e6a (+0 -0 ~0 ) rocmPackages: sha256 = -> hash =
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.