Skip to content

vulkan: handle misalignment in conv_2d and conv_3d - #29365

Merged
0cc4m merged 2 commits into
masterfrom
0cc4m/vulkan-conv-mm-misalignment-fix
Sep 24, 2026
Merged

0cc4m merged 2 commits into
masterfrom
0cc4m/vulkan-conv-mm-misalignment-fix

Conversation

@0cc4m

@0cc4m 0cc4m commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Overview

Fixes the Vulkan CI issues in the tests added by #29137. Also fix the same issue in CONV_2D and add tests for it.

Requirements

@0cc4m
0cc4m requested review from a team and ggerganov as code owners September 24, 2026 09:43
@github-actions github-actions Bot added testing Everything test related Vulkan Issues specific to the Vulkan backend ggml changes relating to the ggml tensor library for machine learning labels Sep 24, 2026
Comment thread tests/test-backend-ops.cpp Outdated

std::string vars() override {
return VARS_TO_STR10(ne_input, ne_kernel, type_kernel, stride0, stride1, padding0, padding1, dilation0, dilation1, cwhn);
return VARS_TO_STR10(ne_input, ne_kernel, type_kernel, stride0, stride1, padding0, padding1, dilation0, dilation1, cwhn) +

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vars_to_str11?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh damn, oops.

@ggerganov

Copy link
Copy Markdown
Member

@0cc4m Is it OK to fast-track merge this ?

@0cc4m
0cc4m merged commit 6b790a9 into master Sep 24, 2026
20 of 21 checks passed
@0cc4m
0cc4m deleted the 0cc4m/vulkan-conv-mm-misalignment-fix branch September 24, 2026 12:08
@0cc4m

0cc4m commented Sep 24, 2026

Copy link
Copy Markdown
Contributor Author

I had just been waiting for the CI to finish.

@BrewTestBot BrewTestBot mentioned this pull request Sep 24, 2026
1 task done
frostyautumnleaf pushed a commit to frostyautumnleaf/llama.cpp that referenced this pull request Oct 5, 2026
* vulkan: handle misalignment in conv_2d and conv_3d

* fix test-backend-ops print
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 7, 2026
* vulkan: handle misalignment in conv_2d and conv_3d

* fix test-backend-ops print

(cherry picked from commit 6b790a9)
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 testing Everything test related Vulkan Issues specific to the Vulkan backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants