fixing resource leaks
[oweals/gnunet.git] / src / ats / perf_ats_solver.c
index 4ad93e0baa40b8d6a5939cc1e77ccbe0d86cf819..1b804d693b532afafa20cfb203a901b9177d4e25 100644 (file)
@@ -116,10 +116,13 @@ struct PerfHandle
   struct Iteration *iterations_results;
 
   /**
-   * Array to store averaged result with length #peers
+   * Array to store averaged full solution result with length #peers
    */
   struct Result *averaged_full_result;
 
+  /**
+   * Array to store averaged updated solution result with length #peers
+   */
   struct Result *averaged_update_result;
 
   /**
@@ -127,7 +130,14 @@ struct PerfHandle
    */
   struct Result *current_result;
 
+  /**
+   * Current number of peers benchmarked
+   */
   int current_p;
+
+  /**
+   * Current number of addresses benchmarked
+   */
   int current_a;
 
   /**
@@ -173,7 +183,7 @@ struct PerfHandle
   /**
    * Number of iterations
    */
-  int iterations;
+  int total_iterations;
 
   /**
    * Current iteration
@@ -189,6 +199,11 @@ struct PerfHandle
    * Is a bulk operation running?
    */
   int expecting_solution;
+
+  /**
+   * Was the problem just updates?
+   */
+  int performed_update;
 };
 
 /**
@@ -347,6 +362,10 @@ static struct PerfHandle ph;
  */
 static int ret;
 
+
+/**
+ * Do shutdown
+ */
 static void
 end_now (int res)
 {
@@ -355,23 +374,22 @@ end_now (int res)
     GNUNET_STATISTICS_destroy (ph.stat, GNUNET_NO);
     ph.stat = NULL;
   }
-  /*
-   if (NULL != addresses)
-   {
-   GNUNET_CONTAINER_multihashmap_iterate (addresses, &addr_it, NULL);
-   GNUNET_CONTAINER_multihashmap_destroy (addresses);
-   addresses = NULL ;
-   }*/
-  if (NULL != ph.peers)
-  {
-    GNUNET_free(ph.peers);
-  }
+
+  GNUNET_free_non_null (ph.peers);
+  GNUNET_free_non_null (ph.iterations_results);
+  GNUNET_free_non_null (ph.averaged_full_result);
+  GNUNET_free_non_null (ph.averaged_update_result);
 
   GAS_normalization_stop ();
   ret = res;
 }
 
 
+/**
+ * Create a peer used for benchmarking
+ *
+ * @param cp the number of the peer
+ */
 static void
 perf_create_peer (int cp)
 {
@@ -383,7 +401,11 @@ perf_create_peer (int cp)
 }
 
 
-
+/**
+ * Perform an update for an address
+ *
+ * @param cur the address to update
+ */
 static void
 perf_update_address (struct ATS_Address *cur)
 {
@@ -475,6 +497,14 @@ perf_address_initial_update (void *solver,
           / 100);
 }
 
+/**
+ * Update a certain percentage of peers
+ *
+ * @param cp the current number of peers
+ * @param ca the current number of addresses
+ * @param percentage_peers the percentage of peers to update
+ */
+
 static void
 perf_update_all_addresses (unsigned int cp, unsigned int ca, unsigned int percentage_peers)
 {
@@ -520,14 +550,19 @@ perf_update_all_addresses (unsigned int cp, unsigned int ca, unsigned int percen
       {
         if (c_cur_a == r)
           perf_update_address (cur_address);
-
         c_cur_a ++;
       }
     }
   }
 }
 
-
+/**
+ * Create an address for a peer
+ *
+ * @param cp index of the peer
+ * @param ca index of the address
+ * @return the address
+ */
 static struct ATS_Address *
 perf_create_address (int cp, int ca)
 {
@@ -540,6 +575,14 @@ perf_create_address (int cp, int ca)
   return a;
 }
 
+
+/**
+ * Information callback for the solver
+ *
+ * @param op the solver operation
+ * @param stat status of the solver operation
+ * @param add additional solver information
+ */
 static void
 solver_info_cb (void *cls,
     enum GAS_Solver_Operation op,
@@ -597,10 +640,15 @@ solver_info_cb (void *cls,
         ph.current_result->d_lp = GNUNET_TIME_UNIT_FOREVER_REL;
         ph.current_result->d_mlp = GNUNET_TIME_UNIT_FOREVER_REL;
         ph.current_result->info = add;
-        if (add == GAS_INFO_UPDATED)
+        if ((add == GAS_INFO_UPDATED) || (GNUNET_YES == ph.performed_update))
+        {
           ph.current_result->update = GNUNET_YES;
+        }
         else
+        {
           ph.current_result->update = GNUNET_NO;
+        }
+
       }
       return;
     case GAS_OP_SOLVE_STOP:
@@ -796,7 +844,7 @@ write_gnuplot_script (char * data_fn, int iteration, int full)
       return;
   }
   if (-1 == iteration)
-    GNUNET_asprintf (&iter_text, "%s_%u", "avg",ph.iterations);
+    GNUNET_asprintf (&iter_text, "%s_%u", "avg",ph.total_iterations);
   else
     GNUNET_asprintf (&iter_text, "%u", iteration);
   if (GNUNET_YES == full)
@@ -883,7 +931,6 @@ write_gnuplot_script (char * data_fn, int iteration, int full)
  *
  * @param iteration the iteration to evaluate
  */
-
 static void
 evaluate (int iteration)
 {
@@ -985,9 +1032,13 @@ evaluate (int iteration)
 
     /* Print log */
     if (GNUNET_NO == cur->update)
+    {
       cur_res = &ph.averaged_full_result[cur->peers - ph.N_peers_start];
+    }
     else
+    {
       cur_res = &ph.averaged_update_result[cur->peers - ph.N_peers_start];
+    }
 
     cur_res->peers = cur->peers;
     cur_res->addresses = cur->addresses;
@@ -1011,12 +1062,12 @@ evaluate (int iteration)
       if (GNUNET_YES == cur->valid)
         cur_res->d_total.rel_value_us += cur->d_total.rel_value_us;
       fprintf (stderr,
-               "Total time to solve %s for %u peers %u addresses: %llu us\n",
-               (GNUNET_YES == cur->update) ? "updated" : "full",
-               cur->peers, cur->addresses, (unsigned long long) cur->d_total.rel_value_us);
+         "Total time to solve %s for %u peers %u addresses: %llu us\n",
+         (GNUNET_YES == cur->update) ? "updated" : "full",
+         cur->peers, cur->addresses,
+         (unsigned long long) cur->d_total.rel_value_us);
       GNUNET_asprintf(&str_d_total,
-                      "%llu",
-                      (unsigned long long) cur->d_total.rel_value_us);
+         "%llu", (unsigned long long) cur->d_total.rel_value_us);
     }
     else
       GNUNET_asprintf(&str_d_total, "-1");
@@ -1028,11 +1079,14 @@ evaluate (int iteration)
         cur_res->d_setup.rel_value_us += cur->d_setup.rel_value_us;
       fprintf (stderr, "Total time to setup %s %u peers %u addresses: %llu us\n",
           (GNUNET_YES == cur->update) ? "updated" : "full",
-          cur->peers, cur->addresses, (unsigned long long )cur->d_setup.rel_value_us);
-      GNUNET_asprintf(&str_d_setup, "%llu", (unsigned long long )cur->d_setup.rel_value_us);
+          cur->peers, cur->addresses,
+          (unsigned long long) cur->d_setup.rel_value_us);
+      GNUNET_asprintf(&str_d_setup, "%llu",
+          (unsigned long long )cur->d_setup.rel_value_us);
     }
     else
       GNUNET_asprintf(&str_d_setup, "-1");
+
     if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_lp.rel_value_us)
     {
       if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us == cur_res->d_lp.rel_value_us)
@@ -1040,17 +1094,17 @@ evaluate (int iteration)
       if (GNUNET_YES == cur->valid)
         cur_res->d_lp.rel_value_us += cur->d_lp.rel_value_us;
       fprintf (stderr,
-               "Total time to solve %s LP for %u peers %u addresses: %llu us\n",
-               (GNUNET_YES == cur->update) ? "updated" : "full",
-               cur->peers,
-               cur->addresses,
-               (unsigned long long )cur->d_lp.rel_value_us);
+         "Total time to solve %s LP for %u peers %u addresses: %llu us\n",
+         (GNUNET_YES == cur->update) ? "updated" : "full",
+         cur->peers,
+         cur->addresses,
+         (unsigned long long )cur->d_lp.rel_value_us);
       GNUNET_asprintf (&str_d_lp,
-                       "%llu",
-                       (unsigned long long )cur->d_lp.rel_value_us);
+          "%llu", (unsigned long long )cur->d_lp.rel_value_us);
     }
     else
       GNUNET_asprintf (&str_d_lp, "-1");
+
     if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_mlp.rel_value_us)
     {
       if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us == cur_res->d_mlp.rel_value_us)
@@ -1060,10 +1114,10 @@ evaluate (int iteration)
 
       fprintf (stderr, "Total time to solve %s MLP for %u peers %u addresses: %llu us\n",
           (GNUNET_YES == cur->update) ? "updated" : "full",
-          cur->peers, cur->addresses, (unsigned long long )cur->d_mlp.rel_value_us);
+          cur->peers, cur->addresses,
+          (unsigned long long )cur->d_mlp.rel_value_us);
       GNUNET_asprintf (&str_d_mlp,
-                       "%llu",
-                       (unsigned long long )cur->d_mlp.rel_value_us);
+          "%llu", (unsigned long long )cur->d_mlp.rel_value_us);
     }
     else
       GNUNET_asprintf (&str_d_mlp, "-1");
@@ -1116,14 +1170,18 @@ evaluate (int iteration)
   GNUNET_free_non_null (data_fn_update);
 }
 
-
+/**
+ * Evaluate average results for all iterations
+ */
 static void
 evaluate_average (void)
 {
-  int c;
+  int c_o;
+  int c_i;
 
   struct GNUNET_DISK_FileHandle *f_full;
   struct GNUNET_DISK_FileHandle *f_update;
+  struct Result *cur;
   char * data_fn_full;
   char * data_fn_update;
   char * data;
@@ -1142,7 +1200,7 @@ evaluate_average (void)
     GNUNET_asprintf (&data_fn_full,
                      "perf_%s_full_avg_%u-%u_%u_%u.data",
                      ph.ats_string,
-                     ph.iterations,
+                     ph.total_iterations,
                      ph.N_peers_start,
                      ph.N_peers_end,
                      ph.N_address);
@@ -1170,7 +1228,7 @@ evaluate_average (void)
   {
     GNUNET_asprintf (&data_fn_update, "perf_%s_update_avg_%u-%u_%u_%u.data",
         ph.ats_string,
-        ph.iterations,
+        ph.total_iterations,
         ph.N_peers_start,
         ph.N_peers_end,
         ph.N_address);
@@ -1195,112 +1253,134 @@ evaluate_average (void)
     write_gnuplot_script (data_fn_update, -1, GNUNET_NO);
   }
 
-  for (c = 0; c <= ph.N_peers_end - ph.N_peers_start; c++)
+  for (c_o = 0; c_o < 2; c_o++)
   {
-    struct Result *cur = &ph.averaged_full_result[c];
-
-    str_d_total = NULL;
-    str_d_setup = NULL;
-    str_d_lp = NULL;
-    str_d_mlp = NULL;
+    if (0 == c_o)
+      fprintf (stderr,
+          "Duration for a full solution averaged over %i iterations\n",
+          ph.total_iterations);
+    if (1 == c_o)
+      fprintf (stderr,
+          "Duration for a full solution averaged over %i iterations\n",
+          ph.total_iterations);
 
-    if (0 >= cur->valid)
+    for (c_i = 0; c_i <= ph.N_peers_end - ph.N_peers_start; c_i++)
     {
-      fprintf (stderr,
-         "No valid results for %s for %u peers %u addresses!\n",
-         (GNUNET_YES == cur->update) ? "updated" : "full",
-             cur->peers, cur->addresses);
+      if (0 == c_o)
+      {
+        cur = &ph.averaged_full_result[c_i];
+      }
+      else if ((GNUNET_YES == ph.measure_updates) && (1 == c_o))
+      {
+        cur = &ph.averaged_update_result[c_i];
+      }
+      else
+         break;
 
-      continue;
-    }
+      if (0 == cur->peers)
+        continue;
 
+      str_d_total = NULL;
+      str_d_setup = NULL;
+      str_d_lp = NULL;
+      str_d_mlp = NULL;
 
-    if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_total.rel_value_us)
-    {
-      fprintf (stderr,
-         "Average total time from %u iterations to solve %s for %u peers %u addresses: %llu us\n",
-         cur->valid,
-         (GNUNET_YES == cur->update) ? "updated" : "full",
-             cur->peers, cur->addresses,
-         (unsigned long long) cur->d_total.rel_value_us / cur->valid);
-      GNUNET_asprintf(&str_d_total, "%llu",
-         (unsigned long long) cur->d_total.rel_value_us / cur->valid);
-    }
-    else
-      GNUNET_asprintf (&str_d_total, "-1");
+      if (0 >= cur->valid)
+      {
+        fprintf (stderr,
+           "No valid results for %s for %u peers %u addresses!\n",
+           (GNUNET_YES == cur->update) ? "updated" : "full",
+               cur->peers, cur->addresses);
+        continue;
+      }
 
-    if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_setup.rel_value_us)
-    {
-      fprintf (stderr,
-         "Average total time from %u iterations to setup for %u peers %u addresses: %llu us\n",
-         cur->valid, cur->peers, cur->addresses,
-         (unsigned long long) cur->d_setup.rel_value_us / cur->valid);
-      GNUNET_asprintf(&str_d_setup, "%llu",
-         (unsigned long long) cur->d_setup.rel_value_us / cur->valid);
+      if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_total.rel_value_us)
+      {
+        fprintf (stderr,
+           "Average total time from %u iterations to solve %s for %u peers %u addresses: %llu us\n",
+           cur->valid,
+           (GNUNET_YES == cur->update) ? "updated" : "full",
+               cur->peers, cur->addresses,
+           (unsigned long long) cur->d_total.rel_value_us / cur->valid);
+        GNUNET_asprintf(&str_d_total, "%llu",
+           (unsigned long long) cur->d_total.rel_value_us / cur->valid);
+      }
+      else
+        GNUNET_asprintf (&str_d_total, "-1");
 
-    }
-    else
-      GNUNET_asprintf (&str_d_setup, "-1");
+      if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_setup.rel_value_us)
+      {
+        fprintf (stderr,
+           "Average total time from %u iterations to setup for %u peers %u addresses: %llu us\n",
+           cur->valid, cur->peers, cur->addresses,
+           (unsigned long long) cur->d_setup.rel_value_us / cur->valid);
+        GNUNET_asprintf(&str_d_setup, "%llu",
+           (unsigned long long) cur->d_setup.rel_value_us / cur->valid);
 
-    if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_lp.rel_value_us)
-    {
-      fprintf (stderr,
-         "Average total time from %u iterations to solve lp %s for %u peers %u addresses: %llu us\n",
-         cur->valid,
-         (GNUNET_YES == cur->update) ? "updated" : "full",
-         cur->peers, cur->addresses,
-         (unsigned long long) cur->d_lp.rel_value_us / cur->valid);
-      GNUNET_asprintf(&str_d_lp, "%llu",
-         (unsigned long long) cur->d_lp.rel_value_us / ph.iterations);
-    }
-    else
-      GNUNET_asprintf (&str_d_lp, "-1");
+      }
+      else
+        GNUNET_asprintf (&str_d_setup, "-1");
 
-    if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_mlp.rel_value_us)
-    {
-      fprintf (stderr,
-         "Average total time from %u iterations to solve mlp %s for %u peers %u addresses: %llu us\n",
-         cur->valid,
-         (GNUNET_YES == cur->update) ? "updated" : "full",
-             cur->peers, cur->addresses,
-         (unsigned long long) cur->d_mlp.rel_value_us / cur->valid);
-      GNUNET_asprintf(&str_d_mlp, "%llu",
-         (unsigned long long) cur->d_mlp.rel_value_us / cur->valid);
-    }
-    else
-      GNUNET_asprintf (&str_d_mlp, "-1");
+      if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_lp.rel_value_us)
+      {
+        fprintf (stderr,
+           "Average total time from %u iterations to solve lp %s for %u peers %u addresses: %llu us\n",
+           cur->valid,
+           (GNUNET_YES == cur->update) ? "updated" : "full",
+           cur->peers, cur->addresses,
+           (unsigned long long) cur->d_lp.rel_value_us / cur->valid);
+        GNUNET_asprintf(&str_d_lp, "%llu",
+           (unsigned long long) cur->d_lp.rel_value_us / ph.total_iterations);
+      }
+      else
+        GNUNET_asprintf (&str_d_lp, "-1");
 
-    data = NULL;
-    if (GNUNET_YES == ph.create_plot)
-    {
-      GNUNET_asprintf (&data,
-                       "%u;%u;%s;%s;%s;%s\n",
-                       cur->peers, cur->addresses,
-                       str_d_total,
-                       str_d_setup,
-                       str_d_lp,
-                       str_d_mlp);
-      if (cur->update == GNUNET_NO)
+      if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us != cur->d_mlp.rel_value_us)
       {
-        if (GNUNET_SYSERR == GNUNET_DISK_file_write (f_full, data, strlen(data)))
-          GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                      "Cannot write data to log file `%s'\n",
-                      data_fn_full);
+        fprintf (stderr,
+           "Average total time from %u iterations to solve mlp %s for %u peers %u addresses: %llu us\n",
+           cur->valid,
+           (GNUNET_YES == cur->update) ? "updated" : "full",
+               cur->peers, cur->addresses,
+           (unsigned long long) cur->d_mlp.rel_value_us / cur->valid);
+        GNUNET_asprintf(&str_d_mlp, "%llu",
+           (unsigned long long) cur->d_mlp.rel_value_us / cur->valid);
       }
-      if ((cur->update == GNUNET_YES) && (NULL != f_update))
+      else
+        GNUNET_asprintf (&str_d_mlp, "-1");
+
+      data = NULL;
+      if (GNUNET_YES == ph.create_plot)
       {
-        if (GNUNET_SYSERR == GNUNET_DISK_file_write (f_update, data, strlen(data)))
-          GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                      "Cannot write data to log file `%s'\n",
-                      data_fn_update);
+        GNUNET_asprintf (&data,
+                         "%u;%u;%s;%s;%s;%s\n",
+                         cur->peers, cur->addresses,
+                         str_d_total,
+                         str_d_setup,
+                         str_d_lp,
+                         str_d_mlp);
+        if (cur->update == GNUNET_NO)
+        {
+          if (GNUNET_SYSERR == GNUNET_DISK_file_write (f_full, data, strlen(data)))
+            GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                        "Cannot write data to log file `%s'\n",
+                        data_fn_full);
+        }
+        if ((cur->update == GNUNET_YES) && (NULL != f_update))
+        {
+          if (GNUNET_SYSERR == GNUNET_DISK_file_write (f_update, data, strlen(data)))
+            GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                        "Cannot write data to log file `%s'\n",
+                        data_fn_update);
+        }
+        GNUNET_free (data);
       }
-      GNUNET_free (data);
-    }
 
-    GNUNET_free_non_null (str_d_total);
-    GNUNET_free_non_null (str_d_setup);
-    GNUNET_free_non_null (str_d_lp);
-    GNUNET_free_non_null (str_d_mlp);
+      GNUNET_free_non_null (str_d_total);
+      GNUNET_free_non_null (str_d_setup);
+      GNUNET_free_non_null (str_d_lp);
+      GNUNET_free_non_null (str_d_mlp);
+    }
   }
 
   if ((NULL != f_full) && (GNUNET_SYSERR == GNUNET_DISK_file_close (f_full)))
@@ -1314,8 +1394,12 @@ evaluate_average (void)
   GNUNET_free_non_null (data_fn_update);
 }
 
+/**
+ * Run a performance iteration
+ */
+
 static void
-perf_run (void)
+perf_run_iteration (void)
 {
   struct ATS_Address *cur;
   struct ATS_Address *next;
@@ -1330,7 +1414,7 @@ perf_run (void)
   for (cp = 0; cp < count_p; cp++)
     perf_create_peer (cp);
   GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-      "Iteration %u of %u, added %u peers\n", ph.current_iteration, ph.iterations, cp);
+      "Iteration %u of %u, added %u peers\n", ph.current_iteration, ph.total_iterations, cp);
 
   for (cp = 0; cp < count_p; cp++)
   {
@@ -1363,9 +1447,7 @@ perf_run (void)
         ph.env.sf.s_bulk_stop (ph.solver);
       }
       else
-      {
         GNUNET_break (0);
-      }
 
       /* Problem is solved by the solver here due to unlocking */
       ph.expecting_solution = GNUNET_NO;
@@ -1378,6 +1460,7 @@ perf_run (void)
             "Updating problem with %u peers and %u addresses\n", cp + 1, ca);
 
         ph.expecting_solution = GNUNET_YES;
+        ph.performed_update = GNUNET_YES;
         if (GNUNET_NO == ph.bulk_running)
         {
           ph.bulk_running = GNUNET_YES;
@@ -1387,6 +1470,7 @@ perf_run (void)
         ph.bulk_running = GNUNET_NO;
         ph.env.sf.s_bulk_stop (ph.solver);
         /* Problem is solved by the solver here due to unlocking */
+        ph.performed_update = GNUNET_NO;
         ph.expecting_solution = GNUNET_NO;
       }
       GNUNET_assert (GNUNET_NO == ph.bulk_running);
@@ -1494,10 +1578,10 @@ run (void *cls, char * const *args, const char *cfgfile,
 
   if (ph.N_peers_start != ph.N_peers_end)
     fprintf (stderr, "Benchmarking solver `%s' with %u to %u peers and %u addresses in %u iterations\n",
-        ph.ats_string, ph.N_peers_start, ph.N_peers_end, ph.N_address, ph.iterations);
+        ph.ats_string, ph.N_peers_start, ph.N_peers_end, ph.N_address, ph.total_iterations);
   else
     fprintf (stderr, "Benchmarking solver `%s' with %u peers and %u addresses in %u iterations\n",
-        ph.ats_string, ph.N_peers_end, ph.N_address, ph.iterations);
+        ph.ats_string, ph.N_peers_end, ph.N_address, ph.total_iterations);
 
   if (0 == ph.opt_update_percent)
     ph.opt_update_percent = DEFAULT_UPDATE_PERCENTAGE;
@@ -1519,9 +1603,8 @@ run (void *cls, char * const *args, const char *cfgfile,
   }
 
   /* Create array of DLL to store results for iterations */
-  ph.iterations_results = GNUNET_malloc (sizeof (struct Iteration) * ph.iterations);
+  ph.iterations_results = GNUNET_malloc (sizeof (struct Iteration) * ph.total_iterations);
   ph.averaged_full_result = GNUNET_malloc (sizeof (struct Result) * ((ph.N_peers_end + 1) - ph.N_peers_start));
-  ph.averaged_update_result = GNUNET_malloc (sizeof (struct Result) * ((ph.N_peers_end + 1) - ph.N_peers_start));
   for (c = 0; c <= ph.N_peers_end - ph.N_peers_start; c++)
   {
     ph.averaged_full_result[c].d_setup = GNUNET_TIME_UNIT_FOREVER_REL;
@@ -1529,6 +1612,14 @@ run (void *cls, char * const *args, const char *cfgfile,
     ph.averaged_full_result[c].d_lp = GNUNET_TIME_UNIT_FOREVER_REL;
     ph.averaged_full_result[c].d_mlp = GNUNET_TIME_UNIT_FOREVER_REL;
   }
+  ph.averaged_update_result = GNUNET_malloc (sizeof (struct Result) * ((ph.N_peers_end + 1) - ph.N_peers_start));
+  for (c = 0; c <= ph.N_peers_end - ph.N_peers_start; c++)
+  {
+    ph.averaged_update_result[c].d_setup = GNUNET_TIME_UNIT_FOREVER_REL;
+    ph.averaged_update_result[c].d_total = GNUNET_TIME_UNIT_FOREVER_REL;
+    ph.averaged_update_result[c].d_lp = GNUNET_TIME_UNIT_FOREVER_REL;
+    ph.averaged_update_result[c].d_mlp = GNUNET_TIME_UNIT_FOREVER_REL;
+  }
 
   /* Load solver */
   ph.env.cfg = solver_cfg;
@@ -1566,9 +1657,9 @@ run (void *cls, char * const *args, const char *cfgfile,
   }
 
   /* Do the benchmark */
-  for (ph.current_iteration = 1; ph.current_iteration <= ph.iterations; ph.current_iteration++)
+  for (ph.current_iteration = 1; ph.current_iteration <= ph.total_iterations; ph.current_iteration++)
   {
-    perf_run ();
+    perf_run_iteration ();
     evaluate (ph.current_iteration);
   }
   evaluate_average ();
@@ -1585,6 +1676,12 @@ run (void *cls, char * const *args, const char *cfgfile,
   ph.solver = NULL;
 }
 
+/**
+ * Main function of the benchmark
+ *
+ * @param argc argument count
+ * @param argv argument values
+ */
 int
 main (int argc, char *argv[])
 {
@@ -1596,7 +1693,7 @@ main (int argc, char *argv[])
   ph.ats_string = NULL;
   ph.create_plot = GNUNET_NO;
   ph.measure_updates = GNUNET_NO;
-  ph.iterations = 1;
+  ph.total_iterations = 1;
 
   static struct GNUNET_GETOPT_CommandLineOption options[] = {
       { 'a', "addresses", NULL,
@@ -1610,7 +1707,7 @@ main (int argc, char *argv[])
           1, &GNUNET_GETOPT_set_uint, &ph.N_peers_end },
       { 'i', "iterations", NULL,
           gettext_noop ("number of iterations used for averaging (default: 1)"),
-          1, &GNUNET_GETOPT_set_uint, &ph.iterations },
+          1, &GNUNET_GETOPT_set_uint, &ph.total_iterations },
       { 'p', "percentage", NULL,
           gettext_noop ("update a fix percentage of addresses"),
           1, &GNUNET_GETOPT_set_uint, &ph.opt_update_percent },
@@ -1624,7 +1721,6 @@ main (int argc, char *argv[])
   };
 
   GNUNET_PROGRAM_run (argc, argv, argv[0], NULL, options, &run, argv[0]);
-
   return ret;
 }