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

libfabric parcelport w/o HPX_PARCELPORT_LIBFABRIC_ENDPOINT_RDM #3163

Conversation

ct-clmsn
Copy link
Contributor

@ct-clmsn ct-clmsn commented Feb 10, 2018

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.

@ct-clmsn ct-clmsn changed the title made some small changes for compiling libfabric parcelport without HP… libfabric parcelport w/o HPX_PARCELPORT_LIBFABRIC_ENDPOINT_RDM Feb 10, 2018
@hkaiser hkaiser added this to the 1.1.0 milestone Feb 11, 2018
@ct-clmsn
Copy link
Contributor Author

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.

@msimberg msimberg removed this from the 1.1.0 milestone Mar 22, 2018
@biddisco
Copy link
Contributor

@ct-clmsn could you please give me access to your clone, so that I can rebase this PR and add a commit to it

@biddisco
Copy link
Contributor

It seems you can make individual PR's editable too
https://help.github.com/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork/

@biddisco biddisco force-pushed the libfabric_passive_to_active_non_endpoint_rdm branch from 7fd7222 to 0af29b9 Compare April 17, 2018 13:54
@biddisco
Copy link
Contributor

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

Copy link
Contributor

@biddisco biddisco left a comment

Choose a reason for hiding this comment

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

LGTM

@ct-clmsn
Copy link
Contributor Author

ct-clmsn commented Apr 17, 2018

@biddisco no problem! thank you for the help (picking up my slack and for the review)!

@biddisco
Copy link
Contributor

Merging this after testing locally. Need it to be ready if GSoC project goes ahead.

@biddisco biddisco merged commit 3388907 into STEllAR-GROUP:master Apr 19, 2018
@msimberg msimberg added this to the 1.2.0 milestone Apr 19, 2018
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.

libfabric support for passive endpoints
4 participants