Skip to content

Commit 222b7f3

Browse files
committed
osc/rdma: Remove duplicate op emulation check
ompi_osc_rdma_btl_op() already includes a check to emulate a remote op with a remote fetch-and-op, so there's no need for a second check in ompi_osc_rdma_acc_single_atomic(). Signed-off-by: Brian Barrett <bbarrett@amazon.com>
1 parent 75e36e5 commit 222b7f3

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

ompi/mca/osc/rdma/osc_rdma_accumulate.c

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -244,12 +244,6 @@ static int ompi_osc_rdma_acc_single_atomic (ompi_osc_rdma_sync_t *sync, const vo
244244
int btl_op, flags;
245245
int64_t origin;
246246

247-
if (!(selected_btl->btl_flags & MCA_BTL_FLAGS_ATOMIC_OPS)) {
248-
/* btl put atomics not supported or disabled. fall back on fetch-and-op */
249-
return ompi_osc_rdma_fetch_and_op_atomic (sync, origin_addr, NULL, dt, extent, peer, target_address, target_handle,
250-
op, req);
251-
}
252-
253247
if ((8 != extent && !((MCA_BTL_ATOMIC_SUPPORTS_32BIT & atomic_flags) && 4 == extent)) ||
254248
(!(OMPI_DATATYPE_FLAG_DATA_INT & dt->super.flags) && !(MCA_BTL_ATOMIC_SUPPORTS_FLOAT & atomic_flags)) ||
255249
!ompi_op_is_intrinsic (op) || (0 == ompi_osc_rdma_op_mapping[op->op_type])) {

0 commit comments

Comments
 (0)