<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div dir="ltr">Pavan:<div><br></div><div>Hopefully this is fixed by c970f0e1d2c82a674e8a235402617d6c60c23811 on the aforementioned branch.</div><div><br></div><div>Jeff</div>







</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 30, 2015 at 10:15 PM, Jeff Hammond <span dir="ltr"><<a href="mailto:jeff.science@gmail.com" target="_blank">jeff.science@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Facepalm.  Indeed, changing the header but not the implementation was pretty dumb on my part.  I'll fix that tomorrow.<div><br></div><div>I have no opinion on the ABI situation, particularly since I know there are other folks that Intel that do.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Jeff</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 30, 2015 at 9:00 PM, Balaji, Pavan <span dir="ltr"><<a href="mailto:balaji@anl.gov" target="_blank">balaji@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Thanks, Jeff.  The actual function definitions should be modified as well, which your patch seems to miss.<br>
<br>
But I think the potentially time consuming part is to get buy-in from all the ABI consortium folks for this change.  It doesn't exactly break ABI for applications, but will be an annoying change for tools.  Ken is looking into this.<br>
<span><font color="#888888"><br>
  -- Pavan<br>
</font></span><div><div><br>
> On Nov 30, 2015, at 10:54 PM, Jeff Hammond <<a href="mailto:jeff.science@gmail.com" target="_blank">jeff.science@gmail.com</a>> wrote:<br>
><br>
> Pavan:<br>
><br>
> See hammond-mpi2-deprecated-functions-noconst branch on mpich-review.  I found three functions where this change should be made.  I do not know if that is the full list, so someone should verify.<br>
><br>
> Best,<br>
><br>
> Jeff<br>
><br>
> On Mon, Nov 30, 2015 at 8:08 PM, Balaji, Pavan <<a href="mailto:balaji@anl.gov" target="_blank">balaji@anl.gov</a>> wrote:<br>
><br>
> MPI_Type_struct is not a part of the MPI standard anymore and is replaced by MPI_Type_create_struct.  Both MPICH and Open MPI decided to retain it, but followed different directions for it.  MPICH decided to follow the same convention as MPI_Type_create_struct with the const additions.  Open MPI decided to keep the old prototypes as is.<br>
><br>
> Unfortunately, MPE got stuck in between the two.  The best approach would be to simply remove those functions from MPE and this problem just goes away.  Would you be able to do that and contribute a patch?<br>
><br>
> More generally, I think MPICH was overzealous in trying to constify prototypes for the deleted functions.  It should have just left them as they are, like what Open MPI did.  We'll figure out what to do about that internally, but that's not going to be a quick change because of ABI and other such stuff.<br>
><br>
> Thanks,<br>
><br>
>   -- Pavan<br>
><br>
> > On Nov 30, 2015, at 8:14 PM, Schilling, Herbert W. (GRC-VE00) <<a href="mailto:hschilling@nasa.gov" target="_blank">hschilling@nasa.gov</a>> wrote:<br>
> ><br>
> > Hi,<br>
> ><br>
> >  I am trying to build MPE for use with OpenMPI. I built from source Version 1.10.1 of OpenMPI.<br>
> ><br>
> >  I am using this version of MPE <a href="ftp://ftp.mcs.anl.gov/pub/mpi/mpe/mpe2.tar.gz" rel="noreferrer" target="_blank">ftp://ftp.mcs.anl.gov/pub/mpi/mpe/mpe2.tar.gz</a><br>
> ><br>
> >  Should this work fine? I followed the instructions to build like this:<br>
> ><br>
> >     ${MPE_SRC_DIR}/configure MPI_CC=${OPENMPI_INSTALL_DIR}/bin/mpicc \<br>
> >                             CC=<C_COMPILER_used_to_build_OpenMPI> \<br>
> >                             MPI_F77=${OPENMPI_INSTALL_DIR}/bin/mpif77 \<br>
> >                             F77=<FORTRAN_COMPILER_used_to_build_OpenMPI> \<br>
> >                             --with-java2=/usr/local/j2sdk-1.4.2<br>
> >                             --prefix=<MPE_INSTALL_DIR><br>
> >    make<br>
> >    make install<br>
> ><br>
> ><br>
> ><br>
> ><br>
> > I keep running into problems with conflicting declarations<br>
> ><br>
> ><br>
> >  log_mpi_core.c:5611: error: conflicting types for 'MPI_Type_struct'<br>
> >  /u/hschilli/local/include/mpi.h:1796: error: previous declaration of 'MPI_Type_struct' was here<br>
> ><br>
> ><br>
> > I think the issue is that MPE's declaration uses some const qualifiers while OpenMPI does not.<br>
> ><br>
> ><br>
> ><br>
> > I tried hacking my way through this with the macro<br>
> ><br>
> > HAVE_CONST_IN_MPI_DECL<br>
> >  and MPE_CONST but no luck so far.<br>
> ><br>
> ><br>
> ><br>
> > What am I doing wrong?<br>
> ><br>
> ><br>
> > Thanks in advance.<br>
> ><br>
> > ---------<br>
> > Herb Schilling<br>
> > Scientific Applications and Visualization Team<br>
> > <a href="http://ocio.grc.nasa.gov/gvis" rel="noreferrer" target="_blank">http://ocio.grc.nasa.gov/gvis</a><br>
> > Office of the Chief Information Officer<br>
> > NASA Glenn Research Center<br>
> > <a href="mailto:hschilling@nasa.gov" target="_blank">hschilling@nasa.gov</a><br>
> ><br>
> ><br>
> ><br>
> ><br>
> > _______________________________________________<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>
><br>
> _______________________________________________<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>
><br>
><br>
><br>
> --<br>
> Jeff Hammond<br>
> <a href="mailto:jeff.science@gmail.com" target="_blank">jeff.science@gmail.com</a><br>
> <a href="http://jeffhammond.github.io/" rel="noreferrer" target="_blank">http://jeffhammond.github.io/</a><br>
> _______________________________________________<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>
<br>
_______________________________________________<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>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>
</div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div 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>
</div>