[mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.2-41-g19db8d8

Service Account noreply at mpich.org
Sun Jan 3 13:39:00 CST 2016

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  19db8d89776c5e8546fda53406fc5fe564b7f4a3 (commit)
       via  7d7438750b249fb79f3190e586193815da1b667b (commit)
      from  f91f92e228bd0dee356473bd7c77ada22d6f9149 (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 -----------------------------------------------------------------

commit 19db8d89776c5e8546fda53406fc5fe564b7f4a3
Author: Jeff Hammond <jeff.science at gmail.com>
Date:   Fri Jan 1 11:23:17 2016 -0800

    test/mpi/coll: abort gather_big if np<ROOT+1
    This test is designed to run on 8+ processes and sets ROOT=7. However,
    it does not test to see if np>7, which means that it fails with an
    invalid root error when run on e.g. np=2.
    Signed-off-by: Ken Raffenetti <raffenet at mcs.anl.gov>

diff --git a/test/mpi/coll/gather_big.c b/test/mpi/coll/gather_big.c
index e649192..ce37c95 100644
--- a/test/mpi/coll/gather_big.c
+++ b/test/mpi/coll/gather_big.c
@@ -31,6 +31,11 @@ int main(int argc, char *argv[])
     MPI_Comm_rank(MPI_COMM_WORLD, &rank);
     MPI_Comm_size(MPI_COMM_WORLD, &size);
+    if (size < (ROOT+1)) {
+        fprintf(stderr, "At least %d processes required\n", ROOT+1);
+        MPI_Abort(MPI_COMM_WORLD, 1);
+    }
     sendbuf = malloc(COUNT * sizeof(long));
     if (sendbuf == NULL) {
         fprintf(stderr, "PE %d:ERROR: malloc of sendbuf failed\n", rank);


commit 7d7438750b249fb79f3190e586193815da1b667b
Author: Jeff Hammond <jeff.science at gmail.com>
Date:   Fri Jan 1 11:49:06 2016 -0800

    test/mpi/rma: abort allgatherv4 if np<3.
    Test will segfault (divide by zero) without sufficient number of procs.
    Signed-off-by: Ken Raffenetti <raffenet at mcs.anl.gov>

diff --git a/test/mpi/coll/allgatherv4.c b/test/mpi/coll/allgatherv4.c
index b1e1c5a..1105407 100644
--- a/test/mpi/coll/allgatherv4.c
+++ b/test/mpi/coll/allgatherv4.c
@@ -53,6 +53,11 @@ int main(int argc, char **argv)
     MPI_Comm_size(MPI_COMM_WORLD, &comm_size);
     MPI_Comm_rank(MPI_COMM_WORLD, &comm_rank);
+    if (comm_size < 3) {
+        fprintf(stderr, "At least 3 processes required\n");
+        MPI_Abort(MPI_COMM_WORLD, 1);
+    }
     if (LARGE_BUF * comm_size > MAX_BUF)
         goto fn_exit;


Summary of changes:
 test/mpi/coll/allgatherv4.c |    5 +++++
 test/mpi/coll/gather_big.c  |    5 +++++
 2 files changed, 10 insertions(+), 0 deletions(-)

MPICH primary repository

More information about the commits mailing list