Fixes #1965

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

instead of assigning and then checking for overflow, check if the
overflow will happen, and then assign.

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

Fixes #1971

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

No reviewer

Changes singe long line function definitions to multiple lines

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

Adds man pages for nonblocking collectives

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

Also added comments to make it less misleading.

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

Fixes #884.

Signed-off-by: Junchao Zhang <jczhang@mcs.anl.gov>

The full path of the test is test/mpi/f90/datatype/get_elem_d.f90

Signed-off-by: Junchao Zhang <jczhang@mcs.anl.gov>

Fixed a set of compiler warnings related to MPI_T code when compiled
with -Wall, including "assignment discards 'const' qualifier from pointer
target type", "unused variable", "shadows a global declaration" etc.

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

Reverted file maint/version.m4 to leave MPICH_RELEASE_DATE with its default
value "unreleased development copy". The proper value is actually generated by
autogen.sh.

Drop payload into a union paired with a double, and payload will end up
double-aligned.  Can't use an anonymous union, though: that's c11.

named-union requires updating any place that used payload.  There aren't too
many, though.

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

Rob Latham authored Nov 01, 2013 and Pavan Balaji committed Nov 03, 2013

MPI_Count in a status object breaks ABI, so we have a scheme to use lo
and hi parts of adjacent struct members.  but it's not foolproof.  Test
case to demonstrate bugs and hopefully catch future regressions.

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

We were checking for (size of datatype < MPIR_COUNT_MAX / count)
earlier.  This is not a good test when the count is very large.  This
patch modifies the assertion to (count * size of datatype <
MPIR_COUNT_MAX) so we don't run into integer division issues.

Signed-off-by: Rob Latham <robl@mcs.anl.gov>

In C, left-shift has well-defined behavior, but right-shift does not
for signed integers.  The implementation is free to pad the shifted
bits with 0's or 1's.  In this patch, we type-cast the values to
unsigned integers to make the behavior well-defined.

Signed-off-by: Rob Latham <robl@mcs.anl.gov>

Antonio J. Pena authored Nov 02, 2013 and Pavan Balaji committed Nov 02, 2013

Fixes ticket #1773

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

Use if (@array) format instead of if (defined(@array)) format, since
the latter is deprecated in perl.

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

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

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

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

To cover the case when multiple communicators are created by
MPI_Comm_idup.

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

Junchao Zhang authored Oct 30, 2013 and Pavan Balaji committed Oct 31, 2013

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

Rob Latham authored Oct 28, 2013 and Pavan Balaji committed Oct 31, 2013

buildbot reports some crazy error messages:
INTERNAL ERROR: invalid error code 3e (Ring ids do not match) in PMPI_T_cvar_handle_alloc:135

valgrind more helpfully points out places where uninitialized varables
were being passed into the MPI_T interface.  Perhaps MPI_T interface
needs to be robust-ified against garbage inputs?

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

Junchao Zhang authored Oct 30, 2013 and Pavan Balaji committed Oct 31, 2013

This is a conservative setting, but should make MPICH "MPI_T-correct".
Once we know more about cvars, we can incrementally relax their scopes.

Fixes #1959

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

The test file was using MPI-I/O functions that the MPICH test suite
allows us to enable/disable.  This patch moves the test to an iodir
format, so it is not run when configured with --disable-romio.

The correct longer term fix for this is to allow multiple conditions
to disable a given test.  For example, we could have configure set the
@enable_io@ macro to either '' or '#', and different tests that rely
on I/O would add them to the start of the line in the test file.  That
way, a test that uses both F77 and I/O would be disabled when support
for either one is not present.

Signed-off-by: Junchao Zhang <jczhang@mcs.anl.gov>

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

MPICH currently blocks in MPI_Test.  This test forces a case where the
program will deadlock if MPI_Test is not nonblocking.

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

Apparently my copy/paste was insufficient and I missed this. Sorry about the
multi-commit.

The guide was a little unclear about how to use local options for mpiexec
executables when running multiple programs (mpiexec -n 1 exec1 : -n 2 exec2).
Now it says:

Usage: mpiexec [global opts] [local opts for exec1] [exec1] [exec1 args] : ...

Fixes #1936

Rob Latham authored Oct 18, 2013 and Pavan Balaji committed Oct 31, 2013

configure determines the right type for MPI_Aint, MPI_Fint, and
MPI_Count, so we have to move the MPIR_Bsend checks later in configure.
Then we can promote a plain int to MPI_Aint.

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

Rob Latham authored Oct 18, 2013 and Pavan Balaji committed Oct 31, 2013

MPI_Pack, MPI_Unpack, and MPI_Pack_size use integers to describe counts
of bytes and posistions in a bytestream.  In a world of 2 GiB+
datatypes, this is not correct.  It will take an army to change the
standard, but we can at least do the right thing internally.

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

Rob Latham authored Oct 28, 2013 and Pavan Balaji committed Oct 31, 2013

MPIR_Type_get_contig_blocks is buggy for certain datatypes (example:
HDF5 multi-dataset IO generated type results in a contig block count of
5 billion.  should be more like 200).  Nothing uses this function except
ROMIO's flattening code.  I'd like to see profiling suggesting the count
of contiguous blocks is a profitable short-circuit before spending a ton
of time debugging it.

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

Rob Latham authored Oct 18, 2013 and Pavan Balaji committed Oct 31, 2013

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

Rob Latham authored Oct 18, 2013 and Pavan Balaji committed Oct 31, 2013

clang carped about too many parens around (( a == b))

Fixes #1929

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

Rob Latham authored Oct 16, 2013 and Pavan Balaji committed Oct 31, 2013

formatting fixes and type promotions to chip away at the 1500 lines of
warnings that pop up with clang's -Wshorten-64-to-32 flag.

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

Jeff Hammond authored Oct 15, 2013 and Pavan Balaji committed Oct 31, 2013

goal: make MPICH count-safe
problem: casting to int when value exceeds INT_MAX
solution: remove all "(int)" in front of values assigned to the count
field

Signed-off-by: Rob Latham <robl@mcs.anl.gov>

Jeff Hammond authored Oct 15, 2013 and Pavan Balaji committed Oct 31, 2013

strlen returns size_t.  since the return argument is localized, there is
no reason to not use the proper type to hold the result.

Signed-off-by: Rob Latham <robl@mcs.anl.gov>