Skip to content

Conversation

clee30
Copy link
Contributor

@clee30 clee30 commented Jul 18, 2025

Earlier 3b84486 commit has caused kernel compilation error for shape of [batch, num_heads, 1, 1]. For this shape, restore to earlier tiled_load_t. The failure only happen if running in static shape.

CVS-170417, CVS-170820

@clee30 clee30 requested review from a team as code owners July 18, 2025 07:51
@clee30
Copy link
Contributor Author

clee30 commented Jul 18, 2025

build_jenkins

@p-durandin p-durandin added this to the 2025.3 milestone Jul 18, 2025
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Jul 18, 2025
@clee30 clee30 force-pushed the sdpa_micro_attn_mask branch from 491976e to 7296cff Compare July 18, 2025 08:30
@clee30 clee30 force-pushed the sdpa_micro_attn_mask branch from 7296cff to 4969e37 Compare July 22, 2025 03:43
@clee30
Copy link
Contributor Author

clee30 commented Jul 22, 2025

build_jenkins

@clee30 clee30 force-pushed the sdpa_micro_attn_mask branch 3 times, most recently from 37277ec to f9aa9cf Compare July 23, 2025 13:34
@clee30
Copy link
Contributor Author

clee30 commented Jul 23, 2025

build_jenkins

@clee30 clee30 force-pushed the sdpa_micro_attn_mask branch from f9aa9cf to e269209 Compare July 25, 2025 02:05
@clee30
Copy link
Contributor Author

clee30 commented Jul 25, 2025

build_jenkins

Earlier 3b84486 commit has caused kernel compilation error for shape
of [batch, num_heads, 1, 1].  For this shape, restore to earlier tiled_load_t.

Besides, add a reshape for attn mask that with only 2 ranks and expand
to 4 ranks by adding 1.
@clee30 clee30 force-pushed the sdpa_micro_attn_mask branch from e269209 to 3a8cb3d Compare July 25, 2025 11:11
@isanghao
Copy link
Contributor

It resolved (at least some) functional issue. Could you rebase your working branch?

@sshlyapn sshlyapn enabled auto-merge July 28, 2025 06:40
@sshlyapn sshlyapn added this pull request to the merge queue Jul 28, 2025
Merged via the queue into openvinotoolkit:master with commit 6892dc5 Jul 28, 2025
184 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