Gracefully deal with a get returning 1 (complete right away).

bosilca · bosilca · commit 2a2db13b321b · 2017-10-01T02:24:02.000-04:00
Kudos to @EmmanuelBRELLE for spotting it. Signed-off-by: George Bosilca <bosilca@icl.utk.edu>
diff --git a/ompi/mca/pml/ob1/pml_ob1_recvreq.c b/ompi/mca/pml/ob1/pml_ob1_recvreq.c
@@ -470,7 +470,7 @@ int mca_pml_ob1_recv_request_get_frag (mca_pml_ob1_rdma_frag_t *frag)
     rc = mca_bml_base_get (bml_btl, frag->local_address, frag->remote_address, local_handle,
                            (mca_btl_base_registration_handle_t *) frag->remote_handle, frag->rdma_length,
                            0, MCA_BTL_NO_ORDER, mca_pml_ob1_rget_completion, frag);
-    if( OPAL_UNLIKELY(OMPI_SUCCESS != rc) ) {
+    if( OPAL_UNLIKELY(OMPI_SUCCESS > rc) ) {
         return mca_pml_ob1_recv_request_get_frag_failed (frag, OMPI_ERR_OUT_OF_RESOURCE);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -470,7 +470,7 @@ int mca_pml_ob1_recv_request_get_frag (mca_pml_ob1_rdma_frag_t *frag)`
`470`	`470`	`rc = mca_bml_base_get (bml_btl, frag->local_address, frag->remote_address, local_handle,`
`471`	`471`	`(mca_btl_base_registration_handle_t *) frag->remote_handle, frag->rdma_length,`
`472`	`472`	`0, MCA_BTL_NO_ORDER, mca_pml_ob1_rget_completion, frag);`
`473`		`- if( OPAL_UNLIKELY(OMPI_SUCCESS != rc) ) {`
	`473`	`+ if( OPAL_UNLIKELY(OMPI_SUCCESS > rc) ) {`
`474`	`474`	`return mca_pml_ob1_recv_request_get_frag_failed (frag, OMPI_ERR_OUT_OF_RESOURCE);`
`475`	`475`	`}`
`476`	`476`