Skip to content

Conversation

@apakbin
Copy link

@apakbin apakbin commented Jun 2, 2025

(This is a cherry-pick of pytorch#154549)

Get current device from Torch rather than HIP in MIOpen handle creation. The device may have already been set from torch side, otherwise device is set to 0 for handle. Additional audits of cudnn vs miopen Handle.cpp file.

Pull Request resolved: pytorch#154549
Approved by: https://github.com/jeffdaily, https://github.com/cyyever

…e creation (pytorch#154549)

Get current device from Torch rather than HIP in MIOpen handle creation. The device may have already been set from torch side, otherwise device is set to 0 for handle.  Additional audits of cudnn vs miopen Handle.cpp file.

Pull Request resolved: pytorch#154549
Approved by: https://github.com/jeffdaily, https://github.com/cyyever

Co-authored-by: Jeff Daily <jeff.daily@amd.com>
@apakbin apakbin requested a review from jerrymannil June 2, 2025 01:02
@apakbin apakbin changed the title [ROCm] MIOpen: Get current device from Torch rather than HIP in handle creation (Mirroring https://github.com/pytorch/pytorch/pull/154549) [release/2.6] MIOpen: Get current device from Torch rather than HIP in handle creation #154549 Jun 2, 2025
@apakbin apakbin marked this pull request as ready for review June 2, 2025 01:03
@apakbin
Copy link
Author

apakbin commented Jun 2, 2025

!cherry-pick --onto release/2.7

@apakbin
Copy link
Author

apakbin commented Jun 2, 2025

!cherry-pick --onto rocm6.4_internal_testing

@apakbin
Copy link
Author

apakbin commented Jun 2, 2025

!cherry-pick --onto rocm6.5_internal_testing

@okakarpa
Copy link
Collaborator

okakarpa commented Jun 2, 2025

The cherry-pick keyword can be used only with the merged PRs

2 similar comments
@okakarpa
Copy link
Collaborator

okakarpa commented Jun 2, 2025

The cherry-pick keyword can be used only with the merged PRs

@okakarpa
Copy link
Collaborator

okakarpa commented Jun 2, 2025

The cherry-pick keyword can be used only with the merged PRs

@pruthvistony pruthvistony merged commit 0ad7380 into release/2.6 Jun 6, 2025
@pruthvistony pruthvistony deleted the miopen_handle_device_fix_release_2.6 branch June 6, 2025 05:19
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.

5 participants