Gah; sorry about that. I thought I had tested that code path, but it appears not. Stupid flexibility of Open MPI :). Jeff's correct, it's supposed to be MPI_SOURCE. Thanks, Jeff!
Brian On Mar 22, 2012, at 6:50 PM, Ralph Castain wrote: > Thanks! > > On Mar 22, 2012, at 6:12 PM, Jeffrey Squyres wrote: > >>> From the context of the code, I'm assuming it's supposed to be MPI_SOURCE. >>> I'll commit shortly. >> >> >> On Mar 22, 2012, at 7:54 PM, Ralph Castain wrote: >> >>> Yo Brian >>> >>> I believe you have an error in this commit: >>> >>> pml_ob1_iprobe.c:113: error: 'ompi_status_public_t' has no member named >>> 'MPI_STATUS' >>> >>> I checked the definition of that struct, and the error is correct - there >>> is no such member. What should it be? >>> >>> Ralph >>> >>> >>> On Mar 22, 2012, at 4:55 PM, brbar...@osl.iu.edu wrote: >>> >>>> Author: brbarret >>>> Date: 2012-03-22 18:55:59 EDT (Thu, 22 Mar 2012) >>>> New Revision: 26180 >>>> URL: https://svn.open-mpi.org/trac/ompi/changeset/26180 >>>> >>>> Log: >>>> * Implement matched probe for the CM PML. Required adding a peer field to >>>> the ompi_message_t structure to properly initialize convertor (the peer >>>> is available in the request in OB1, and wasn't needed when I did the >>>> original implementation). >>>> * Implement matched probe for the Portals4 MTL and add NULL function >>>> pointers >>>> for the other MTLs. >>>> * Add add_comm and del_comm functions to portals4 MTL so that direct call >>>> almost works again. >>>> * Add NEWS item that we've implemented matched probe >>>> >>>> Added: >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4_message.c >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4_message.h >>>> Text files modified: >>>> trunk/NEWS | 4 + >>>> >>>> trunk/ompi/mca/mtl/mtl.h | 16 ++++++ >>>> >>>> trunk/ompi/mca/mtl/mx/mtl_mx.c | 6 + >>>> >>>> trunk/ompi/mca/mtl/mxm/mtl_mxm.c | 2 >>>> >>>> trunk/ompi/mca/mtl/portals/mtl_portals.c | 2 >>>> >>>> trunk/ompi/mca/mtl/portals4/Makefile.am | 4 + >>>> >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4.c | 24 ++++++++- >>>> >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4.h | 24 ++++++++ >>>> >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4_component.c | 13 ++++ >>>> >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4_probe.c | 105 >>>> ++++++++++++++++++++++++++++++++++++++- >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4_recv.c | 44 >>>> ++++++++++++++++ >>>> trunk/ompi/mca/mtl/portals4/mtl_portals4_request.h | 4 + >>>> >>>> trunk/ompi/mca/mtl/psm/mtl_psm.c | 4 + >>>> >>>> trunk/ompi/mca/pml/cm/pml_cm_cancel.c | 3 >>>> >>>> trunk/ompi/mca/pml/cm/pml_cm_probe.c | 28 +++++++++- >>>> >>>> trunk/ompi/mca/pml/cm/pml_cm_recv.c | 65 >>>> +++++++++++++++++++++++- >>>> trunk/ompi/mca/pml/cm/pml_cm_recvreq.h | 23 ++++++++ >>>> >>>> trunk/ompi/mca/pml/ob1/pml_ob1_iprobe.c | 3 >>>> >>>> trunk/ompi/message/message.h | 18 +++--- >>>> >>>> 19 files changed, 352 insertions(+), 40 deletions(-) >>>> >>>> >>>> Diff not shown due to size (33150 bytes). >>>> To see the diff, run the following command: >>>> >>>> svn diff -r 26179:26180 --no-diff-deleted >>>> >>>> _______________________________________________ >>>> svn mailing list >>>> s...@open-mpi.org >>>> http://www.open-mpi.org/mailman/listinfo.cgi/svn >>> >>> >>> _______________________________________________ >>> 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 >
smime.p7s
Description: S/MIME cryptographic signature