Skip to content

Update to hwloc 2.4.0 #8314

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

Merged
merged 1 commit into from
Feb 9, 2021
Merged

Update to hwloc 2.4.0 #8314

merged 1 commit into from
Feb 9, 2021

Conversation

jjhursey
Copy link
Member

No description provided.

Signed-off-by: Joshua Hursey <[email protected]>
@jjhursey
Copy link
Member Author

This picks up a PGI fix from upstream hwloc

@jjhursey jjhursey requested a review from jsquyres December 22, 2020 17:11
@jjhursey
Copy link
Member Author

I updated the PGI in our containers - retesting just to confirm that nothing is broken 🤞
bot:ibm:retest

@jjhursey
Copy link
Member Author

bot:ibm:retest

@open-mpi open-mpi deleted a comment from ibm-ompi Dec 22, 2020
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 22, 2020
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 22, 2020
@jjhursey
Copy link
Member Author

bot:ibm:pgi:retest

@open-mpi open-mpi deleted a comment from ibm-ompi Dec 22, 2020
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 23, 2020
@jjhursey
Copy link
Member Author

OK. I fixed the PGI issue - it was a CI setup issue, not PGI or this PR. So now we are using PGI 20.11-0

@open-mpi open-mpi deleted a comment from ibm-ompi Dec 23, 2020
@jjhursey
Copy link
Member Author

I found another PGI'ism in hwloc and filed a bug for it.

So though the 2.4.0 release fixes the detection of PGI, there was still a bit more to do. We can take this in as a good thing to do. We might want to update again before branching for v5.

@jjhursey jjhursey merged commit 56ed3a6 into open-mpi:master Feb 9, 2021
@jjhursey jjhursey deleted the update-hwloc branch February 9, 2021 15:27
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.

2 participants