Skip to content

UCX: replace ucp_ep_flush() with ucp_ep_fence() - #219

Closed
michal-shalev wants to merge 1 commit into
ai-dynamo:mainfrom
michal-shalev:replace-ucp-ep-flush-with-ucp-ep-fence
Closed

michal-shalev wants to merge 1 commit into
ai-dynamo:mainfrom
michal-shalev:replace-ucp-ep-flush-with-ucp-ep-fence

Conversation

@michal-shalev

Copy link
Copy Markdown
Contributor

WIP - Do not code review

@copy-pr-bot

copy-pr-bot Bot commented Apr 29, 2025

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@michal-shalev
michal-shalev marked this pull request as draft April 29, 2025 15:11
@github-actions

Copy link
Copy Markdown

👋 Hi michal-shalev! Thank you for contributing to ai-dynamo/nixl.

Your PR reviewers will review your contribution then trigger the CI to test your changes.

🚀

@artpol84

Copy link
Copy Markdown
Contributor

@michal-shalev I originally used fence but was told that it’s not fully implanted (IIRC). Has that changed now? Is it part of new features in UCX 1.19?

@michal-shalev

Copy link
Copy Markdown
Contributor Author

@artpol84 We're currently adding a new API: openucx/ucx#10665

@michal-shalev

Copy link
Copy Markdown
Contributor Author

Closing this PR, since openucx/ucx#10665 won't be relevant to NIXL.

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