Hi Jeff,
Thanks for having a look in my unified diff (see comments in the text)
I have commited all my last changes in bitbucket, including those that
follows.
Pascal
Jeff Squyres a écrit :
Some questions about the patch:
configure.in:
@@ -2002,9 +1987,8 @@
# Turn off the building of the Fortran interface and the Info routines
EXTRA_DIRS=""
AC_DEFINE(HAVE_STATUS_SET_BYTES,1,[Define if status_set_bytes available])
- DEFINE_HAVE_MPI_GREQUEST="#define HAVE_MPI_GREQUEST"
- # Add the MPICH2_INCLUDE_FLAGS to CPPFLAGS
- CPPFLAGS="$CPPFLAGS $MPICH2_INCLUDE_FLAGS"
+ DEFINE_HAVE_MPI_GREQUEST="#define HAVE_MPI_GREQUEST 1"
+ AC_DEFINE(HAVE_MPIU_FUNCS,1,[Define if MPICH2 memory tracing macros
defined])
fi
#
#
Do we have the MPIU functions? Or is that an MPICH2-specific thing?
I have put in comments this last "AC_DEFINE":
# Open MPI does not have the MPIU functions
# AC_DEFINE(HAVE_MPIU_FUNCS,1,[Define if MPICH2 memory tracing macros
defined])
I see that you moved confdb/aclocal_cc.m4 to acinclude.m4. Shoudn't we just -I
confdb instead to get all of their .m4 files?
This has been done during the last porting (years ago).
I have now changed this: All confdb/.m4 files are now copied from
MPICH2. Only the definition of PAC_FUNC_NEEDS_DECL is still kept in
acinclude.m4.
If I do not so, configure is still blocking on this macro.
All seems working well so. If you have any clue about this, I will take it !
In mpipr.h, why remove the #if 0?
-/* Open MPI: these functions are not supposed to be profiled */
-#if 0
#undef MPI_Wtick
#define MPI_Wtick PMPI_Wtick
#undef MPI_Wtime
#define MPI_Wtime PMPI_Wtime
-#endif
OK, I put the #if 0 again.
In configure.in, please update the version number in AM_INIT_AUTOMAKE.
AM_INIT_AUTOMAKE(io-romio, 1.0.0, 'no')
is changed to
AM_INIT_AUTOMAKE(io-romio, 1.0.1, 'no')
I thought there was one other thing that I saw, but I can't recall it right
now...
This is just from looking at your diff; I didn't try to run it yet because you
said there were some things that weren't pushed back up to bitbucket yet.
On Nov 24, 2010, at 10:48 AM, Pascal Deveze wrote:
Hi Jeff,
Here is the unified diff.
As only the romio subtree is modified, I made the following command:
diff -u -r -x .svn ompi-trunk/ompi/mca/io/romio/romio/
NEW-ROMIO-FOR-OPENMPI/ompi/mca/io/romio/romio/ > DIFF_UPDATE
tar cvzf DIFF_UPDATE.TGZ DIFF_UPDATE
Compilation is OK. I run the ROMIO tests.
There are a few new modifications that are not in bitbucket. I think it is not
necessary to update bitbucket
(http://bitbucket.org/devezep/new-romio-for-openmpi/ ).
Pascal
Jeff Squyres a écrit :
Thanks Pascal!
Is there any change you could send a unified diff of the tip of your hg vs. the
SVN trunk HEAD?
E.g., if you have an hg+ssh combo tree, could you "hg up" in there to get all your work, and
then "svn diff > diff.out" and then compress and send the diff.out?
Thanks!
On Nov 10, 2010, at 8:43 AM, Pascal Deveze wrote:
WHAT: Port the lastest ROMIO version from MPICH2-1.3 into the trunk.
WHY: There is a considerable interest in updating the ROMIO branch that was
ported from mpich2-1.0.7
WHERE: ompi/mca/io/romio/
WHEN: Before 1.5.2, so asap
TIMEOUT: Next Tuesday teleconf, 23 Nov 2010
-----
I am in charge of ticket 1888 (see at
https://svn.open-mpi.org/trac/ompi/ticket/1888
).
I have made the porting of ROMIO available in bitbucket since September 17th
2010. (
http://bitbucket.org/devezep/new-romio-for-openmpi/
)
Until now, I do not have any report on this porting and it's now time to bring
it into the trunk.
All modified files are located under the romio subtree.
Pascal Devèze
_______________________________________________
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel
<DIFF_UPDATE.TGZ>_______________________________________________
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel