Skip to content

Add opal_recursive_mutex_t support to qthreads #8500

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
Mar 31, 2021

Conversation

janciesko
Copy link
Contributor

  • Adds the opal_recursive_mutex_t type to qthreads.
  • Fixes some segfaults when ompi is configured to qthreads.

@gpaulsen gpaulsen requested a review from hppritcha March 2, 2021 14:41
@awlauria
Copy link
Contributor

bot:retest

@ompiteam-bot
Copy link

Can one of the admins verify this patch?

@awlauria
Copy link
Contributor

ok to test

@awlauria
Copy link
Contributor

bot:aws:retest

@hppritcha
Copy link
Member

ompi:bot:retest

@hppritcha
Copy link
Member

not sure what's happening at aws jenkins
bot:ompi:retest

@hppritcha
Copy link
Member

@janciesko looks like you need to rebase

@janciesko
Copy link
Contributor Author

@janciesko looks like you need to rebase

Rebased to v5.0.x

@hppritcha
Copy link
Member

oops, @janciesko i meant to rebase on top of master since there was some conflict.

@janciesko janciesko force-pushed the opal_recursive_mutex_t_2 branch from 56ca827 to 8c499a3 Compare March 25, 2021 18:33
@janciesko
Copy link
Contributor Author

oops, @janciesko i meant to rebase on top of master since there was some conflict.

Rebased to master.

@hppritcha hppritcha merged commit 92389c3 into open-mpi:master Mar 31, 2021
@awlauria
Copy link
Contributor

@janciesko @hppritcha is this a v5.0.x item?

@hppritcha
Copy link
Member

yes i'll cherry-pick over to the 5.0.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants