Skip to content

disable ibverbs build with env variable#5513

Merged
ezyang merged 3 commits intopytorch:masterfrom
ailzhang:fix_ibverbs_build
Mar 7, 2018
Merged

disable ibverbs build with env variable#5513
ezyang merged 3 commits intopytorch:masterfrom
ailzhang:fix_ibverbs_build

Conversation

@ailzhang
Copy link
Copy Markdown
Contributor

@ailzhang ailzhang commented Mar 2, 2018

This is to fix #5504
Disable ibverbs build with environment variable.

@ailzhang
Copy link
Copy Markdown
Contributor Author

ailzhang commented Mar 2, 2018

@pytorchbot retest this please

Comment thread setup.py Outdated

DEBUG = check_env_flag('DEBUG')
ENABLE_IBVERBS_ENV = check_env_flag('WITH_GLOO_IBVERBS')
WITH_GLOO_IBVERBS = WITH_GLOO_IBVERBS and ENABLE_IBVERBS_ENV

This comment was marked as off-topic.

@ailzhang
Copy link
Copy Markdown
Contributor Author

ailzhang commented Mar 2, 2018

@pytorchbot retest this please

Copy link
Copy Markdown
Contributor

@apaszke apaszke left a comment

Choose a reason for hiding this comment

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

I don’t think this is a good fix. In general we want people to be able to say that they forcefully want to build ibverbs even when we can’t auto detect the program, and we also want them to be able to forbid us from doing this. So it seems like what we should do is:

  • if the env variable is specified, use its value to determine what to do
  • otherwise use the heuristic we have (should_build_ib)

@ezyang ezyang merged commit fcaa3bf into pytorch:master Mar 7, 2018
@ailzhang ailzhang deleted the fix_ibverbs_build branch March 7, 2018 17:45
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
if the env variable is specified, use its value to determine what to do
otherwise use the heuristic we have (should_build_ib)
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.

Disabling ibverbs fails

3 participants