Copy-engine support - #426
Conversation
… dev/dasidler/sdma
… dev/dasidler/sdma
… dev/dasidler/sdma
… dev/dasidler/sdma
This reverts commit fde8982.
|
@mawad-amd not sure how useful the changes and copy egine path for examples 07 and 10 are. I might revert those and only keep the changes for message-passing example |
|
Thanks, David! I think we will want to keep the benchmark you used for your results. The message passing one would be a great introductory example. Will review this PR soon. |
| "ruff", | ||
| "tritonblas @ git+https://github.com/ROCm/tritonBLAS.git@muhaawad/iris", | ||
|
|
||
| "rocm-xio @ git+https://github.com/ROCm/rocm-xio.git", |
There was a problem hiding this comment.
This will likely break the simulation backend but let's merge it and see.
There was a problem hiding this comment.
Also let's pin this to a known good sha
CI failure triageTwo distinct classes here — 1 real, 10 infra. TL;DR: re-run the 10, the Gluon one needs a fix. The real failure — External Gluon Validation TestBoth ranks die at the same instant right after
HSAKMT_STATUS createStatus = hsaKmtCreateQueueExt(
localNodeId, HSA_QUEUE_SDMA_BY_ENG_ID, DEFAULT_QUEUE_PERCENTAGE,
DEFAULT_PRIORITY, engineId, queueBuffer_, SDMA_QUEUE_SIZE, nullptr, &queue_);
CHECK_HSAKMT_SUCCESS(createStatus, "Failed"); // <-- 239
Suspected cause — an unguarded There is no guard on a negative Why it would trigger on this runner: This is inferred from reading the source — I have not reproduced it on hardware. Cheap confirmation: re-run with Possible fix: validate before the cast — if The other 10 look like infra flakesAll the
The exact mechanism is not recoverable from the logs since the child's stdout is lost when it dies — but (1) is fairly strong evidence it is not the diff. Unrelated to this PR, but worth noting: This comment was AI-generated. The source trace is mechanical and checkable; the root-cause attribution is a hypothesis, not a reproduced result — please verify before acting on it. |
|
b603a74 is a workaround. i'll plan test a fix on the xio side such that HIP_VISIBLE_DEVICES works. |
No description provided.