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 b2870c55d476a7fe2498f1a231c2d62b3a548332 (commit) from c70ed954dfd595f26f16895c0eb685fbe98c72a9 (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/b2870c55d476a7fe2498f1a231c2d62b3a... commit b2870c55d476a7fe2498f1a231c2d62b3a548332 Author: Lena Oden <[email protected]> Date: Sun Jul 12 14:03:34 2015 -0500 Fix memory leaks in test/mpi/comm/comm_idup_nb Fix memory leaks, remove xfail for comm_idup_isend and comm_idup_comm, since 2269 was fixed. comm_idup_nb still fails because of 2284 No reviewer diff --git a/test/mpi/comm/comm_idup_nb.c b/test/mpi/comm/comm_idup_nb.c index fe8e354..67dc07b 100644 --- a/test/mpi/comm/comm_idup_nb.c +++ b/test/mpi/comm/comm_idup_nb.c @@ -11,7 +11,7 @@ #include "mpitest.h" -#define ITERS 3 +#define ITERS 5 @@ -102,9 +102,15 @@ int main(int argc, char **argv) MPI_Ibcast(&in[i], 1, MPI_INT, 0,test_comm, &sreq[i]); MPI_Comm_idup(test_comm, &new_comm[i], &sreq[i+ITERS]); } - sol = 815; MPI_Waitall(ITERS*2, sreq, MPI_STATUS_IGNORE); + sol = 815; + for(i = 0; i< ITERS; i++) { + if(in[i]!= sol) errs++; + errs += MTestTestComm(new_comm[i]); + MPI_Comm_free(&new_comm[i]); + } + MTestFreeComm(&test_comm); } /* Now the test for inter-communicators */ @@ -118,17 +124,16 @@ int main(int argc, char **argv) /* Ibarrier */ - /* for(i = 0; i< ITERS; i++) { + for(i = 0; i< ITERS; i++) { MPI_Ibarrier(test_comm, &sreq[i]); MPI_Comm_idup(test_comm, &new_comm[i], &sreq[i+ITERS]); } MPI_Waitall(ITERS*2, sreq, MPI_STATUS_IGNORE); for(i = 0; i< ITERS; i++) { - if(in[i]!= sol) errs++; errs += MTestTestComm(new_comm[i]); MPI_Comm_free(&new_comm[i]); } -*/ + /*Ibcast */ int root; for(i = 0; i< ITERS; i++) { @@ -172,6 +177,7 @@ int main(int argc, char **argv) MPI_Comm_free(&new_comm[i]); } + MTestFreeComm(&test_comm); } MTest_Finalize( errs ); diff --git a/test/mpi/comm/testlist b/test/mpi/comm/testlist index 13780cd..606a038 100644 --- a/test/mpi/comm/testlist +++ b/test/mpi/comm/testlist @@ -30,9 +30,9 @@ comm_idup 9 mpiversion=3.0 comm_idup_mul 2 mpiversion=3.0 comm_idup_overlap 2 mpiversion=3.0 comm_idup_iallreduce 6 mpiversion=3.0 -comm_idup_nb 6 mpiversion=3.0 xfail=ticket2283 -comm_idup_isend 6 mpiversion=3.0 xfail=ticket2269 -comm_idup_comm 6 mpiversion=3.0 xfail=ticket2269 +comm_idup_nb 6 mpiversion=3.0 xfail=ticket2284 +comm_idup_isend 6 mpiversion=3.0 +comm_idup_comm 6 mpiversion=3.0 dup_with_info 2 mpiversion=3.0 dup_with_info 4 mpiversion=3.0 dup_with_info 9 mpiversion=3.0 ----------------------------------------------------------------------- Summary of changes: test/mpi/comm/comm_idup_nb.c | 16 +++++++++++----- test/mpi/comm/testlist | 6 +++--- 2 files changed, 14 insertions(+), 8 deletions(-) hooks/post-receive -- MPICH primary repository