This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "MPICH primary repository". The branch, master has been updated via df15b7f3d3683d67af651d436b5816ca5f0383e1 (commit) from 9c4b3bbd6a9c26d76e4e38bbacede668b0e08659 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.mpich.org/mpich.git/commitdiff/df15b7f3d3683d67af651d436b5816ca5f... commit df15b7f3d3683d67af651d436b5816ca5f0383e1 Author: Pavan Balaji <[email protected]> Date: Fri May 9 12:37:44 2014 -0500 Fix ibsend failures with the pathscale compilers. A bunch of arbitrary changes were made to the ibsend.c test in [85e4a507]. It included using different pthread handles, which is correct. But the same commit also included: (1) setting a joinable pthread attribute, (2) using pthread_exit to force kill threads, and (3) several updates to the documentation. The first change isunnecessary since it's set by default. The second change is plainly wrong. pthreads should be cleanly joined, not force killed. The third change is fine, but should have been a separate commit. This patch fixes the second change and uses "return NULL" instead of pthread_exit. Signed-off-by: Ken Raffenetti <[email protected]> diff --git a/test/mpi/threads/pt2pt/ibsend.c b/test/mpi/threads/pt2pt/ibsend.c index 6050c00..1e634c4 100644 --- a/test/mpi/threads/pt2pt/ibsend.c +++ b/test/mpi/threads/pt2pt/ibsend.c @@ -43,7 +43,7 @@ void *receiver(void *ptr) MPI_Recv(buf, MSGSIZE, MPI_CHAR, MPI_ANY_SOURCE, MPI_ANY_TAG, MPI_COMM_WORLD, MPI_STATUS_IGNORE); - pthread_exit(NULL); + return NULL; } @@ -53,7 +53,7 @@ void *sender_bsend(void *ptr) MPI_Bsend(buffer, MSGSIZE, MPI_CHAR, (rank + 1) % size, 0, MPI_COMM_WORLD); - pthread_exit(NULL); + return NULL; } void *sender_ibsend(void *ptr) @@ -64,7 +64,7 @@ void *sender_ibsend(void *ptr) MPI_COMM_WORLD, &req); MPI_Wait(&req, MPI_STATUS_IGNORE); - pthread_exit(NULL); + return NULL; } void *sender_isend(void *ptr) @@ -75,7 +75,7 @@ void *sender_isend(void *ptr) MPI_COMM_WORLD, &req); MPI_Wait(&req, MPI_STATUS_IGNORE); - pthread_exit(NULL); + return NULL; } void *sender_send(void *ptr) @@ -84,7 +84,7 @@ void *sender_send(void *ptr) MPI_Send(buffer, MSGSIZE, MPI_CHAR, (rank + 1) % size, 0, MPI_COMM_WORLD); - pthread_exit(NULL); + return NULL; } int main(int argc, char *argv[]) ----------------------------------------------------------------------- Summary of changes: test/mpi/threads/pt2pt/ibsend.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) hooks/post-receive -- MPICH primary repository