<div dir="ltr">Has Lisandro provided any feedback on this anywhere?  I agree with Hui.  This appears to be an artifact of the way MPI uses magic values for MPI_IN_PLACE and how those get encoded in binaries.  I naively speculate that MPICH could encode the magic values differently to work around this, but that would likely break ABI compatibility, at least on the Fortran side.<div><br><div>Jeff</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 10, 2020 at 3:31 PM Zhou, Hui via discuss <<a href="mailto:discuss@mpich.org">discuss@mpich.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="EN-US">
<div class="gmail-m_-101756718602840352WordSection1">
<p class="MsoNormal">I am not sure this is an issue for mpich. Seems more to be an issue of `mpi4py`. The issue is exactly as you suspected -- `libmpifort.so` need be loaded before `libmpi.so`, or the external symbol in `libmpifort.so` won’t get resolved.
<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">>  It works if you load something linked to libmpifort first or load everything with RTLD_GLOBAL. <br>
<br>
Seems plausible. I don’t have any better idea other than fixing `mpi4py` so it always load `libmpifort.so`  first.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<div>
<p class="MsoNormal">-- <br>
Hui Zhou<u></u><u></u></p>
</div>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(181,196,223);padding:3pt 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12pt;color:black">From: </span></b><span style="font-size:12pt;color:black">Patrick McNally via discuss <<a href="mailto:discuss@mpich.org" target="_blank">discuss@mpich.org</a>><br>
<b>Reply-To: </b>"<a href="mailto:discuss@mpich.org" target="_blank">discuss@mpich.org</a>" <<a href="mailto:discuss@mpich.org" target="_blank">discuss@mpich.org</a>><br>
<b>Date: </b>Wednesday, June 10, 2020 at 1:09 PM<br>
<b>To: </b>"<a href="mailto:discuss@mpich.org" target="_blank">discuss@mpich.org</a>" <<a href="mailto:discuss@mpich.org" target="_blank">discuss@mpich.org</a>><br>
<b>Cc: </b>Patrick McNally <<a href="mailto:rpmcnally@gmail.com" target="_blank">rpmcnally@gmail.com</a>><br>
<b>Subject: </b>Re: [mpich-discuss] Using MPICH in Python breaks Fortran MPI_IN_PLACE<u></u><u></u></span></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I hate to bug, but this is a pretty serious issue.  I suspect it is why we get segfaults trying to use similar variables like MPI_STATUSES_IGNORE.  Any insight would be appreciated.<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Thanks,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Patrick<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">On Wed, May 27, 2020 at 10:25 AM Patrick McNally <<a href="mailto:rpmcnally@gmail.com" target="_blank">rpmcnally@gmail.com</a>> wrote:<u></u><u></u></p>
</div>
<blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in">
<div>
<p class="MsoNormal">Our application consists primarily of a Python head calling into Fortran routines to do the heavy lifting.  We have never been able to successfully use MPI_IN_PLACE in Fortran but weren't sure why.  Recently, we discovered that it works
 fine in standalone Fortran code and is only broken when the Fortran code is run through our Python modules.<br>
<br>
The issue appears to be related to having code that only links to the C libmpi library loaded first and with RTLD_LOCAL, as happens when we load mpi4py.  It works if you load something linked to libmpifort first or load everything with RTLD_GLOBAL.  I'm assuming
 this has something to do with how MPICH tests the address of MPIR_F08_MPI_IN_PLACE but I don't understand SO loading well enough to fully grasp the issue.  Below is some standalone code to show the issue.  I'd appreciate any insight you can provide into why
 this is happening.<br>
<br>
Relevant system details:<br>
RHEL 7.8<br>
Python 2.7<br>
GCC 7.3.0<br>
MPICH 3.3.2 (and 3.2)<br>
<br>
The below files are also available towards the end of the bug report at the following link:<br>
<a href="https://bitbucket.org/mpi4py/mpi4py/issues/162/mpi4py-initialization-breaks-fortran" target="_blank">https://bitbucket.org/mpi4py/mpi4py/issues/162/mpi4py-initialization-breaks-fortran</a><u></u><u></u></p>
<div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Thanks,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Patrick<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><br>
makefile<br>
-----------<br>
libs = testc.so testf.so<br>
all: $(libs)<br>
<br>
testc.so: testc.c<br>
        mpicc   -shared -fPIC $< -o $@<br>
<br>
testf.so: testf.f90<br>
        mpifort -shared -fPIC $< -o $@<br>
<br>
clean:<br>
        $(RM) $(libs)<br>
<br>
testc.c<br>
---------<br>
#include <stddef.h><br>
#include <stdio.h><br>
#include <mpi.h><br>
<br>
extern void initc(void);<br>
extern void testc(void);<br>
<br>
void initc(void)<br>
{<br>
  MPI_Init(NULL,NULL);<br>
}<br>
<br>
void testc(void)<br>
{<br>
  int val = 1;<br>
  MPI_Allreduce(MPI_IN_PLACE, &val, 1, MPI_INT, MPI_SUM, MPI_COMM_WORLD);<br>
  printf("C val: %2d\n",val);<br>
}<br>
<br>
testf.f90<br>
-----------<br>
subroutine initf() bind(C)<br>
  use mpi<br>
  integer ierr<br>
  call MPI_Init(ierr)<br>
end subroutine initf<br>
<br>
subroutine testf() bind(C)<br>
  use mpi<br>
  integer ierr<br>
  integer val<br>
  val = 1<br>
  call MPI_Allreduce(MPI_IN_PLACE, val, 1, MPI_INTEGER, MPI_SUM, MPI_COMM_WORLD, ierr)<br>
  print '(A,I2)', 'F val: ', val<br>
end subroutine testf<br>
<br>
test.py<br>
---------<br>
from ctypes import CDLL, RTLD_LOCAL, RTLD_GLOBAL<br>
<br>
mode = RTLD_LOCAL<br>
cfirst = True<br>
<br>
if cfirst: # it does not work!<br>
    libc = CDLL("./testc.so", mode)<br>
    libf = CDLL("./testf.so", mode)<br>
else: # it works!<br>
    libf = CDLL("./testf.so", mode)<br>
    libc = CDLL("./testc.so", mode)<br>
<br>
libc.initc.restype  = None<br>
libc.testc.argtypes = []<br>
libf.initf.restype  = None<br>
libf.testf.argtypes = []<br>
<br>
libc.initc()<br>
libc.testc()<br>
libf.testf()<u></u><u></u></p>
</div>
</div>
</div>
</blockquote>
</div>
</div>
</div>

_______________________________________________<br>
discuss mailing list     <a href="mailto:discuss@mpich.org" target="_blank">discuss@mpich.org</a><br>
To manage subscription options or unsubscribe:<br>
<a href="https://lists.mpich.org/mailman/listinfo/discuss" rel="noreferrer" target="_blank">https://lists.mpich.org/mailman/listinfo/discuss</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">Jeff Hammond<br><a href="mailto:jeff.science@gmail.com" target="_blank">jeff.science@gmail.com</a><br><a href="http://jeffhammond.github.io/" target="_blank">http://jeffhammond.github.io/</a></div>