1. Make the manpage entries consistent with the function prototype.

2. Consistently use "Parameters" instead of the inconsistent
"Parameter" and "Parameters" usage.

3. Correct the manpages for MPI_Type_get_contents and
MPI_Type_get_envelope.

Reviewed by dinan.

for MPI_Type_create_f90_{integer,real,complex}.

Reviewed by dinan.

Made a cleanup pass on mpi.h.in and the function definitions to make
mpi.h.in consistent with the MPI standard.

1. The usage of "*" vs. "[]" was not consistent with the MPI standard
(aside from the consistency changes added in tickets 125 and 126 in
MPI-3).  This commit makes this notation consistent.

2. Consistently provide function parameter names for all MPI
functions.  Some functions had parameter names, while others did not.

3. Consistently use line-breaks for all function prototypes.  Some
prototypes were on a single line, while others were split across
lines.

4. Reorder prototypes to better place them in sections (e.g., all
collectives go to the same section).  Move all PMPI definitions to the
PMPI section.  Move all MPIX_ extension functions to the same section.

The following prototypes are different from that given in the
standard; we use "indx" instead of "index" or "binding" instead of
"bind", to avoid conflicts with preused variable names: MPI_Waitany,
MPI_Testany, MPI_Graph_create, MPI_Graph_get, MPI_Graph_map,
MPI_T_enum_get_item, MPI_T_cvar_get_info, MPI_T_pvar_get_info.

Reviewed by dinan.

Reviewed by dinan.

notation to match the MPI-3 standard: io.

Reviewed by dinan.

notation to match the MPI-3 standard: mpi_t.

Reviewed by dinan.

notation to match the MPI-3 standard: topo.

Reviewed by dinan.

notation to match the MPI-3 standard: pt2pt.

Reviewed by dinan.

notation to match the MPI-3 standard: groups.

Reviewed by dinan.

notation to match the MPI-3 standard: error handlers.

Reviewed by dinan.

notation to match the MPI-3 standard: datatypes.

Reviewed by dinan.

notation to match the MPI-3 standard: collectives.

Reviewed by dinan.

notation to match the MPI-3 standard: attributes.

Reviewed by dinan.

This patch adds missing attributes to MPI-3 RMA functions.  Patch provided by
Dmitri Gribenko.

Reviewer: dinan

Updated RMA code to remove trailing "_e" and "_s" on enum and struct type
names to match the MPICH style.

Reviewer: goodell

Reviewer: balaji

Code review cleanups that couldn't be merged into earlier patches because of
later changes.

Reviewer: goodell

Temporarily reduced the communication volume in manyrma2.  For right now, it's
valuable to get correctness testing from this test.  We are still tracking the
performance issue under #1669 and will tackle that for the next release.

Reviewer: goodell

Reviewer: goodell

Updated get_accumulate implementation to properly handle NO_OP operations;
currently, we dispatch these as get operations.  As a part of this change, the
op issue code was also gathered into a macro to reduce copy-paste across all
of the synchronization functions.

Reviewer: goodell

Removed scattered initializations of the RMA op dataloop field that are
redundant, since the dataloop field is now initialized when the op is
allocated.

Reviewer: goodell

Fetch-and-op has been updated to ignore the origin data and perform no
operation when the given op is MPI_NO_OP.

Reviewer: goodell

Updated op error checking routines to check for correct usage of MPI_Op
argument in accumulate and reduce operations.  Also updated a handful of other
error checks in accumulate ops, and corrected some to ensure that origin
arguments are ignored in the GACC/FOP NO_OP case.

Reviewer: goodell

Reviewer: balaji

Reviewer: buntinas

Reviewer: buntinas

Reviewer: goodell

Removed calls to higher-level MPIU_RMA_CALL() macro.  This macro dispatches
either from the RMAFns table or directly to the MPID_ function; since CH3
always uses the RMAFns table, this macro shouldn't be used inside of CH3.

Reviewer: balaji

If flush is used as a part of polling for incoming data, we can deadlock, since
local RMA calls never poke the progress engine.  Extra progress was added to
local flushes to better fake passive progress.

Reviewer: balaji

Reviewer: buntinas

Updated RMA implementation to track the passive target status individually, for
each target.  Includes new implementation for lock/unlock_all.  Lock_all is
currently unoptimized, see #1734 for future plans.

Reviewer: buntinas

Remove ugly rank 0 hack and updated the active target code to maintain and
process a proper RMA ops list.

Reviewer: goodell

The use of a dense array is a temporary measure to support the reference
implementation.  This will be much improved by ticket #1735.

Reviewer: goodell

A couple of the RMA synchronization error checks were returning the wrong error
class.

reviewer: buntinas

The old "lockRank" error checking is no longer sufficient in MPI 3.0 and must
be removed to add support for locking multiple targets.

Reviewer: balaji

Removed the use of lockRank in the passive target RMA code.  This was a hack to
start with, since lockRank belonged to the the front-end error checking and
should not have been used below the ADI.

Reviewer: buntinas

This is a temporary measure for the reference implementation that should be
improved by ticket #1733.

Reviewer: buntinas

In this patch, I have refactored the RMA ops list again to use the MPL UTList
doubly-linked list and to treat the list as a proper object.  This should set
us up to work with multiple lists, as we will soon have one list per target.
Doubly-linking the list is a big help in terms of maintainability (no more
prevNext pointers) and flexibility (better implementation of request-based
ops and other optimizations).

Reviewer: goodell