[mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.2b3-218-gb2870c5

[Service Account]

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/b2870c55d476a7fe2498f1a231c2d62b3a548332

commit b2870c55d476a7fe2498f1a231c2d62b3a548332
Author: Lena Oden <loden at anl.gov>
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