Re: [OMPI devel] OMPI devel] RFC: semantic change of opal_hwloc_base_get_relative_locality

Saw it and will review - thanks! On Tue, Jun 24, 2014 at 9:51 PM, Gilles Gouaillardet < gilles.gouaillar...@iferc.org> wrote: > Ralph, > > i pushed the change (r32079) and updated the wiki. > > the RFC can be now closed and the consensus is semantic of > opal_hwloc_base_get_relative_locality

Re: [OMPI devel] OMPI devel] RFC: semantic change of opal_hwloc_base_get_relative_locality

Ralph, i pushed the change (r32079) and updated the wiki. the RFC can be now closed and the consensus is semantic of opal_hwloc_base_get_relative_locality will not be changed since this is not needed : the hang is a coll/ml bug, so it will be fixed within coll/ml. Cheers, Gilles On 2014/06/25

Re: [OMPI devel] OMPI devel] RFC: semantic change of opal_hwloc_base_get_relative_locality

Yeah, we should make that change, if you wouldn't mind doing it. On Tue, Jun 24, 2014 at 9:43 AM, Gilles GOUAILLARDET < gilles.gouaillar...@gmail.com> wrote: > Ralph, > > That makes perfect sense. > > What about FCA_IS_LOCAL_PROCESS ? > Shall we keep it or shall we use directly

Re: [OMPI devel] OMPI devel] RFC: semantic change of opal_hwloc_base_get_relative_locality

Ralph, That makes perfect sense. What about FCA_IS_LOCAL_PROCESS ? Shall we keep it or shall we use directly OPAL_PROC_ON_LOCAL_NODE directly ? Cheers Gilles Ralph Castain wrote: >Hi Gilles > > >We discussed this at the devel conference this morning. The root cause of the

Re: [OMPI devel] RFC: semantic change of opal_hwloc_base_get_relative_locality

Hi Gilles We discussed this at the devel conference this morning. The root cause of the problem is a test in coll/ml that we feel is incorrect - it basically checks to see if the proc itself is bound, and then assumes that all other procs are similarly bound. This in fact is never guaranteed to

[OMPI devel] RFC: semantic change of opal_hwloc_base_get_relative_locality

WHAT: semantic change of opal_hwloc_base_get_relative_locality WHY: make is closer to what coll/ml expects. Currently, opal_hwloc_base_get_relative_locality means "at what level do these procs share cpus" however, coll/ml is using it as "at what level are these procs commonly