Skip to content
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

Update ROCm artifacts to newer version #440

Open
pxl-th opened this issue Jul 6, 2023 · 2 comments
Open

Update ROCm artifacts to newer version #440

pxl-th opened this issue Jul 6, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@pxl-th
Copy link
Member

pxl-th commented Jul 6, 2023

Since #423 we now require at least ROCm 5.3, but we probably should update to 5.4 which works fine with Julia 1.9.

@pxl-th pxl-th added the enhancement New feature or request label Jul 6, 2023
@vchuravy
Copy link
Member

I don't think removing the artifacts is the right answer.
This breaks down-stream JLL being able to build against ROCM and makes it harder for us to understand what version the user is using.

julia> AMDGPU.versioninfo()
Using ROCm provided by: System
HSA Runtime (ready)
- Path: /opt/rocm/hsa/lib/libhsa-runtime64.so
- Version: 1.1.0
ld.lld (ready)
- Path: /opt/rocm/llvm/bin/ld.lld
ROCm-Device-Libs (ready)
- Path: /opt/rocm/amdgcn/bitcode
HIP Runtime (ready)
- Path: /opt/rocm/hip/lib/libamdhip64.so
rocBLAS (ready)
- Path: /opt/rocm/lib/librocblas.so
rocSOLVER (ready)
- Path: /opt/rocm/lib/librocsolver.so
rocALUTION (ready)
- Path: /opt/rocm/lib/librocalution.so
rocSPARSE (ready)
- Path: /opt/rocm/lib/librocsparse.so
rocRAND (ready)
- Path: /opt/rocm/lib/librocrand.so
rocFFT (ready)
- Path: /opt/rocm/lib/librocfft.so
MIOpen (ready)
- Path: /opt/rocm/lib/libMIOpen.so
HIP Devices (2):
- HIPDevice(name="AMD Radeon Graphics", id=1)
- HIPDevice(name="AMD Radeon RX 6800 XT", id=2)

Should at least provide what version of HSA/HIP I am using.

@pxl-th
Copy link
Member Author

pxl-th commented Jul 26, 2023

Existing artifacts do not work properly with stream-ordered allocator, so I saw no point in keeping them around.
Once we update artifacts we can add them back, but for now we might as well save some space.

#452 will print versions for all currently used libraries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants