Skip to content

feat(hwq_heuristic): add stream-priority interaction test - #60

Draft
y1k2chaitanya wants to merge 1 commit into
mainfrom
users/kyenumul/hwq-stream-priority
Draft

feat(hwq_heuristic): add stream-priority interaction test#60
y1k2chaitanya wants to merge 1 commit into
mainfrom
users/kyenumul/hwq-stream-priority

Conversation

@y1k2chaitanya

Copy link
Copy Markdown
Contributor

Summary

Adds test_stream_priority.py (TMS 1002695 hwq_stream_priority_interaction, P1) to the hwq_heuristic suite:

  • test_stream_priority_semantics — with no override, the queue heuristic preserves requested stream priorities: hipStreamGetPriority echoes the range-clamped request, the high-priority stream is numerically ≤ the low-priority stream, and both workloads compute correct results.
  • test_stream_priority_ignore_override — with DEBUG_HIP_IGNORE_STREAM_PRIORITY=1, the override is honored gracefully (priority streams still create and both workloads execute correctly).

New vendored public source src/hwq_stream_priority.cpp; added as a target=-scoped CMake executable so existing fixtures are untouched.

Provenance / OSS-safety

Re-authored from standard public HIP stream-priority APIs — no internal/NDA payload.

Test plan

  • pytest --collect-only --no-gpu — 2 collected, markers valid
  • Local GPU validation on quanta MI325 (gfx942): 2/2 passed
  • e2e CI dispatch

@y1k2chaitanya

Copy link
Copy Markdown
Contributor Author

CI validation — e2e-tests.yml on linux-gfx94x-MI325 (gfx942)

Ran via the integration branch users/kyenumul/wave12-ci (all three Wave 1+2 areas), isolated with keyword_filter="deviceside_malloc or multi_instance or stream_priority or zero_mat":

12 passed, 288 deselected in 27.79s

Run: https://github.com/ROCm/rocm-tests/actions/runs/30346057930

(test_multi_instance_two_gpu is hw.multi_gpu and is correctly deselected on the 1-GPU runner.)

test_stream_priority.py validates the queue heuristic preserves HIP stream
priority semantics (hipStreamGetPriority echoes the range-clamped request, high
<= low, workloads correct) and that DEBUG_HIP_IGNORE_STREAM_PRIORITY is honored
gracefully. New src hwq_stream_priority.cpp. Validated 2/2 on quanta MI325.
@y1k2chaitanya
y1k2chaitanya force-pushed the users/kyenumul/hwq-stream-priority branch from 8ade32a to 26bc8cf Compare July 30, 2026 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants