Skip to content

Fix static linkage cases and NO_DISTRIBUTED=1 + CUDA#16705

Closed
soumith wants to merge 3 commits intomasterfrom
buildsfix_static
Closed

Fix static linkage cases and NO_DISTRIBUTED=1 + CUDA#16705
soumith wants to merge 3 commits intomasterfrom
buildsfix_static

Conversation

@soumith
Copy link
Copy Markdown
Collaborator

@soumith soumith commented Feb 3, 2019

cc: @pjh5

Fixes case where:

  • CuDNN is statically linked, but CUDA is dynamically linked (needed for new conda workflow)
  • NO_DISTRIBUTED=1 with CUDA builds works (forces CUDART to be dynamic linkage by default)

Copy link
Copy Markdown
Contributor

@pjh5 pjh5 left a comment

Choose a reason for hiding this comment

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

Confirmed that builds pass with these changes

@soumith
Copy link
Copy Markdown
Collaborator Author

soumith commented Feb 4, 2019

@pjh5 do you know why CI "binary" builds are failing?

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@soumith is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Feb 5, 2019
Differential Revision: D13952085

Pulled By: soumith

fbshipit-source-id: 410c4e117a44c08eadc6f3ded91fafc320a7c696
@ezyang
Copy link
Copy Markdown
Contributor

ezyang commented Feb 5, 2019

The CUDA failures were real. I'm reverting this.

soumith added a commit that referenced this pull request Feb 21, 2019
Differential Revision: D13952085

Pulled By: soumith

fbshipit-source-id: 410c4e117a44c08eadc6f3ded91fafc320a7c696
soumith added a commit that referenced this pull request Feb 21, 2019
Differential Revision: D13952085

Pulled By: soumith

fbshipit-source-id: 410c4e117a44c08eadc6f3ded91fafc320a7c696
soumith added a commit that referenced this pull request Feb 21, 2019
Differential Revision: D13952085

Pulled By: soumith

fbshipit-source-id: 410c4e117a44c08eadc6f3ded91fafc320a7c696
@soumith soumith closed this Feb 21, 2019
@soumith soumith deleted the buildsfix_static branch February 21, 2019 22:35
facebook-github-bot pushed a commit that referenced this pull request Feb 22, 2019
Summary:
Attempt #2 (attempt 1 is #16705 and got reverted because of CI failures)

Fixes #14805
Pull Request resolved: #17337

Differential Revision: D14175626

Pulled By: soumith

fbshipit-source-id: 66f2e10e219a1bf88ed342ec5c89da6f2994d8eb
zdevito pushed a commit to zdevito/ATen that referenced this pull request Feb 22, 2019
Summary:
Attempt #2 (attempt 1 is pytorch/pytorch#16705 and got reverted because of CI failures)

Fixes pytorch/pytorch#14805
Pull Request resolved: pytorch/pytorch#17337

Differential Revision: D14175626

Pulled By: soumith

fbshipit-source-id: 66f2e10e219a1bf88ed342ec5c89da6f2994d8eb
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.

4 participants