Skip to content

Conversation

@rocm-devops
Copy link
Contributor

Associated JIRA ticket number/Github issue number

SWDEV-524745

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Continuous Integration

What were the changes?

  • The graph nodes have been updated to capture the device ID from the capture stream or the current device when explicitly added.
  • Update the device ID for the memcpy node, ensuring that the device where the memory is allocated is taken into account for H2D and D2H pinned operations.

Why are these changes needed?

For Graph multi device support.

Updated CHANGELOG?

  • Yes
  • No, Does not apply to this PR.

Added/Updated documentation?

  • Yes
  • No, Does not apply to this PR.

Additional Checks

  • I have added tests relevant to the introduced functionality, and the unit tests are passing locally.
  • Any dependent changes have been merged.

…nodes with DevId.

- The graph nodes have been updated to capture the device ID from the capture stream or the current device when explicitly added.
- Update the device ID for the memcpy node, ensuring that the device where the memory is allocated is taken into account for H2D and D2H pinned operations.
@rocm-devops
Copy link
Contributor Author

Manual review required for 52c7c7
Assigned Auditors: @deepatel

@amd-hsivasun
Copy link

amd-hsivasun commented Aug 18, 2025

Unable to import to rocm-systems, merge conflict

@rocm-devops
Copy link
Contributor Author

Error importing due to merge conflicts – please reopen the PR on ROCm/rocm-systems

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants