<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">
Hi Jeff and Pavan,
<div><br>
</div>
<div> Thanks for all the help on this. Jeff, when you have the fix up there, I can give it a shot!</div>
<div><br>
</div>
<div><br>
</div>
<div>
<div>
<div>
<div>On Dec 1, 2015, at 1:15 AM, Jeff Hammond <<a href="mailto:jeff.science@gmail.com">jeff.science@gmail.com</a>></div>
<div> wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<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>
<div><br>
</div>
<div>Jeff</div>
</div>
<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 class="HOEnZb"><font color="#888888"><br>
-- Pavan<br>
</font></span>
<div class="HOEnZb">
<div class="h5"><br>
> On Nov 30, 2015, at 10:54 PM, Jeff Hammond <<a href="mailto:jeff.science@gmail.com">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">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">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">hschilling@nasa.gov</a><br>
> ><br>
> ><br>
> ><br>
> ><br>
> > _______________________________________________<br>
> > discuss mailing list <a href="mailto:discuss@mpich.org">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">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">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">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">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 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>
_______________________________________________<br>
discuss mailing list <a href="mailto:discuss@mpich.org">discuss@mpich.org</a><br>
To manage subscription options or unsubscribe:<br>
<a href="https://lists.mpich.org/mailman/listinfo/discuss">https://lists.mpich.org/mailman/listinfo/discuss</a></blockquote>
</div>
<br>
<div apple-content-edited="true">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">
<div>---------</div>
<div>Herb Schilling</div>
<div>Scientific Applications and Visualization Team</div>
<div><a href="http://ocio.grc.nasa.gov/gvis">http://ocio.grc.nasa.gov/gvis</a></div>
<div>Office of the Chief Information Officer</div>
<div>NASA Glenn Research Center</div>
<div><a href="mailto:hschilling@nasa.gov">hschilling@nasa.gov</a></div>
<div><br>
</div>
</div>
<br class="Apple-interchange-newline">
<br class="Apple-interchange-newline">
</div>
<br>
</div>
</div>
</body>
</html>