cosmetics on tracing mechanism

[simgrid.git] / src / smpi / smpi_bench.c

diff --git a/src/smpi/smpi_bench.c b/src/smpi/smpi_bench.c
index 3608df626a51524111d8580b0bf9e5d14f441c65..387cac49b4258e17cf3ade716ba64a08dd4091b4 100644 (file)
--- a/src/smpi/smpi_bench.c
+++ b/src/smpi/smpi_bench.c
@@ -35,32 +35,36 @@ void smpi_bench_destroy(void) {
    }
 }
 
-static void smpi_execute(double duration) {
+static void smpi_execute_flops(double flops) {
   smx_host_t host;
   smx_action_t action;
   smx_mutex_t mutex;
   smx_cond_t cond;
   e_surf_action_state_t state;
 
+  host = SIMIX_host_self();
+  mutex = SIMIX_mutex_init();
+  cond = SIMIX_cond_init();
+  DEBUG1("Handle real computation time: %f flops", flops);
+  action = SIMIX_action_execute(host, "computation", flops);
+  SIMIX_mutex_lock(mutex);
+  SIMIX_register_action_to_condition(action, cond);
+  for(state = SIMIX_action_get_state(action);
+      state == SURF_ACTION_READY ||
+      state == SURF_ACTION_RUNNING; state = SIMIX_action_get_state(action)) {
+    SIMIX_cond_wait(cond, mutex);
+  }
+  SIMIX_unregister_action_to_condition(action, cond);
+  SIMIX_mutex_unlock(mutex);
+  SIMIX_action_destroy(action);
+  SIMIX_cond_destroy(cond);
+  SIMIX_mutex_destroy(mutex);
+}
+
+static void smpi_execute(double duration) {
   if(duration >= xbt_cfg_get_double(_surf_cfg_set, "smpi/cpu_threshold")) {
-    host = SIMIX_host_self();
-    mutex = SIMIX_mutex_init();
-    cond = SIMIX_cond_init();
     DEBUG1("Sleep for %f to handle real computation time", duration);
-    duration *= xbt_cfg_get_double(_surf_cfg_set, "smpi/running_power");
-    action = SIMIX_action_execute(host, "computation", duration);
-    SIMIX_mutex_lock(mutex);
-    SIMIX_register_action_to_condition(action, cond);
-    for(state = SIMIX_action_get_state(action);
-        state == SURF_ACTION_READY ||
-        state == SURF_ACTION_RUNNING; state = SIMIX_action_get_state(action)) {
-      SIMIX_cond_wait(cond, mutex);
-    }
-    SIMIX_unregister_action_to_condition(action, cond);
-    SIMIX_mutex_unlock(mutex);
-    SIMIX_action_destroy(action);
-    SIMIX_cond_destroy(cond);
-    SIMIX_mutex_destroy(mutex);
+    smpi_execute_flops(duration * xbt_cfg_get_double(_surf_cfg_set, "smpi/running_power"));
   }
 }
 
@@ -83,6 +87,7 @@ void smpi_bench_end(int rank, const char* mpi_call) {
 
 unsigned int smpi_sleep(unsigned int secs) {
    smpi_execute((double)secs);
+   return secs;
 }
 
 int smpi_gettimeofday(struct timeval* tv, struct timezone* tz) {
@@ -126,7 +131,6 @@ void smpi_sample_1(int global, const char* file, int line, int max) {
 int smpi_sample_2(int global, const char* file, int line) {
    char* loc = sample_location(global, file, line);
    local_data_t* data;
-   double* simu;
 
    xbt_assert0(samples, "You did something very inconsistent, didn't you?");
    data = xbt_dict_get_or_null(samples, loc);
@@ -153,7 +157,6 @@ int smpi_sample_2(int global, const char* file, int line) {
 void smpi_sample_3(int global, const char* file, int line) {
    char* loc = sample_location(global, file, line);
    local_data_t* data;
-   double spent;
 
    xbt_assert0(samples, "You did something very inconsistent, didn't you?");
    data = xbt_dict_get_or_null(samples, loc);
@@ -165,6 +168,10 @@ void smpi_sample_3(int global, const char* file, int line) {
    DEBUG2("Average mean after %d steps is %f", data->count, data->time / (double)data->count);
 }
 
+void smpi_sample_flops(double flops) {
+   smpi_execute_flops(flops);
+}
+
 void* smpi_shared_malloc(size_t size, const char* file, int line) {
    char* loc = bprintf("%s:%d:%zu", file, line, size);
    shared_data_t* data;