Skip to content

clr: fix segment-scheduled graph handoffs (single kernel/hostnode) — coverage experiment (clone of ROCm#8055)#3

Open
jainprad wants to merge 4 commits into
developfrom
jainprad/segsched-handoff-coverage
Open

clr: fix segment-scheduled graph handoffs (single kernel/hostnode) — coverage experiment (clone of ROCm#8055)#3
jainprad wants to merge 4 commits into
developfrom
jainprad/segsched-handoff-coverage

Conversation

@jainprad

@jainprad jainprad commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Summary

Clone of upstream ROCm/rocm-systems#8055 (by @LeoZhao-BJ) for a code-coverage experiment. Upstream PR is left untouched — this copies its 3 commits (git cherry-pick -x) onto this fork's develop.

Fixes HIP graph segment-scheduling visibility: applies system-scope release when a captured AQL batch hands off to an uncaptured host node, and for single-packet AQL batches applies the release fence to firstHeader.

Runtime changes

  • projects/clr/hipamd/src/hip_graph_internal.hpp
  • projects/clr/rocclr/device/rocm/rocvirtual.cpp

Tests (existing + new)

  • hipLaunchHostFuncWithGraph.cc — adds Unit_hipLaunchHostFunc_SameStreamCapturedDtoH_UncapturedHost
  • hipGraphInstantiateWithFlags.cc, graph.yaml

Test plan

Made with Cursor

Add this PR's own coverage gate (build_and_test.sh, collect_and_gate.sh,
pr-diff-coverage.yml) so the segsched-handoff clone is measured on its
own changed lines independently of other PRs.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants