On Feb 15, 2011, at 9:24 AM, Eugene Loh wrote:

> Ralph Castain wrote:
> 
>> On Feb 14, 2011, at 9:26 PM, Abhishek Kulkarni wrote:
>> 
>>> On Mon, 14 Feb 2011, Ralph Castain wrote:
>>>   
>>>> If the ability to turn "on" thread support is missing from 1.5, then that 
>>>> is an error.
>>>>     
>>> No, it was changed from "--enable-mpi-threads" to 
>>> "--enable-opal-multi-threads" on the trunk in r22841 [1].
>>>   
>>> If the changeset has not been brought over to v1.5, it indeed looks like an 
>>> anachronism in the README.
>>> 
>>> [1] https://svn.open-mpi.org/trac/ompi/changeset/22841
>>>   
>> My point is that it isn't an anachronism in the README, but an error in 1.5 
>> - it needs to have the ability to turn on thread safety.
>> 
> I'm not sure if we're making progress here.  So far as I can tell, the v1.5 
> README talks about --enable-opal-multi-threads.  This option does not 
> otherwise appear in v1.5, but only in the trunk.  So, to make the v1.5 README 
> consistent with the v1.5 source code (as opposed to talking about features 
> that will appear in unspecified future releases), either:
> 
> *) the comment should be removed from the README, or
> 
> *) opal-multi-threads should be CMRed to v1.5

My point is that option 2 needs to be done - not sure how much clearer I can be 
:-)


> 
>>>> On Feb 14, 2011, at 5:36 PM, Eugene Loh wrote:
>>>>     
>>>>> In the v1.5 README, I see this:
>>>>> 
>>>>> --enable-opal-multi-threads
>>>>> Enables thread lock support in the OPAL and ORTE layers. Does
>>>>> not enable MPI_THREAD_MULTIPLE - see above option for that feature.
>>>>> This is currently disabled by default.
>>>>> 
>>>>> I don't otherwise find opal-multi-threads at all in this branch.  It 
>>>>> seems to me, for such an option, one needs to move to the trunk.
>>>>> 
>>>>> Is this an error (anachronism) in the v1.5 README?
>>>>>       
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel


Reply via email to