Xin Zhao authored Sep 25, 2013 and Pavan Balaji committed Sep 26, 2013

Do a memory barrier when winow is allocated by MPI_Win_allocate_shared,
if this fence is (1) not call with MPI_MODE_NO_PROCEDE; (2) not the very
first fence; (3) not following a fence with MPI_MODE_NO_SUCCEED.

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

Kenneth Raffenetti authored Sep 24, 2013 and Pavan Balaji committed Sep 25, 2013

When a key is not found locally, pass the "get" command upstream.
This allows us to handle special cases like dead processes.
Fixes #1917.

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

Xin Zhao authored Sep 24, 2013 and Pavan Balaji committed Sep 25, 2013

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

As reported by Michael Raymond @ SGI, we were never checking if the
disp_unit was set correctly.  This patch improves the error checks.

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

1. Cleanup check for local IPs.  We check to see if the host IP
matches that of the list of local IP addresses.  If it does, it's a
local host.

2. Do not try to detect remotely accessible nodes.  Our logic for
identifying whether a node is remotely accessible is quite primitive
and hacky.  It doesn't serve any real purpose either.  This patch
deletes that code.

3. Cleanup the is_local lookup function.  If getifaddrs and inet_ntop
are available, try to detect local processes.  If they aren't there,
simply return non-local.  In this case, the launcher will fall back to
using a remote launch even for local processes.  Bad, but workable.

4. Make the hostname lookups more comprehensive, by matching the local
hostname and the IP address the local hostname resolves to as well.
This works well when the /etc/hosts file contains an IP address match
that is not a part of the network addresses.  In this case, the
hostname IP lookup will give that IP address, but it will not match
any of the local network IP addresses.

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

In verbose mode, mpit_vars.c prints all MPI_T control variables, performance
variables and categories defined in the MPI implementation.

It is adapted from the code provided Bill Gropp.
It needs more refinement when printing hierarchical categories.

Also added mpit_vars to .gitignore.

Junchao Zhang authored Aug 14, 2013 and Pavan Balaji committed Sep 22, 2013

This commit will increase the ABI version. Fix #1904

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

Junchao Zhang authored Aug 16, 2013 and Pavan Balaji committed Sep 21, 2013

Some MPI_T functions return MPI_SUCCESS even when they are not implemented.
Now return MPI_ERR_INTERN instead. Of course it is a temp fix.

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

Junchao Zhang authored Aug 13, 2013 and Pavan Balaji committed Sep 21, 2013

Add missing MPI_T error codes in mpi.h. Will increase the ABI version.

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

No reviewer.

Picked up the relevant parts of [2bda6bc7], [49b74176] and [59cbc846].

No reviewer.

No reviewer.

If data sieving is not supported by the underlying file system (e.g. PVFS,
PLFS, BGLOCKLESS) then there is no reason to turn write-only requests
into read-write requests.  Suggestion first reported by David Knaak of Cray.

Reviewed-by: David Knaak <knaak@cray.com>

Kenneth Raffenetti authored Sep 18, 2013 and Pavan Balaji committed Sep 18, 2013

FreeBSD does not support this attribute. Including this runtime
test ensures nemesis will not override the allocate_shm function
pointer unless it is supported. Fixes #1926

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

- large_type now conforms to mpich test guidelines.
- additional datatype cases that do not rely on "large MPI_Aint" (but
  32 bit platforms probably still cannot process these types)

Fixup the f90tof77 failcheck patch to eliminate a stray blank

Add a test for success in the executions of the f77tof90 script in autogen.sh

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

RobL wears the "bad commit of shame" badge for missing these in the last
commit

An complimentary test to large_count.  Whereas large_count hits an
assertion early on about size being negative, this test progresses
further into the dataloop code... where it asserts about a count of
blocks being negative:

Assertion failed in file ../src/mpid/common/datatype/dataloop/segment_count.c at line 100: *blocks_p > 0

refer to ticket #1893

RobL's large-count/large-type combo branch will address this once it
lands.

Initialized a variable in mpi_t_str.c to a garbage value so that the test will
more easily trigger MPI_T implementation bugs.

Current MPI_T implementation is incorrect, so I also added the test to xfail.

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.