-
Notifications
You must be signed in to change notification settings - Fork 45
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
LB Julia to 1.7, upgrade to ROCm 4.2 #187
Conversation
BLAS/RNG failures will probably be solvable by shipping rocRAND_jll (JuliaPackaging/Yggdrasil#4250) and rocBLAS_jll (JuliaPackaging/Yggdrasil#4255) or by updating the CI container to a newer ROCm. |
Next step would be to use methods overlays for device functionality? |
Yeah! Do we want that in this PR, or separately? |
Seperatly is fine |
@jpsamaroo, testing current branch on Vega20 test system:
|
cdbdc65
to
91c7e37
Compare
@jpsamaroo - Reporting about tests of Observations:
ERROR stack
ERROR stack
|
I still get ┌ Warning: ld.lld was not found, compilation functionality will be unavailable.
│ Please run Pkg.build("AMDGPU") and reload AMDGPU.
│ Reason: ld.lld executable not found
└ @ AMDGPU ~/Documents/github/dotFiles/julia/.julia/dev/AMDGPU/src/AMDGPU.jl:207
┌ Warning: ROCm-Device-Libs were not found, device intrinsics will be unavailable.
│ Please run Pkg.build("AMDGPU") and reload AMDGPU.
│ Reason: unknown
└ @ AMDGPU ~/Documents/github/dotFiles/julia/.julia/dev/AMDGPU/src/AMDGPU.jl:226
┌ Warning: rocRAND failed to load, RNG functionality will be unavailable.
│ Please run Pkg.build("AMDGPU") and reload AMDGPU.
│ Reason: false
└ @ AMDGPU ~/Documents/github/dotFiles/julia/.julia/dev/AMDGPU/src/AMDGPU.jl:235
┌ Error: Exception while generating log record in module Main at /home/akako/Documents/github/dotFiles/julia/.julia/dev/AMDGPU/test/runtests.jl:26
│ exception =
│ UndefRefError: access to undefined reference is there special driver requirement? I'm not using AMDPRO on Linux |
You need to do The power kernel failures are related to us not using method overlay tables like CUDA.jl (intentionally so, because on Julia 1.6 we would break CUDA by using them). Since we're lower-bounding Julia to 1.7, and I've already implemented this locally, I'll push those changes to this PR for testing. |
@Moelf can you please post |
@vchuravy method table overlays added! |
|
We can run without it for many things, and since it often blocks usage of certain cards due to questionable behavior in its library initializer, it is worth not requiring that HIP be available. Fix hipStream_t type in rocBLAS wrapper Add rocSOLVER stubs
2bb4390
to
79ec593
Compare
Co-authored-by: Valentin Churavy <vchuravy@users.noreply.github.com>
79ec593
to
c22fd01
Compare
@luraess
Todo:
rocRAND_jll
Addupgrade CI to 4.2rocBLAS_jll
orld.lld
fromLLVM_jll