Skip to content

Commit

Permalink
Merge pull request open-mpi#2367 from ggouaillardet/topic/v2.x/pml_ob…
Browse files Browse the repository at this point in the history
…1_recv_memchecker

v2.x: pml/ob1: mca_pml_ob1_recv must have memchecker mark the buffer as def…
  • Loading branch information
jsquyres authored Nov 9, 2016
2 parents 8779796 + 842b6fb commit 4b718dd
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions ompi/mca/pml/ob1/pml_ob1_irecv.c
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@
#include "pml_ob1_recvfrag.h"
#include "ompi/peruse/peruse-internal.h"
#include "ompi/message/message.h"
#include "ompi/memchecker.h"

/**
* Single usage request. As we allow recursive calls to recv
Expand Down Expand Up @@ -133,6 +134,17 @@ int mca_pml_ob1_recv(void *addr,

rc = recvreq->req_recv.req_base.req_ompi.req_status.MPI_ERROR;

if (recvreq->req_recv.req_base.req_pml_complete) {
/* make buffer defined when the request is compeleted,
and before releasing the objects. */
MEMCHECKER(
memchecker_call(&opal_memchecker_base_mem_defined,
recvreq->req_recv.req_base.req_addr,
recvreq->req_recv.req_base.req_count,
recvreq->req_recv.req_base.req_datatype);
);
}

#if OMPI_ENABLE_THREAD_MULTIPLE
MCA_PML_OB1_RECV_REQUEST_RETURN(recvreq);
#else
Expand Down

0 comments on commit 4b718dd

Please sign in to comment.