Skip to content

Commit d8dd525

Browse files
shamispalex-mikheev
authored andcommitted
For UCX it is legal to return UCS_INPROGRESS (1) code for non-blocking function
calls, which means that the operation was successfully started but not immediately completed. This is a "good" return code that should not be handled as an error. Signed-off-by: Pavel Shamis (Pasha) <[email protected]> (cherry picked from commit 92b0ebd)
1 parent 803a11f commit d8dd525

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

oshmem/mca/spml/ucx/spml_ucx.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
* All rights reserved.
44
* Copyright (c) 2014-2016 Research Organization for Information Science
55
* and Technology (RIST). All rights reserved.
6+
* Copyright (c) 2016 ARM, Inc. All rights reserved.
67
* $COPYRIGHT$
78
*
89
* Additional copyrights may follow
@@ -483,7 +484,7 @@ int mca_spml_ucx_get_nb(void *src_addr, size_t size, void *dst_addr, int src, vo
483484
status = ucp_get_nbi(mca_spml_ucx.ucp_peers[src].ucp_conn, dst_addr, size,
484485
(uint64_t)rva, ucx_mkey->rkey);
485486

486-
return ucx_status_to_oshmem(status);
487+
return ucx_status_to_oshmem_nb(status);
487488
}
488489

489490
int mca_spml_ucx_put(void* dst_addr, size_t size, void* src_addr, int dst)
@@ -509,7 +510,7 @@ int mca_spml_ucx_put_nb(void* dst_addr, size_t size, void* src_addr, int dst, vo
509510
status = ucp_put_nbi(mca_spml_ucx.ucp_peers[dst].ucp_conn, src_addr, size,
510511
(uint64_t)rva, ucx_mkey->rkey);
511512

512-
return ucx_status_to_oshmem(status);
513+
return ucx_status_to_oshmem_nb(status);
513514
}
514515

515516
int mca_spml_ucx_fence(void)

oshmem/mca/spml/ucx/spml_ucx.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
* All rights reserved.
44
* Copyright (c) 2016 Research Organization for Information Science
55
* and Technology (RIST). All rights reserved.
6+
* Copyright (c) 2016 ARM, Inc. All rights reserved.
67
* $COPYRIGHT$
78
*
89
* Additional copyrights may follow

0 commit comments

Comments
 (0)