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