Skip to content

Commit d6d73b7

Browse files
hppritchajsquyres
authored andcommitted
mtl/ofi: replace OMPI_UNLIKELY with OPAL version
one off patch for v4.0.x. for some reason commit on master didn't have this problem. Signed-off-by: Howard Pritchard <[email protected]> (cherry picked from commit 5f3dbdb) Note that this commit is actually a cherry-pick from the v4.0.x branch. This is the opposite direction than what we nornmally do: we usually commit to master first and then cherry-pick to the release branches (vs. the other way around). As is probably evident from the original commit message above, through a comedy of errors, this commit was actually applied to the v4.0.x branch first and then cherry-picked back to master (i.e., the problem *did* exist in the original master commit 3aca4af, but it was not recongized at the time). Signed-off-by: Jeff Squyres <[email protected]>
1 parent 2808569 commit d6d73b7

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

ompi/mca/mtl/ofi/mtl_ofi.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22
* Copyright (c) 2013-2018 Intel, Inc. All rights reserved
33
* Copyright (c) 2017 Los Alamos National Security, LLC. All rights
44
* reserved.
5+
* Copyright (c) 2019 Triad National Security, LLC. All rights
6+
* reserved.
57
*
68
* Copyright (c) 2018 Amazon.com, Inc. or its affiliates. All Rights reserved.
79
* $COPYRIGHT$
@@ -562,7 +564,7 @@ ompi_mtl_ofi_isend_generic(struct mca_mtl_base_module_t *mtl,
562564
sep_peer_fiaddr = fi_rx_addr(endpoint->peer_fiaddr, ctxt_id, ompi_mtl_ofi.rx_ctx_bits);
563565

564566
ompi_ret = ompi_mtl_datatype_pack(convertor, &start, &length, &free_after);
565-
if (OMPI_UNLIKELY(OMPI_SUCCESS != ompi_ret)) return ompi_ret;
567+
if (OPAL_UNLIKELY(OMPI_SUCCESS != ompi_ret)) return ompi_ret;
566568

567569
ofi_req->buffer = (free_after) ? start : NULL;
568570
ofi_req->length = length;

0 commit comments

Comments
 (0)