e968ddfe64
* mpi/start: fix bugs in cm and ob1 start functions There were several problems with the implementation of start in Open MPI: - There are no checks whatsoever on the state of the request(s) provided to MPI_Start/MPI_Start_all. It is erroneous to provide an active request to either of these calls. Since we are already looping over the provided requests there is little overhead in verifying that the request can be started. - Both ob1 and cm were always throwing away the request on the initial call to start and start_all with a particular request. Subsequent calls would see that the request was pml_complete and reuse it. This introduced a leak as the initial request was never freed. Since the only pml request that can be mpi complete but not pml complete is a buffered send the code to reallocate the request has been moved. To detect that a request is indeed mpi complete but not pml complete isend_init in both cm and ob1 now marks the new request as pml complete. - If a new request was needed the callbacks on the original request were not copied over to the new request. This can cause osc/pt2pt to hang as the incoming message callback is never called. Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov> * osc/pt2pt: add request for gc after starting a new request Starting a new receive may cause a recursive call into the pt2pt frag receive function. If this happens and the prior request is on the garbage collection list it could cause problems. This commit moves the gc insert until after the new request has been posted. Signed-off-by: Nathan Hjelm <hjelmn@me.com> |
||
|---|---|---|
| .. | ||
| Makefile.am | ||
| owner.txt | ||
| pml_cm_cancel.c | ||
| pml_cm_component.c | ||
| pml_cm_component.h | ||
| pml_cm_recvreq.c | ||
| pml_cm_recvreq.h | ||
| pml_cm_request.c | ||
| pml_cm_request.h | ||
| pml_cm_sendreq.c | ||
| pml_cm_sendreq.h | ||
| pml_cm_start.c | ||
| pml_cm.c | ||
| pml_cm.h | ||
| post_configure.sh | ||