Skip to content

Conversation

@jjhursey
Copy link
Member

  • The command line options --display-map --display-devel-allocation --display-topo
    would crash because the wrong object was being passed to the internal
    print function when processing the --display-topo
    • Passing orte_topology_t instead of hwloc_topology_t to the opal_hwloc_print function

bot:notacherrypick

 * The command line options `--display-map --display-devel-allocation --display-topo`
   would crash because the wrong object was being passed to the internal
   print function when processing the `--display-topo`
   - Passing `orte_topology_t` instead of `hwloc_topology_t` to the `opal_hwloc_print` function

Signed-off-by: Joshua Hursey <[email protected]>
@jjhursey jjhursey added this to the v4.1.5 milestone Aug 15, 2022
@jjhursey jjhursey requested review from gpaulsen and hppritcha August 15, 2022 19:52
@jjhursey
Copy link
Member Author

FYI: This works fine on main/v5.0.x with the command line:

mpirun --display map-devel,topo,allocation -np 2 ./hello

You can see that PRRTE already has the fix here

@awlauria awlauria requested review from bwbarrett and jsquyres and removed request for gpaulsen and hppritcha August 16, 2022 17:51
@awlauria
Copy link
Contributor

bot:aws:retest

@jsquyres jsquyres merged commit 5abd86c into open-mpi:v4.1.x Aug 17, 2022
@jjhursey jjhursey deleted the v41-fix-display-topo branch August 17, 2022 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants