Ok, thanks for the confirmation. On Apr 25, 2013, at 5:48 AM, George Bosilca <[email protected]> wrote:
> Sure, it should be included in the 1.6 as well. > > George. > > On Apr 25, 2013, at 03:39 , Jeff Squyres (jsquyres) <[email protected]> > 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 <[email protected]> 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)" <[email protected]> >>> wrote: >>> >>>> George / Brian / Nathan -- >>>> >>>> Can you guys comment on this patch? >>>> >>>> >>>> On Apr 4, 2013, at 4:40 PM, Martin SCHREIBER <[email protected]> >>>> 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 >>>>> [email protected] >>>>> http://www.open-mpi.org/mailman/listinfo.cgi/devel >>>> >>>> >>>> -- >>>> Jeff Squyres >>>> [email protected] >>>> For corporate legal information go to: >>>> http://www.cisco.com/web/about/doing_business/legal/cri/ >>>> >>>> >>>> _______________________________________________ >>>> devel mailing list >>>> [email protected] >>>> http://www.open-mpi.org/mailman/listinfo.cgi/devel >>> >>> >>> _______________________________________________ >>> devel mailing list >>> [email protected] >>> http://www.open-mpi.org/mailman/listinfo.cgi/devel >> >> >> -- >> Jeff Squyres >> [email protected] >> For corporate legal information go to: >> http://www.cisco.com/web/about/doing_business/legal/cri/ >> >> >> _______________________________________________ >> devel mailing list >> [email protected] >> http://www.open-mpi.org/mailman/listinfo.cgi/devel > > > _______________________________________________ > devel mailing list > [email protected] > http://www.open-mpi.org/mailman/listinfo.cgi/devel -- Jeff Squyres [email protected] For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/
