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 d089353f5514a61062bd34bde2c0c5135902b97c (commit) from a5aa7920ba7045ff66a3c3f5cfec7d8037b40fea (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/d089353f5514a61062bd34bde2c0c51359... commit d089353f5514a61062bd34bde2c0c5135902b97c Author: Rob Latham <[email protected]> Date: Fri Nov 1 16:51:20 2013 -0500 testcase for status object manipulations MPI_Count in a status object breaks ABI, so we have a scheme to use lo and hi parts of adjacent struct members. but it's not foolproof. Test case to demonstrate bugs and hopefully catch future regressions. Signed-off-by: Pavan Balaji <[email protected]> diff --git a/test/mpi/pt2pt/Makefile.am b/test/mpi/pt2pt/Makefile.am index 5031100..4929daa 100644 --- a/test/mpi/pt2pt/Makefile.am +++ b/test/mpi/pt2pt/Makefile.am @@ -48,5 +48,6 @@ noinst_PROGRAMS = \ waittestnull \ sendall \ large_message \ - mprobe + mprobe \ + big_count_status diff --git a/test/mpi/pt2pt/big_count_status.c b/test/mpi/pt2pt/big_count_status.c new file mode 100644 index 0000000..3b93109 --- /dev/null +++ b/test/mpi/pt2pt/big_count_status.c @@ -0,0 +1,51 @@ +#include <mpi.h> +#include <assert.h> +#include <stdio.h> + +int test_count(MPI_Count count) +{ + MPI_Status stat; + int cancelled, cancelled2; + MPI_Count bcount, bcount2; + int nerrs = 0; + + bcount = count; + cancelled = 0; + MPI_Status_set_cancelled(&stat, cancelled); + MPI_Status_set_elements_x(&stat, MPI_BYTE, bcount); + MPI_Get_elements_x(&stat, MPI_BYTE, &bcount2); + MPI_Test_cancelled(&stat, &cancelled2); + if (bcount != bcount2) { + fprintf(stderr, "Count Error: expected %lx, got %lx\n", bcount, bcount2); + nerrs++; + } + if (cancelled != cancelled2) { + fprintf(stderr, "Cancelled Error: expected %d, got %d\n", cancelled, cancelled2); + nerrs++; + } + return nerrs; +} + +int main(int argc, char **argv) +{ + int nerrors = 0; + + MPI_Init(&argc, &argv); + /* baseline: this tiny value should pose no problems */ + nerrors += test_count(60); + /* one with no next-to-high-bits set */ + nerrors += test_count(0x3654321f71234567); + /* masking after shift can help the count_high, but count_low is still + * wrong */ + nerrors += test_count(0x7654321f71234567); + /* original problematic count reported by Artem Yalozo */ + nerrors += test_count(0x7654321ff1234567); + + if (nerrors != 0) { + fprintf(stderr, "found %d errors\n", nerrors); + } + else { + printf(" No Errors\n"); + } + MPI_Finalize(); +} diff --git a/test/mpi/pt2pt/testlist b/test/mpi/pt2pt/testlist index f5c86aa..89de3e2 100644 --- a/test/mpi/pt2pt/testlist +++ b/test/mpi/pt2pt/testlist @@ -37,3 +37,4 @@ waitany-null 1 # perhaps disable in the release tarball large_message 3 mprobe 2 mpiversion=3.0 +big_count_status 1 mpiversion=3.0 ----------------------------------------------------------------------- Summary of changes: test/mpi/pt2pt/Makefile.am | 3 +- test/mpi/pt2pt/big_count_status.c | 51 +++++++++++++++++++++++++++++++++++++ test/mpi/pt2pt/testlist | 1 + 3 files changed, 54 insertions(+), 1 deletions(-) create mode 100644 test/mpi/pt2pt/big_count_status.c hooks/post-receive -- MPICH primary repository