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 9a7b024b4ff2b1b758b6199d8e5f0297e35415cc (commit) from 902bb9e890329482cf084663b25ee9dc45a96403 (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/9a7b024b4ff2b1b758b6199d8e5f0297e3... commit 9a7b024b4ff2b1b758b6199d8e5f0297e35415cc Author: James Dinan <[email protected]> Date: Thu Feb 7 16:48:06 2013 -0600 Fix incorrect timing reporting in llist bench The linked list benchmarks were reportin the sum of the timings across all nodes, rather than the maximum. Reviewer: none diff --git a/test/mpi/rma/linked_list_bench_lock_all.c b/test/mpi/rma/linked_list_bench_lock_all.c index 9e7c5c0..90df021 100644 --- a/test/mpi/rma/linked_list_bench_lock_all.c +++ b/test/mpi/rma/linked_list_bench_lock_all.c @@ -242,7 +242,7 @@ int main(int argc, char **argv) { if (print_perf) { double max_time; - MPI_Reduce(&time, &max_time, 1, MPI_DOUBLE, MPI_SUM, 0, MPI_COMM_WORLD); + MPI_Reduce(&time, &max_time, 1, MPI_DOUBLE, MPI_MAX, 0, MPI_COMM_WORLD); if (procid == 0) { printf("Total time = %0.2f sec, elem/sec = %0.2f, sec/elem = %0.2f usec\n", max_time, NUM_ELEMS/max_time, max_time/NUM_ELEMS*1.0e6); diff --git a/test/mpi/rma/linked_list_bench_lock_excl.c b/test/mpi/rma/linked_list_bench_lock_excl.c index 275360f..7fe5240 100644 --- a/test/mpi/rma/linked_list_bench_lock_excl.c +++ b/test/mpi/rma/linked_list_bench_lock_excl.c @@ -245,7 +245,7 @@ int main(int argc, char **argv) { if (print_perf) { double max_time; - MPI_Reduce(&time, &max_time, 1, MPI_DOUBLE, MPI_SUM, 0, MPI_COMM_WORLD); + MPI_Reduce(&time, &max_time, 1, MPI_DOUBLE, MPI_MAX, 0, MPI_COMM_WORLD); if (procid == 0) { printf("Total time = %0.2f sec, elem/sec = %0.2f, sec/elem = %0.2f usec\n", max_time, NUM_ELEMS/max_time, max_time/NUM_ELEMS*1.0e6); diff --git a/test/mpi/rma/linked_list_bench_lock_shr.c b/test/mpi/rma/linked_list_bench_lock_shr.c index c96cf33..2ddb791 100644 --- a/test/mpi/rma/linked_list_bench_lock_shr.c +++ b/test/mpi/rma/linked_list_bench_lock_shr.c @@ -234,7 +234,7 @@ int main(int argc, char **argv) { if (print_perf) { double max_time; - MPI_Reduce(&time, &max_time, 1, MPI_DOUBLE, MPI_SUM, 0, MPI_COMM_WORLD); + MPI_Reduce(&time, &max_time, 1, MPI_DOUBLE, MPI_MAX, 0, MPI_COMM_WORLD); if (procid == 0) { printf("Total time = %0.2f sec, elem/sec = %0.2f, sec/elem = %0.2f usec\n", max_time, NUM_ELEMS/max_time, max_time/NUM_ELEMS*1.0e6); ----------------------------------------------------------------------- Summary of changes: test/mpi/rma/linked_list_bench_lock_all.c | 2 +- test/mpi/rma/linked_list_bench_lock_excl.c | 2 +- test/mpi/rma/linked_list_bench_lock_shr.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) hooks/post-receive -- MPICH primary repository