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

fabric: Move ep_type from fi_info into ep_attr #698

Merged
merged 1 commit into from
Feb 27, 2015
Merged

Conversation

shefty
Copy link
Member

@shefty shefty commented Feb 26, 2015

Conceptually, this is a simple change, but the impact is significant. Please review the provider code to see that what was changed is correct and that nothing was missed!

@goodell @jithinjosepkl @j-xiong @shantonu @a-ilango

The ep_type should really be an EP attribute.  Let's move
it!

Signed-off-by: Sean Hefty <sean.hefty@intel.com>
@j-xiong
Copy link
Contributor

j-xiong commented Feb 26, 2015

PSM provider changes look fine to me.

@jithinjosepkl
Copy link

Sockets provider changes also look good.

@arn314
Copy link

arn314 commented Feb 26, 2015

Verbs provider changes look fine.

@goodell
Copy link
Member

goodell commented Feb 26, 2015

usnic provider changes look OK to me

shefty added a commit that referenced this pull request Feb 27, 2015
fabric: Move ep_type from fi_info into ep_attr
@shefty shefty merged commit 7f5b752 into ofiwg:master Feb 27, 2015
sungeunchoi added a commit to sungeunchoi/libfabric that referenced this pull request Feb 27, 2015
Signed-off-by: Sung-Eun Choi <sungeunchoi@users.noreply.github.com>
sungeunchoi referenced this pull request in ofi-cray/libfabric-cray Feb 27, 2015
Changes for moving ep_type to ep_attr (upstream pull request #698)

Hopefully this will make everything all good.
hppritcha added a commit to hppritcha/libfabric that referenced this pull request Mar 9, 2016
prov/gni: fix of FABRIC_DIRECT_ENABLED
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.

5 participants