Skip to content

Conversation

@wzamazon
Copy link
Contributor

This patch added call to opal_set_using_threads() in orted/main.c,
which is to enable OPAL's multi-thread support.

This is because orted used multiple threads.

Without OPAL's multi-thread support, OPAL_RELEASE will not use
atomic operations to update an object's reference count, which
will lead to double free.

This patch is applied to 4.1.x directly because orte has been
removed from the master branch.

Signed-off-by: Wei Zhang wzam@amazon.com

This patch added call to opal_set_using_threads() in orted/main.c,
which is to enable OPAL's multi-thread support.

This is because orted used multiple threads.

Without OPAL's multi-thread support, OPAL_RELEASE will not use
atomic operations to update an object's reference count, which
will lead to double free.

This patch is applied to 4.1.x directly because orte has been
removed from the master branch.

Signed-off-by: Wei Zhang <wzam@amazon.com>
@wzamazon wzamazon requested a review from rhc54 January 11, 2021 16:07
@wzamazon
Copy link
Contributor Author

address #8360

@rajachan rajachan added this to the v4.1.1 milestone Jan 11, 2021
@wzamazon
Copy link
Contributor Author

Note this patch is applied to 4.1.x directly because orte has been removed from the master branch.

@jsquyres jsquyres merged commit 2c583de into open-mpi:v4.1.x Jan 11, 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.

4 participants