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-boun...@open-mpi.org [devel-boun...@open-mpi.org] On Behalf Of 
Sylvain Jeaugey [sylvain.jeau...@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
de...@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.
-----------------------------------------------------------------------------------

Reply via email to