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

Fix compilation of verbs provider with ofed 3.12-1 #6

Merged
merged 1 commit into from
Jul 28, 2015
Merged

Fix compilation of verbs provider with ofed 3.12-1 #6

merged 1 commit into from
Jul 28, 2015

Conversation

pmmccorm
Copy link

@pmmccorm pmmccorm commented Jul 7, 2015

This commit works around the multiple definition errors and allows compilation on RHEL 6.6 without
modifying the installed headers as described in the README.

The correct fix for this is to modify the makefiles to properly depend on the compat-rdma kernel
modules/symbols. I can't figure this out however and the only apparent documentation (OFED_tips.txt
section 4) is out of date and wrong.

Signed-off-by: pmmccorm [email protected]

@stanfordlightfoot
Copy link
Contributor

Please update the top-level README file, leaving tracks such that those who follow may be able to figure out the correct Makefile solution.

This commit works around the multiple definition errors and allows compilation on RHEL 6.6 without
modifying the installed headers as described in the README.

The correct fix for this is to modify the makefiles to properly depend on the compat-rdma kernel
modules/symbols. I can't figure this out however and the only apparent documentation (OFED_tips.txt
section 4) is out of date and wrong.

Updated the TODO and README to catpure this info.

Signed-off-by: pmmccorm <[email protected]>
@pmmccorm
Copy link
Author

pmmccorm commented Jul 8, 2015

Updated and put a detailed description in the TODO. Filed bug: http://bugs.openfabrics.org/bugzilla/show_bug.cgi?id=2560

stanfordlightfoot added a commit that referenced this pull request Jul 28, 2015
Fix compilation of verbs provider with ofed 3.12-1
@stanfordlightfoot stanfordlightfoot merged commit 3f41e1f into ofiwg:master Jul 28, 2015
@stanfordlightfoot
Copy link
Contributor

Done.

This pull request was closed.
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