Hi Tim,
Tim Mattox wrote:
Hmm, why should an external debugger dictate the members
of our communicator struct? Does MPICH, etc. have the exact same
structure for their comunicators? I doubt it.
Yes, only for mqs_communicator which shares between the DLL and TV. In
fact, the whole mpi_interface.h should not be touched. Nikolay Piskun of
Totalview Tech confirmed it.
If George's r16177 change is correct, mqs_taddr_t unique_id is never
used or set in OMPI.
Yes, it is not used for now. But George seems to have second thought
about using this unique_id instead of our own cid to make things
consistent. We'll stay tuned on what he has to do...
I'm sure TotalView & DLL already need to adjust what binary interface
they expect based on which version of OMPI they are interacting with...
what is one more struct difference?
If it is very important for this struct to not change, please add a comment
to the struct explaining why it shouldn't be changed.
I will, once I've figured out how to modify my existing comment for
r16200. Just to show you the kind of badness you'll see by taking things
out from the struct should show you a messed up comm name and comm_size,
like this on both Solaris AMD and SPARC. By putting this member back to
the struct, at least it can band aid the problem.
_WORLD
Comm_size 5571033412557294925
Comm_rank 4
Pending receives : none
Unexpected messages : no information available
Pending sends : none
_SELF
Comm_size 5571033412557294925
Comm_rank 1
Pending receives : none
Unexpected messages : no information available
Pending sends : none
_NULL
Comm_size 5571033412557294925
Comm_rank 0
Pending receives : none
Unexpected messages : no information available
Pending sends : none
On 9/24/07, pak...@osl.iu.edu <pak...@osl.iu.edu> wrote:
Author: paklui
Date: 2007-09-24 15:02:56 EDT (Mon, 24 Sep 2007)
New Revision: 16200
URL: https://svn.open-mpi.org/trac/ompi/changeset/16200
Log:
mqs_communicator type should not be changed as it serves as the interface
between Totalview and DLL.
Text files modified:
trunk/ompi/debuggers/mpi_interface.h | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
Modified: trunk/ompi/debuggers/mpi_interface.h
==============================================================================
--- trunk/ompi/debuggers/mpi_interface.h (original)
+++ trunk/ompi/debuggers/mpi_interface.h 2007-09-24 15:02:56 EDT (Mon,
24 Sep 2007)
@@ -259,7 +259,8 @@
/* A structure to represent a communicator */
typedef struct
{
- mqs_tword_t local_rank; /* The rank of this process Comm_rank */
+ mqs_taddr_t unique_id; /* A unique tag for the
communicator */
+ mqs_tword_t local_rank; /* The rank of this process
Comm_rank */
mqs_tword_t size; /* Comm_size */
char name[64]; /* the name if it has one */
} mqs_communicator;
_______________________________________________
svn-full mailing list
svn-f...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/svn-full
--
- Pak Lui
pak....@sun.com
