Skip to content

opal/mca/threads: fix Argobots support [5.0.x] #9375

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 5 commits into from
Sep 16, 2021

Conversation

devreal
Copy link
Contributor

@devreal devreal commented Sep 14, 2021

Cherry-pick of #8767 to v5.0.x

"opal_list.h" happens to be included by "tsd.h" in mca/threads when a
thread type is POSIX, but "mca/threads/threads.h" of other thread types
do indirectly not include tsd.h and thus no "opal_list.h".

"pmix_internal.h" directly uses opal_list_item_t and thus should include
"opal_list.h" in its header.

Signed-off-by: Shintaro Iwasaki <[email protected]>
(cherry picked from commit 8baab96)
Signed-off-by: Shintaro Iwasaki <[email protected]>
(cherry picked from commit 43c21f4)
Argobots 1.1 started to support static mutex/cond initialization.  This
patch introduces them to clean up the current implementation.

Signed-off-by: Shintaro Iwasaki <[email protected]>
(cherry picked from commit 8500fc4)
This patch removes unused variables and types.

Signed-off-by: Shintaro Iwasaki <[email protected]>
(cherry picked from commit f26eb9b)
Signed-off-by: Shintaro Iwasaki <[email protected]>
(cherry picked from commit 0be6865)
@devreal devreal requested a review from hppritcha September 14, 2021 21:59
@awlauria awlauria added this to the v5.0.0 milestone Sep 16, 2021
@gpaulsen gpaulsen merged commit cc3515b into open-mpi:v5.0.x Sep 16, 2021
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.

5 participants