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

PR #16921: [PJRT:GPU] Treat GPU collective memory space as device memory space #17030

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Sep 10, 2024

PR #16921: [PJRT:GPU] Treat GPU collective memory space as device memory space

Imported from GitHub PR #16921

This is a regression fix when using --xla_gpu_enable_nccl_user_buffers=true.
Return device memory space when collective memory space is used as an output on GPU.
Copybara import of the project:

--
8113e6f by Jane Liu [email protected]:

Treat collective memory space as device memory space when using as an output

--
b5e43d6 by Jane Liu [email protected]:

fix the test

Merging this change closes #16921

FUTURE_COPYBARA_INTEGRATE_REVIEW=#16921 from zhenying-liu:nccl-buffer-output b5e43d6

@copybara-service copybara-service bot force-pushed the test_672618973 branch 14 times, most recently from 36b2d0e to 348d7f4 Compare September 13, 2024 00:06
…ory space

Imported from GitHub PR #16921

This is a regression fix when using --xla_gpu_enable_nccl_user_buffers=true.
Return device memory space when collective memory space is used as an output on GPU.
Copybara import of the project:

--
8113e6f by Jane Liu <[email protected]>:

Treat collective memory space as device memory space when using as an output

--
b5e43d6 by Jane Liu <[email protected]>:

fix the test

Merging this change closes #16921

COPYBARA_INTEGRATE_REVIEW=#16921 from zhenying-liu:nccl-buffer-output b5e43d6
PiperOrigin-RevId: 674073363
@copybara-service copybara-service bot merged commit c5346ef into main Sep 13, 2024
@copybara-service copybara-service bot deleted the test_672618973 branch September 13, 2024 00:37
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.

1 participant