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

Protect against using CUDA without VMM support. #13016

Open
wants to merge 1 commit into
base: v4.1.x
Choose a base branch
from

Conversation

bosilca
Copy link
Member

@bosilca bosilca commented Jan 3, 2025

Fixes #13005.

bot:notacherrypick

@github-actions github-actions bot added this to the v4.1.7 milestone Jan 3, 2025
Copy link

github-actions bot commented Jan 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

f3513c0: Protect against using CUDA without VMM support.

  • check_cherry_pick: does not include a cherry pick message (did you need to bot:notacherrypick?)

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

Fixes open-mpi#13005.

bot:notacherrypick

Signed-off-by: George Bosilca <[email protected]>
Copy link

github-actions bot commented Jan 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

9ae79b1: Protect against using CUDA without VMM support.

  • check_cherry_pick: does not include a cherry pick message (did you need to bot:notacherrypick?)

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@jsquyres
Copy link
Member

jsquyres commented Jan 3, 2025

@bosilca Is this needed in main / v5.0.x?

@jsquyres jsquyres added the bug label Jan 3, 2025
@bosilca
Copy link
Member Author

bosilca commented Jan 3, 2025

No, in the 5.0 the code is fully contained in the CUDA accelerator component.

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