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

TL/MLX5: add nonblocking cudaMemcpy support #1040

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MamziB
Copy link
Collaborator

@MamziB MamziB commented Oct 24, 2024

adding support for nonblocking cuda memcpy in mcast

@MamziB MamziB force-pushed the mamzi/mcast-nb-cudacopy branch from cd6c065 to 363a044 Compare October 24, 2024 20:31
@MamziB MamziB changed the title add nonblocking cudaMemcpy support TL/MLX5: add nonblocking cudaMemcpy support Oct 24, 2024
@MamziB MamziB self-assigned this Oct 24, 2024
@janjust
Copy link
Collaborator

janjust commented Nov 1, 2024

Please fix linter

@janjust janjust self-requested a review November 1, 2024 18:03
@janjust
Copy link
Collaborator

janjust commented Nov 14, 2024

@MamziB rebase, and fix CI, then good to go

@MamziB MamziB force-pushed the mamzi/mcast-nb-cudacopy branch 2 times, most recently from d754374 to ff3cecf Compare November 22, 2024 23:11
@MamziB
Copy link
Collaborator Author

MamziB commented Nov 23, 2024

@Sergei-Lebedev can you please take a look? all the tests now are passing

@MamziB
Copy link
Collaborator Author

MamziB commented Nov 25, 2024

@Sergei-Lebedev thanks for the new comments. please take a look at my responses and let me know your thoughts.

@swx-jenkins3
Copy link

Can one of the admins verify this patch?

@MamziB MamziB force-pushed the mamzi/mcast-nb-cudacopy branch 2 times, most recently from bdf3b74 to 4104cc9 Compare December 16, 2024 18:03
@MamziB
Copy link
Collaborator Author

MamziB commented Dec 16, 2024

@Sergei-Lebedev i made the two requested changes. please take a look

@janjust
Copy link
Collaborator

janjust commented Dec 19, 2024

bot: retest

@MamziB MamziB force-pushed the mamzi/mcast-nb-cudacopy branch from 4104cc9 to 084a9c5 Compare December 20, 2024 16:17
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.

4 participants