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.

Sign up for GitHub

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

Jump to bottom

Fix performance regression caused by enabling opal thread support #1909

Merged
merged 1 commit into from
Jul 28, 2016
Merged

Fix performance regression caused by enabling opal thread support #1909

merged 1 commit into from
Jul 28, 2016

Commits on Jul 28, 2016

  1. Fix performance regression caused by enabling opal thread support 

    This commit adds opal_using_threads() protection around the atomic
operation in OBJ_RETAIN/OBJ_RELEASE. This resolves the performance
issues seen when running psm with MPI_THREAD_SINGLE.

To avoid issues with header dependencies opal_using_threads() has been
moved to a new header (thread_usage.h). The OPAL_THREAD_ADD* and
OPAL_THREAD_CMPSET* macros have also been relocated to this header.

This commit is cherry-picked off a fix that was submitted for the v1.8
release series but never applied to master. This fixes part of the
problem reported by @nysal in open-mpi#1902.

(cherry picked from commit open-mpi/ompi-release@ce91307)

Signed-off-by: Nathan Hjelm <hjelmn@me.com>
    @hjelmn
    hjelmn committed Jul 28, 2016
    Configuration menu
    Copy the full SHA
    a8c3699 View commit details
    Browse the repository at this point in the history