OSHMEM/UCX: implements atomic support #913
Conversation
|
Test PASSed. |
|
Sorry, we are already feature complete for v2.0.0. Deferring to v2.1. |
|
👍 |
|
@hppritcha is it ready to be merged? |
|
This PR will be merged in to v2.x branch once we have branched off 2.0.x for 2.0 release. |
|
bot:mellanox:retest |
|
Test PASSed. |
|
@hppritcha Since this is a bug fix, we would very much like to have this included into 2.0.1. |
|
This doesn't look like a bug fix. I think this should go in to 2.1. |
|
@jladd-mlnx Is there a bug fix in this PR that can be separated out into a v2.0.1 PR? |
|
@alex-mikheev @jladd-mlnx This has merge conflicts. Can someone rebase? |
ucx atomic component has a real code now. fixes bug in spml ucx add_procs removes redundant parameter checks from atomic components. (cherry picked from commit f627608)
alex-mikheev
force-pushed
the
topic/oshmem_ucx_atomic_v2.x
branch
from
c7b1156 to
908cf4e
Compare
|
Test PASSed. |
|
@jsquyres I think this is ready to go and it looks like some other oshmem prs depend on this one first. |
@miked-mellanox
ucx atomic component has a real code now.
fixes bug in spml ucx arr_procs
removes redundant parameter checks from atomic components.
(cherry picked from commit open-mpi/ompi@f627608)