Ok, thanks for the confirmation.
On Apr 25, 2013, at 5:48 AM, George Bosilca <bosi...@icl.utk.edu> wrote:
> Sure, it should be included in the 1.6 as well.
>
> George.
>
> On Apr 25, 2013, at 03:39 , Jeff Squyres (jsquyres) <jsquy...@cisco.com>
> wrote:
>
>> Ok; thanks.
>>
>> It looks like this should go to v1.6, too -- right (Nathan/George/Brian)?
>>
>>
>>
>> On Apr 24, 2013, at 9:31 PM, Ralph Castain <r...@open-mpi.org> wrote:
>>
>>> This was already resolved - Nathan applied it, and it has been moved into
>>> v1.7
>>>
>>> On Apr 24, 2013, at 5:53 PM, "Jeff Squyres (jsquyres)" <jsquy...@cisco.com>
>>> wrote:
>>>
>>>> George / Brian / Nathan --
>>>>
>>>> Can you guys comment on this patch?
>>>>
>>>>
>>>> On Apr 4, 2013, at 4:40 PM, Martin SCHREIBER <martin.schrei...@in.tum.de>
>>>> wrote:
>>>>
>>>>> Dear developers,
>>>>>
>>>>> it seems that for messages of size 0 no convertor is created due to
>>>>> optimizations issues.
>>>>> However, this is not considered in MCA_PML_OB1_SEND_REQUEST_RESET when
>>>>> processing pending send requests.
>>>>>
>>>>> A fix (or something similar) for this issue is provided below. Please be
>>>>> aware, that I'm not an OpenMPI developer and that e. g. req_bytes_packed
>>>>> is the wrong field to check.
>>>>>
>>>>> If you have any questions or if you need any further information (stack
>>>>> trace, etc.), don't hesitate to ask!
>>>>>
>>>>> Best regards & thank you for developing OpenMPI,
>>>>>
>>>>> Martin
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> affected versions:
>>>>> openmpi-1.7
>>>>> openmpi-1.6.4
>>>>> and probably versions below
>>>>>
>>>>>
>>>>> file which needs a fix:
>>>>> pml_ob1_sendreq.h
>>>>>
>>>>>
>>>>> Inserting the if statement of the code-snippet below fixes the problem
>>>>> for me.
>>>>>
>>>>>
>>>>> #define MCA_PML_OB1_SEND_REQUEST_RESET(sendreq)
>>>>> \
>>>>> /* check for zero-package since convertor is not created for zero-size
>>>>> packages */ \
>>>>> if (sendreq->req_send.req_bytes_packed > 0)
>>>>> \
>>>>> {
>>>>> \
>>>>> size_t _position = 0;
>>>>> \
>>>>> opal_convertor_set_position(&sendreq->req_send.req_base.req_convertor,
>>>>> \
>>>>> &_position);
>>>>> \
>>>>> assert( 0 == _position );
>>>>> \
>>>>> }
>>>>>
>>>>> --
>>>>> Dipl.-Inf. Martin Schreiber
>>>>> Chair of Scientific Computing, http://www5.in.tum.de
>>>>> Technische Universität München, Fakultät für Informatik
>>>>> Boltzmannstr. 3 / Zi. 2.5.57, 85748 Garching, Germany
>>>>> Phone: +49-89-289-18630, Fax: +49-89-289-18607
>>>>>
>>>>> _______________________________________________
>>>>> devel mailing list
>>>>> de...@open-mpi.org
>>>>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>>>
>>>>
>>>> --
>>>> Jeff Squyres
>>>> jsquy...@cisco.com
>>>> For corporate legal information go to:
>>>> http://www.cisco.com/web/about/doing_business/legal/cri/
>>>>
>>>>
>>>> _______________________________________________
>>>> devel mailing list
>>>> de...@open-mpi.org
>>>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>>
>>>
>>> _______________________________________________
>>> devel mailing list
>>> de...@open-mpi.org
>>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>
>>
>> --
>> Jeff Squyres
>> jsquy...@cisco.com
>> For corporate legal information go to:
>> http://www.cisco.com/web/about/doing_business/legal/cri/
>>
>>
>> _______________________________________________
>> devel mailing list
>> de...@open-mpi.org
>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>
>
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
--
Jeff Squyres
jsquy...@cisco.com
For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/
