Skip to content

UPSTREAM PR #19254: Add test for vk_buffer from host memory#1126

Open
loci-dev wants to merge 1 commit intomainfrom
loci/pr-19254-work-sredman-vulkan-external-memory-host
Open

UPSTREAM PR #19254: Add test for vk_buffer from host memory#1126
loci-dev wants to merge 1 commit intomainfrom
loci/pr-19254-work-sredman-vulkan-external-memory-host

Conversation

@loci-dev
Copy link
Copy Markdown

@loci-dev loci-dev commented Feb 1, 2026

Note

Source pull request: ggml-org/llama.cpp#19254

Add a test to validate external host memory can correctly be allocated, written, used, and read.

I already had this test for my own external memory buffer solution, but then I noticed #18467 had done the work. I thought the test might still be useful.

@loci-review
Copy link
Copy Markdown

loci-review bot commented Feb 1, 2026

No meaningful performance changes were detected across 115327 analyzed functions in the following binaries: build.bin.libllama.so, build.bin.llama-tts, build.bin.llama-cvector-generator, build.bin.libmtmd.so, build.bin.libggml-base.so, build.bin.libggml-cpu.so, build.bin.libggml.so, build.bin.llama-gguf-split, build.bin.llama-llava-cli, build.bin.llama-minicpmv-cli, build.bin.llama-quantize, build.bin.llama-gemma3-cli, build.bin.llama-bench, build.bin.llama-tokenize, build.bin.llama-qwen2vl-cli.

🔎 Full breakdown: Loci Inspector.
💬 Questions? Tag @loci-dev.

@loci-dev loci-dev force-pushed the main branch 26 times, most recently from 6efe498 to 8db062d Compare February 3, 2026 06:25
@loci-dev loci-dev force-pushed the main branch 11 times, most recently from 4a5a4c2 to 45aacad Compare February 24, 2026 02:17
@loci-dev loci-dev force-pushed the main branch 8 times, most recently from 13648e6 to 1d064d0 Compare March 3, 2026 02:17
@loci-dev loci-dev force-pushed the main branch 8 times, most recently from 551dfb5 to 55a969e Compare March 11, 2026 02:16
@loci-dev loci-dev force-pushed the main branch 3 times, most recently from 61601b2 to 56aaa36 Compare March 13, 2026 02:16
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.

2 participants