Browse Source

fix warnings

(cherry picked from commit 06943d364b18a2c4819037713f1339935c98819d)
Samuel Thibault 6 years ago
parent
commit
7a12085827

+ 0 - 2
mpi/examples/user_datatype/my_interface.c

@@ -111,8 +111,6 @@ static void data_register_data_handle(starpu_data_handle_t handle, unsigned home
 {
 	struct starpu_my_data_interface *my_data_interface = (struct starpu_my_data_interface *) data_interface;
 
-	struct starpu_my_data *data = (struct starpu_my_data *)my_data_interface->ptr;
-
 	unsigned node;
 	for (node = 0; node < STARPU_MAXNODES; node++)
 	{

+ 0 - 1
mpi/src/starpu_mpi_cache.c

@@ -98,7 +98,6 @@ void _starpu_mpi_cache_shutdown()
 	}
 	STARPU_PTHREAD_MUTEX_UNLOCK(&_cache_mutex);
 	STARPU_PTHREAD_MUTEX_DESTROY(&_cache_mutex);
-	free(_cache_data);
 	_starpu_mpi_cache_stats_shutdown();
 }
 

+ 1 - 1
src/core/perfmodel/perfmodel_history.c

@@ -1046,7 +1046,7 @@ void starpu_perfmodel_dump_xml(FILE *f, struct starpu_perfmodel *model)
 		int nimpls = model->state->nimpls[comb];
 		for (impl = 0; impl < nimpls; impl++)
 		{
-			fprintf(f, "    <implementation id=\"%u\">\n", impl);
+			fprintf(f, "    <implementation id=\"%d\">\n", impl);
 			char archname[STR_SHORT_LENGTH];
 			starpu_perfmodel_get_arch_name(arch_combs[comb], archname,  sizeof(archname), impl);
 			fprintf(f, "      <!-- %s -->\n", archname);

+ 14 - 8
src/debug/traces/anim.c

@@ -409,10 +409,13 @@ void _starpu_fxt_component_push(FILE *output, struct starpu_fxt_options *options
 	}
 	else
 		nflowing++;
-	if (prio)
-		to_p->npriotasks++;
-	else
-		to_p->ntasks++;
+	if (to_p)
+	{
+		if (prio)
+			to_p->npriotasks++;
+		else
+			to_p->ntasks++;
+	}
 
 	// fprintf(stderr,"push from %s to %s\n", from_p?from_p->name:"none", to_p?to_p->name:"none");
 	fxt_component_print_step(output, options, timestamp, workerid, 1, from_p, to_p);
@@ -435,10 +438,13 @@ void _starpu_fxt_component_pull(FILE *output, struct starpu_fxt_options *options
 		COMPONENT_FIND(components, to, to_p);
 		STARPU_ASSERT(to_p);
 	}
-	if (prio)
-		from_p->npriotasks--;
-	else
-		from_p->ntasks--;
+	if (from_p)
+	{
+		if (prio)
+			from_p->npriotasks--;
+		else
+			from_p->ntasks--;
+	}
 	if (to_p)
 	{
 		if (prio)

+ 2 - 2
tests/datawizard/partition_init.c

@@ -27,7 +27,7 @@ void my_func(void *buffers[], void *cl_arg)
 	for(i=0 ; i<nb ; i++)
 	{
 		v[i] = i+42;
-		FPRINTF(stderr, "setting v[%d] to %d\n",i, v[i]);
+		FPRINTF(stderr, "setting v[%u] to %d\n",i, v[i]);
 	}
 }
 
@@ -44,7 +44,7 @@ void display_func(void *buffers[], void *cl_arg)
         int *v = (int *)STARPU_VECTOR_GET_PTR(buffers[0]);
 
 	unsigned i;
-	for(i=0 ; i<nb ; i++) FPRINTF(stderr, "v[%d] = %d\n", i, v[i]);
+	for(i=0 ; i<nb ; i++) FPRINTF(stderr, "v[%u] = %d\n", i, v[i]);
 }
 
 struct starpu_codelet display_codelet =

+ 0 - 2
tests/parallel_tasks/parallel_kernels_trivial.c

@@ -94,8 +94,6 @@ int main(void)
 	starpu_malloc((void **)&v, VECTORSIZE*sizeof(unsigned));
 	starpu_vector_data_register(&v_handle, STARPU_MAIN_RAM, (uintptr_t)v, VECTORSIZE, sizeof(unsigned));
 
-	unsigned nworker = starpu_worker_get_count() + starpu_combined_worker_get_count();
-
 	/* First submit a sequential task */
 	ret = starpu_task_insert(&cl_seq, STARPU_R, v_handle, 0);
 	if (ret == -ENODEV) goto enodev;