Skip to content

test(hipblaslt): quarantine the gfx950 alpha = 0, beta = 1 complex batch edge cases (ROCM-33823) - #14867

Merged
tony-davis merged 2 commits into
developfrom
users/todavis_amdeng/ROCM-33823-gfx950-complex-alpha0-quarantine
Nov 2, 2026
Merged

tony-davis merged 2 commits into
developfrom
users/todavis_amdeng/ROCM-33823-gfx950-complex-alpha0-quarantine

Conversation

@tony-davis

@tony-davis tony-davis commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Jira ID: ROCM-33823

Motivation

Every TheRock rocm-libraries bump since ROCm/TheRock#9651 fails gfx950 hipBLASLt shard 4 on matmul_batch_complex_general_batch_edge (cf64 TN 1x1024x512, alpha = 0, beta = 1, -nan in D). This was the intermittent #14282. #14743 now fills D with a NaN sentinel before every verified launch, so a kernel that leaves D elements unwritten fails every time. That's the fix working as intended, and it's exposing a real kernel bug tracked in ROCM-33823.

Technical Details

Adds one known_bugs.yaml entry that quarantines the alpha = 0, beta = 1 cases of that test on gfx950 only. I scoped it to the alpha/beta pair rather than the single failing case because the bug has moved between shapes and types (cf64 NN and TN, cf32 NN and NC). All other alpha/beta combinations and all other arches still run.

Test Plan

Ran hipblaslt_gentest.py over matmul_gtest.yaml and confirmed the entry marks only the alpha = 0, beta = 1 cases of this test for gfx950. CI.

Test Result

TBD (CI)

Submission Checklist

…tch edge cases (ROCM-33823)

Since #14743 fills D with a NaN sentinel before each verified launch, the gfx950 kernels that leave some D elements unwritten in matmul_batch_complex_general_batch_edge fail every time instead of only when leftover memory happens to be NaN (#14282). That blocks every TheRock rocm-libraries bump since ROCm/TheRock#9651. Quarantine the alpha = 0, beta = 1 cases on gfx950 until ROCM-33823 is fixed.

Co-authored-by: Cursor <cursoragent@cursor.com>
@therock-pr-bot

therock-pr-bot Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass —
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass —
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —
🤖 therock-pr-bot ✅ Pass —

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

@therock-pr-bot

therock-pr-bot Bot commented Nov 2, 2026

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

Copilot AI left a comment

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.

🟢 Approval recommended

No active review issues were identified for the gfx950-only quarantine of the tracked complex batched edge-case failures.

No open issues 🎉

What changed in this PR

Quarantines the known gfx950 complex batched edge-case failures tracked by ROCM-33823, which became reproducible after verification began resetting D with a NaN sentinel.

Changes:

  • Adds a gfx950-only known-bug entry for matmul_batch_complex_general_batch_edge with alpha = 0 and beta = 1, both with zero imaginary components.
  • Documents the failure, removal condition, and November 16 review deadline.
File Description
projects/​hipblaslt/​clients/​tests/​data/​known_bugs.yaml Adds the scoped quarantine and tracking comments.

Review effort: Balanced

@rahulc-gh rahulc-gh added the ci:gfx950-dcgpu Opt-in to gfx950-dcgpu builds/tests label Nov 2, 2026
@tony-davis
tony-davis enabled auto-merge (squash) November 2, 2026 17:11
@rahulc-gh rahulc-gh removed the ci:skip Skip all CI builds/tests for this PR label Nov 2, 2026
@tony-davis tony-davis added ci:skip Skip all CI builds/tests for this PR and removed ci:gfx950-dcgpu Opt-in to gfx950-dcgpu builds/tests labels Nov 2, 2026
@tony-davis
tony-davis merged commit f16ebe7 into develop Nov 2, 2026
88 of 98 checks passed
@tony-davis
tony-davis deleted the users/todavis_amdeng/ROCM-33823-gfx950-complex-alpha0-quarantine branch November 2, 2026 20:04
@assistant-librarian

Copy link
Copy Markdown
Contributor

TheRock Submodule Bump Activity

Newest first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip Skip all CI builds/tests for this PR organization: ROCm project: hipblaslt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants