October 2016 - commits - mailman3.cels.anl.gov

Argonne Spam Quarantine Notification
by Argonne Anti Spam Quarantine 29 Oct '16

29 Oct '16

++ Note: This message has been sent by a notification only system. Please do not reply ++ IronPort Spam Quarantine Notification If a spam message was missed, or if a message was incorrectly flagged as spam, you can add the sender to a personal safelist or blocklist via the quarantine. Senders on the safelist will not be flagged as spam, and senders on the blocklist will be automatically quarantined. For more information, visit: https://wiki.inside.anl.gov/inside/Ironport_Anti-Spam_Appliances/Documentat… The message(s) below have been quarantined by the Argonne National Laboratory spam filtering system. There are 1 new messages in your Email Quarantine since you received your last Spam Quarantine Notification. If the messages below are spam, you do not need to take any action. Messages will be automatically deleted from the quarantine after 28 day(s). If any of the messages below are not spam, click the 'Release' link to have them sent to your Inbox. Messages that have been correctly flagged as spam do not need to be reported. ----------- New Quarantine Messages --------------- Message 1 From: Robert Powers <robertp(a)ocoosbusiness.com> Subject: *****SPAM*****Your wordpress website and security Date: 28 Oct 2016 Release: http://mailgateway.anl.gov:8080/Message?action=Release&mid=134605294&h=3cd0… --------------------------------------------------- To manage your quarantine please visit the URL below: http://mailgateway.anl.gov:8080/Search?h=fc87c670d8c3e2454112880d759ad6c4&e… If you have any questions regarding the Laboratory spam filtering product, please contact the CIS helpdesk at help(a)anl.gov or x-9999 option 2.

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-477-g4fcffbe
by noreply＠mpich.org 14 Oct '16

14 Oct '16

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 4fcffbe695cb360db588b4c2e67a379ead5c9c9d (commit) from 245f4c1fdce2f93d98c1ed137733419648ad6e86 (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/4fcffbe695cb360db588b4c2e67a379ea… commit 4fcffbe695cb360db588b4c2e67a379ead5c9c9d Author: Ken Raffenetti <raffenet(a)mcs.anl.gov> Date: Wed Oct 12 16:17:24 2016 -0500 ROMIO: Fix header include order adio.h must be included first before any system headers so that config settings are taken into account. Fixes an issue with inconsistent types for off_t when using the fallback versions of pwrite/pread. Signed-off-by: Rob Latham <robl(a)mcs.anl.gov> diff --git a/src/mpi/romio/adio/common/ad_read.c b/src/mpi/romio/adio/common/ad_read.c index 19f2908..75b6c48 100644 --- a/src/mpi/romio/adio/common/ad_read.c +++ b/src/mpi/romio/adio/common/ad_read.c @@ -6,9 +6,9 @@ */ +#include "adio.h" #include <unistd.h> -#include "adio.h" #ifdef AGGREGATION_PROFILE #include "mpe.h" #endif diff --git a/src/mpi/romio/adio/common/ad_write.c b/src/mpi/romio/adio/common/ad_write.c index 34a31d9..d1c24ac 100644 --- a/src/mpi/romio/adio/common/ad_write.c +++ b/src/mpi/romio/adio/common/ad_write.c @@ -6,9 +6,9 @@ */ +#include "adio.h" #include <unistd.h> -#include "adio.h" #ifdef AGGREGATION_PROFILE #include "mpe.h" #endif ----------------------------------------------------------------------- Summary of changes: src/mpi/romio/adio/common/ad_read.c | 2 +- src/mpi/romio/adio/common/ad_write.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-476-g245f4c1
by noreply＠mpich.org 11 Oct '16

11 Oct '16

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 245f4c1fdce2f93d98c1ed137733419648ad6e86 (commit) from 41d9cd4e3484bdd66b33e7f667483aeee92b3801 (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/245f4c1fdce2f93d98c1ed13773341964… commit 245f4c1fdce2f93d98c1ed137733419648ad6e86 Author: Rob Latham <robl(a)mcs.anl.gov> Date: Mon Oct 10 14:30:32 2016 -0500 move ASYNCHRONOUS attributes for all tests commit ac4e1f9ab27c was not thorough enough and only fixed one test. Other tests had same declaration: fix all. No Reviewer diff --git a/test/mpi/f08/subarray/test11.f90 b/test/mpi/f08/subarray/test11.f90 index a33fdc4..f484aa2 100644 --- a/test/mpi/f08/subarray/test11.f90 +++ b/test/mpi/f08/subarray/test11.f90 @@ -15,7 +15,7 @@ program main character (len=80) :: title='test 11: Isend/Irecv - 1d array - iar(10)' integer :: sint, i - integer, dimension(10) :: iar + integer, ASYNCHRONOUS, dimension(10) :: iar type(MPI_Request) request type(MPI_Status) status @@ -33,7 +33,6 @@ program main end do block - ASYNCHRONOUS :: iar call mpi_isend(iar, 10, MPI_INTEGER, 1, 567, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then @@ -52,7 +51,6 @@ program main else if (rank .eq. 1) then block - ASYNCHRONOUS :: iar call mpi_irecv(iar, 10, MPI_INTEGER, 0, 567, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then diff --git a/test/mpi/f08/subarray/test12.f90 b/test/mpi/f08/subarray/test12.f90 index 9ad3341..ee576bf 100644 --- a/test/mpi/f08/subarray/test12.f90 +++ b/test/mpi/f08/subarray/test12.f90 @@ -15,7 +15,8 @@ program main character (len=80) :: title='test 12: Isend/Irecv array slice - iar(2:7)' integer :: sint, i - integer, dimension(10) :: iar, iar_check + integer, dimension(10) :: iar_check + integer, ASYNCHRONOUS, dimension(10) :: iar type(MPI_Request) request type(MPI_Status) status @@ -33,7 +34,6 @@ program main end do block - ASYNCHRONOUS :: iar call mpi_isend(iar(2:7), 6, MPI_INTEGER, 1, 678, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then @@ -59,7 +59,6 @@ program main end do block - ASYNCHRONOUS :: iar call mpi_irecv(iar(2:7), 6, MPI_INTEGER, 0, 678, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then diff --git a/test/mpi/f08/subarray/test14.f90 b/test/mpi/f08/subarray/test14.f90 index d11bb10..2c6311b 100644 --- a/test/mpi/f08/subarray/test14.f90 +++ b/test/mpi/f08/subarray/test14.f90 @@ -15,7 +15,8 @@ program main character (len=80) :: title='test 14: Isend/Irecv 2d array column slice - iar_2d(:,2:6:2)' integer :: i, j - integer, dimension(9,9) :: iar_2d, iar_2dch + integer, dimension(9,9) :: iar_2dch + integer, ASYNCHRONOUS, dimension(9,9) :: iar_2d type(MPI_Status) status type(MPI_Request) request @@ -57,7 +58,6 @@ program main if (rank .eq. 0) then block - ASYNCHRONOUS :: iar_2d call mpi_isend(iar_2d(:,2:6:2), 27, MPI_INTEGER, 1, 123, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then @@ -76,7 +76,6 @@ program main else if (rank .eq. 1) then block - ASYNCHRONOUS :: iar_2d call mpi_irecv(iar_2d(:,2:6:2), 27, MPI_INTEGER, 0, 123, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then diff --git a/test/mpi/f08/subarray/test15.f90 b/test/mpi/f08/subarray/test15.f90 index 2e183bb..7c3d3ee 100644 --- a/test/mpi/f08/subarray/test15.f90 +++ b/test/mpi/f08/subarray/test15.f90 @@ -15,7 +15,8 @@ program main character (len=80) :: title='test 15: Isend/Irecv 2d array column slice - iar_2d(1:7:3,2:6:2)' integer :: i, j - integer, dimension(9,9) :: iar_2d, iar_2dch + integer, dimension(9,9) :: iar_2dch + integer, ASYNCHRONOUS, dimension(9,9) :: iar_2d type(MPI_Status) status type(MPI_Request) request @@ -56,7 +57,6 @@ program main if (rank .eq. 0) then block - ASYNCHRONOUS :: iar_2d call mpi_isend(iar_2d(1:7:3,2:6:2), 9, MPI_INTEGER, 1, 123, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then if (verbose) print *,"PE ",rank,": ",name,": mpi_isend exited in error (",ierr,")" @@ -72,7 +72,6 @@ program main end block else if (rank .eq. 1) then block - ASYNCHRONOUS :: iar_2d call mpi_irecv(iar_2d(1:7:3,2:6:2), 9, MPI_INTEGER, 0, 123, MPI_COMM_WORLD, request, ierr); if (ierr .ne. MPI_SUCCESS) then ----------------------------------------------------------------------- Summary of changes: test/mpi/f08/subarray/test11.f90 | 4 +--- test/mpi/f08/subarray/test12.f90 | 5 ++--- test/mpi/f08/subarray/test14.f90 | 5 ++--- test/mpi/f08/subarray/test15.f90 | 5 ++--- 4 files changed, 7 insertions(+), 12 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-475-g41d9cd4
by noreply＠mpich.org 10 Oct '16

10 Oct '16

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 41d9cd4e3484bdd66b33e7f667483aeee92b3801 (commit) via 32cbcba941243b80afcfc69df6efcbbe801e5646 (commit) from ed612b0a744661e974b0e56c3d121a99627543da (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/41d9cd4e3484bdd66b33e7f667483aeee… commit 41d9cd4e3484bdd66b33e7f667483aeee92b3801 Author: Yanfei Guo <yguo(a)anl.gov> Date: Thu Sep 29 17:07:10 2016 -0500 autoconf: fix strict build on Solaris. 1. GCC on solaris use C89 as default which does not support POSIX 2001. The support for POSIX 2001 must be checked after checking the support of C99. 2. AC_CHECK_FUNCS has false positive when checking gethrtime on Solaris with strict configuration. Use AC_CHECK_DECL to double check the availability of gethrtime. No reviewer. diff --git a/confdb/aclocal_cc.m4 b/confdb/aclocal_cc.m4 index fd7152b..23d7365 100644 --- a/confdb/aclocal_cc.m4 +++ b/confdb/aclocal_cc.m4 @@ -626,13 +626,6 @@ if test "$enable_strict_done" != "yes" ; then pac_cc_strict_flags="-O2" fi pac_cc_strict_flags="$pac_cc_strict_flags $pac_common_strict_flags" - case "$enable_posix" in - no) : ;; - 1995) PAC_APPEND_FLAG([-D_POSIX_C_SOURCE=199506L],[pac_cc_strict_flags]) ;; - 2001) PAC_APPEND_FLAG([-D_POSIX_C_SOURCE=200112L],[pac_cc_strict_flags]) ;; - 2008) PAC_APPEND_FLAG([-D_POSIX_C_SOURCE=200809L],[pac_cc_strict_flags]) ;; - *) AC_MSG_ERROR([internal error, unexpected POSIX version: '$enable_posix']) ;; - esac # We only allow one of strict-C99 or strict-C89 to be # enabled. If C99 is enabled, we automatically disable C89. if test "${enable_c99}" = "yes" ; then @@ -645,6 +638,16 @@ if test "$enable_strict_done" != "yes" ; then PAC_APPEND_FLAG([-std=c89],[pac_cc_strict_flags]) PAC_APPEND_FLAG([-Wdeclaration-after-statement],[pac_cc_strict_flags]) fi + # POSIX 2001 should be used with C99. But the default standard for some + # compilers are not C99. We must test the support of POSIX 2001 after + # testing C99. + case "$enable_posix" in + no) : ;; + 1995) PAC_APPEND_FLAG([-D_POSIX_C_SOURCE=199506L],[pac_cc_strict_flags]) ;; + 2001) PAC_APPEND_FLAG([-D_POSIX_C_SOURCE=200112L],[pac_cc_strict_flags]) ;; + 2008) PAC_APPEND_FLAG([-D_POSIX_C_SOURCE=200809L],[pac_cc_strict_flags]) ;; + *) AC_MSG_ERROR([internal error, unexpected POSIX version: '$enable_posix']) ;; + esac fi # See if the above options work with the compiler diff --git a/src/mpl/configure.ac b/src/mpl/configure.ac index cf34469..9438e03 100644 --- a/src/mpl/configure.ac +++ b/src/mpl/configure.ac @@ -273,8 +273,14 @@ pac_cv_hrtime_works=yes,pac_cv_hrtime_works=no)]) # hrtime_t to int64_t works, and even more ambitious # would check whether long or long long was 64 bits (or even # better, the sizeof hrtime_t). + + # AC_CHECK_FUNCS has false positive when checking whether gethrtime is + # available on Solaris with strict configuration. We must use + # AC_CHECK_DECL to confirm it. + AC_CHECK_DECL(gethrtime) fi if test "$ac_cv_func_gethrtime" = "yes" -a \ + "$ac_cv_has_decl_gethrtime" = "yes" -a \ "$pac_cv_hrtime_works" = "yes" ; then timer_type=gethrtime elif test "$ac_cv_func_clock_gettime" = "yes" -a \ http://git.mpich.org/mpich.git/commitdiff/32cbcba941243b80afcfc69df6efcbbe8… commit 32cbcba941243b80afcfc69df6efcbbe801e5646 Author: Rob Latham <robl(a)mcs.anl.gov> Date: Tue Sep 27 14:15:32 2016 -0500 A more standard check for large file support Remove the explicit setting of _LARGEFILE64_SOURCE and friends. Let autoconf's AC_SYS_LARGEFILE do it for us. (solaris strict headers do not like the LARGEFILE defines) Signed-off-by: Yanfei Guo <yguo(a)anl.gov> diff --git a/src/mpi/romio/configure.ac b/src/mpi/romio/configure.ac index eaec4f2..e9539b2 100644 --- a/src/mpi/romio/configure.ac +++ b/src/mpi/romio/configure.ac @@ -1306,35 +1306,11 @@ AC_CHECK_TYPE([blksize_t],[],[AC_DEFINE_UNQUOTED([blksize_t],[__blksize_t],[Prov #endif]] ) # -# Check for large file support. Make sure that we can use the off64_t -# type (in some cases, it is an array, and the ROMIO code isn't prepared for -# that). +# in 2004, solairs defined off_t as a union. Today (2016) that is not the case +# and we can simplify this check # -AC_CHECK_HEADERS(unistd.h) -if test "$ac_cv_header_unistd_h" = "yes" ; then - AC_CACHE_CHECK([for large file defines],ac_cv_has_large_file_defines,[ - AC_TRY_COMPILE([ -#include <unistd.h> - ], [ -#ifndef _LFS64_LARGEFILE -#error no largefile defines -#endif - ],ac_cv_has_large_file_defines=yes,ac_cv_has_large_file_defines=no)]) - if test "$ac_cv_has_large_file_defines" = "yes" ; then - # See if we can use them - AC_CACHE_CHECK([whether off64_t is an scalar type],ac_cv_off64_t_scalar,[ - AC_TRY_COMPILE([ -#define _LARGEFILE64_SOURCE -#define _FILE_OFFSET_BITS 64 -#include <sys/types.h> -#include <unistd.h> -],[off64_t a = 3;],ac_cv_off64_t_scalar=yes,ac_cv_off64_t_scalar=no)]) - if test "$ac_cv_off64_t_scalar" = "yes" ; then - CFLAGS="${CFLAGS} -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64" - fi - fi -fi +AC_SYS_LARGEFILE() # pread and pwrite are useful to ROMIO: if implemented well, they help us avoid # an extra system call. But --enable-strict or CFLAGS="--std=c99" does not @@ -1631,20 +1607,6 @@ else F77MPIOINC="include 'mpiof.h'" fi -AC_CHECK_HEADERS(unistd.h, - AC_MSG_CHECKING([for large file defines]) - AC_TRY_COMPILE([ - #include <unistd.h> - ], [ - #ifndef _LFS64_LARGEFILE - #error no largefile defines - #endif - ], - CFLAGS="${CFLAGS} -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64" - AC_MSG_RESULT(yes), - AC_MSG_RESULT(none) ) - ) - echo "setting SYSDEP_INC to $SYSDEP_INC" AC_SUBST(SYSDEP_INC) diff --git a/test/mpi/io/simple_collective.c b/test/mpi/io/simple_collective.c index 54e32fa..5bf5512 100644 --- a/test/mpi/io/simple_collective.c +++ b/test/mpi/io/simple_collective.c @@ -19,8 +19,6 @@ * I am surprised src/mpi/romio/test/create_excl.c did not uncover the bug */ -#define _LARGEFILE64_SOURCE - #include <assert.h> #include <stdio.h> #include <stdlib.h> ----------------------------------------------------------------------- Summary of changes: confdb/aclocal_cc.m4 | 17 ++++++++------ src/mpi/romio/configure.ac | 44 ++------------------------------------ src/mpl/configure.ac | 6 +++++ test/mpi/io/simple_collective.c | 2 - 4 files changed, 19 insertions(+), 50 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-473-ged612b0
by noreply＠mpich.org 06 Oct '16

06 Oct '16

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 ed612b0a744661e974b0e56c3d121a99627543da (commit) from 701056db57aea45096d4468291abe36df45d5f87 (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/ed612b0a744661e974b0e56c3d121a996… commit ed612b0a744661e974b0e56c3d121a99627543da Author: Wesley Bland <wesley.bland(a)intel.com> Date: Thu Oct 6 09:09:54 2016 -0500 CH4/OFI: Avoid messing up compile/link flags The link flags for libfabric were getting messed up if the user passed in embedded as the value because they were being added to LDFLAGS too early. Avoid this by only doing it when we determine that we are not embedding. Also avoid nuking too much of the user inputed flags so we can keep things like visibility preserved. diff --git a/src/mpid/ch4/netmod/ofi/subconfigure.m4 b/src/mpid/ch4/netmod/ofi/subconfigure.m4 index 9579175..dc3522a 100644 --- a/src/mpid/ch4/netmod/ofi/subconfigure.m4 +++ b/src/mpid/ch4/netmod/ofi/subconfigure.m4 @@ -118,8 +118,6 @@ AM_COND_IF([BUILD_CH4_NETMOD_OFI],[ ofilib="" AC_SUBST([ofilib]) - PAC_SET_HEADER_LIB_PATH(libfabric) - ofi_embedded="" dnl Use embedded libfabric if we specify to do so or we didn't specify and the source is present if test "${with_libfabric}" = "embedded" ; then @@ -129,9 +127,11 @@ AM_COND_IF([BUILD_CH4_NETMOD_OFI],[ ofi_embedded="yes" else ofi_embedded="no" + PAC_SET_HEADER_LIB_PATH(libfabric) fi else ofi_embedded="no" + PAC_SET_HEADER_LIB_PATH(libfabric) fi if test "${ofi_embedded}" = "yes" ; then @@ -140,13 +140,7 @@ AM_COND_IF([BUILD_CH4_NETMOD_OFI],[ dnl Unset all of these env vars so they don't pollute the libfabric configuration PAC_PUSH_ALL_FLAGS() - unset CFLAGS - unset CPPFLAGS - unset CXXFLAGS - unset FFLAGS - unset FCFLAGS - unset LDFLAGS - unset LIBS + PAC_RESET_ALL_FLAGS() PAC_CONFIG_SUBDIR_ARGS([src/mpid/ch4/netmod/ofi/libfabric],[$ofi_subdir_args],[],[AC_MSG_ERROR(libfabric configure failed)]) PAC_POP_ALL_FLAGS() PAC_APPEND_FLAG([-I${master_top_builddir}/src/mpid/ch4/netmod/ofi/libfabric/include], [CPPFLAGS]) ----------------------------------------------------------------------- Summary of changes: src/mpid/ch4/netmod/ofi/subconfigure.m4 | 12 +++--------- 1 files changed, 3 insertions(+), 9 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-472-g701056d
by noreply＠mpich.org 05 Oct '16

05 Oct '16

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 701056db57aea45096d4468291abe36df45d5f87 (commit) from 8de843d4587ec9a8da1be4a6320f4a66d0f25b91 (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/701056db57aea45096d4468291abe36df… commit 701056db57aea45096d4468291abe36df45d5f87 Author: Wesley Bland <wesley.bland(a)intel.com> Date: Wed Oct 5 10:30:21 2016 -0500 CH4/OFI: Fix embedded build Fixes embedded build by completely unsetting the environment variables set by the build system. Signed-off-by: Ken Raffenetti <raffenet(a)mcs.anl.gov> diff --git a/src/mpid/ch4/netmod/ofi/subconfigure.m4 b/src/mpid/ch4/netmod/ofi/subconfigure.m4 index 9eb0ed6..9579175 100644 --- a/src/mpid/ch4/netmod/ofi/subconfigure.m4 +++ b/src/mpid/ch4/netmod/ofi/subconfigure.m4 @@ -139,9 +139,16 @@ AM_COND_IF([BUILD_CH4_NETMOD_OFI],[ ofi_subdir_args="--enable-embedded" dnl Unset all of these env vars so they don't pollute the libfabric configuration - PAC_PUSH_FLAG(CPPFLAGS) + PAC_PUSH_ALL_FLAGS() + unset CFLAGS + unset CPPFLAGS + unset CXXFLAGS + unset FFLAGS + unset FCFLAGS + unset LDFLAGS + unset LIBS PAC_CONFIG_SUBDIR_ARGS([src/mpid/ch4/netmod/ofi/libfabric],[$ofi_subdir_args],[],[AC_MSG_ERROR(libfabric configure failed)]) - PAC_POP_FLAG(CPPFLAGS) + PAC_POP_ALL_FLAGS() PAC_APPEND_FLAG([-I${master_top_builddir}/src/mpid/ch4/netmod/ofi/libfabric/include], [CPPFLAGS]) PAC_APPEND_FLAG([-I${use_top_srcdir}/src/mpid/ch4/netmod/ofi/libfabric/include], [CPPFLAGS]) ----------------------------------------------------------------------- Summary of changes: src/mpid/ch4/netmod/ofi/subconfigure.m4 | 11 +++++++++-- 1 files changed, 9 insertions(+), 2 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-471-g8de843d
by noreply＠mpich.org 05 Oct '16

05 Oct '16

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 8de843d4587ec9a8da1be4a6320f4a66d0f25b91 (commit) from 887e0d30f8fe8e939a509aceed02324fef7eca38 (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/8de843d4587ec9a8da1be4a6320f4a66d… commit 8de843d4587ec9a8da1be4a6320f4a66d0f25b91 Author: Jeff Hammond <jeff.science(a)gmail.com> Date: Wed Oct 5 05:39:55 2016 -0700 test/mpi: Use correct type for MPI_Datatype Reported-by: Chris Ward <TJCW(a)uk.ibm.com> Signed-off-by: Ken Raffenetti <raffenet(a)mcs.anl.gov> diff --git a/test/mpi/coll/nonblocking.c b/test/mpi/coll/nonblocking.c index 057480b..9d8dffb 100644 --- a/test/mpi/coll/nonblocking.c +++ b/test/mpi/coll/nonblocking.c @@ -34,7 +34,7 @@ int main(int argc, char **argv) int *rcounts = NULL; int *sdispls = NULL; int *rdispls = NULL; - int *types = NULL; + MPI_Datatype *types = NULL; MPI_Comm comm; MPI_Request req; @@ -62,7 +62,7 @@ int main(int argc, char **argv) my_assert(sdispls); rdispls = malloc(size * sizeof(int)); my_assert(rdispls); - types = malloc(size * sizeof(int)); + types = malloc(size * sizeof(MPI_Datatype)); my_assert(types); for (i = 0; i < size; ++i) { diff --git a/test/mpi/errors/coll/noalias2.c b/test/mpi/errors/coll/noalias2.c index 5b99ce6..b66e516 100644 --- a/test/mpi/errors/coll/noalias2.c +++ b/test/mpi/errors/coll/noalias2.c @@ -35,7 +35,7 @@ int main(int argc, char **argv) int *rcounts = NULL; int *sdispls = NULL; int *rdispls = NULL; - int *types = NULL; + MPI_Datatype *types = NULL; MPI_Comm comm; /* intentionally not using MTest_Init/MTest_Finalize in order to make it @@ -64,7 +64,7 @@ int main(int argc, char **argv) my_assert(sdispls); rdispls = malloc(size * sizeof(int)); my_assert(rdispls); - types = malloc(size * sizeof(int)); + types = malloc(size * sizeof(MPI_Datatype)); my_assert(types); for (i = 0; i < size; ++i) { diff --git a/test/mpi/errors/coll/noalias3.c b/test/mpi/errors/coll/noalias3.c index 6b8cf69..c95da3a 100644 --- a/test/mpi/errors/coll/noalias3.c +++ b/test/mpi/errors/coll/noalias3.c @@ -35,7 +35,7 @@ int main(int argc, char **argv) int *rcounts = NULL; int *sdispls = NULL; int *rdispls = NULL; - int *types = NULL; + MPI_Datatype *types = NULL; MPI_Comm comm; MPI_Request req; @@ -65,7 +65,7 @@ int main(int argc, char **argv) my_assert(sdispls); rdispls = malloc(size * sizeof(int)); my_assert(rdispls); - types = malloc(size * sizeof(int)); + types = malloc(size * sizeof(MPI_Datatype)); my_assert(types); for (i = 0; i < size; ++i) { ----------------------------------------------------------------------- Summary of changes: test/mpi/coll/nonblocking.c | 4 ++-- test/mpi/errors/coll/noalias2.c | 4 ++-- test/mpi/errors/coll/noalias3.c | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-470-g887e0d3
by noreply＠mpich.org 03 Oct '16

03 Oct '16

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 887e0d30f8fe8e939a509aceed02324fef7eca38 (commit) from 72c266f9eca8697e60384897b339f54b334be3f9 (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/887e0d30f8fe8e939a509aceed02324fe… commit 887e0d30f8fe8e939a509aceed02324fef7eca38 Author: Wesley Bland <wesley.bland(a)intel.com> Date: Thu Aug 4 10:28:04 2016 -0500 CH4/OFI: Enable forced embedding If the user wants to force using an embedded OFI library, we shouldn't check for the default library locations because "embedded/lib" won't be a valid location. This is triggered by using the configure flag --with-libfabric=embedded. Signed-off-by: Ken Raffenetti <raffenet(a)mcs.anl.gov> diff --git a/autogen.sh b/autogen.sh index c5dd677..3215a68 100755 --- a/autogen.sh +++ b/autogen.sh @@ -164,6 +164,10 @@ MAKE=${MAKE-make} # external packages that require autogen.sh to be run for each of them externals="src/pm/hydra src/mpi/romio src/openpa" +if [ -e src/mpid/ch4/netmod/ofi/libfabric ]; then + externals="${externals} src/mpid/ch4/netmod/ofi/libfabric" +fi + # amdirs are the directories that make use of autoreconf amdirs=". src/mpl src/util/logging/rlog" diff --git a/src/mpid/ch4/netmod/ofi/subconfigure.m4 b/src/mpid/ch4/netmod/ofi/subconfigure.m4 index 0c07763..9eb0ed6 100644 --- a/src/mpid/ch4/netmod/ofi/subconfigure.m4 +++ b/src/mpid/ch4/netmod/ofi/subconfigure.m4 @@ -119,23 +119,53 @@ AM_COND_IF([BUILD_CH4_NETMOD_OFI],[ AC_SUBST([ofilib]) PAC_SET_HEADER_LIB_PATH(libfabric) - PAC_PUSH_FLAG(LIBS) - PAC_CHECK_HEADER_LIB([rdma/fabric.h], [fabric], [fi_getinfo], [have_libfabric=yes], [have_libfabric=no]) - PAC_POP_FLAG(LIBS) - if test "${have_libfabric}" = "yes" ; then - AC_MSG_NOTICE([CH4 OFI Netmod: Using an external libfabric]) - PAC_APPEND_FLAG([-lfabric],[WRAPPER_LIBS]) - elif test ! -z "${with_libfabric}" ; then - AC_MSG_ERROR([Provided libfabric installation (--with-libfabric=${with_libfabric}) could not be configured.]) + + ofi_embedded="" + dnl Use embedded libfabric if we specify to do so or we didn't specify and the source is present + if test "${with_libfabric}" = "embedded" ; then + ofi_embedded="yes" + elif test -z ${with_libfabric} ; then + if test -f ${use_top_srcdir}/src/mpid/ch4/netmod/ofi/libfabric/configure ; then + ofi_embedded="yes" + else + ofi_embedded="no" + fi else - # fallback to embedded libfabric + ofi_embedded="no" + fi + + if test "${ofi_embedded}" = "yes" ; then AC_MSG_NOTICE([CH4 OFI Netmod: Using an embedded libfabric]) - PAC_CONFIG_SUBDIR_ARGS([src/mpid/ch4/netmod/ofi/libfabric],[],[],[AC_MSG_ERROR(libfabric configure failed)]) + ofi_subdir_args="--enable-embedded" + + dnl Unset all of these env vars so they don't pollute the libfabric configuration + PAC_PUSH_FLAG(CPPFLAGS) + PAC_CONFIG_SUBDIR_ARGS([src/mpid/ch4/netmod/ofi/libfabric],[$ofi_subdir_args],[],[AC_MSG_ERROR(libfabric configure failed)]) + PAC_POP_FLAG(CPPFLAGS) PAC_APPEND_FLAG([-I${master_top_builddir}/src/mpid/ch4/netmod/ofi/libfabric/include], [CPPFLAGS]) PAC_APPEND_FLAG([-I${use_top_srcdir}/src/mpid/ch4/netmod/ofi/libfabric/include], [CPPFLAGS]) + if test "$do_direct_provider" = "true" ; then + if test "x${enable_direct}" != "x" && test "${enable_direct}" != "no" ; then + PAC_APPEND_FLAG([-I${master_top_builddir}/src/mpid/ch4/netmod/ofi/libfabric/prov/${enable_direct}/include], [CPPFLAGS]) + PAC_APPEND_FLAG([-I${use_top_srcdir}/src/mpid/ch4/netmod/ofi/libfabric/prov/${enable_direct}/include], [CPPFLAGS]) + else + AC_MSG_ERROR([Enabled direct OFI provider but didn't specify which one]) + fi + fi + ofisrcdir="${master_top_builddir}/src/mpid/ch4/netmod/ofi/libfabric" ofilib="src/mpid/ch4/netmod/ofi/libfabric/src/libfabric.la" + else + PAC_PUSH_FLAG(LIBS) + PAC_CHECK_HEADER_LIB([rdma/fabric.h], [fabric], [fi_getinfo], [have_libfabric=yes], [have_libfabric=no]) + PAC_POP_FLAG(LIBS) + if test "${have_libfabric}" = "yes" ; then + AC_MSG_NOTICE([CH4 OFI Netmod: Using an external libfabric]) + PAC_APPEND_FLAG([-lfabric],[WRAPPER_LIBS]) + else + AC_MSG_ERROR([Provided libfabric installation (--with-libfabric=${with_libfabric}) could not be configured.]) + fi fi ])dnl end AM_COND_IF(BUILD_CH4_NETMOD_OFI,...) ----------------------------------------------------------------------- Summary of changes: autogen.sh | 4 ++ src/mpid/ch4/netmod/ofi/subconfigure.m4 | 50 ++++++++++++++++++++++++------ 2 files changed, 44 insertions(+), 10 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-469-g72c266f
by noreply＠mpich.org 03 Oct '16

03 Oct '16

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 72c266f9eca8697e60384897b339f54b334be3f9 (commit) from 6ab817f8ef69c02509cf423ae53121ea64c55685 (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/72c266f9eca8697e60384897b339f54b3… commit 72c266f9eca8697e60384897b339f54b334be3f9 Author: Hajime Fujita <hajime.fujita(a)intel.com> Date: Tue Sep 20 14:28:15 2016 -0500 test/mpi: Fix error messages in uoplong Fix minor typo and use a variable to store the expected test output rather than recalculate on error. Signed-off-by: Ken Raffenetti <raffenet(a)mcs.anl.gov> diff --git a/test/mpi/coll/uoplong.c b/test/mpi/coll/uoplong.c index 280a8a7..ab8a43f 100644 --- a/test/mpi/coll/uoplong.c +++ b/test/mpi/coll/uoplong.c @@ -45,7 +45,7 @@ int main(int argc, char *argv[]) int wsize, wrank, i, count; MPI_Datatype tripleType; double *inVal, *outVal; - double maxval, sumval; + double maxval, sumval, minval; MPI_Op op; MTest_Init(&argc, &argv); @@ -62,7 +62,7 @@ int main(int argc, char *argv[]) inVal = (double *) malloc(3 * count * sizeof(double)); outVal = (double *) malloc(3 * count * sizeof(double)); if (!inVal || !outVal) { - fprintf(stderr, "Unable to allocated %d words for data\n", 3 * count); + fprintf(stderr, "Unable to allocate %d words for data\n", 3 * count); MPI_Abort(MPI_COMM_WORLD, 1); } for (i = 0; i < count * 3; i++) { @@ -75,6 +75,7 @@ int main(int argc, char *argv[]) for (i = 0; i < 3 * count; i += 3) { sumval = wsize * (1 + (i & 0x3)); maxval = 1 + ((i + 1) & 0x3); + minval = 1 + ((i + 2) & 0x3); if (outVal[i] != sumval) { if (errs < MAX_ERRS) fprintf(stderr, "%d: outval[%d] = %f, expected %f (sum)\n", @@ -87,10 +88,10 @@ int main(int argc, char *argv[]) count, i + 1, outVal[i + 1], maxval); errs++; } - if (outVal[i + 2] != 1 + ((i + 2) & 0x3)) { + if (outVal[i + 2] != minval) { if (errs < MAX_ERRS) fprintf(stderr, "%d: outval[%d] = %f, expected %f (min)\n", - count, i + 2, outVal[i + 2], (double) (1 + ((i + 2) ^ 0x3))); + count, i + 2, outVal[i + 2], minval); errs++; } } ----------------------------------------------------------------------- Summary of changes: test/mpi/coll/uoplong.c | 9 +++++---- 1 files changed, 5 insertions(+), 4 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.2-468-g6ab817f
by noreply＠mpich.org 03 Oct '16

03 Oct '16

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 6ab817f8ef69c02509cf423ae53121ea64c55685 (commit) via da684aa7e335c4b4d084ff3a1cc5a0cf4d5d6705 (commit) from ac4e1f9ab27cb31b61fd66fb9d5138e7d08f864b (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/6ab817f8ef69c02509cf423ae53121ea6… commit 6ab817f8ef69c02509cf423ae53121ea64c55685 Author: Hajime Fujita <hajime.fujita(a)intel.com> Date: Thu Sep 22 10:12:51 2016 -0500 CH4/OFI: RMA: Remove redundant casts Signed-off-by: Ken Raffenetti <raffenet(a)mcs.anl.gov> diff --git a/src/mpid/ch4/netmod/ofi/ofi_rma.h b/src/mpid/ch4/netmod/ofi/ofi_rma.h index 05e462b..0ba5ffb 100644 --- a/src/mpid/ch4/netmod/ofi/ofi_rma.h +++ b/src/mpid/ch4/netmod/ofi/ofi_rma.h @@ -343,7 +343,7 @@ static inline int MPIDI_OFI_do_put(const void *origin_addr, MPIDI_OFI_WIN(win).syncQ = req; MPIDI_OFI_init_iovec_state(&req->noncontig->iovs, (uintptr_t) origin_addr, - (uintptr_t) MPIDI_OFI_winfo_base(win, req->target_rank) + offset, + MPIDI_OFI_winfo_base(win, req->target_rank) + offset, req->noncontig->origin_dt.num_contig, req->noncontig->target_dt.num_contig, INT64_MAX, @@ -489,7 +489,7 @@ static inline int MPIDI_OFI_do_get(void *origin_addr, MPIDI_OFI_WIN(win).syncQ = req; MPIDI_OFI_init_iovec_state(&req->noncontig->iovs, (uintptr_t) origin_addr, - (uintptr_t) MPIDI_OFI_winfo_base(win, req->target_rank) + offset, + MPIDI_OFI_winfo_base(win, req->target_rank) + offset, req->noncontig->origin_dt.num_contig, req->noncontig->target_dt.num_contig, INT64_MAX, @@ -846,7 +846,7 @@ static inline int MPIDI_OFI_do_accumulate(const void *origin_addr, MPIDI_OFI_init_iovec_state(&req->noncontig->iovs, (uintptr_t) origin_addr, - (uintptr_t) MPIDI_OFI_winfo_base(win, req->target_rank) + offset, + MPIDI_OFI_winfo_base(win, req->target_rank) + offset, req->noncontig->origin_dt.num_contig, req->noncontig->target_dt.num_contig, max_size, @@ -1026,8 +1026,7 @@ static inline int MPIDI_OFI_do_get_accumulate(const void *origin_addr, MPIDI_OFI_init_iovec_state2(&req->noncontig->iovs, (uintptr_t) origin_addr, (uintptr_t) result_addr, - (uintptr_t) MPIDI_OFI_winfo_base(win, - req->target_rank) + offset, + MPIDI_OFI_winfo_base(win, req->target_rank) + offset, req->noncontig->origin_dt.num_contig, req->noncontig->result_dt.num_contig, req->noncontig->target_dt.num_contig, max_size, @@ -1036,7 +1035,7 @@ static inline int MPIDI_OFI_do_get_accumulate(const void *origin_addr, else MPIDI_OFI_init_iovec_state(&req->noncontig->iovs, (uintptr_t) result_addr, - (uintptr_t) MPIDI_OFI_winfo_base(win, req->target_rank) + offset, + MPIDI_OFI_winfo_base(win, req->target_rank) + offset, req->noncontig->result_dt.num_contig, req->noncontig->target_dt.num_contig, max_size, http://git.mpich.org/mpich.git/commitdiff/da684aa7e335c4b4d084ff3a1cc5a0cf4… commit da684aa7e335c4b4d084ff3a1cc5a0cf4d5d6705 Author: Hajime Fujita <hajime.fujita(a)intel.com> Date: Tue Sep 20 14:22:59 2016 -0500 CH4/OFI: RMA: Fix casts to supress warnings This patch addresses warning messages regarding casts in the OFI RMA code, which appear when --enable-strict is specified. Signed-off-by: Ken Raffenetti <raffenet(a)mcs.anl.gov> diff --git a/src/mpid/ch4/netmod/ofi/ofi_rma.h b/src/mpid/ch4/netmod/ofi/ofi_rma.h index 7d1b87a..05e462b 100644 --- a/src/mpid/ch4/netmod/ofi/ofi_rma.h +++ b/src/mpid/ch4/netmod/ofi/ofi_rma.h @@ -426,8 +426,7 @@ static inline int MPIDI_NM_mpi_put(const void *origin_addr, (char *) origin_addr + origin_true_lb, target_bytes, MPIDI_OFI_comm_to_phys(win->comm_ptr, target_rank, MPIDI_OFI_API_CTR), - (uint64_t) (char *) MPIDI_OFI_winfo_base(win, - target_rank) + (uint64_t) MPIDI_OFI_winfo_base(win, target_rank) + target_disp * MPIDI_OFI_winfo_disp_unit(win, target_rank) + target_true_lb, MPIDI_OFI_winfo_mr_key(win, @@ -586,7 +585,7 @@ static inline int MPIDI_NM_mpi_get(void *origin_addr, iov.iov_base = (char *) origin_addr + origin_dt.true_lb; iov.iov_len = target_dt.size; riov.addr = - (uint64_t) ((char *) MPIDI_OFI_winfo_base(win, target_rank) + offset + + (uint64_t) (MPIDI_OFI_winfo_base(win, target_rank) + offset + target_dt.true_lb); riov.len = target_dt.size; riov.key = MPIDI_OFI_winfo_mr_key(win, target_rank); @@ -700,7 +699,7 @@ static inline int MPIDI_NM_mpi_compare_and_swap(const void *origin_addr, buffer = (char *) origin_addr + origin_dt.true_lb; rbuffer = (char *) result_addr + result_dt.true_lb; - tbuffer = (char *) MPIDI_OFI_winfo_base(win, target_rank) + offset; + tbuffer = (void *) (MPIDI_OFI_winfo_base(win, target_rank) + offset); MPIDI_CH4U_EPOCH_START_CHECK(win, mpi_errno, goto fn_fail); ----------------------------------------------------------------------- Summary of changes: src/mpid/ch4/netmod/ofi/ofi_rma.h | 18 ++++++++---------- 1 files changed, 8 insertions(+), 10 deletions(-) hooks/post-receive -- MPICH primary repository

1 0