I am afraid the suggestion on the mailing list only addressed half of the
problem. Indeed, alloca is used in two files (isend and irecv) while the
suggested patch only only fixed the one in isend.

  George.



On Fri, Aug 1, 2014 at 12:12 AM, Gilles Gouaillardet <
gilles.gouaillar...@iferc.org> wrote:

>  Paul,
>
> As Ralph pointed, this issue was reported last month on the user mailing
> list.
>
> #include <alloca.h> did not help :
> http://www.open-mpi.org/community/lists/users/2014/07/24883.php
>
> I will try if i can reproduce and fix this issue on a solaris10 (but x86)
> VM
>
> BTW, are you using the GNU compiler ?
>
> Cheers,
>
> Gilles
>
>
> On 2014/08/01 13:08, Paul Hargrove wrote:
>
> In general I am only setup to build from tarballs, not svn.
> However, I can (and will) apply this change manually w/o difficulty.
>
> I will report back when I've had a chance to try that.
>
> I already have many builds in-flight to test George's atomics patch and am
> in danger of confusing myself if I am not careful.
>
> -Paul
>
>
> On Thu, Jul 31, 2014 at 8:29 PM, Ralph Castain <r...@open-mpi.org> 
> <r...@open-mpi.org> wrote:
>
>
>  FWIW: we had Siegmar try that and it didn't solve the problem. Paul?
>
>
> On Jul 31, 2014, at 8:28 PM, svn-commit-mai...@open-mpi.org wrote:
>
>
>  Author: bosilca (George Bosilca)
> Date: 2014-07-31 23:28:23 EDT (Thu, 31 Jul 2014)
> New Revision: 32388
> URL: https://svn.open-mpi.org/trac/ompi/changeset/32388
>
> Log:
> Missing alloca.h. Thanks Paul for catching this.
>
> Text files modified:
>   trunk/ompi/mca/pml/ob1/pml_ob1_irecv.c |     3 +++
>   trunk/ompi/mca/pml/ob1/pml_ob1_isend.c |     3 +++
>   2 files changed, 6 insertions(+), 0 deletions(-)
>
> Modified: trunk/ompi/mca/pml/ob1/pml_ob1_irecv.c
>
>
>  
> ==============================================================================
>
>  --- trunk/ompi/mca/pml/ob1/pml_ob1_irecv.c    Thu Jul 31 21:00:42 2014
>
>       (r32387)
>
>  +++ trunk/ompi/mca/pml/ob1/pml_ob1_irecv.c    2014-07-31 23:28:23 EDT
>
>  (Thu, 31 Jul 2014)      (r32388)
>
>  @@ -28,6 +28,9 @@
> #include "pml_ob1_recvfrag.h"
> #include "ompi/peruse/peruse-internal.h"
> #include "ompi/message/message.h"
> +#if HAVE_ALLOCA_H
> +#include <alloca.h>
> +#endif  /* HAVE_ALLOCA_H */
>
> int mca_pml_ob1_irecv_init(void *addr,
>                            size_t count,
>
> Modified: trunk/ompi/mca/pml/ob1/pml_ob1_isend.c
>
>
>  
> ==============================================================================
>
>  --- trunk/ompi/mca/pml/ob1/pml_ob1_isend.c    Thu Jul 31 21:00:42 2014
>
>       (r32387)
>
>  +++ trunk/ompi/mca/pml/ob1/pml_ob1_isend.c    2014-07-31 23:28:23 EDT
>
>  (Thu, 31 Jul 2014)      (r32388)
>
>  @@ -26,6 +26,9 @@
> #include "pml_ob1_sendreq.h"
> #include "pml_ob1_recvreq.h"
> #include "ompi/peruse/peruse-internal.h"
> +#if HAVE_ALLOCA_H
> +#include <alloca.h>
> +#endif  /* HAVE_ALLOCA_H */
>
> int mca_pml_ob1_isend_init(void *buf,
>                            size_t count,
> _______________________________________________
> svn mailing 
> listsvn@open-mpi.orghttp://www.open-mpi.org/mailman/listinfo.cgi/svn
>
>  _______________________________________________
> devel mailing listde...@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/07/15424.php
>
>
>
>
> _______________________________________________
> devel mailing listde...@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/08/15427.php
>
>
>
> _______________________________________________
> 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/08/15428.php
>

Reply via email to