Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable CUDA Fuser for ROCm #45965

Closed
wants to merge 1 commit into from
Closed

Enable CUDA Fuser for ROCm #45965

wants to merge 1 commit into from

Conversation

t-vi
Copy link
Collaborator

@t-vi t-vi commented Oct 7, 2020

This enables the cuda fuser on ROCm and enables tests for them.

Part of this patch is based on work of Rohith Nallamaddi, thank you.
Errors are my own, of course.

@t-vi t-vi requested a review from apaszke as a code owner October 7, 2020 09:59
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Oct 7, 2020
Part of this patch is based on work of Rohith Nallamaddi, thank you.
Errors are my own, of course.
@t-vi
Copy link
Collaborator Author

t-vi commented Oct 7, 2020

@jeffdaily @rohithkrn @walterddr @malfet @sunway513 (in part based on ROCm#712 )

@t-vi
Copy link
Collaborator Author

t-vi commented Oct 7, 2020

Oh, and @csarofeen @jjsjann123 , I hope I didn't screw up your code too much.

Copy link
Contributor

@csarofeen csarofeen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me.

@jeffdaily jeffdaily added the module: rocm AMD GPU support for Pytorch label Oct 7, 2020
@jeffdaily
Copy link
Collaborator

The caffe2 ROCm CI is timing out due to an unrelated change to master, scheduled to be reverted. Also, ROCm CI moved to the 3.8 image this morning, in case we want to verify this PR against the newer image, too.

@t-vi
Copy link
Collaborator Author

t-vi commented Oct 7, 2020

I think ROCm 3.8 should be relatively safe given that I'm running it with success there. But I can also push an empty commit to rerun the CI.

@jeffdaily
Copy link
Collaborator

No need to push empty commit. We can use pytorchbot.

@jeffdaily
Copy link
Collaborator

Let's wait until the caffe2 PR is reverted, though.

@jeffdaily
Copy link
Collaborator

@pytorchbot retest this please

@jeffdaily
Copy link
Collaborator

CI looks good.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@walterddr has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@walterddr walterddr requested a review from a team October 7, 2020 22:40
@t-vi
Copy link
Collaborator Author

t-vi commented Oct 8, 2020

If I understand the messages correctly, this is about the Caffe2 breakage. If it helps, I can rebase, just shout.

@facebook-github-bot
Copy link
Contributor

@walterddr merged this pull request in d3d8da7.

jjsjann123 pushed a commit to jjsjann123/nvfuser that referenced this pull request Oct 29, 2022
Summary:
This enables the cuda fuser on ROCm and enables tests for them.

Part of this patch is based on work of Rohith Nallamaddi, thank you.
Errors are my own, of course.

Pull Request resolved: pytorch/pytorch#45965

Reviewed By: seemethere

Differential Revision: D24170457

Pulled By: walterddr

fbshipit-source-id: 3dd25b3501a41d2f00acba3ce8642ce51c49c9a6
jjsjann123 pushed a commit to jjsjann123/nvfuser that referenced this pull request Nov 10, 2022
Summary:
This enables the cuda fuser on ROCm and enables tests for them.

Part of this patch is based on work of Rohith Nallamaddi, thank you.
Errors are my own, of course.

Pull Request resolved: pytorch/pytorch#45965

Reviewed By: seemethere

Differential Revision: D24170457

Pulled By: walterddr

fbshipit-source-id: 3dd25b3501a41d2f00acba3ce8642ce51c49c9a6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: rocm AMD GPU support for Pytorch oncall: jit Add this issue/PR to JIT oncall triage queue open source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants