Skip to content

Conversation

sungeunk
Copy link
Contributor

@sungeunk sungeunk commented Jul 31, 2025

Description of the issue(symptom, root-cause, how it was resolved)

convolution in/out tensor
input tensor: f16:b_fs_yx_fsv16:1x512x78:nopad
input tensor: f16:bfyx:512x512x3:nopad
input tensor: f16:bfyx:1x512x1:nopad
input tensor: f16:b_fs_yx_fsv16:1x512x78:nopad --> binary_add
input tensor: f16:b_fs_yx_fsv16:1x1x1:nopad --> binary_mul
output tensor: f16:b_fs_yx_fsv16:1x512x78:nopad

  • onednn memory desc of conv/deconv needs setting onednn::mem_flags::need_blocked if the format is blocked.

The code and line that caused this issue (if it is not changed directly)

  • src/plugins/intel_gpu/thirdparty/onednn_gpu/src/common/primitive_attr.cpp (validate_binary)

Reproduction step and snapshot (model is attached in the ticket)

  • benchmark_app.exe -d GPU --hint latency -niter 1 -m kokoro.xml -shape "ref_s[1,256],input_ids[1,78]"

Problematic graph

  • This issue could not check with graph.

Checklist

  • Is it a proper fix? (not a workaround)
  • Did you include test case for this fix, if necessary?
  • Did you review existing test that can be extended to cover this scenario? Which test did you review?
    -- unit/func test can not cover this case due to target shape is 3d and blocked format.

Tickets:

  • 169876

@sungeunk sungeunk requested review from a team as code owners July 31, 2025 09:03
@sungeunk sungeunk added category: GPU OpenVINO GPU plugin do_not_merge and removed do_not_merge labels Jul 31, 2025
Copy link
Contributor

@jade-cho jade-cho left a comment

Choose a reason for hiding this comment

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

LGTM

@p-durandin p-durandin added this to the 2025.3 milestone Aug 4, 2025
@p-durandin p-durandin added this pull request to the merge queue Aug 4, 2025
Merged via the queue into openvinotoolkit:master with commit 0fd24f9 Aug 4, 2025
197 of 198 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: GPU OpenVINO GPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants