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
libfabric parcelport w/o HPX_PARCELPORT_LIBFABRIC_ENDPOINT_RDM #3163
libfabric parcelport w/o HPX_PARCELPORT_LIBFABRIC_ENDPOINT_RDM #3163
Conversation
just found a method that converts endpoints to active endpoints. will fix circleci/appveyor issues and swap out the logic to use the existing method. |
…X_PARCELPORT_LIBFABRIC_ENDPOINT_RDM
@ct-clmsn could you please give me access to your clone, so that I can rebase this PR and add a commit to it |
It seems you can make individual PR's editable too |
7fd7222
to
0af29b9
Compare
Apologies. I already had permission. I didn't try it right. I've rebased this branch onto current master and force pushed. I hope this does not cause trouble at your end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@biddisco no problem! thank you for the help (picking up my slack and for the review)! |
Merging this after testing locally. Need it to be ready if GSoC project goes ahead. |
HPX_PARCELPORT_LIBFABRIC_ENDPOINT_RDM fixes.
Fixes #2650
Any background context you want to provide?
platforms without LIBFABRIC_ENDPOINT_RDM turned on have compile time errors. this patch resolves compile issues. this patch also fixes an issue where the passive endpoint is not turned into an active endpoint for IPC. without this fix active endpoints are never created; active endpoints are created only in blocks of code with the HPX_PARCELPORT_LIBFABRIC_ENDPOINT_RDM #define guard. active endpoints are necessary for IPC.