-
Notifications
You must be signed in to change notification settings - Fork 885
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
remove --enable-heterogenous configury option #2802
Comments
the test provided in #168 runs fine with master + a few patches of mine. there was a previous RFC to get rid of heterogeneous support at https://www.open-mpi.org/community/lists/devel/2014/04/14584.php |
fwiw, this was mentionned (and in a negative way) in that post that got some audience at that time |
@jsquyres Do you want me to remove the dead code disabling this option will produce as well? Or I can leave it in incase we decided to support this option again in the future. I have a fix ready if you just want the option disabled. |
Maintaining means testing. If we are going to maintain I'd like to see either a MTT or CI test of the hetero feature. |
the first step is to configure with the second step is to configure with |
@ggouaillardet We talked about this on the telecon this morning, and bottom line is that we will apply our usual policy here: if someone is willing to maintain and test this feature, then no problem leaving it in the code. This probably applies sooner to v2.1 than v2.0.3, but if you can update the support and at least post on the PR your MTT results, then we are content to leave this alive. |
At present I don't care about BE/LE support but one scenario of my use cases is to make remote calls to x86_64 servers (LE) with high electric power consumption from a low power 32-bit system (LE). Will this be affected if heterogeneous support is removed? |
I think this issue is now moot, per #2802 (comment) and #2940. |
Mixed big endian/little endian no longer works in Open MPI and there are no plans currently to fix it. To avoid user confusion, remove the configure --enable-heterogenous option. Related to issue #639.
Rather than just removing entirely from the configury, it would be better to have configure print out an error message if this option is attempted and abort.
The text was updated successfully, but these errors were encountered: