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.

Sign up for GitHub

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

Jump to bottom

Disable dynamic add procs for 32 bit and big endian environments #1348

Closed
jsquyres opened this issue Feb 9, 2016 · 2 comments
Closed

Disable dynamic add procs for 32 bit and big endian environments #1348

jsquyres opened this issue Feb 9, 2016 · 2 comments
Assignees
@hjelmn
Labels
bug Severity: blocker
Milestone
v2.0.0

Comments

@jsquyres
Copy link
Member

jsquyres commented Feb 9, 2016

Per discussion on the Feb 9 2016 webex, we decided that the quickest / easiest solution for v2.0.0 is to simply disable the dynamic add procs behavior for big endian and 32 bit environments.

This will allow discussion to continue on #1345 to come to a better long-term solution (which may take a little time to get consensus, implement, test, ...etc. -- we may also want to discuss this at the upcoming face to face Dallas meeting).

It was decided that @hjelmn will do this quick fix for v2.0.0.

FYI/heads up for: @bosilca @rhc54 @ggouaillardet
@jsquyres jsquyres added this to the v2.0.0 milestone Feb 9, 2016
hjelmn added a commit to hjelmn/ompi that referenced this issue Feb 9, 2016
@hjelmn
ompi: temporarily disable dynamic add_procs on big-endian and 32-bit
3bea485 
This commit temporarily disables the dynamic add_procs support on
big-endian and 32-bit systems. There is an issue with the sentinel
used to indicate when proc needs to be generated that causes issues on
these systems. This will be removed once the underlying issue has been
resolved.

Fixes open-mpi#1348

Signed-off-by: Nathan Hjelm <[email protected]>
@hjelmn hjelmn mentioned this issue Feb 9, 2016
Closed
@jsquyres jsquyres mentioned this issue Feb 9, 2016
Closed
@jsquyres
Copy link
Member Author

jsquyres commented Feb 9, 2016

@hjelmn Can you refer to #1350 in the comments in your code?

@jsquyres
Copy link
Member Author

This issue is now moot -- we do not need to disable dynamic add procs for big endian and 32 bit machines. Closing this issue.

jsquyres pushed a commit to jsquyres/ompi that referenced this issue Sep 19, 2016
Merge pull request open-mpi#1348 from jsquyres/pr/v1.10.x/fix-fortran…
4d04802 
…-wrapper-compiler-rpath-detection

opal_setup_wrappers.m4: fix fortran rpath detection
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hjelmn hjelmn

Labels
bug Severity: blocker
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging a pull request may close this issue.

ompi: temporarily disable dynamic add_procs on big-endian and 32-bit hjelmn/ompi
2 participants
@jsquyres @hjelmn