-
Notifications
You must be signed in to change notification settings - Fork 932
display-devel-map: minor output tweak #551
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
display-devel-map: minor output tweak #551
Conversation
hwloc output can get fairly long, especially on machines with lots of cores and/or hyperthreads. So put the Locale and Binding output on separate lines.
|
|
Refer to this link for build results (access rights to CI server needed): Build Log Test FAILed. |
|
@miked-mellanox Something must be wrong with the Jenkins setup; this is another false failure. Can you have a look? Thanks! |
|
bot:retest |
|
@miked-mellanox Ah - your comment on open-mpi/ompi-release#254 (comment) just came in. Thanks! bot:retest |
|
yep, should be fixed now |
|
You beat me to it - I realized after I'd gone back to bed that we needed to break these lines. We probably need to come up with some kind of regex or something as the number of cores/socket keep going up, but that's for another day. Thx! 👍 |
…ut-tweak display-devel-map: minor output tweak
|
|
Refer to this link for build results (access rights to CI server needed): Build Log Test FAILed. |
opal/mutex: remove unnecessary ()s from OPAL_SCOPED_LOCK macro
add container status to -prot output
hwloc output can get fairly long, especially on machines with lots of cores and/or hyperthreads. So put the Locale and Binding output on separate lines.
Specifically, I just tweaked one line of code to make the output look like this:
instead of
@rhc54 Is this ok? If so, this PR can be merged, and this commit can be added to open-mpi/ompi-release#255