Ok, got --disable-dlopen working again. I removed the code in question and changed how coll/sm shares the segment data.
-Nathan On Mon, Jul 28, 2014 at 02:41:37PM -0600, Nathan Hjelm wrote: > > Or pull it into coll/sm. Though I think we can do better here since > point-to-point messaging can be used in coll/sm. We can use the > netpatterns code to share the segment information. > > -Nathan > > On Mon, Jul 28, 2014 at 08:37:15PM +0000, Jeff Squyres (jsquyres) wrote: > > Perhaps that RML code can go back up in ompi/common/sm...? (since only > > ompi/coll/sm uses it) > > > > > > On Jul 28, 2014, at 4:34 PM, Nathan Hjelm <hje...@lanl.gov> wrote: > > > > > > > > Damn, spoke too soon. coll/sm uses it: > > > > > > ./ompi/mca/coll/sm/coll_sm_module.c: > > > mca_common_sm_init_group(comm->c_local_group, size, fullpath, > > > ./ompi/mca/coll/sm/coll_sm_module.c: > > > "coll:sm:enable:bootstrap comm (%d/%s): mca_common_sm_init_group failed", > > > > > > > > > Let me see if I can resolve that one. > > > > > > -Nathan > > > > > > On Mon, Jul 28, 2014 at 02:14:36PM -0600, Nathan Hjelm wrote: > > >> > > >> Looks like you are correct. The function that calls the rml code is > > >> mca_common_sm_init which is no longer called by anything (other than > > >> mca_common_sm_init_group.. which isn't called either). Let me see if I > > >> can fix this. I need this build working again with --disable-dlopen. > > >> > > >> mu-fey:/usr/projects/hpctools/hjelmn/ompi-trunk-git hjelmn$ find opal > > >> -name \*.[ch] | xargs grep mca_common_sm_init > > >> opal/mca/common/sm/common_sm.c:mca_common_sm_init(opal_proc_t **procs, > > >> opal/mca/common/sm/common_sm.c:mca_common_sm_init_group(ompi_group_t > > >> *group, > > >> opal/mca/common/sm/common_sm.c: /* let mca_common_sm_init take care > > >> of the rest ... */ > > >> opal/mca/common/sm/common_sm.c: ret = mca_common_sm_init(procs, > > >> group_size, size, file_name, > > >> opal/mca/common/sm/common_sm.h:mca_common_sm_init(opal_proc_t **procs, > > >> opal/mca/common/sm/common_sm.h:mca_common_sm_init_group(ompi_group_t > > >> *group, > > >> > > >> > > >> On Mon, Jul 28, 2014 at 01:05:50PM -0700, Ralph Castain wrote: > > >>> I'm not sure the sm actually relies on the RML any more - I thought we > > >>> had removed that dependency, though the file may not have been deleted. > > >>> > > >>> On Jul 28, 2014, at 1:02 PM, Nathan Hjelm <hje...@lanl.gov> wrote: > > >>> > > >>>> > > >>>> The trunk is totally broken and it might not be easy to fix. I am > > >>>> seeing > > >>>> this error when building with --disable-dlopen (the LANL default): > > >>>> > > >>>> /usr/projects/hpctools/hjelmn/ompi-trunk-git/opal/mca/common/sm/common_sm_rml.c: > > >>>> In function 'mca_common_sm_rml_info_bcast': > > >>>> /usr/projects/hpctools/hjelmn/ompi-trunk-git/opal/mca/common/sm/common_sm_rml.c:110: > > >>>> warning: passing argument 1 of 'orte_rml.send_buffer_nb' from > > >>>> incompatible pointer type > > >>>> /usr/projects/hpctools/hjelmn/ompi-trunk-git/opal/mca/common/sm/common_sm_rml.c:110: > > >>>> note: expected 'struct orte_process_name_t *' but argument is of type > > >>>> 'opal_process_name_t *' > > >>>> /usr/projects/hpctools/hjelmn/ompi-trunk-git/opal/mca/common/sm/common_sm_rml.c:127: > > >>>> warning: passing argument 1 of 'orte_rml.recv_buffer_nb' from > > >>>> incompatible pointer type > > >>>> /usr/projects/hpctools/hjelmn/ompi-trunk-git/opal/mca/common/sm/common_sm_rml.c:127: > > >>>> note: expected 'struct orte_process_name_t *' but argument is of type > > >>>> 'opal_process_name_t *' > > >>>> /usr/projects/hpctools/hjelmn/ompi-trunk-git/opal/runtime/opal_init.c:69:2: > > >>>> warning: #ident is a GCC extension > > >>>> ../../../opal/.libs/libopen-pal.so: undefined reference to `orte_rml' > > >>>> ../../../opal/.libs/libopen-pal.so: undefined reference to > > >>>> `orte_proc_is_bound' > > >>>> ../../../opal/.libs/libopen-pal.so: undefined reference to > > >>>> `orte_rml_send_callback' > > >>>> > > >>>> > > >>>> common/sm relies on the RML for some communication. This will have to > > >>>> be > > >>>> rewritten to get it to work. > > >>>> > > >>>> -Nathan > > >>>> _______________________________________________ > > >>>> devel mailing list > > >>>> de...@open-mpi.org > > >>>> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel > > >>>> Link to this post: > > >>>> http://www.open-mpi.org/community/lists/devel/2014/07/15292.php > > >>> > > >>> _______________________________________________ > > >>> devel mailing list > > >>> de...@open-mpi.org > > >>> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel > > >>> Link to this post: > > >>> http://www.open-mpi.org/community/lists/devel/2014/07/15294.php > > > > > > > > > > > >> _______________________________________________ > > >> devel mailing list > > >> de...@open-mpi.org > > >> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel > > >> Link to this post: > > >> http://www.open-mpi.org/community/lists/devel/2014/07/15295.php > > > > > > _______________________________________________ > > > devel mailing list > > > de...@open-mpi.org > > > Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel > > > Link to this post: > > > http://www.open-mpi.org/community/lists/devel/2014/07/15296.php > > > > > > -- > > Jeff Squyres > > jsquy...@cisco.com > > For corporate legal information go to: > > http://www.cisco.com/web/about/doing_business/legal/cri/ > > > > _______________________________________________ > > devel mailing list > > de...@open-mpi.org > > Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel > > Link to this post: > > http://www.open-mpi.org/community/lists/devel/2014/07/15297.php > _______________________________________________ > devel mailing list > de...@open-mpi.org > Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel > Link to this post: > http://www.open-mpi.org/community/lists/devel/2014/07/15298.php
pgp7hBqfa2g8x.pgp
Description: PGP signature