[PR #10150] discover: make unique_id check optional for AMD GPU detection #44408

Open
opened 2026-04-24 23:54:16 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/10150
Author: @vlttnv
Created: 4/6/2025
Status: 🔄 Open

Base: mainHead: main


📝 Commits (1)

  • 0a0752e discover: make unique_id check optional for AMD GPU detection

📊 Changes

1 file changed (+9 additions, -3 deletions)

View changed files

📝 discover/amd_linux.go (+9 -3)

📄 Description

This fix handles systems where DRMUniqueIDFile is absent, allowing GPU detection to continue based on vendor and device ID matching.

Background

Ollama was failing to detect an RX 9070 XT when running on Ubuntu 24.04 while llama-cpp was detecting it.
While debugging I noticed that the match was failing since it was unable to find DRMUniqueIDFile which from what I understand is optional and not guaranteed to be there.

Once I implemented the change the card was immediately detected.


🔄 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/10150 **Author:** [@vlttnv](https://github.com/vlttnv) **Created:** 4/6/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (1) - [`0a0752e`](https://github.com/ollama/ollama/commit/0a0752e714e7a8259c50e104217961036893ec8e) discover: make unique_id check optional for AMD GPU detection ### 📊 Changes **1 file changed** (+9 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `discover/amd_linux.go` (+9 -3) </details> ### 📄 Description This fix handles systems where `DRMUniqueIDFile` is absent, allowing GPU detection to continue based on vendor and device ID matching. ### Background Ollama was failing to detect an RX 9070 XT when running on Ubuntu 24.04 while llama-cpp was detecting it. While debugging I noticed that the match was failing since it was unable to find `DRMUniqueIDFile` which from what I understand is optional and not guaranteed to be there. Once I implemented the change the card was immediately detected. --- <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 23:54:16 -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#44408