Skip to content

UCT/CUDA/CUDA_IPC: Separated get and put remote cache methods.#11501

Open
rakhmets wants to merge 1 commit into
openucx:masterfrom
rakhmets:topic/cuda-ipc-get-remote-cache
Open

UCT/CUDA/CUDA_IPC: Separated get and put remote cache methods.#11501
rakhmets wants to merge 1 commit into
openucx:masterfrom
rakhmets:topic/cuda-ipc-get-remote-cache

Conversation

@rakhmets
Copy link
Copy Markdown
Contributor

What?

Separated uct_cuda_ipc_get_remote_cache into get and put parts. To use uct_cuda_ipc_put_remote_cache in
uct_cuda_ipc_map_memhandle, and uct_cuda_ipc_get_remote_cache in uct_cuda_ipc_unmap_memhandle.

@rakhmets rakhmets force-pushed the topic/cuda-ipc-get-remote-cache branch from 8f6c42e to 3870eee Compare May 28, 2026 14:48
@rakhmets rakhmets marked this pull request as ready for review May 28, 2026 14:49
@rakhmets
Copy link
Copy Markdown
Contributor Author

@svc-nvidia-pr-review

@svc-nvidia-pr-review
Copy link
Copy Markdown

🤖 Starting review — findings will be posted here when done.

Copy link
Copy Markdown

@svc-nvidia-pr-review svc-nvidia-pr-review left a comment

Choose a reason for hiding this comment

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

Code Review

2 findings — 🚨 2 blockers

2 findings posted as inline comments.

Comment thread src/uct/cuda/cuda_ipc/cuda_ipc_cache.c Outdated
Comment thread src/uct/cuda/cuda_ipc/cuda_ipc_cache.c
@rakhmets rakhmets force-pushed the topic/cuda-ipc-get-remote-cache branch from 45caec2 to 17616e4 Compare May 29, 2026 13:02
@rakhmets
Copy link
Copy Markdown
Contributor Author

@svc-nvidia-pr-review

@svc-nvidia-pr-review
Copy link
Copy Markdown

🤖 Starting review — findings will be posted here when done.

Copy link
Copy Markdown

@svc-nvidia-pr-review svc-nvidia-pr-review left a comment

Choose a reason for hiding this comment

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

Code Review

No issues found. ✅

@rakhmets rakhmets requested review from gleon99 and ofirfarjun7 May 29, 2026 13:04
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.

2 participants