Ah, thats good to know. I have no problem with Gilles committing these
sorts of fixes to my components. I went ahead and committed this one
myself though.

-Nathan

On Mon, May 12, 2014 at 01:29:50PM +0000, Jeff Squyres (jsquyres) wrote:
> FWIW, Gilles has singed the OMPI IP agreement, has demonstrated care and 
> knowledge of the OMPI code base, and is an OMPI SVN committer now.
> 
> Just be aware that Gilles is about 12 hours off from North America.
> 
> 
> 
> On May 12, 2014, at 9:13 AM, "Hjelm, Nathan T" <hje...@lanl.gov> wrote:
> 
> > Hah. Thanks for catching that. I will commit your patch later today.
> > 
> > -Nathan
> > ________________________________________
> > From: devel [devel-boun...@open-mpi.org] on behalf of Gilles Gouaillardet 
> > [gilles.gouaillar...@iferc.org]
> > Sent: Monday, May 12, 2014 4:42 AM
> > To: Open MPI Developers
> > Subject: Re: [OMPI devel] scif btl side effects
> > 
> > i wrote this too early ...
> > 
> > the attached program produces incorrect results when ran with
> > --mca btl scif,vader,self
> > 
> > once the most up-to-date patch of #4610 has been applied, (at least) one
> > bug remain, and it is in the scif btl
> > 
> > the attached patch fixes it.
> > 
> > Gilles
> > 
> > On 2014/05/12 16:17, Gilles Gouaillardet wrote:
> >> Nathan,
> >> 
> >> On 2014/05/08 4:21, Hjelm, Nathan T wrote:
> >>> c) that being said, that should work so there is a bug
> >>> d) there is a regression in v1.8 and a bug that might have been always 
> >>> here
> >>> This is probably not a regression. The SCIF btl has been part of the 1.7 
> >>> series for some time. The nightly MTTs are probably missing one of the 
> >>> cases that causes this problem. Hopefully we can get this fixed before 
> >>> 1.8.2.
> >> as explained in #4610 (https://svn.open-mpi.org/trac/ompi/ticket/4610)
> >> the root cause is in the way data are unpacked.
> >> 
> >> The scif btl is ok :-)
> >> 
> >> when using --mca btl scif,self fragments can be received out of order,
> >> and that can trigger a bug introduced by r31496
> >> 
> >> that being said, --mca btl scif,vader,self does not work with r31496
> >> reverted.
> >> the root cause is an other bug in the way data are unpacked, it happen
> >> also when fragments are received out of order
> >> *and* fragments contain a subpart of a predefined datatype.
> >> in this case, the vader btl received a fragment of size 1325 *and* out
> >> of order and that caused the bug.
> >> 
> >> Gilles
> > 
> > _______________________________________________
> > devel mailing list
> > de...@open-mpi.org
> > Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> > Link to this post: 
> > http://www.open-mpi.org/community/lists/devel/2014/05/14772.php
> 
> 
> -- 
> 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
> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> Link to this post: 
> http://www.open-mpi.org/community/lists/devel/2014/05/14773.php

Attachment: pgpJil_3kF_vW.pgp
Description: PGP signature

Reply via email to