Skip to content

test(rocwmma): default CTest --parallel to 1 (ROCM-24171) - #5061

Closed
tony-davis wants to merge 2 commits into
mainfrom
users/todavis/ROCM-24171
Closed

tony-davis wants to merge 2 commits into
mainfrom
users/todavis/ROCM-24171

Conversation

@tony-davis

@tony-davis tony-davis commented May 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Set rocWMMA TheRock CTest --parallel default from 2 to 1 in build_tools/github_actions/test_executable_scripts/test_rocwmma.py.
  • Bump rocm-libraries submodule to include the matching test/therock/test_rocwmma.py change.

Context

TheRock CI Nightly (Linux gfx94X-dcgpu) has seen intermittent rocwmma test steps hit the 60-minute timeout (shard 4/5). Running CTest with a single parallel job avoids concurrent GPU test processes on single-GPU runners while the root cause is investigated.

Related

Made with Cursor

ROCM-24171

Update rocm-libraries submodule and mirror the change in build_tools
test_rocwmma.py. Intermittent lockups / 60m timeouts on gfx942 TheRock
CI; serialize CTest until root cause is found.

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

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.

Pull request overview

Adjusts rocWMMA’s CI test invocation to avoid concurrent GPU test processes by defaulting CTest parallelism to a single job, addressing intermittent timeouts seen on gfx94x single-GPU runners.

Changes:

  • Set rocWMMA CTest --parallel default from 2 to 1 (no longer varies by AMDGPU_FAMILIES).
  • Remove now-unused AMDGPU_FAMILIES environment read from the rocWMMA test script.
  • Add inline context comment referencing ROCM-24171 and the motivation for reduced parallelism.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants