Michael Blocksome authored May 08, 2014 and Kenneth Raffenetti committed May 08, 2014

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

A partial revert of the portion of commit
50f3d580 : I did not mean to modify
anything in the struct case.  I did, though, and that modification
caused a bug in darray datatypes.  The zero-length blocklens in the
struct case indicate upper bound and lower bounds and must be respected.

Closes: #2089

No Reviewer

be less verbose, returning only ' No Error\n' if all is well, and return
a non-zero exit code so i may git-bisect with it.

ROMIO was not handling a particular darray pattern well.  Test case
taken from openmpi mailing list.

See ticket #2089

ROMIO has DBG_FPRINTF throughout the code, but those DBG_FPRINTF
statements will not do anything unless ROMIO registers itself with the
MPICH debug event logging system.

still todo: the ROMIO logging is way too verbose and needs to implement
TRACE, DEBUG, and VERBOSE levels.

not every file system lives in a posix-like environment, but many do.
for those file systems, open and delete will return -1 and set errno.
The translation from unix erno to MPI error class was haphazard.  Get
all aplicable file systems using ADIOI_Err_create_code so we have one
place to update error code conversion.

Closes: #2075

Signed-off-by: Wei-keng Liao <wkliao@ece.northwestern.edu>

In preparation for using ADIOI_Err_create_code everywhere,
ADIOI_Err_create_code did not handle some errno values that fs-specific
drivers were handling.

Signed-off-by: Wei-keng Liao <wkliao@ece.northwestern.edu>

ROMIO flattening code presently does not handle resized-of-resized
types.

See #2088

Bill Gropp reminds us not to forget this text from MPI-3
MPI-3, page 84, lines 18-20.

  "Most datatype constructors have replication count or block length
  arguments. Allowed values are non-negative integers. If the value is
  zero, no elements are generated in the type map and there is no effect
  on datatype bounds or extent."

The ROMIO flattening codes was treating blocklen elments of zero as
significant.

the struct case probably has the same bug, but I'm deeply nervous about
touching too much of this old code with a release imminent.

Closes: #2073

Wei-keng Liao authored Apr 30, 2014 and Rob Latham committed May 06, 2014

The problem is when defining a filetype using MPI_Type_indexed and the
first few elements of argument blocklens[] are zeros, a collective write
will miss writing some data.

The test program first fills a file with 9 integers with values all
-999.  It then defines a filetype and writes to the file in parallel
with user buffers with value all 1s. Lastly, the file is read back and
checked for contents.

(it's Wei-keng's test case. I just hooked it into ROMIO's test suite)

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

MPI-IO presumes concurrent parallel access, so make that the default.
Applications in strange situations where an MPI-IO program is
manipulating a file while another program manipulates the file can
disable with a hint, but let's make the common, likely case fast.

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

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

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

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

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

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

This reverts commit 12531694.

Jenkins, for one platform/toolchain, still fails these datatype tests. The
results are still correct and performance is no worse, so we need not revert
the whole batch of changes, but the tests are still expected to fail.

No Reviewer

William Gropp authored Mar 10, 2014 and Rob Latham committed May 02, 2014

The best approach for some of these optimizations is through the new
system, so this commit removes some of the work in progress to allow
this patch for dataloop performance to be integrated with MPICH
until the dataloop code is replaced.

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

William Gropp authored Mar 08, 2014 and Rob Latham committed May 02, 2014

A probably copy and paste error in the description of the block indexed
structure.

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

William Gropp authored Mar 08, 2014 and Rob Latham committed May 02, 2014

Statement was placed before one of the values was defined.

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

William Gropp authored Mar 08, 2014 and Rob Latham committed May 02, 2014

This test was created from another one that used communication - in that
process, the use of status.MPI_SOURCE needed to be changed and wasn't in
the original version of this test.

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

William Gropp authored Feb 19, 2014 and Rob Latham committed May 02, 2014

This adds a step that optimizes the dataloop representation, primarily
merging CONTIG blocks with the parent dataloop, such as a VECTOR loop.
It also performs a change of VECTOR of CONTIG with extent > size to
VECTOR of VECTOR; this reduces the stack operations needed to perform
the move.

This is a temporary fix for the dataloop performance.  See the DAME
wiki page (http://wiki.mpich.org/mpich/index.php/DAME

) for current work
on a replacement, higher performance datatype system.

A partial, but not complete, fix for ticket #1788

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

William Gropp authored Feb 19, 2014 and Rob Latham committed May 02, 2014

These tests were inspired by tests in the MPICH-1 test suite
(structpack2.c) or the Intel test suite (vecblklen.c and hvecblklen.c)
that failed with early versions of the dataloop optimization code.

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

William Gropp authored Feb 19, 2014 and Rob Latham committed May 02, 2014

While debugging dataloop optimizations, I needed a little more information
about the errors from this test.  No change to output when there are
no errors.

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

William Gropp authored Feb 19, 2014 and Rob Latham committed May 02, 2014

Mostly minor changes in copyright message, but also retained correction
to erroneous use of the status returned by a send request, and correctly
freeing only allocated buffers.

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

William Gropp authored Feb 19, 2014 and Rob Latham committed May 02, 2014

While adding dataloop optimizations, a number of places where there
were errors in text or in nomunclature were found and fixed.

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

William Gropp authored Feb 03, 2014 and Rob Latham committed May 02, 2014

With these optimizations, most of the datatype performance tests should
pass.  However, indexperf still fails, since these optimizations to not
address the pattern in indexperf.

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

  MPI_Comm_split hang w/ collective_offload=yes on plinux w/o fca

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

No review since F08 binding is experimental now.

No review since F08 binding is experimental now.

No review since F08 binding is experimental now.

Copies a sampling of tests from the f90 directory and converts them
to use the f08 bindings.

No review since it is experimental.

Adds configure logic and a stub Makefile for Fortran 2008 tests.

No review since F08 binding is experimental now.

No review since F08 binding is experimental now.

This includes a configure test to check if the compiler supports F08.

No review since F08 binding is experimental now.