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


Reply via email to