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:
> 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, <rhc_at_[hidden]> 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
>> 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/
> timattox_at_[hidden] || tmattox_at_[hidden]
> I'm a bright... http://www.the-brights.net/
> devel mailing list