Actually, I could have fixed it without removing anything. All that was required was to change the flag passed to a function from false to true.
But I figured I would go ahead and remove some code (support for ompi-profiler) that nobody is using. Will probably remove it from the rest of the codebase over time. :-) On Mar 17, 2011, at 6:49 AM, Tim Mattox wrote: > Ralph, > I love how this bugfix results in a net reduction of almost 200 lines > of code! Very nice. > (I now return to deep lurking mode...) > > On Wed, Mar 16, 2011 at 10:22 PM, <r...@osl.iu.edu> wrote: >> Author: rhc >> Date: 2011-03-16 22:22:23 EDT (Wed, 16 Mar 2011) >> New Revision: 24536 >> URL: https://svn.open-mpi.org/trac/ompi/changeset/24536 >> >> Log: >> Fix the hier grpcomm module so modex results in correct data. The prior >> implementation stored the modex data as node-based attributes. This worked >> fine for BTL's such as openib where the interfaces were associated with the >> node. However, BTL's such as TCP have interfaces associated with a specific >> process, not a node. Thus, store the data in the modex database so it is >> correctly indexed. >> >> >> Text files modified: >> trunk/orte/mca/grpcomm/hier/grpcomm_hier_module.c | 209 >> +-------------------------------------- >> 1 files changed, 7 insertions(+), 202 deletions(-) > > -- > Tim Mattox, Ph.D. - http://homepage.mac.com/tmattox/ > timat...@open-mpi.org || tmat...@gmail.com > I'm a bright... http://www.the-brights.net/ > > _______________________________________________ > devel mailing list > de...@open-mpi.org > http://www.open-mpi.org/mailman/listinfo.cgi/devel
