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

Fixed rdma multi device connect failed issue #1402

Closed
wants to merge 2 commits into from

Conversation

tbago
Copy link
Contributor

@tbago tbago commented May 13, 2021

The root cause is that if you specified the rdma_device in GFLAGS, the rdma_helper will alloc the pd in the specified device. But in RdmaCommunicationManager::Create() the cm_id will always bind to the first device. So the pd is not matched and create qp will return failed. #1401

tbago added 2 commits May 13, 2021 18:35
 * The root cause is that if you specified the rdma_device in GFLAGS,
   the rdma_helper will alloc the pd in the specified device. But when
   create the cm_id it's bind to the first device. So the pd is not
   match and create qp will return failed.
@wwbmmm
Copy link
Contributor

wwbmmm commented Aug 9, 2022

Hi, 我们后续计划不再维护单独的RDMA分支,而是将RDMA功能合入master,相关PR:#1836
所以这个PR先close了。如果有关于RDMA的改进建议,可以在新的PR下讨论

@wwbmmm wwbmmm closed this Aug 9, 2022
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