feat(ccu): add four-rank alltoall mesh - #79
Conversation
|
CI note: Host Checks for this PR never reached checkout or repository tests. Attempt 1 failed in apt-get with Ubuntu mirror connection errors; attempts 2 and 3 remained in Install host dependencies for more than 20 minutes and were cancelled to release runner capacity. The cumulative successor #80, which contains all changes from this PR, passed Host Checks and PR Gate. Local cumulative validation also passes: 142 core tests, 19 microcode/signal/memory tests, workflow policy test, source guard, and git diff --check. Re-run this PR when the Ubuntu mirror is healthy. |
| } | ||
| program->push_back(instr); | ||
|
|
||
| if (AppendSetSourceCke(shared, TILEXR_CCU_ALLTOALL_SOURCE_CKE_INIT_MASK, program, report) != TILEXR_SUCCESS) { |
There was a problem hiding this comment.
Only peers.front().route.sourceCke is initialized here, but the post phase later emits SyncCke using each peer route's own sourceCke. ValidateMeshSpec permits distinct nonzero source CKEs, so such a spec returns success while peers after the first reference uninitialized source CKEs and can stall. Please initialize every unique (sourceCke, mask) pair or require all routes to share the initialized pair.
Summary
Stack
8 of 9. Depends on
codex/ccu-stack-07-alltoall-2rank.Validation
git diff --checkScope
Prior four-rank hardware evidence belongs to #71 and was not rerun from this host.