[mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.1.1-12-gd505ece

Service Account noreply at mpich.org
Mon Jun 16 15:33:10 CDT 2014


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  d505ece8ea259a6334f35dd4023a762b9d73d0ba (commit)
      from  d016c5c5f396a0496a295310cd2cc8a9ffceca3a (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/d505ece8ea259a6334f35dd4023a762b9d73d0ba

commit d505ece8ea259a6334f35dd4023a762b9d73d0ba
Author: Wesley Bland <wbland at anl.gov>
Date:   Mon Jun 16 14:04:22 2014 -0500

    Add a CVAR to turn off collective alias checking
    
    Add a new CVAR (MPIR_CVAR_COLL_ALIAS_CHECK) that allows the user to turn off
    alias checking in collective operations. In 5b4bdef9, alias checking was
    expanded to be more correct according to the standard, but will cause some
    previously misbehaving applications to fail even when they might not have
    displayed that behavior before. This patch lets users turn off checking if
    they accept the risk and don't want this specific check to be done. All error
    checking can still be turned off at configure time as usual. By default, this
    new CVAR will be set to `1` to indicate that checking is on.
    
    Fixes #2117
    
    Signed-off-by: Antonio J. Pena <apenya at mcs.anl.gov>

diff --git a/src/include/mpierrs.h b/src/include/mpierrs.h
index 082b087..3eeb243 100644
--- a/src/include/mpierrs.h
+++ b/src/include/mpierrs.h
@@ -9,6 +9,23 @@
 /* mpierrs.h */
 /* ------------------------------------------------------------------------- */
 
+/*
+=== BEGIN_MPI_T_CVAR_INFO_BLOCK ===
+
+cvars:
+    - name        : MPIR_CVAR_COLL_ALIAS_CHECK
+      category    : COLLECTIVE
+      type        : int
+      default     : 1
+      class       : device
+      verbosity   : MPI_T_VERBOSITY_USER_BASIC
+      scope       : MPI_T_SCOPE_ALL_EQ
+      description : >-
+        Enable checking of aliasing in collective operations
+
+=== END_MPI_T_CVAR_INFO_BLOCK ===
+*/
+
 /* Error checking (see --enable-error-checking for control of this) */
 #ifdef HAVE_ERROR_CHECKING
 
@@ -113,7 +130,7 @@
 /* FIXME this test is probably too broad.  Comparing the (buffer,type,count)
  * tuples is really what is needed to check for aliasing. */
 #define MPIR_ERRTEST_ALIAS_COLL(ptr1,ptr2,err)			\
-    if ((ptr1)==(ptr2)) {		                        \
+    if (MPIR_CVAR_COLL_ALIAS_CHECK && (ptr1)==(ptr2)) {        \
         err = MPIR_Err_create_code(MPI_SUCCESS,			\
 				   MPIR_ERR_RECOVERABLE,	\
 				   FCNAME, __LINE__,		\

-----------------------------------------------------------------------

Summary of changes:
 src/include/mpierrs.h |   19 ++++++++++++++++++-
 1 files changed, 18 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
MPICH primary repository


More information about the commits mailing list