GitLab

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

Use PAC macro to AC_SUBST and export WRAPPER_XXX variables. Ensures
that they are available to the testsuite configure script.

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

Two tag bits are reserved for error propagation. We need to make sure
they are ignored by the network matching capabilities. Refs #2260

No reviewer.

Halim Amer authored May 26, 2015 and Kenneth Raffenetti committed May 27, 2015

If the destination of a LMT nonblocking send, which
follows a rendez-vous protocol, receives the RTS packet to the
unexpected queue followed by a cancelation request, the request in
the unexpected queue never gets freed. To solve the issue, we
forcefully free the rendez-vous request by adding an additional
request release operation. Refs #287
Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

Frees memory allocated to track large sends when those sends are
successfully cancelled.

No reviewer.

Junchao Zhang authored May 26, 2015 and Kenneth Raffenetti committed May 26, 2015

The configure test was dropped long time ago in MPICH, and to keep ABI compatible,
we don't want to restore it.

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

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

Change the bindings to make it as if MPI_BOTTOM in Fortran is
at address zero as it in C. See discussion on p.652 of MPI-3.0

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

According to the standard, collectives using the same communicator must
implement thread safety in the application level. So, if using
MPI_Comm_spawn in THREAD_MULTIPLE safety level, it must use different
communicators.

I've modified the test to use different communicators per thread so that
we can still test its thread safety.

Fixes #2256

It was created but never destroyed.

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

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

Simplifies request completion logic and ensures operations are properly
accounted for in the cancel send case. Removes pt2pt send tracking in the
VC, as requests are enough to guarantee completion.
Signed-off-by: Antonio J. Pena <apenya@mcs.anl.gov>

This is currently a workaround to avoid weird errors, e.g., stack
fault, occurred in the NBC I/O implementation on Linux. When the host
OS is Linux and aio-lite is not used, a blocking ADIO function is used
in the NBC I/O implementation.

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

configure defines ROMIO_RUN_ON_LINUX as 1 when the host OS is Linux.
Signed-off-by: Rob Latham <robl@mcs.anl.gov>

Defines usleep when needed. This build error was discovered via the
random config tests in jenkins.
Signed-off-by: Rob Latham <robl@mcs.anl.gov>

Pavan Balaji authored May 13, 2015 and Kenneth Raffenetti committed May 13, 2015

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

Pavan Balaji authored May 12, 2015 and Kenneth Raffenetti committed May 13, 2015

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

we were not correctly handling the case where the error handler
MPI_ERRORS_ARE_FATAL was called.  credit Lisandro Dalcin with finidng
the issue.
Signed-off-by: Junchao Zhang <jczhang@mcs.anl.gov>

Lisandro Dalcin found we returned the wrong value from
MPI_File_call_errhandler.  it should return MPI_SUCCESS unless something
goes wrong with MPI_File_call_errhandler itself.

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

Signed-off-by: Jithin Jose <jithin.jose@intel.com>
Signed-off-by: Charles J Archer <charles.j.archer@intel.com>

Signed-off-by: Jithin Jose <jithin.jose@intel.com>
Signed-off-by: Charles J Archer <charles.j.archer@intel.com>

For tests that call pthread routines, add an explicit dependency.
This was not an issue previously, because either the compile wrappers
always added -lpthread, or the linker discovered the pthread symbols
via an interlibrary dependency on libmpi. However on FreeBSD 10, the
interlibrary dependency resolution appears more strict. Applications
must specify any libraries directly used, thus this fix.
Signed-off-by: Halim Amer <aamer@anl.gov>

This reverts commit 7c0f2967

.
Signed-off-by: Halim Amer <aamer@anl.gov>

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

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

No reviewer

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

If the user passes the -static flag, we disable interlibrary
dependencies, since the linker would pick the static versions of the
libraries in that case.

We are still making an assumption that the default mode of the linker
is shared, and the user can specify static library builds with
-static.  However, this is not always true.  On BG/Q, for example, the
default is static.

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

This includes several changes:

1. Merged WRAPPER and EXTERNAL LIBS.  There is no reason to maintain
two names for these flags.  These are eventually appended and added to
the compiler wrappers anyway.

2. Updated mpicc and friend to add the necessary flags directly
instead of trying to merge these flags in configure.
Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

It does not make sense for the user to specify LDFLAGS/LIBS that would
only be used to build the MPICH library, but not the user application.
Doing do will create a false dependency of the MPICH library on these
external libraries resulting in weird interactions with the user
application in cases where interlibrary dependencies are not
supported.  We were anyway silently modifying the WRAPPER LDFLAGS/LIBS
to include these anyway, so the flags themselves were weirdly screwed
up.

This patch simply removes them instead of pretending to allow them,
but not really allowing them.
Signed-off-by: Ken Raffenetti <raffenet@mcs.anl.gov>

Kenneth Raffenetti authored Apr 28, 2015 and Sangmin Seo committed Apr 28, 2015

The random configuration tests revealed that MPI name publishing
features could be disabled at configure time, but the test suite
would still run tests for them.
Signed-off-by: Sangmin Seo <sseo@anl.gov>

Min Si authored Apr 27, 2015 and Pavan Balaji committed Apr 28, 2015

The latest FreeBSD production release (10.1-RELEASE) does not support
inter-library dependency but libtool cannot detect it correctly. This
patch sets the deplibs_check_method to unknown for FreeBSD, thus all
dependent shared liraries (i.e. -lpthread) can be added in mpicc.
Signed-off-by: Pavan Balaji <balaji@anl.gov>

Min Si authored Apr 26, 2015 and Pavan Balaji committed Apr 28, 2015

On FreeBSD-10.1, function hstrerror is implemented in libc but its
declaration is disabled in netdb.h by default. Thus error "implicit
declaration of function hstrerrori" was reported when compiling hydra.
This patch adds two steps in hydra configure in order to transparently
fix this issue:
1. Check if function hstrerror exists (in libc).
2. Add the declaration of hstrerror if it exists in libc but the
declaration is disabled.
Signed-off-by: Pavan Balaji <balaji@anl.gov>

Min Si authored Apr 26, 2015 and Pavan Balaji committed Apr 28, 2015

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

MPID_nem_ofi_data_callback used to check sreq->cc in order to track progress of
the RTS/CTS/DATA protocol. The was an implicit assumption that fi_tsend with RTS
completes first. However this would cause a hang if fi_trecv completed earlier.
The fix is: don't rely on the cc but rather check the tag bits explicitly.
Note, the RTS/CTS/DATA bits are no longer accumulated (i.e., no more
"wc->tag | CTS/DATA").
Signed-off-by: Charles J Archer <charles.j.archer@intel.com>

It uses nemesis shared memory which is already cleaned up at this stage.
However, w/o any synchronization a hang in the close protocol is possible
since rts/cts/data messages may be on the fly. This change fixes the issue.
Signed-off-by: Charles J Archer <charles.j.archer@intel.com>

Signed-off-by: Charles J Archer <charles.j.archer@intel.com>

Signed-off-by: Charles J Archer <charles.j.archer@intel.com>

This patch modifies the OFI netmod to support large tag layouts, while preserving the old
tag layout.  OFI defines a 64 bit tag, but also provides for a 64 bit tag and immediate data.
In some OFI providers, we may want to select different tag layouts.  This patch currently
does not query for the proper tag layout or attempt to make a choice of the optimal layout,
it provides macro/templatized support for different tag formats.  Additional selection
criteria will be added in subsequent patches.

  * Tag layout is moved to a separate file.
    Added init_sendtag_M2, init_recvtag_M2 (M2 stands for MODE #2, i.e. the mode
    that uses fi_tsenddata and does not pack source into tag).

  * Created a template file for ofi_tagged.c
    Moved do_isend into template file which is included twice into ofi_tagged.c thus providing for the two
    versions of do_isend and do_isend_2 corresponding to the two API sets.

  * All send functions are available in two versions.
    Added macro that declares a function for the two API sets. The first set has the namings inherited from
    the previous netmod version. The functions of the second API set have the "_2" suffix.

  * Recv_posted, anysource_posted, recv_callback, ofi_probe  are templatized.

  * ofi_tag_to_vc renamed ofi_wc_to_vc
    Note, for the API_SET_2 the pgid is stored in the imm data while
    psource and port will be packed the same way as in API_SET_1.

  * Adds api_set member in gl_data struct.  Initialize routines based on api_set

  * Added RCD (RtsCtsData) protocol identifiers

  * Added support for OFI MEM_TAG_FORMAT

  * PGID placement modified
Signed-off-by: Charles J Archer <charles.j.archer@intel.com>