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


Reply via email to