Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
First step in the implementation of topologies in SMPI
[simgrid.git] / src / smpi / smpi_comm.c
index 1c36589..a1b5939 100644 (file)
 XBT_LOG_NEW_DEFAULT_SUBCATEGORY(smpi_comm, smpi,
                                 "Logging specific to SMPI (comm)");
 
+
+
+/* Support for cartesian topology was added, but there are 2 other types of
+ * topology, graph et dist graph. In order to support them, we have to add a
+ * field MPIR_Topo_type, and replace the MPI_Topology field by an union. */
+
 typedef struct s_smpi_mpi_communicator {
   MPI_Group group;
+  MPIR_Topo_type topoType; 
+  MPI_Topology topo; // to be replaced by an union
   int refcount;
 } s_smpi_mpi_communicator_t;
 
@@ -37,7 +45,7 @@ static int smpi_compare_rankmap(const void *a, const void *b)
   return 1;
 }
 
-MPI_Comm smpi_comm_new(MPI_Group group)
+MPI_Comm smpi_comm_new(MPI_Group group, MPI_Topology topo)
 {
   MPI_Comm comm;
 
@@ -45,12 +53,14 @@ MPI_Comm smpi_comm_new(MPI_Group group)
   comm->group = group;
   smpi_group_use(comm->group);
   comm->refcount=1;
+  comm->topo = topo;
   return comm;
 }
 
 void smpi_comm_destroy(MPI_Comm comm)
 {
   smpi_group_unuse(comm->group);
+  smpi_topo_destroy(comm->topo); // there's no use count on topos
   smpi_comm_unuse(comm);
 }
 
@@ -59,6 +69,12 @@ MPI_Group smpi_comm_group(MPI_Comm comm)
   return comm->group;
 }
 
+MPI_Topology smpi_comm_topo(MPI_Comm comm) {
+  if (comm != MPI_COMM_NULL)
+    return comm->topo;
+  return NULL;
+}
+
 int smpi_comm_size(MPI_Comm comm)
 {
   return smpi_group_size(smpi_comm_group(comm));
@@ -157,7 +173,7 @@ MPI_Comm smpi_comm_split(MPI_Comm comm, int color, int key)
       }
     } /* otherwise, exit with group_out == NULL */
   }
-  return group_out ? smpi_comm_new(group_out) : MPI_COMM_NULL;
+  return group_out ? smpi_comm_new(group_out, NULL) : MPI_COMM_NULL;
 }
 
 void smpi_comm_use(MPI_Comm comm){
@@ -169,3 +185,4 @@ void smpi_comm_unuse(MPI_Comm comm){
   if(comm->refcount==0)
     xbt_free(comm);
 }
+