This reverts commit 7a366031

.

At one point (circa MPICH2-1.5) this clever optimization behaved as
expected. Once ported forward to more recent MPICH, segfaults on trivial
MPI applications.

Closes: #2217
Reopens: #1835

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

     This allows the systems that don't have GPUs installed and no libcuda
     to use the libmpi.so w/o getting unable to load lib errors

(ibm) D203056

Signed-off-by: Tsai-Yang (Alan) Jea <tjea@us.ibm.com>
Signed-off-by: Sameh Sharkawi <sssharka@us.ibm.com>

 (ibm) D203137

Signed-off-by: Sameh Sharkawi <sssharka@us.ibm.com>

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

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

The max_eqs limit returned at PtlNIInit time is actually a limit on the
number of event queues a particular interface can support. It is not the
number of events a single queue can hold, which is how we were using it.

Since there is no way to query for the max events a queue can hold, we
simply request a conservative 32K. Thanks to Sayantan Sur for pointing out
our error.

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

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

FreeBSD seems to take slightly more than 20 minutes to run these tests
primarily because the poll() call is somehow superslow on that OS.
This new timeout should be sufficient to cover that case.

No reviewer.

When configured with --enable-thread-cs=per-object, MPICH build will go
wrong because MPID_cc_t will be defined as OPA_int_t, which can not be
directly compared with or assigned with int.

This patch adds one macro,  MPID_cc_get, for correctly reading
MPID_cc_t. Also, change writing to MPID_cc_t to use MPID_cc_set.

Fixes #2250

The nonblocking4 test was renamed and moved to the errors directory
in [a0ebb441].

No reviewer.

The abort fix for gforker was not agressive enough about cleaning up
MPI processes. An abort could lead to a situation where some MPI
processes were left running on the node after mpiexec has exited. Now
we force kill all processes before exit. Refs #2249

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

Since [b1e89abf

], PMI_Abort sends an abort command to the PMI server
and relies on it to cleanup the application. In the case of gforker,
nothing was done with this command, leaving the PMI client hanging
waiting for a response. Gforker will now exit with the requested abort
code upon receiving the command. Fixes #2249

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

This test is run with 64 processes.  On platforms that do not schedule
that many processes well (such as FreeBSD), this can slow down the
test a lot and cause timeouts.

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

These tests take a longer time on OSX due to the poor performance of
malloc/calloc on OSX platforms. The extended timeout gives them a chance
to complete.

This could be better fixed in the future by only allocating the memory
for the window once instead of over and over (~5000 times).

No reviewer

back in december 2013 I apparently did not test a standalone ROMIO
against a const-ified (MPI-3.0 compliant) MPICH

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

The standalone case calls PAC_TEST_NEEDS_CONST, but I never implemented
it.  here is that implementation.

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

if we use this aio-lite implementation of aio routines, the MPICH test
suites all pass.

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

Autoconf provides nice routines for "does this struct have this member",
so use that instead of our home-grown nonstandrd checks

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

there is no need for ROMIO to try to compile header files.  modern (and
not-so-modern: since 2.56) autoconf will do exactly this with
AC_CHECK_HEADERS

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

MPICH collectives use MPIR_Localcopy to move data from src to destination
buffer for same task. MPIR_Localcopy can't handle GPU buffers. A change
in MPIR_Localcopy would affect all common code. This change is to handle
the checking of GPU buffers in the PAMID collectives layer and allocate
host buffer and copy data from GPU buffer to Host buffer and vice versa so
MPIR_Localcopy would work w/o issues.
This is not performance optimized code.

(ibm) D202834

Signed-off-by: Su Huang <suhuang@us.ibm.com>

Track this library separately so as not to impose its requirements
when building other components like ROMIO or Hydra.

No reviewer.

Commit [7dfe2840

] broke builds with error checking disabled. We fix this
by moving the guards to ensure the fn_exit and fn_fail labels are always
present and only the error checking code is preprocessed out.

Fixes #2225

Signed-off-by: Rajeev Thakur <thakur@mcs.anl.gov>

This code was using uint32_t as a fixed size variable to simplify the
code, but we don't currently require C99. The patch removes that usage
and does everything with int's and more math.

Fixes #2245

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

This is another function that is no longer used as it was replaced by a
static function instead. This also removes an internal test that is no
longer useful for this code.

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

Originally this function was implemented in the CH3 layer, but it was
moved up to the MPI layer as an MPIR function. The old MPID version
doesn't need to exist anymore.

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

The Myrinet MX network module, which had a life cyle from 1.1 till
3.1.2, has now been deprecated.

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

No reviewer.

File ch3u_rma_oplist.c contains functions that make progress
on RMA on the origin side. Here we change the name of file to
a more suitable one.

No reviewer.

Removes additional whitespace in testlist files to make them easier
to manipulate with tools like sed.

No reviewer.

Put tests that check if MPI correctly detects aliased buffers in
collective operations into the errors section of the testsuite.

Fixes #2211

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

No reviewer

Sangmin Seo authored Mar 05, 2015 and Rob Latham committed Mar 05, 2015

Although MPIDI_CH3I_progress_blocked is a variable only used in CH3,
it was referenced in the ROMIO glue code. This caused a build problem
when pamid is used as a device. This patch removed the reference to
MPIDI_CH3I_progress_blocked, but it degrades the efficiency of
MPIR_Ext_cs_yield_allfunc_if_progress_blocked() since we do not have
a way to check if the progress engine is blocked for now (related to
ticket #2202). For a better solution for ticket #2202, we need to
fix a wait function of the extended generalized request.

Fixes #2242

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

comm_idup was caught failing on mpich-portals4 with configuration
"intel,strict,ib”. It was not fully tested on portals4 because portals
was added after comm_dup patch. On other platforms comm_idup seems to be
OK.

Ticket #2243

No reviewer

Since MPI-3.1 has not been voted this time.

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