docs: add note about needing hwloc 2.8.0 for PV #11494
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
is this the right place for this kind of note? |
jsquyres
reviewed
Mar 16, 2023
|
Sorry -- when I made the previous suggestion, I failed to notice that we called it "Hardware Locality" on this page, not "Hwloc". That being said, I wonder if we should make a more sweeping comment, because other libraries have problems with specific versions, too (UCX and libfabric jump to mind). For example, how about adding this below the table in .. note:: The versions listed in this table are the *minimum* versions needed. In general, the Open MPI community recommends using more recent versions of both the :ref:`required support libraries <label-install-required-support-libraries>` and any other optional support libraries. This is because more recent versions typically tend to include bug fixes, sometimes affecting Open MPI functionality. As a specific example, there is a known issue with `Hardware Locality <https://www.open-mpi.org/projects/hwloc/>`_ releases older than v2.8.0 on systems with Intel Ponte Vecchio accelerators. If you run Open MPI on such systems, you need to use Hwloc v2.8.0 or newer, or you will experience undefined behavior.
This effect is not unique to the Hardware Locality library; this is why the Open MPI community recommends using as recent as possible versions of all support libraries. |
3f2f15f to
06ce9ab
Compare
|
@jsquyres could you recheck? does the last sentence look okay? |
jsquyres
requested changes
Mar 21, 2023
06ce9ab to
4146538
Compare
accelerators. related to issue open-mpi#11246 Signed-off-by: Howard Pritchard <howardp@lanl.gov>
4146538 to
c1b5e6e
Compare
|
@jsquyres check again |
jsquyres
approved these changes
Mar 23, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
accelerators.
related to issue #11246