Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] [EXTERNAL] Re: [OMPI svn] svn:open-mpi r26180
From: Barrett, Brian W (bwbarre_at_[hidden])
Date: 2012-03-23 09:52:02


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, brbarret_at_[hidden] 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
>>>> svn_at_[hidden]
>>>> http://www.open-mpi.org/mailman/listinfo.cgi/svn
>>>
>>>
>>> _______________________________________________
>>> devel mailing list
>>> devel_at_[hidden]
>>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>
>>
>> --
>> Jeff Squyres
>> jsquyres_at_[hidden]
>> For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/
>>
>>
>> _______________________________________________
>> devel mailing list
>> devel_at_[hidden]
>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>
>
> _______________________________________________
> devel mailing list
> devel_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>



  • application/pkcs7-signature attachment: smime.p7s