Skip to content

vulkan: fix output corruption on GCN 2.0/3.0 (Vulkan 1.2)#21787

Open
rafikb wants to merge 1 commit intoggml-org:masterfrom
rafikb:fix/vulkan-gcn2-sync
Open

vulkan: fix output corruption on GCN 2.0/3.0 (Vulkan 1.2)#21787
rafikb wants to merge 1 commit intoggml-org:masterfrom
rafikb:fix/vulkan-gcn2-sync

Conversation

@rafikb
Copy link
Copy Markdown

@rafikb rafikb commented Apr 12, 2026

Overview

This PR resolves output corruption (token repetition/double-wording) on legacy AMD GCN 2.0 and 3.0 hardware (e.g., R9 390, Fury X).

The issue was introduced in b8394 due to the implementation of modern Vulkan synchronization primitives that are not fully supported by the final legacy Adrenalin drivers (Vulkan 1.2.170). By restoring backward-compatible memory barrier logic in ggml-vulkan.cpp, we restore coherent generation for these cards without regressing performance on modern Vulkan 1.3+ hardware.

Additional information

  • Hardware Tested: Radeon R9 390 (8GB VRAM) on Windows 10.

  • Performance: Verified 22.0 t/s using gemma-4-E4B-it-Q4_K_M.gguf with 100% GPU offload.

  • Context: Many legacy 8GB cards are still highly capable for inference in 2026, but are limited by driver-level Vulkan support. This fix ensures continued accessibility for users on vintage or budget-restricted hardware.

Requirements

@rafikb rafikb requested a review from a team as a code owner April 12, 2026 02:13
@github-actions github-actions bot added Vulkan Issues specific to the Vulkan backend ggml changes relating to the ggml tensor library for machine learning labels Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ggml changes relating to the ggml tensor library for machine learning Vulkan Issues specific to the Vulkan backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant