[PR #11140] [MERGED] ggml: Check return status for computation. #44701

Closed
opened 2026-04-25 00:19:54 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/11140
Author: @jessegross
Created: 6/19/2025
Status: Merged
Merged: 6/20/2025
Merged by: @jessegross

Base: mainHead: jessegross/compute


📝 Commits (1)

  • 8ace61f ggml: Check return status for computation.

📊 Changes

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

View changed files

📝 ml/backend/ggml/ggml.go (+3 -1)

📄 Description

We don't check the return status after computing the graph, which can silently lead to bad outputs if we try to keep going and future computation succeeds. This appears to happens in certain cases on Apple M2 devices.

Fixes #11070


🔄 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/11140 **Author:** [@jessegross](https://github.com/jessegross) **Created:** 6/19/2025 **Status:** ✅ Merged **Merged:** 6/20/2025 **Merged by:** [@jessegross](https://github.com/jessegross) **Base:** `main` ← **Head:** `jessegross/compute` --- ### 📝 Commits (1) - [`8ace61f`](https://github.com/ollama/ollama/commit/8ace61fa4f1ceb7642e6bf2ea13388f252c190be) ggml: Check return status for computation. ### 📊 Changes **1 file changed** (+3 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `ml/backend/ggml/ggml.go` (+3 -1) </details> ### 📄 Description We don't check the return status after computing the graph, which can silently lead to bad outputs if we try to keep going and future computation succeeds. This appears to happens in certain cases on Apple M2 devices. Fixes #11070 --- <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-25 00:19:54 -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#44701