Browse Source

Fix using MAX_MPI_NODES for create_ordered_stream_id, not the number of starpu memory nodes

Samuel Thibault 5 years ago
parent
commit
7f30784a0e

+ 2 - 2
src/debug/traces/starpu_fxt.c

@@ -1196,8 +1196,8 @@ static void handle_new_mem_node(struct fxt_ev_64 *ev, struct starpu_fxt_options
  */
 static int create_ordered_stream_id (int nodeid, int devid)
 {
-	static int stable[STARPU_MAXNODES][STARPU_MAXCUDADEVS];
-	STARPU_ASSERT(nodeid < STARPU_MAXNODES);
+	static int stable[MAX_MPI_NODES][STARPU_MAXCUDADEVS];
+	STARPU_ASSERT(nodeid < MAX_MPI_NODES);
 	STARPU_ASSERT(devid < STARPU_MAXCUDADEVS);
 	return stable[nodeid][devid]++;
 }

+ 2 - 0
src/debug/traces/starpu_fxt.h

@@ -41,6 +41,8 @@
 #include <starpu.h>
 #include "../../../include/starpu_fxt.h"
 
+#define MAX_MPI_NODES 64
+
 extern char _starpu_last_codelet_symbol[STARPU_NMAXWORKERS][(FXT_MAX_PARAMS-5)*sizeof(unsigned long)];
 
 void _starpu_fxt_dag_init(char *dag_filename);

+ 0 - 2
src/debug/traces/starpu_fxt_mpi.c

@@ -26,8 +26,6 @@
 #define STARPU_POTI_STR_LEN 200
 #endif
 
-#define MAX_MPI_NODES 64
-
 LIST_TYPE(mpi_transfer,
 	unsigned matched;
 	int src;