Skip to content

Commit 97d66bc

Browse files
committed
This fixes trac:2154 - CSUM PML false positive. Needs to go to both cmr:v1.4.2 and cmr:v1.5
This commit was SVN r22590. The following Trac tickets were found above: Ticket 2154 --> https://svn.open-mpi.org/trac/ompi/ticket/2154
1 parent d40d216 commit 97d66bc

File tree

2 files changed

+0
-13
lines changed

2 files changed

+0
-13
lines changed

ompi/mca/pml/csum/pml_csum_recvreq.c

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1054,17 +1054,6 @@ void mca_pml_csum_recv_req_start(mca_pml_csum_recv_request_t *req)
10541054
if(req->req_recv.req_base.req_peer == OMPI_ANY_SOURCE) {
10551055
frag = recv_req_match_wild(req, &proc);
10561056
queue = &comm->wild_receives;
1057-
#if !OPAL_ENABLE_HETEROGENEOUS_SUPPORT
1058-
/* As we are in a homogeneous environment we know that all remote
1059-
* architectures are exactly the same as the local one. Therefore,
1060-
* we can safely construct the convertor based on the proc
1061-
* information of rank 0.
1062-
*/
1063-
if( NULL == frag ) {
1064-
req->req_recv.req_base.req_proc = ompi_proc_local_proc;
1065-
prepare_recv_req_converter(req);
1066-
}
1067-
#endif /* !OPAL_ENABLE_HETEROGENEOUS_SUPPORT */
10681057
} else {
10691058
proc = &comm->procs[req->req_recv.req_base.req_peer];
10701059
req->req_recv.req_base.req_proc = proc->ompi_proc;

ompi/mca/pml/csum/pml_csum_recvreq.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -224,12 +224,10 @@ static inline void recv_req_matched(mca_pml_csum_recv_request_t *req,
224224
opal_atomic_wmb();
225225
#endif
226226
if(req->req_recv.req_bytes_packed > 0) {
227-
#if OPAL_ENABLE_HETEROGENEOUS_SUPPORT
228227
if(MPI_ANY_SOURCE == req->req_recv.req_base.req_peer) {
229228
/* non wildcard prepared during post recv */
230229
prepare_recv_req_converter(req);
231230
}
232-
#endif /* OPAL_ENABLE_HETEROGENEOUS_SUPPORT */
233231
PERUSE_TRACE_COMM_EVENT(PERUSE_COMM_REQ_XFER_BEGIN,
234232
&req->req_recv.req_base, PERUSE_RECV);
235233
}

0 commit comments

Comments
 (0)