Repository navigation
AIRUNTIME-2896 - Fix streams perf tests - #13682
Conversation
❌ PR Check — Action Required
📖 Need help? See the Policy FAQ for details on every check and how to fix failures. |
|
🚫 Please fix the failed policies before requesting reviews. The following policy checks failed:
The |
There was a problem hiding this comment.
🟡 Changes recommended
Calibration can exhaust memory before budgeting applies, and checked event cleanup can prevent the intended skip.
2 open issues
What changed in this PR
This PR aims to prevent resource-exhaustion hangs in HIP multi-stream performance tests by skipping unsupported stream counts.
Changes:
- Adds shared stream-cost calibration, memory budgeting, and exhaustion reporting.
- Integrates skip handling into both performance scenarios.
- Removes the 5,000-stream case.
| File | Description |
|---|---|
| projects/hip-tests/catch/performance/scenarios/stream/stream_create_helper.hh | Adds memory-aware stream creation. |
| projects/hip-tests/catch/performance/scenarios/stream/hipPerfMultiStreamKernelLaunch.cc | Uses the helper to skip unsupported experiments. |
| projects/hip-tests/catch/performance/scenarios/stream/hipPerfIncreasingNumberOfStreams.cc | Adds skip handling and removes the largest stream count. |
Review effort: Balanced
5995cdc to
aa2f218
Compare
aa2f218 to
fbbaccf
Compare
dd57f92 to
44e0f5b
Compare
|
🤖 AI-assisted pre-review (for @mangupta) Re-reviewed the updated head ( |
Fix the following tests' hang in some devices, StreamScenarioPerformance.exe Performance_KernelLaunchLatency_IncreasingNumberOfStreams StreamScenarioPerformance.exe Performance_hipPerfMultiStreamKernelLaunch
44e0f5b to
6dcc339
Compare
|
🤖 AI-assisted pre-review (for @mangupta) Re-reviewed the updated head ( |
TheRock Submodule Bump ActivityNewest first
|


Motivation
Fix the following tests' hang in some devices,
StreamScenarioPerformance.exe Performance_KernelLaunchLatency_IncreasingNumberOfStreams StreamScenarioPerformance.exe Performance_hipPerfMultiStreamKernelLaunch
Technical Details
When memory is not enough, some test cases will be skipped.
Issue Tracking
AIRUNTIME-2896
Test Plan
All tests should pass
Test Result
Pass
Submission Checklist