[OMPI devel] IP address to verb interface mapping

2016-04-07 Thread dpchoudh .
Hello all (Newbie warning! Sorry :-( ) Let's say my cluster has 7 nodes, connected via IP-over-Ethernet for control traffic and some kind of raw verbs (or anything else such as SRIO) interface for data transfer. Let's say my host file chooses 4 out of the 7 nodes for an MPI job, based on the IP

Re: [OMPI devel] [OMPI commits] Git: open-mpi/ompi branch master updated. dev-3792-g92290b9

2016-04-07 Thread George Bosilca
Thanks for the suggestion. Unfortunately, things are never as simple as they look. The original code was wrong for weeks/months. This unlock is on an execution path that would require an MPI_Cancel happening in exactly the same time as the PML completes the same receive request, or in other words a

Re: [OMPI devel] [OMPI commits] Git: open-mpi/ompi branch master updated. dev-3792-g92290b9

2016-04-07 Thread Paul Hargrove
In the same spirit as Ralph intended, I want to suggest that code changes be *run* before pushing to master. If necessary, add temporary debugging output to help determine that your changes are reached in testing. This can save one the public embarrassment of having a thread like one this dedicated

Re: [OMPI devel] [OMPI commits] Git: open-mpi/ompi branch master updated. dev-3792-g92290b9

2016-04-07 Thread Ralph Castain
Apologies - didn’t mean to make it sound that way. Just wanted to inform a new person on the recommended procedures in case they were unaware of them. > On Apr 7, 2016, at 11:07 AM, George Bosilca wrote: > > Let's not blown this out of proportion, it was nothing more than a typo > pinpointed

Re: [OMPI devel] [OMPI commits] Git: open-mpi/ompi branch master updated. dev-3792-g92290b9

2016-04-07 Thread George Bosilca
Let's not blown this out of proportion, it was nothing more than a typo pinpointed and fixed in a matter of seconds. George. On Thu, Apr 7, 2016 at 1:58 PM, Ralph Castain wrote: > Just as a suggestion: please express such changes in the form of a Pull > Request instead of a direct commit to

Re: [OMPI devel] [OMPI commits] Git: open-mpi/ompi branch master updated. dev-3792-g92290b9

2016-04-07 Thread Ralph Castain
Just as a suggestion: please express such changes in the form of a Pull Request instead of a direct commit to avoid getting such mistakes into the code base. I’m not advocating it for truly trivial stuff - but changing the thread_unlock to an OB1 call probably should be given a chance for commen

Re: [OMPI devel] [OMPI commits] Git: open-mpi/ompi branch master updated. dev-3792-g92290b9

2016-04-07 Thread Nathan Hjelm
Hah, just caught that as well. Commented on the commit on github. Definitely looks wrong. -Nathan On Thu, Apr 07, 2016 at 05:43:17PM +, Dave Goodell (dgoodell) wrote: > [inline] > > On Apr 7, 2016, at 12:53 PM, git...@crest.iu.edu wrote: > > > > This is an automated email from the git hook

Re: [OMPI devel] [OMPI commits] Git: open-mpi/ompi branch master updated. dev-3792-g92290b9

2016-04-07 Thread Dave Goodell (dgoodell)
[inline] On Apr 7, 2016, at 12:53 PM, git...@crest.iu.edu wrote: > > This is an automated email from the git hooks/post-receive script. It was > generated because a ref change was pushed to the repository containing > the project "open-mpi/ompi". > > The branch, master has been updated > v