Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] MPI_Graph_create
From: Jeff Squyres (jsquyres_at_[hidden])
Date: 2009-12-01 16:54:42


You are absolutely correct. I've filed CMRs for v1.4 and v1.5.

Thanks for the heads up!

On Dec 1, 2009, at 4:26 PM, David Singleton wrote:

>
> It looks like MPI_Cart_create argument checking was fixed in 1.3.4
> but not MPI_Graph_create.
>
> vayu1:~/openmpi-1.3.4 > diff -w -u ompi/mpi/c/cart_create.c ompi/mpi/
> c/graph_create.c
>
> ...
>
> -int MPI_Cart_create(MPI_Comm old_comm, int ndims, int *dims,
> - int *periods, int reorder, MPI_Comm *comm_cart) {
> +int MPI_Graph_create(MPI_Comm old_comm, int nnodes, int *index,
> + int *edges, int reorder, MPI_Comm *comm_graph)
> +{
>
> ...
>
> + if ((0 > reorder) || (1 < reorder)) {
>
>
> David
>
> David Singleton wrote:
> >
> > Kiril Dichev has already pointed a problem with MPI_Cart_create
> > http://www.open-mpi.org/community/lists/devel/2009/08/6627.php
> > MPI_Graph_create has the same problem. I checked all other
> > functions with logical in arguments and no others do anything
> > similar.
> >
> _______________________________________________
> devel mailing list
> devel_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>

-- 
Jeff Squyres
jsquyres_at_[hidden]