Skip to content

fix: parse vulkan VRAM from text#9669

Open
eglia wants to merge 1 commit intomudler:masterfrom
eglia:vulkan-vram
Open

fix: parse vulkan VRAM from text#9669
eglia wants to merge 1 commit intomudler:masterfrom
eglia:vulkan-vram

Conversation

@eglia
Copy link
Copy Markdown
Contributor

@eglia eglia commented May 5, 2026

Description
This PR fixes the vulkan VRAM parsing. As far as I can tell, the current implementation is nonsense, because vulkaninfo --json does not write to stdout:

[-j, --json]         Produce a json version of vulkaninfo output conforming to the Vulkan
                     Profiles schema, saved as 
                     "VP_VULKANINFO_[DEVICE_NAME]_[DRIVER_VERSION].json"
                     of the first gpu in the system.

Also, on my machine the json info did not actually contain the heap information, which is present in the text output.
This PR therefore changes to vulkaninfo --text and parses the output. On my 32GB AMD GPU, this correctly detects the available VRAM.
New output:
Total available VRAM vram=34208743424 caller={caller.file="/build/pkg/system/state.go" caller.L=79 }
Old output:
Total available VRAM vram=0 caller={caller.file="/build/pkg/system/state.go" caller.L=79 }

This PR fixes #9289

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Assisted-by: opencode:gpt-5.5
Signed-off-by: Andreas Egli <github@kharan.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Intel Arc Pro B70 VRAM not parsed

1 participant