No reviewer

Now, when we pop an event, we queue up the buddy event (e.g., ACK for
SEND) to return next.  This way, we don't need to search for the event
everytime.  Since we know that there'll be at most one such pending
event, we maintain a single event structure for this.

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

We were stashing events when the origin receives a NACK.  This is
unnecessary since we retransmit the op and never use those stashed
events.

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

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

1. Moved op management to a different file.

2. Move rptl_info to an extern, so it can be shared by multiple files.

3. Separate out rptl initialization routines.

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

ReqHandler_GaccumLikeSendComplete is used for GACC-like operations,
including GACC, CAS and FOP. Here we split it into following three
functions:

ReqHandler_GaccumSendComplete
ReqHandler_CASSendComplete
ReqHandler_FOPSendComplete

It is convenient for us to add different actions in future for those
three kinds of operations.

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

Here we wrap up common action when one RMA op is finished on target
into a function to make code structure cleaner.

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

Originally do_accumulate_op() only accepts request pointer as
argument which is too restrict to be reused. Here we modify
it to access buffer address, count, datatype and op, so that
it can be reused in more general cases.

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

Here we initalize packet flag as FLAG_NONE when creating
this packet, and add flags later when needed.

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

When operation pending list and request lists are all empty, FLUSH message
needs to be sent by origin only when origin issued PUT/ACC operations since
the last synchronization calls, otherwise origin does not need to issue FLUSH
at all and does not need to wait for FLUSH ACK message.

Similiarly, origin waits for ACK of UNLOCK message only when origin issued
PUT/ACC operations since the last synchronization calls. However, UNLOCK
message always needs to be sent out because origin needs to unlock the
target process. This patch avoids issuing unnecessary
FLUSH / FLUSH ACK / UNLOCK ACK messages.

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

It makes testlist.in files more flexible and easier to read

Signed-off-by: Sangmin Seo <sseo@anl.gov>

No review since F08 binding is experimental now.

No review since F08 binding is experimental now.

Without doing so, the script wrongly thinks #ifdef etc. are part of
a subroutine's prototype line.

No review since F08 binding is experimental now.

No review since F08 binding is experimental now.

Stack variables should not have been used as sendbuf for MPI_Iallgather
because we do not wait the completion of MPI_Iallgather in caller
functions. This fix moved them to the struct used for keeping track
of operation state and uses variables in the struct for MPI_Iallgather.

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

Helps clarity since we no longer use ACKs in the netmod code.

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

The rportals layer is taking care of retransmissions, so we should only
be interested in delivery events in the netmod layer.

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

No reviewer

Nightly testing reported timeout on octopus with 12 timelimit. Each of
them took 12:05 ~ 12:10 mins.

This tests the behavior after a failure when using revoke+shrink. Right
now this test still fails so it is marked as xfail.

See #2198

No reviewer

Those were introduced for a robust protocol during development.
No longer needed.

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

The user pointer was set, but later overwritten with an internal value.

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

Sangmin Seo authored Nov 10, 2014 and Rob Latham committed Nov 13, 2014

Ported test programs using collective I/O in the ROMIO test directory
to the nonblocking collective I/O version. They were temporarily added
to the MPICH test directory to run with Jenkins and nightly tests.
However, they may need to be moved to the ROMIO test directory later.

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

Sangmin Seo authored Nov 10, 2014 and Rob Latham committed Nov 13, 2014

Added nonblocking version of bigtype, hindexed_io, rdwrord, and setviewcur
for testing nonblocking collective I/O functions.

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

Sangmin Seo authored Nov 07, 2014 and Rob Latham committed Nov 13, 2014

This patch implemented four functions for nonblocking collective I/O,
which will be added to MPI 3.1 standard. Details for these functions
can be found in the MPI-Forum ticket,
https://svn.mpi-forum.org/trac/mpi-forum-web/ticket/273

.
Currently, they are implemented as MPIX functions.

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

Some of the FT tests were not correctly setting their error handlers to
MPI_ERRORS_RETURN. While this doesn't seem to have caused problems, it's
safer to do so.

This commit also cleans up some unused variables, reorders communicator
creation, and correctly frees some variables to avoid some debugging
output.

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

Depending on how some uninitialized data was prepopulated, the ibsend
check was periodically crashing out in the call to
MPID_Request_is_pending_failure. Some simple sanity checking to make
sure the input data wasn't NULL takes care of this.

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

The previous code only detected a datatype mismatch when the message
was copied out of the unexpected queue. Now it will throw an error
in both cases.

We also set the error in the status object to match the default ch3
behavior. This fixed an issue where the request would not be freed
and cause extra debugging output at MPI_Finalize.

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

Antonio Pena Monferrer authored Nov 12, 2014 and Pavan Balaji committed Nov 12, 2014

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

The terminology "flow_control" was a bit of a misnomer since we do
more than just enable/disable flow control based on whether messages
are on the data or control portal.

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

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

We now use a target structure for each target ID that we want to send
data to.  This allows us to separate out target-specific states and
more cleanly manage operations to a single target.

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

Kenneth Raffenetti authored Nov 11, 2014 and Pavan Balaji committed Nov 12, 2014

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

Antonio Pena Monferrer authored Nov 11, 2014 and Pavan Balaji committed Nov 12, 2014

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

Antonio J. Pena authored Oct 22, 2014 and Kenneth Raffenetti committed Nov 12, 2014

Full redesign, mainly of the functions in ptl_nm.c and the
communications involving the "control" portal. Still some
problems with flow control.

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

Timeout is reported on some overloaded machines with 10 minutes time
limitation.

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

Free the group and communicator created in the test so it does not
complain when memory debug is on.

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

Fixes #1945

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