[PR #2469] [CLOSED] Adjust ROCm v5 build #10901

Closed
opened 2026-04-12 23:14:48 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/2469
Author: @dhiltgen
Created: 2/12/2024
Status: Closed

Base: mainHead: rocm_v5


📝 Commits (1)

📊 Changes

2 files changed (+18 additions, -4 deletions)

View changed files

📝 .github/workflows/test.yaml (+2 -1)
📝 Dockerfile (+16 -3)

📄 Description

It looks like within the major version 5 family of ROCm releases, there were two significant library major version bumps that seem to be causing problems for some users on some distros. In 5.6, they bumped hipblas and rocblas libraries from .so.0 to newer versions, and do not bundle the old versions in new packages. While you can install multiple versions, this complicates the UX. Building on either side of this library change should hopefully yield a llm variant that can link on their system.

Related to #738

Marking draft for now until I can test this on more permutations of drivers/rocm libraries to see if it truly makes things better both for the pre-built binaries, and the container image.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/2469 **Author:** [@dhiltgen](https://github.com/dhiltgen) **Created:** 2/12/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `rocm_v5` --- ### 📝 Commits (1) - [`186ee35`](https://github.com/ollama/ollama/commit/186ee3560123c966a640e4e4f01f6b560ed3ccb4) Adjust ROCm v5 build ### 📊 Changes **2 files changed** (+18 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test.yaml` (+2 -1) 📝 `Dockerfile` (+16 -3) </details> ### 📄 Description It looks like within the major version 5 family of ROCm releases, there were two significant library major version bumps that seem to be causing problems for some users on some distros. In 5.6, they bumped hipblas and rocblas libraries from .so.0 to newer versions, and do not bundle the old versions in new packages. While you can install multiple versions, this complicates the UX. Building on either side of this library change should hopefully yield a llm variant that can link on their system. Related to #738 Marking draft for now until I can test this on more permutations of drivers/rocm libraries to see if it truly makes things better both for the pre-built binaries, and the container image. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-12 23:14:48 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#10901