Skip to content

Commit ee149fc

Browse files
committed
MPI3 (unchanged in 4) says that errors after MPI_REQUEST_FREE are FATAL
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
1 parent bec7dfc commit ee149fc

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

ompi/mca/pml/ob1/pml_ob1_recvreq.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,9 @@ recv_request_pml_complete(mca_pml_ob1_recv_request_t *recvreq)
179179

180180
if(true == recvreq->req_recv.req_base.req_free_called) {
181181
if( MPI_SUCCESS != recvreq->req_recv.req_base.req_ompi.req_status.MPI_ERROR ) {
182-
OMPI_ERRHANDLER_NOHANDLE_INVOKE(MPI_ERR_REQUEST, "Recv error after request freed.");
182+
/* An error after freeing the request MUST be fatal
183+
* MPI3 ch3.7: MPI_REQUEST_FREE */
184+
ompi_mpi_errors_are_fatal_comm_handler(NULL, MPI_ERR_REQUEST, "Recv error after request freed");
183185
}
184186
MCA_PML_OB1_RECV_REQUEST_RETURN(recvreq);
185187
} else {

ompi/mca/pml/ob1/pml_ob1_sendreq.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,9 @@ send_request_pml_complete(mca_pml_ob1_send_request_t *sendreq)
276276
MCA_PML_OB1_SEND_REQUEST_MPI_COMPLETE(sendreq, true);
277277
} else {
278278
if( MPI_SUCCESS != sendreq->req_send.req_base.req_ompi.req_status.MPI_ERROR ) {
279-
OMPI_ERRHANDLER_NOHANDLE_INVOKE(MPI_ERR_REQUEST, "Send error after request freed");
279+
/* An error after freeing the request MUST be fatal
280+
* MPI3 ch3.7: MPI_REQUEST_FREE */
281+
ompi_mpi_errors_are_fatal_comm_handler(NULL, MPI_ERR_REQUEST, "Send error after request freed");
280282
}
281283
}
282284
} else {

0 commit comments

Comments
 (0)