r31829 caused many sigsegv :-(
/* i am testing on a RHEL6.3 like VM with --mca btl,tcp */
this is now fixed in r31830,
i think i get the intent of the code and i believe we are all set now.
bottom line :
- we agree on line 487 (e.g. use btl_send)
- your update of line 485 is correct (e.g. use btl_send)
- my suggested update of line 498 (e.g. use btl_send) was correct.
On 2014/05/20 4:06, Nathan Hjelm wrote:
> On Mon, May 19, 2014 at 02:14:57PM +0900, Gilles Gouaillardet wrote:
>> do you mean the bug/typo was not at line 487
>> (e.g. btl_send was ok)
>> but at line 498 ?
>> (e.g. btl_send must be used instead of btl_eager)
> Yup. If you look at the next loop (L497) it looks through the btl_send
> list and then calls del_procs if it finds the btl in that list.
>> at first sight, that make sense.
>> i'd rather let the author/maintainer of this part comment on that
> I don't know if the original author still works on Open MPI. I think we
> will have to guess the intent of the code. Let me take a closer look and
> see if I can determine for sure what was intended. If I can determine
> for sure I will include this change with another bml fix that needs to
> go in.
> devel mailing list
> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> Link to this post: http://www.open-mpi.org/community/lists/devel/2014/05/14828.php