Skip to content

[v4.1.x] mtl/ofi: Disable CUDA convertor for specified ofi providers #8821

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

Merged
merged 1 commit into from
Apr 19, 2021

Conversation

acgoldma
Copy link
Contributor

@acgoldma acgoldma commented Apr 16, 2021

This patch is only in v4.x as code in v5.x was rewritten to use FI_HMEM
and there is no plan to backport the related patches.

Refs: #8762
Signed-off-by: Goldman, Adam [email protected]

bot:notacherrypick

This patch is only in v4.x as code in v5.x was rewritten to use FI_HMEM
and there is no plan to backport the related patches.

Refs: 8762
Signed-off-by: Goldman, Adam <[email protected]>
@ompiteam-bot
Copy link

Can one of the admins verify this patch?

@acgoldma
Copy link
Contributor Author

@rajachan @mwheinz @wckzhang, Can you please review?

Also, I am not sure how to go about this as there is no way to really cherry-pick this from master:
(GitHub Action CI / Git commit checker (pull_request) )

Error: The following commits caused this test to fail

Error: * 70eec6ac: mtl/ofi: Disable CUDA convertor for specified ofi ...
Error:     * check_cherry_pick: does not include a cherry pick message

Test failed: sad panda
Error: Process completed with exit code 1.

@rajachan
Copy link
Member

rajachan commented Apr 16, 2021

You can add bot:notacherrypick to your PR description for such changes.

@rajachan rajachan requested a review from a team April 16, 2021 19:38
@rajachan
Copy link
Member

ok to test

@jsquyres jsquyres added this to the v4.1.1 milestone Apr 16, 2021
@jsquyres
Copy link
Member

@acgoldma Is it worthwhile, once this PR is merged, to cherry-pick to v4.0.x?

@acgoldma
Copy link
Contributor Author

@acgoldma Is it worthwhile, once this PR is merged, to cherry-pick to v4.0.x?

Yes, I think it is worth while.

Copy link

@mwheinz mwheinz left a comment

Choose a reason for hiding this comment

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

Looks good.

@jsquyres jsquyres merged commit dc42f40 into open-mpi:v4.1.x Apr 19, 2021
@acgoldma acgoldma deleted the v4.1.x-ofi-cuda-perf branch April 19, 2021 16:39
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.

6 participants