Browse Source

remove unused variables

Nathalie Furmento 7 years ago
parent
commit
3db2dec72e

+ 1 - 2
mpi/tests/user_defined_datatype_value.h

@@ -1,6 +1,6 @@
 /* StarPU --- Runtime system for heterogeneous multicore architectures.
  *
- * Copyright (C) 2013-2015,2017                           CNRS
+ * Copyright (C) 2013-2015,2017,2018                      CNRS
  * Copyright (C) 2014                                     Université de Bordeaux
  *
  * StarPU is free software; you can redistribute it and/or modify
@@ -129,7 +129,6 @@ static int copy_any_to_any(void *src_interface, unsigned src_node,
 {
 	struct starpu_value_interface *src_value = src_interface;
 	struct starpu_value_interface *dst_value = dst_interface;
-	int ret = 0;
 
 	return starpu_interface_copy((uintptr_t) src_value->value, 0, src_node,
 				     (uintptr_t) dst_value->value, 0, dst_node,

+ 0 - 3
src/core/sched_ctx.c

@@ -2432,7 +2432,6 @@ static void _starpu_sched_ctx_block_workers_in_parallel(unsigned sched_ctx_id, u
 	int master, temp_master = 0;
 	struct starpu_worker_collection *workers = sched_ctx->workers;
 	struct starpu_sched_ctx_iterator it;
-	int workers_count = 0;
 
 	/* temporarily put a master if needed */
 	if (sched_ctx->main_master == -1)
@@ -2442,7 +2441,6 @@ static void _starpu_sched_ctx_block_workers_in_parallel(unsigned sched_ctx_id, u
 	}
 	master = sched_ctx->main_master;
 
-	workers_count = 0;
 	workers->init_iterator(workers, &it);
 	while(workers->has_next(workers, &it))
 	{
@@ -2456,7 +2454,6 @@ static void _starpu_sched_ctx_block_workers_in_parallel(unsigned sched_ctx_id, u
 			_starpu_worker_request_blocking_in_parallel(worker);
 			STARPU_PTHREAD_MUTEX_UNLOCK_SCHED(&worker->sched_mutex);
 		}
-		workers_count++;
 	}
 
 	if (temp_master)

+ 1 - 4
src/sched_policies/deque_modeling_policy_data_aware.c

@@ -4,7 +4,7 @@
  * Copyright (C) 2013                                     Simon Archipoff
  * Copyright (C) 2009-2017                                Université de Bordeaux
  * Copyright (C) 2013                                     Joris Pablo
- * Copyright (C) 2010-2017                                CNRS
+ * Copyright (C) 2010-2018                                CNRS
  * Copyright (C) 2011                                     Télécom-SudParis
  * Copyright (C) 2016                                     Uppsala University
  *
@@ -486,7 +486,6 @@ static int _dm_push_task(struct starpu_task *task, unsigned prio, unsigned sched
 {
 	struct _starpu_dmda_data *dt = (struct _starpu_dmda_data*)starpu_sched_ctx_get_policy_data(sched_ctx_id);
 	int best = -1;
-	unsigned worker_ctx = 0;
 
 	double best_exp_end = 0.0;
 	double model_best = 0.0;
@@ -527,7 +526,6 @@ static int _dm_push_task(struct starpu_task *task, unsigned prio, unsigned sched
 			if (!(impl_mask & (1U << nimpl)))
 			{
 				/* no one on that queue may execute this task */
-				//			worker_ctx++;
 				continue;
 			}
 
@@ -607,7 +605,6 @@ static int _dm_push_task(struct starpu_task *task, unsigned prio, unsigned sched
 				best_impl = nimpl;
 			}
 		}
-		worker_ctx++;
 	}
 
 	if (unknown)

+ 1 - 3
tools/starpu_replay.c

@@ -1,7 +1,7 @@
 /* StarPU --- Runtime system for heterogeneous multicore architectures.
  *
  * Copyright (C) 2017                                     Erwan Leria
- * Copyright (C) 2017                                     CNRS
+ * Copyright (C) 2017, 2018                               CNRS
  * Copyright (C) 2016-2017                                Université de Bordeaux
  *
  * StarPU is free software; you can redistribute it and/or modify
@@ -709,7 +709,6 @@ int main(int argc, char **argv)
 		{
 			char * buffer = s + 7;
 			int mode_i = 0;
-			int i = 0;
 			const char * delim = " ";
 			char * token = strtok(buffer, delim);
 
@@ -720,7 +719,6 @@ int main(int argc, char **argv)
 				{
 					*(modes_ptr+mode_i) = STARPU_RW;
 					mode_i++;
-					i++;
 				}
 				else if (!strncmp(token, "R", 1))
 				{