Skip to content

fix(openvino): explicit memset in buffer_context allocation#20857

Merged
ggerganov merged 2 commits intoggml-org:masterfrom
thedanhoffman:openvino_buffer_context_memset
Mar 23, 2026
Merged

fix(openvino): explicit memset in buffer_context allocation#20857
ggerganov merged 2 commits intoggml-org:masterfrom
thedanhoffman:openvino_buffer_context_memset

Conversation

@thedanhoffman
Copy link
Copy Markdown
Contributor

This was causing a variety of Valgrind errors when running locally. Weight-tensor initialization seems to be done via a copy from the GGUF, but non-weight tensor information is done via copies from uninitialized memory. The initialization right here is pretty defensive, but it solves the issue on my end at least

@github-actions github-actions Bot added ggml changes relating to the ggml tensor library for machine learning OpenVINO labels Mar 22, 2026
@ggerganov ggerganov requested a review from wine99 March 22, 2026 06:31
Comment thread ggml/src/ggml-openvino/ggml-openvino.cpp Outdated
@ggerganov ggerganov merged commit cc18f96 into ggml-org:master Mar 23, 2026
1 check passed
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
…#20857)

* fix(openvino): explicit memset in buffer_context allocation

* minor

---------

Co-authored-by: Dan Hoffman <dhoffman@cyket.net>
Co-authored-by: Georgi Gerganov <ggerganov@gmail.com>
rsenthilkumar6 pushed a commit to rsenthilkumar6/llama.cpp that referenced this pull request May 1, 2026
…#20857)

* fix(openvino): explicit memset in buffer_context allocation

* minor

---------

Co-authored-by: Dan Hoffman <dhoffman@cyket.net>
Co-authored-by: Georgi Gerganov <ggerganov@gmail.com>
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 OpenVINO

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants