Skip to content
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

v2.0.x: coll/libnbc: fix race condition with multi threaded apps #2442

Merged
merged 2 commits into from
Dec 3, 2016

Conversation

ggouaillardet
Copy link
Contributor

protect the mca_coll_libnbc_component.active_requests list with
the new mca_coll_libnbc_component.lock mutex.

Fixes open-mpi/ompi@2427

Thanks Jie Hu for the report

Signed-off-by: Gilles Gouaillardet gilles@rist.or.jp

(cherry picked from commit 2c94a3a)

protect the mca_coll_libnbc_component.active_requests list with
the new mca_coll_libnbc_component.lock mutex.

Fixes open-mpi/ompi@2427

Thanks Jie Hu for the report

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>

(cherry picked from commit open-mpi/ompi@2c94a3a)
no code change

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>

(cherry picked from commit open-mpi/ompi@1509816)
@jsquyres
Copy link
Member

jsquyres commented Dec 2, 2016

@bosilca @jjhursey Can you review? Thanks.

Copy link
Member

@jjhursey jjhursey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per discussion of same patch for PR #2443 - this looks good. Thanks!

@hppritcha
Copy link
Member

@jsquyres let's merge this one too.

@jsquyres jsquyres merged commit 1237c2a into open-mpi:v2.0.x Dec 3, 2016
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