One was discovered on reading the code; the correct test for the enum
case was not made.  The second was discovered by the Blue Waters team when
testing the most recent Cray MPI release.  Its not clear how this passed our
tests.

The MPI_Errhandler callback function is defined as the MPI Standard as having
a valist which is implementation specific. This adds a specification for those
parameters to the user doc and the man page for MPI_Comm_create_errhandler.

Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>

As of MPI-2, MPI_Init will accept NULL as input parameters. This wasn't
mentioned in the man page though this is pertinent information.

Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>

Wesley Bland authored Aug 23, 2013 and Pavan Balaji committed Aug 23, 2013

We don't provide support for Windows anymore. There shouldn't be
instructions for installing and using it in the install guide.

Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>

Wesley Bland authored Aug 21, 2013 and Pavan Balaji committed Aug 23, 2013

We don't provide support for Windows anymore. There shouldn't be instructions
for installing and using it in the user guide.

Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>

The default "aggregators per peset" value of 8 had not been changed
since Blue Gene /L.  Further, allow one to set the environment variable
BGMPIO_NAGG_PSET to have the same effect as setting the "bg_nodes_pset"
hint.  Since this is all very confusing on /Q, toss in a few more (but
no doubt not enough) comments.

Reviewed-by: Bob Cernohous <bobc@us.ibm.com>

Signed-off-by: Wesley Bland <wbland@mcs.anl.gov>

Use _exit instead of kill to more reliably fail processes in
fault tolerance tests. _exit ends a process immediately rather than
relying on kill/sleep combo. Also, tests now output " No Errors"
correctly upon success.

Signed-off-by: Wesley Bland <wbland@mcs.anl.gov>

We were listing some of the configure options in the README.
Unfortunately, these are not consistent with the configure file since
they are not autogenerated.  Fixes #1921.

Signed-off-by: Antonio J. Pena <apenya@mcs.anl.gov>

Signed-off-by: Antonio J. Pena <apenya@mcs.anl.gov>

sizeof() is used inconsistently in pm/hydra/utils/sock/sock.c. In some
places this is size of type, in some - size of variable.

Thanks to Yauheni Zelenko for reporting this error.

Signed-off-by: Wesley Bland <wbland@mcs.anl.gov>

Pavan Balaji authored Aug 14, 2013 and Kenneth Raffenetti committed Aug 14, 2013

We need to make sure that the xfail tickets that are added to the test
list are not left in there permanently.  This script checks whether
these tickets are still open and points out of they are not, but the
xfail is still listed.

Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

Pavan Balaji authored Aug 13, 2013 and Kenneth Raffenetti committed Aug 14, 2013

Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

FreeBSD 9.1-RELEASE has aio routines that appear to work but
when called return ENOSYS.  We can handle that, though, if we
check for that condition.

Yet more error handling routines added to another I/O test that does not check errors.

freebsd did not like this expr syntax for some reason. Since we require
autoconf-2.67 or newer, let's use AS_VAR_ARITH and make portable shell scripts
autoconf's problem

This reverts commit 4614eb44.

The environment description did not mention what the port range 0:0
meant.

Signed-off-by: Wesley Bland <wbland@mcs.anl.gov>

The genparams script was displaying the range format as {low,high},
while the format the environment variables accept is low:high.  This
commit fixes the script to display the default in the correct format.

Signed-off-by: Wesley Bland <wbland@mcs.anl.gov>

We might add some MPIX functions to the interface (for active
messages), but those are not a part of the libmpich ABI.  This value
needs to be verified when we are making the actual release.

Signed-off-by: Wesley Bland <wbland@mcs.anl.gov>

Xin Zhao authored Aug 08, 2013 and Pavan Balaji committed Aug 08, 2013

Initialize "list_complete" before entering MPIDI_CH3I_RMAListPartialComplete
because it is used in that function.

Fixes ticket #1906.

Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>

The error message in MPID_nem_tcp_bind line 580 was incorrectly reporting the
port that was being attempted due to incrementing the variable at the end of
the for loop and not accounting for that in the error message. This patch
subtracts one if the port was not succesfully bound when printing out the
value for the error message.

Thanks to Yauheni Zelenko for reporting this error.

Signed-off-by: Huiwei Lu <huiweilu@mcs.anl.gov>

Michael Blocksome authored Aug 08, 2013 and Pavan Balaji committed Aug 08, 2013

There are three configure options and two install cases that need to be
handled.

The configure could have be done on a bgq system that has V1R2M1+
installed or on a bgq system that has a pre-V1R2M1 installation. In
V1R2M1 the location and names of the pami libraries changed.

The three configure options are:
  --with-bgq-install-dir ..... forces use of a specific bgq install
  --with-pami[-include|-lib] . forces use of a specific pami install
  PAMILIBNAME ................ forces use of a specific pami library name

Signed-off-by: Bob Cernohous <bobc@us.ibm.com>
Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>

Increase the number of RMA operations issued within the epoch.  The
error seems to happen rarely, so increasing the number of RMA
operations increases the probability of it occurring.

Signed-off-by: Xin Zhao <xinzhao3@illinois.edu>

We didn't change ABI from 3.0.x, so the 'c' value should still be 10.

Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>

We were jumping to exit early causing us to try to free an unitialized
string stash list.  We should really use C++ constructor-style
allocation functions for this.  This commit just moves the
initialization early, so we don't try to free garbage when we jump to
exit.

Signed-off-by: Antonio J. Pena <apenya@mcs.anl.gov>

Pavan Balaji authored Aug 06, 2013 and Kenneth Raffenetti committed Aug 07, 2013

We need the environment-passed parameters to determine what
thread-level needs to be used.  Fixes ticket #1892.

Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

Michael Blocksome authored Jul 18, 2013 and Kenneth Raffenetti committed Aug 07, 2013

Do not overload the 'mpid.next' pointer in the request as this field is
defined by the mpid layer and therefore is specific to the device.  Not
all adi implementations have this field.

Signed-off-by: Haizhu Liu <haizhu@us.ibm.com>
Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

Douglas Miller authored Jun 28, 2012 and Kenneth Raffenetti committed Aug 07, 2013

This patch overloads the MPID_Request.mpid.next pointer to use for
referencing the debugger SEND queue structure to facilitate
faster removal during completion. MPID_Request.mpid.next is only
used by receives and free-pools otherwise.

(ibm) Issue 8178
(ibm) e1c4d5cfe88fccc67624f81497a74bc55d774e43

Signed-off-by: Su Huang <suhuang@us.ibm.com>
Signed-off-by: Michael Blocksome <blocksom@us.ibm.com>
Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

Keira Zhang authored Dec 18, 2011 and Kenneth Raffenetti committed Aug 07, 2013

(ibm) bd92b21a52afd8f4f6ef018d764d50060966f20b

Signed-off-by: Michael Blocksome <blocksom@us.ibm.com>
Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

Allows the minimal configure ../configure --host=powerpc64-bgq-linux
--with-device=pamid --enable-thread-cs=per-object.  Without this
change the MPID_REQUEST_PREALLOC symbol would be undefined.

Signed-off-by: Michael Blocksome <blocksom@us.ibm.com>

Michael Blocksome authored Aug 06, 2013 and Pavan Balaji committed Aug 06, 2013

Removed the processing of the (undocumented) environment variable
'PAMID_CORE_ON_ABORT' which was being checked to determine if the user
does *not* want the process to core dump. On Blue Gene/Q the core dump
was accomplished by calling 'abort()' which sends SIGSBRT to all
processes and all processes would then write a core file. This is not
scalable.

Instead, MPID_Abort() will invoke 'exit(1)' which will terminate all
processes in the job. This behavior is identical for both the POE and
the Blue Gene/Q control systems.

On Blue Gene/Q the user may replicate the previous core dump behavior by
using the environment variables 'BG_COREDUMPONERROR=1' or
'BG_COREDUMPONEXIT=1'.

Finally, the 'DYNAMIC_TASKING' #ifdef is moved up so it is checked first.
'MPIDI_NO_ASSERT' and 'DYNAMIC_TASKING' are typically defined for PE. It
appears that the dynamic tasking code was never being invoked.

(ibm) CPS 99YURA

Signed-off-by: Bob Cernohous <bobc@us.ibm.com>

Wesley Bland authored Aug 06, 2013 and Pavan Balaji committed Aug 06, 2013

These tests should not be distributed with the test suite when grabbing a
release. They are still enabled for developer builds.

Signed-off-by: Pavan Balaji <balaji@mcs.anl.gov>