True. I'm very sorry. I could have sworn it was this patch. And I doubled checked in SVN _and_ HG it was this one. But now I confirm it's Ralph's (very explicit) patch, and the patch I was refering to is really doing what it pretends.

Weird.

Btw, commit done.

Sylvain

devel-bounces@open-mpi.org a écrit sur 07/09/2011 16:00:18 :

> De : Rolf vandeVaart <rvandevaart@nvidia.com>
> A : Open MPI Developers <devel@open-mpi.org>
> Date : 07/09/2011 16:00
> Objet : Re: [OMPI devel] Bull Vendor ID disappeared from IB ini file
> Envoyé par : devel-bounces@open-mpi.org
>
>
> Actually, I think you are off by which commit undid the change.  It
> was this one.  And the message does suggest it might have caused problems.
>
> https://svn.open-mpi.org/trac/ompi/changeset/23764
> Timestamp:
>     09/17/10 19:04:06 (12 months ago)
> Author:
>     rhc
> Message:
>     WARNING: Work on the temp branch being merged here encountered
> problems with bugs in subversion. Considerable effort has gone into
> validating the branch. However, not all conditions can be checked,
> so users are cautioned that it may be advisable to not update from
> the trunk for a few days to allow MTT to identify platform-specific issues.
>    This merges the branch containing the revamped build system based
> around converting autogen from a bash script to a Perl program. Jeff
> has provided emails explaining the features contained in the change.
>     Please note that configure requirements on components HAVE
> CHANGED. For example. a configure.params file is no longer required
> in each component directory. See Jeff's emails for an explanation.
>
>
>
> ________________________________________
> From: devel-bounces@open-mpi.org [devel-bounces@open-mpi.org] On
> Behalf Of Sylvain Jeaugey [sylvain.jeaugey@bull.net]
> Sent: Wednesday, September 07, 2011 8:56 AM
> To: Open MPI Developers
> Subject: [OMPI devel] Bull Vendor ID disappeared from IB ini file
>
> Hi All,
>
> I just realized that Bull Vendor IDs for Infiniband cards disappeared from
> the trunk. Actually, they were removed shortly after we included them in
> last September.
>
> The original commit was :
> r23715 | derbeyn | 2010-09-03 16:13:19 +0200 (Fri, 03 Sep 2010) | 1 line
> Added Bull vendor id for ConnectX card
>
> An here is the commit that undid Nadia's patch :
> r23791 | swise | 2010-09-22 20:16:53 +0200 (Wed, 22 Sep 2010) | 2 lines
> Add T4 device IDs to openib btl params ini file.
>
> It does indeed add some T4 device IDs and removes our vendor ID. The other
> thing that bugs me is that unlike the commit message suggests, this patch
> does a lot more than adding T4 device ids. So, It looks like something
> went wrong on this commit (something like : I forgot to update and forced
> the commit) and it may be worth checking nothing else were reverted with
> this commit ...
>
> Sylvain
> _______________________________________________
> devel mailing list
> devel@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
> -----------------------------------------------------------------------------------
> This email message is for the sole use of the intended recipient(s)
> and may contain
> confidential information.  Any unauthorized review, use, disclosure
> or distribution
> is prohibited.  If you are not the intended recipient, please
> contact the sender by
> reply email and destroy all copies of the original message.
> -----------------------------------------------------------------------------------
>
> _______________________________________________
> devel mailing list
> devel@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel