Skip to content

Conversation

@vspetrov
Copy link

@vspetrov vspetrov commented Oct 6, 2017

Adds new hcoll type mappings: lb/ub/wchar/bool (for legacy mpi apps).
Small type create optimization: if hcoll type creation fails we set zero dte on the ompi_datatype for subsequent queries to avoid repeating attempts to create hcoll type.

Valentin Petrov added 2 commits October 6, 2017 10:29
    Adds support for legacy MPI_UB/LB types (old apps may use it) as
    well as for BOOL/WCHAR.

Signed-off-by: Valentin Petrov <valentinp@mellanox.com>
    If hcoll fails to create mpi derived type let's set zero_dte on this dtype.
    This will save cycles on subsequent collective calls with the same derived
    type since we will not try to create hcoll type again.

Signed-off-by: Valentin Petrov <valentinp@mellanox.com>
@vspetrov vspetrov requested review from bureddy and jladd-mlnx October 6, 2017 13:00
Copy link
Member

@jladd-mlnx jladd-mlnx left a comment

Choose a reason for hiding this comment

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

👍 Good to go.

@jladd-mlnx
Copy link
Member

@vspetrov Please cherry pick into 3.0.1 and 3.1.x branches.

@jladd-mlnx jladd-mlnx merged commit bb4bbbf into open-mpi:master Oct 9, 2017
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.

3 participants