[PR #5503] [MERGED] Workaround broken ROCm p2p copy #43052

Closed
opened 2026-04-24 22:45:21 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/5503
Author: @dhiltgen
Created: 7/5/2024
Status: Merged
Merged: 7/9/2024
Merged by: @dhiltgen

Base: mainHead: dual_rocm


📝 Commits (1)

  • 0bacb30 Workaround broken ROCm p2p copy

📊 Changes

2 files changed (+2 additions, -1 deletions)

View changed files

📝 llm/generate/gen_linux.sh (+1 -1)
📝 llm/generate/gen_windows.ps1 (+1 -0)

📄 Description

Enable the build flag for llama.cpp to use CPU copy for multi-GPU scenarios.

Marking draft until I can validate this doesn't cause a significant performance hit for single GPU ROCm scenarios.

Update: perf results look good - no regressions detected in my testing.

Fixes #5087 #5450


🔄 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/5503 **Author:** [@dhiltgen](https://github.com/dhiltgen) **Created:** 7/5/2024 **Status:** ✅ Merged **Merged:** 7/9/2024 **Merged by:** [@dhiltgen](https://github.com/dhiltgen) **Base:** `main` ← **Head:** `dual_rocm` --- ### 📝 Commits (1) - [`0bacb30`](https://github.com/ollama/ollama/commit/0bacb300071ba4baa928075b142633f2e85281ab) Workaround broken ROCm p2p copy ### 📊 Changes **2 files changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `llm/generate/gen_linux.sh` (+1 -1) 📝 `llm/generate/gen_windows.ps1` (+1 -0) </details> ### 📄 Description Enable the build flag for llama.cpp to use CPU copy for multi-GPU scenarios. ~~Marking draft until I can validate this doesn't cause a significant performance hit for single GPU ROCm scenarios.~~ Update: perf results look good - no regressions detected in my testing. Fixes #5087 #5450 --- <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-24 22:45:21 -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#43052