Skip to content

Conversation

@boberfly
Copy link

@boberfly boberfly commented Nov 12, 2021

Hello all!

This is the first time figuring out how the aswf-docker image works, so I apologise if I didn't do it quite right!

This adds AMD ROCm https://rocmdocs.amd.com/en/latest/ to the image. ROCm/HIP is GPU compute on AMD hardware, similar to Nvidia's CUDA.

I would like to build Cycles-X with ROCm support so this would be great to add into the main image. I've yet to get this running yet (it does build) but I thought I'd put up a PR to see if I've got this working mostly correct...

@bsavery

Cheers!

@bsavery
Copy link

bsavery commented Nov 12, 2021

The supported linux driver is upcoming. Let me see if I can find more info

@aloysbaillet
Copy link
Contributor

Hi! Thanks for the contribution, do you know if rocm would work in the same docker image as the one we use which already contain the NVIDIA drivers? Have you been able to test the resulting image?
Also you'll need to get the DCO check passing (just adding you email address to your checkin comment, you can follow the instructions in the contributing doc).

ENV LD_LIBRARY_PATH=/usr/local/lib:/usr/local/lib64:/opt/rh/httpd24/root/usr/lib64:/opt/rh/devtoolset-${ASWF_DTS_VERSION}/root/usr/lib64:/opt/rh/devtoolset-${ASWF_DTS_VERSION}/root/usr/lib:${LD_LIBRARY_PATH} \
PATH=/opt/rh/rh-git218/root/usr/bin:/usr/local/bin:/opt/rh/devtoolset-${ASWF_DTS_VERSION}/root/usr/bin:/opt/app-root/src/bin:/opt/rh/devtoolset-${ASWF_DTS_VERSION}/root/usr/bin/:/usr/local/sbin:/usr/sbin:/usr/bin:/sbin:/bin \
ENV LD_LIBRARY_PATH=/opt/rocm/lib:/usr/local/lib:/usr/local/lib64:/opt/rh/httpd24/root/usr/lib64:/opt/rh/devtoolset-${ASWF_DTS_VERSION}/root/usr/lib64:/opt/rh/devtoolset-${ASWF_DTS_VERSION}/root/usr/lib:${LD_LIBRARY_PATH} \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For safety I would prefer if you added the rocm path at the end of the LD_LIBRARY_PATH

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants