Skip to content

Conversation

channeladam
Copy link
Contributor

@channeladam channeladam commented Sep 14, 2025

https://rocm.docs.amd.com/projects/install-on-linux/en/docs-6.4.1/reference/system-requirements.html#rdna-os states the Radeon RX 9000 series is supported from Ubuntu 24.04.2, and the dockerfile is using 24.04 which is ROCm 6.4.

This fixed the ROCm error: invalid device function I was getting when trying to use the rocm container.

Make sure to read the contributing guidelines before submitting a PR

…adeon RX 9000 series

https://rocm.docs.amd.com/projects/install-on-linux/en/docs-6.4.1/reference/system-requirements.html#rdna-os
states the Radeon RX 9000 series is supported support from Ubuntu 24.04.2, and the dockerfile is using 24.04 which is ROCm 6.4.

This fixed the `ROCm error: invalid device function` I was getting when trying to use the rocm container.
@channeladam channeladam requested a review from ngxson as a code owner September 14, 2025 18:20
@github-actions github-actions bot added the devops improvements to build systems and github actions label Sep 14, 2025
@channeladam
Copy link
Contributor Author

@slaren as discussed in #15106

ARG AMDGPU_VERSION=6.4

# Target the CUDA build image
# Target the ROCM build image
Copy link
Member

Choose a reason for hiding this comment

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

There is another comment in line 23 about "nvcc" that could also be updated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@slaren
Copy link
Member

slaren commented Sep 14, 2025

I also noticed that this image is built without rocWMMA, so performance is likely not going to be optimal.

@channeladam
Copy link
Contributor Author

I also noticed that this image is built without rocWMMA, so performance is likely not going to be optimal.

Sorry, I'm not familiar with that... best I don't make that change!

@slaren slaren merged commit 0fa154e into ggml-org:master Sep 14, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants